Add cross-targeting coverage

dotnet · May 18, 2017 · 7ba7a66 · 7ba7a66
1 parent 25c7fce
commit 7ba7a66
Showing 1 changed file with 20 additions and 5 deletions.
diff --git a/test/Microsoft.NET.Build.Tests/GivenThatWeWantToBuildADesktopExe.cs b/test/Microsoft.NET.Build.Tests/GivenThatWeWantToBuildADesktopExe.cs
@@ -378,13 +378,28 @@ public void It_generates_binding_redirects_if_needed()
             });
         }
 
-        [WindowsOnlyFact]
-        public void It_places_package_satellites_correctly()
+        [WindowsOnlyTheory]
+        [InlineData(true)]
+        [InlineData(false)]
+        public void It_places_package_satellites_correctly(bool crossTarget)
         {
             var testAsset = _testAssetsManager
-              .CopyTestAsset("DesktopUsingPackageWithSatellites")
-              .WithSource()
-              .Restore(Log);
+              .CopyTestAsset(
+                  "DesktopUsingPackageWithSatellites", 
+                  identifier: crossTarget ? "_cross" : "")
+              .WithSource();
+
+            if (crossTarget)
+            {
+                 testAsset = testAsset.WithProjectChanges(project =>
+                 {
+                     var ns = project.Root.Name.Namespace;
+                     var propertyGroup = project.Root.Elements(ns + "PropertyGroup").First();
+                     propertyGroup.Element(ns + "TargetFramework").Name += "s";
+                 });
+            }
+
+            testAsset.Restore(Log);
 
             var buildCommand = new BuildCommand(Log, testAsset.TestRoot);
             buildCommand