re-implement hide of nuget api key in build logs, add more tests for it
[teamcity/dotNetPackagesSupport.git] / nuget-tests / src / jetbrains / buildServer / nuget / tests / agent / NuGetPackActionFactoryTest.java
index 1d670cbd253e38aa8b2f1db5a79fe7bd195867bc..37da9d799410efb047b132bceec77243d73b68e9 100644 (file)
@@ -19,6 +19,7 @@ package jetbrains.buildServer.nuget.tests.agent;
 import jetbrains.buildServer.BaseTestCase;\r
 import jetbrains.buildServer.RunBuildException;\r
 import jetbrains.buildServer.agent.AgentRunningBuild;\r
+import jetbrains.buildServer.agent.BuildParametersMap;\r
 import jetbrains.buildServer.agent.BuildRunnerContext;\r
 import jetbrains.buildServer.nuget.agent.commands.impl.CommandFactoryImpl;\r
 import jetbrains.buildServer.nuget.agent.commands.impl.NuGetActionFactoryImpl;\r
@@ -55,6 +56,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
   private Collection<String> myExcludes;\r
   private Collection<String> myProperties;\r
   private Collection<String> myExtra;\r
+  private String cmd;\r
+  private BuildParametersMap myBuildParametersMap;\r
+\r
 \r
 \r
   @BeforeMethod\r
@@ -78,8 +82,14 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
     myExcludes = new ArrayList<String>();\r
     myProperties = new ArrayList<String>();\r
     myExtra = new ArrayList<String>();\r
+    myBuildParametersMap = m.mock(BuildParametersMap.class);\r
+    cmd = System.getenv("ComSpec");\r
 \r
     m.checking(new Expectations(){{\r
+      allowing(ctx).getBuildParameters(); will(returnValue(myBuildParametersMap));\r
+      allowing(myBuildParametersMap).getEnvironmentVariables(); will(returnValue(Collections.singletonMap("ComSpec", cmd)));\r
+\r
+\r
       allowing(ctx).getBuild(); will(returnValue(build));\r
       allowing(build).getCheckoutDirectory(); will(returnValue(myWorkingDir));\r
       allowing(myPackParameters).getSpecFile(); will(returnValue(myFile));\r
@@ -101,9 +111,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packTool(); will(returnValue(false));\r
       allowing(myPackParameters).packSymbols(); will(returnValue(false));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r
@@ -120,9 +130,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packTool(); will(returnValue(false));\r
       allowing(myPackParameters).packSymbols(); will(returnValue(false));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Properties", "p1=p2", "-Properties", "p3=p24")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Properties", "p1=p2", "-Properties", "p3=p24")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r
@@ -141,9 +151,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packSymbols();\r
       will(returnValue(false));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "arg1", "arg2")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "arg1", "arg2")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r
@@ -160,9 +170,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packTool(); will(returnValue(false));\r
       allowing(myPackParameters).packSymbols(); will(returnValue(false));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Exclude", "aaa", "-Exclude", "d/v/de")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Exclude", "aaa", "-Exclude", "d/v/de")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r
@@ -177,9 +187,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packTool(); will(returnValue(true));\r
       allowing(myPackParameters).packSymbols(); will(returnValue(false));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Tool")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Tool")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r
@@ -194,9 +204,9 @@ public class NuGetPackActionFactoryTest extends BaseTestCase {
       allowing(myPackParameters).packTool(); will(returnValue(false));\r
       allowing(myPackParameters).packSymbols(); will(returnValue(true));\r
 \r
-      oneOf(myProcessFactory).executeCommandLine(ctx, myNuGet,\r
+      oneOf(myProcessFactory).executeCommandLine(ctx, cmd,\r
               Arrays.asList(\r
-                      "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Symbols")\r
+                      "/c", myNuGet.getPath(), "pack", myFile.getPath(), "-OutputDirectory", myOut.getPath(), "-BasePath", myRoot.getPath(), "-Verbose", "-Version", "45.239.32.12", "-Symbols")\r
               , myWorkingDir,\r
               Collections.<String, String>emptyMap());\r
     }});\r