Fixing unit tests in ExecuteCommandTests (#556)

saibulusu · saibulusu · web-flow · commit d4f99e41cb6c · 2025-07-29T19:59:53.000Z
* Fixing ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnUnixSystems.

* Debugging ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnWindowsSystems.

* Debugging ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnWindowsSystems.

* Debugging ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnWindowsSystems.

* Printing full command in GetCommandsToExecute.

* Adding platform win.

* Removing unnecessary line.

---------

Co-authored-by: saibulusu &lt;saibulusu@microsoft.com&gt;
diff --git a/src/VirtualClient/VirtualClient.Dependencies.UnitTests/ExecuteCommandTests.cs b/src/VirtualClient/VirtualClient.Dependencies.UnitTests/ExecuteCommandTests.cs
@@ -368,6 +368,7 @@ public async Task ExecuteCommandResolvesPackagePathExpressionsOnInitializationIn
         }
 
         [Test]
+        [Platform("Win")]
         public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnWindowsSystems()
         {
             this.SetupDefaults(PlatformID.Win32NT);
@@ -407,7 +408,7 @@ public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCo
         [Test]
         public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnUnixSystems()
         {
-            this.SetupDefaults(PlatformID.Win32NT);
+            this.SetupDefaults(PlatformID.Unix);
             string packagePath = this.mockFixture.GetPackagePath("anypackage");
 
             // The package MUST exist on the system.

Original file line number	Diff line number	Diff line change
`@@ -368,6 +368,7 @@ public async Task ExecuteCommandResolvesPackagePathExpressionsOnInitializationIn`
`368`	`368`	`}`
`369`	`369`
`370`	`370`	`[Test]`
	`371`	`+ [Platform("Win")]`
`371`	`372`	`public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnWindowsSystems()`
`372`	`373`	`{`
`373`	`374`	`this.SetupDefaults(PlatformID.Win32NT);`
`@@ -407,7 +408,7 @@ public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCo`
`407`	`408`	`[Test]`
`408`	`409`	`public async Task ExecuteCommandTheResolvedPackagePathExpressionsWhenExecutingCommandsOnUnixSystems()`
`409`	`410`	`{`
`410`		`- this.SetupDefaults(PlatformID.Win32NT);`
	`411`	`+ this.SetupDefaults(PlatformID.Unix);`
`411`	`412`	`string packagePath = this.mockFixture.GetPackagePath("anypackage");`
`412`	`413`
`413`	`414`	`// The package MUST exist on the system.`