Enable test ExecuteCommandAsync_WithMultiplePackagesAndInvalidCertifi…

…cate_RaisesErrorsOnceAsync (#3342) * enable a test, change the comments; extend test limitation on Linux
NuGet · Jun 12, 2020 · ebf0ad1 · ebf0ad1
1 parent 2b06877
commit ebf0ad1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/build/vsts_build.yaml b/build/vsts_build.yaml
@@ -669,7 +669,7 @@ jobs:
 
 - job: Tests_On_Linux
   dependsOn: Initialize_Build
-  timeoutInMinutes: 15
+  timeoutInMinutes: 45
   variables:
     FULLVSTSBUILDNUMBER: $[dependencies.Initialize_Build.outputs['updatebuildnumber.FullVstsBuildNumber']]
     SDKVersionForBuild: $[dependencies.Initialize_Build.outputs['getSDKVersionForBuild.SDKVersionForBuild']]

diff --git a/test/NuGet.Core.Tests/NuGet.Commands.Test/SignCommandRunnerTests.cs b/test/NuGet.Core.Tests/NuGet.Commands.Test/SignCommandRunnerTests.cs
@@ -121,8 +121,8 @@ public async Task ExecuteCommandAsync_WithAmbiguousMatch_ThrowsAsync()
             }
         }
 
-        //skip this test as the signing APIs are not yet implemented. We should enable this test when signing APIs are implemented. Tracking issue:https://github.com/NuGet/Home/issues/8807
-#if IS_DESKTOP
+        //skip this test when signing is not supported, as the signing APIs are not implemented.
+#if IS_SIGNING_SUPPORTED
         [Fact]
         public async Task ExecuteCommandAsync_WithMultiplePackagesAndInvalidCertificate_RaisesErrorsOnceAsync()
         {