Merge pull request #4185 from brthor/brthor/migrate-x-targetting

Migration X-Targeting

TestAssets/TestProjects/TestAppWithMigrateableScripts/project.json

@@ -14,16 +14,16 @@
   },
   "scripts": {
     "prepublish": [
-      "echoscript prepublish_output ?%publish:ProjectPath%? ?%publish:Configuration%? ?%publish:OutputPath%? ?%publish:FullTargetFramework%?"
+      "./echoscript prepublish_output ?%publish:ProjectPath%? ?%publish:Configuration%? ?%publish:OutputPath%? ?%publish:FullTargetFramework%?"
     ],
     "postpublish": [
-      "echoscript postpublish_output ?%publish:ProjectPath%? ?%publish:Configuration%? ?%publish:OutputPath%? ?%publish:FullTargetFramework%?"
+      "./echoscript postpublish_output ?%publish:ProjectPath%? ?%publish:Configuration%? ?%publish:OutputPath%? ?%publish:FullTargetFramework%?"
     ],
     "precompile": [
-      "echoscript precompile_output ?%compile:Configuration%? ?%compile:OutputDir%? ?%compile:FullTargetFramework%?"
+      "./echoscript precompile_output ?%compile:Configuration%? ?%compile:OutputDir%? ?%compile:FullTargetFramework%?"
     ],
     "postcompile": [
-      "echoscript postcompile_output ?%compile:Configuration%? ?%compile:OutputDir%? ?%compile:FullTargetFramework%?"
+      "./echoscript postcompile_output ?%compile:Configuration%? ?%compile:OutputDir%? ?%compile:FullTargetFramework%?"
     ]
   }
 }

src/Microsoft.DotNet.ProjectJsonMigration/ProjectMigrator.cs

@@ -68,11 +68,6 @@ namespace Microsoft.DotNet.ProjectJsonMigration
 
         private void VerifyProject(IEnumerable<ProjectContext> projectContexts, string projectDirectory)
         {
-            if (projectContexts.Count() > 1)
-            {
-                MigrationErrorCodes.MIGRATE20011($"Multi-TFM projects currently not supported.").Throw();
-            }
-
             if (!projectContexts.Any())
             {
                 MigrationErrorCodes.MIGRATE1013($"No projects found in {projectDirectory}").Throw();

src/Microsoft.DotNet.ProjectJsonMigration/Rules/MigrateScriptsRule.cs

@@ -13,9 +13,6 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 {
     public class MigrateScriptsRule : IMigrationRule
     {
-        private static readonly string s_unixScriptExtension = ".sh";
-        private static readonly string s_windowsScriptExtension = ".cmd";
-
         private readonly ITransformApplicator _transformApplicator;
 
         public MigrateScriptsRule(ITransformApplicator transformApplicator = null)
@@ -41,79 +38,17 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
             string scriptSetName)
         {
             var target = CreateTarget(csproj, scriptSetName);
-            var count = 0;
             foreach (var scriptCommand in scriptCommands)
             {
-                var scriptExtensionPropertyName = AddScriptExtension(propertyGroup, scriptCommand, $"{scriptSetName}_{++count}");
+                AddExec(target, FormatScriptCommand(scriptCommand));
-                AddExec(target, FormatScriptCommand(scriptCommand, scriptExtensionPropertyName));
             }
 
             return target;
         }
 
-        private string AddScriptExtension(ProjectPropertyGroupElement propertyGroup, string scriptCommandline, string scriptId)
+        internal string FormatScriptCommand(string scriptCommandline)
         {
-            var scriptArguments = CommandGrammar.Process(
+            return ReplaceScriptVariables(scriptCommandline);
-                scriptCommandline,
-                (s) => null,
-                preserveSurroundingQuotes: false);
-
-            scriptArguments = scriptArguments.Where(argument => !string.IsNullOrEmpty(argument)).ToArray();
-            var scriptCommand = scriptArguments.First();
-            var propertyName = $"MigratedScriptExtension_{scriptId}";
-
-            var windowsScriptExtensionProperty = propertyGroup.AddProperty(propertyName,
-                s_windowsScriptExtension);
-            var unixScriptExtensionProperty = propertyGroup.AddProperty(propertyName,
-                s_unixScriptExtension);
-
-            windowsScriptExtensionProperty.Condition =
-                $" '$(OS)' == 'Windows_NT' and Exists('{scriptCommand}{s_windowsScriptExtension}') ";
-            unixScriptExtensionProperty.Condition =
-                $" '$(OS)' != 'Windows_NT' and Exists('{scriptCommand}{s_unixScriptExtension}') ";
-
-            return propertyName;
-        }
-
-        internal string FormatScriptCommand(string scriptCommandline, string scriptExtensionPropertyName)
-        {
-            var command = ReplaceScriptVariables(scriptCommandline);
-            command = AddScriptExtensionPropertyToCommandLine(command, scriptExtensionPropertyName);
-            return command;
-        }
-
-        internal string AddScriptExtensionPropertyToCommandLine(string scriptCommandline,
-            string scriptExtensionPropertyName)
-        {
-            var scriptArguments = CommandGrammar.Process(
-                scriptCommandline,
-                (s) => null,
-                preserveSurroundingQuotes: true);
-
-            scriptArguments = scriptArguments.Where(argument => !string.IsNullOrEmpty(argument)).ToArray();
-
-            var scriptCommand = scriptArguments.First();
-            var trimmedCommand = scriptCommand.Trim('\"').Trim('\'');
-
-            // Path.IsPathRooted only looks at paths conforming to the current os,
-            // we need to account for all things
-            if (!IsPathRootedForAnyOS(trimmedCommand))
-            {
-                scriptCommand = @".\" + scriptCommand;
-            }
-
-            if (scriptCommand.EndsWith("\"") || scriptCommand.EndsWith("'"))
-            {
-                var endChar = scriptCommand.Last();
-                scriptCommand = $"{scriptCommand.TrimEnd(endChar)}$({scriptExtensionPropertyName}){endChar}";
-            }
-            else
-            {
-                scriptCommand += $"$({scriptExtensionPropertyName})";
-            }
-
-            var command = string.Join(" ", new[] {scriptCommand}.Concat(scriptArguments.Skip(1)));
-            return command;
         }
 
         internal string ReplaceScriptVariables(string scriptCommandline)
@@ -165,6 +100,9 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
                 target.AfterTargets = targetHookInfo.TargetName;
             }
 
+            // Run Scripts After each inner build
+            target.Condition = " '$(IsCrossTargetingBuild)' != 'true' ";
+
             return target;
         }
 
@@ -185,13 +123,13 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 
         private static Dictionary<string, string> ScriptVariableToMSBuildMap => new Dictionary<string, string>
         {
-            { "compile:TargetFramework", null },  // TODO: Need Short framework name in CSProj
             { "compile:ResponseFile", null },     // Not migrated
             { "compile:CompilerExitCode", null }, // Not migrated
             { "compile:RuntimeOutputDir", null }, // Not migrated
             { "compile:RuntimeIdentifier", null },// Not Migrated
             
-            { "publish:TargetFramework", null },  // TODO: Need Short framework name in CSProj
+            { "compile:TargetFramework", "$(TargetFramework)" },
+            { "publish:TargetFramework", "$(TargetFramework)" },
             { "publish:Runtime", "$(RuntimeIdentifier)" },
 
             { "compile:FullTargetFramework", "$(TargetFrameworkIdentifier),Version=$(TargetFrameworkVersion)" },

src/Microsoft.DotNet.ProjectJsonMigration/Rules/MigrateTFMRule.cs

@@ -8,6 +8,7 @@ using System.Text;
 using Microsoft.Build.Construction;
 using Microsoft.DotNet.ProjectJsonMigration.Transforms;
 using NuGet.Frameworks;
+using System.Collections.Generic;
 
 namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 {
@@ -36,6 +37,13 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 
             CleanExistingProperties(csproj);
 
+            if (migrationRuleInputs.ProjectContexts.Count() > 1)
+            {
+                _transformApplicator.Execute(
+                    FrameworksTransform.Transform(migrationRuleInputs.ProjectContexts.Select(p => p.TargetFramework)),
+                    propertyGroup);
+            }
+
             foreach (var transform in _transforms)
             {
                 _transformApplicator.Execute(
@@ -90,5 +98,9 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
             new AddPropertyTransform<NuGetFramework>("TargetFrameworkVersion",
                 f => "v" + GetDisplayVersion(f.Version),
                 f => true);
+        private AddPropertyTransform<IEnumerable<NuGetFramework>> FrameworksTransform =>
+            new AddPropertyTransform<IEnumerable<NuGetFramework>>("TargetFrameworks",
+                frameworks => string.Join(";", frameworks.Select(f => f.GetShortFolderName())),
+                frameworks => true);
     }
 }

src/Microsoft.DotNet.ProjectJsonMigration/Rules/SaveOutputProjectRule.cs

@@ -2,6 +2,7 @@
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
 using System.IO;
+using Microsoft.Build.Construction;
 
 namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 {
@@ -14,7 +15,22 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Rules
 
             var outputProject = Path.Combine(migrationSettings.OutputDirectory, outputName + ".csproj");
 
+            CleanEmptyPropertyAndItemGroups(migrationRuleInputs.OutputMSBuildProject);
+
             migrationRuleInputs.OutputMSBuildProject.Save(outputProject);
         }
+
+        private void CleanEmptyPropertyAndItemGroups(ProjectRootElement msbuildProject)
+        {
+            foreach (var propertyGroup in msbuildProject.PropertyGroups)
+            {
+                propertyGroup.RemoveIfEmpty();
+            }
+
+            foreach (var itemGroup in msbuildProject.ItemGroups)
+            {
+                itemGroup.RemoveIfEmpty();
+            }
+        }
     }
 }

test/Microsoft.DotNet.ProjectJsonMigration.Tests/Rules/GivenThatIWantToMigratePublishOptions.cs

@@ -79,10 +79,6 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
                 }",
                 testDirectory: testDirectory);
 
-            Console.WriteLine(string.Join(";", mockProj.Items.Select(i => " ;; " + i.ItemType)));
-            Console.WriteLine(string.Join(";", mockProj.Items.Select(i => " ;; " + i.Include)));
-            Console.WriteLine(string.Join(";", mockProj.Items.Select(i => " ;; " + i.Exclude)));
-
             mockProj.Items.Count(i => i.ItemType.Equals("Content", StringComparison.Ordinal)).Should().Be(3);
 
             // From ProjectReader #L725 (Both are empty)

test/Microsoft.DotNet.ProjectJsonMigration.Tests/Rules/GivenThatIWantToMigrateRuntimeOptions.cs

@@ -41,7 +41,6 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
             var migratedRuntimeOptionsPath = Path.Combine(projectDir, s_runtimeConfigFileName);
 
             File.Exists(migratedRuntimeOptionsPath).Should().BeTrue();
-            Console.WriteLine(migratedRuntimeOptionsPath);
 
             var migratedRuntimeOptionsContent = JObject.Parse(File.ReadAllText(migratedRuntimeOptionsPath));
             JToken.DeepEquals(rawRuntimeOptions, migratedRuntimeOptionsContent).Should().BeTrue();

test/Microsoft.DotNet.ProjectJsonMigration.Tests/Rules/GivenThatIWantToMigrateScripts.cs

@@ -16,6 +16,8 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
     public class GivenThatIWantToMigrateScripts : TestBase
     {
         [Theory]
+        [InlineData("compile:TargetFramework", "$(TargetFramework)")]
+        [InlineData("publish:TargetFramework", "$(TargetFramework)")]
         [InlineData("compile:FullTargetFramework", "$(TargetFrameworkIdentifier),Version=$(TargetFrameworkVersion)")]
         [InlineData("compile:Configuration", "$(Configuration)")]
         [InlineData("compile:OutputFile", "$(TargetPath)")]
@@ -37,12 +39,10 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
         }
 
         [Theory]
-        [InlineData("compile:TargetFramework")]
         [InlineData("compile:ResponseFile")]
         [InlineData("compile:CompilerExitCode")]
         [InlineData("compile:RuntimeOutputDir")]
         [InlineData("compile:RuntimeIdentifier")]
-        [InlineData("publish:TargetFramework")]
         public void Formatting_script_commands_throws_when_variable_is_unsupported(string unsupportedVariable)
         {
             var scriptMigrationRule = new MigrateScriptsRule();
@@ -122,7 +122,7 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
             var scriptMigrationRule = new MigrateScriptsRule();
             ProjectRootElement mockProj = ProjectRootElement.Create();
 
-            var commands = new[] { "compile:FullTargetFramework", "compile:Configuration"};
+            var commands = new[] { "%compile:FullTargetFramework%", "%compile:Configuration%"};
 
             var target = scriptMigrationRule.MigrateScriptSet(mockProj, mockProj.AddPropertyGroup(), commands, scriptName);
             target.Tasks.Count().Should().Be(commands.Length);
@@ -136,76 +136,23 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
             }
         }
 
-        [Theory]
-        [InlineData("precompile")]
-        [InlineData("postcompile")]
-        [InlineData("prepublish")]
-        [InlineData("postpublish")]
-        public void Migrated_ScriptSet_has_two_MigratedScriptExtensionProperties_for_each_script(string scriptName)
-        {
-            var scriptMigrationRule = new MigrateScriptsRule();
-            ProjectRootElement mockProj = ProjectRootElement.Create();
-
-            var commands = new string[] {"compile:FullTargetFramework", "compile:Configuration"};
-            var propertyGroup = mockProj.AddPropertyGroup();
-            var target = scriptMigrationRule.MigrateScriptSet(mockProj, propertyGroup, commands,
-                scriptName);
-
-            Console.WriteLine(string.Join(";", propertyGroup.Properties.Select(n => n.Name)));
-            propertyGroup.Properties.Count().Should().Be(commands.Length * 2);
-
-            var count = 0;
-            foreach (var command in commands)
-            {
-                count += 1;
-                var scriptExtensionProperties =
-                    propertyGroup.Properties.Where(p => p.Name.Contains($"MigratedScriptExtension_{scriptName}_{count}")).ToArray();
-
-                scriptExtensionProperties.All(p => p.Value == ".sh" || p.Value == ".cmd").Should().BeTrue();
-                scriptExtensionProperties.Count().Should().Be(2);
-            }
-        }
-
-        [Theory]
-        [InlineData("echo", ".\\echo$(MigratedScriptExtension_1)")]
-        [InlineData("echo hello world", ".\\echo$(MigratedScriptExtension_1) hello world")]
-        [InlineData("\"echo\"", ".\\\"echo$(MigratedScriptExtension_1)\"")]
-        [InlineData("\"echo space\"", ".\\\"echo space$(MigratedScriptExtension_1)\"")]
-        [InlineData("\"echo space\" other args", ".\\\"echo space$(MigratedScriptExtension_1)\" other args")]
-        [InlineData("\"echo space\" \"other space\"", ".\\\"echo space$(MigratedScriptExtension_1)\" \"other space\"")]
-        public void Migrated_ScriptSet_has_ScriptExtension_added_to_script_command(string scriptCommandline, string expectedOutputCommand)
-        {
-            var scriptMigrationRule = new MigrateScriptsRule();
-
-            var formattedCommand = scriptMigrationRule.AddScriptExtensionPropertyToCommandLine(scriptCommandline,
-                "MigratedScriptExtension_1");
-
-            formattedCommand.Should().Be(expectedOutputCommand);
-        }
-
-        [Theory]
-        [InlineData("echo", @".\echo")]
-        [InlineData("/usr/echo", "/usr/echo")]
-        [InlineData(@"C:\usr\echo", @"C:\usr\echo")]
-        [InlineData("\"echo\"", @".\""echo")]
-        [InlineData("\"/usr/echo\"", @"""/usr/echo")]
-        [InlineData(@"""C:\usr\echo", @"""C:\usr\echo")]
-        public void Migrated_ScriptSet_has_dotSlash_prepended_when_command_is_not_rooted(string scriptCommandline,
-            string expectedOutputCommandPrefix)
-        {
-            var scriptMigrationRule = new MigrateScriptsRule();
-
-            var formattedCommand = scriptMigrationRule.FormatScriptCommand(scriptCommandline,
-                "MigratedScriptExtension_1");
-
-            formattedCommand.Should().StartWith(expectedOutputCommandPrefix);
-        }
-
         [Fact]
         public void Formatting_script_commands_replaces_unknown_variables_with_MSBuild_Property_for_environment_variable_support()
         {
             var scriptMigrationRule = new MigrateScriptsRule();
             scriptMigrationRule.ReplaceScriptVariables($"%UnknownVariable%").Should().Be("$(UnknownVariable)");
         }
+
+        [Fact]
+        public void Migrating_scripts_creates_target_with_IsCrossTargettingBuild_not_equal_true_Condition()
+        {
+            var scriptMigrationRule = new MigrateScriptsRule();
+            ProjectRootElement mockProj = ProjectRootElement.Create();
+
+            var commands = new[] { "compile:FullTargetFramework", "compile:Configuration"};
+
+            var target = scriptMigrationRule.MigrateScriptSet(mockProj, mockProj.AddPropertyGroup(), commands, "prepublish");
+            target.Condition.Should().Be(" '$(IsCrossTargetingBuild)' != 'true' ");
+        }
     }
 }

test/Microsoft.DotNet.ProjectJsonMigration.Tests/Rules/GivenThatIWantToMigrateTFMs.cs

@@ -15,8 +15,59 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
 {
     public class GivenThatIWantToMigrateProjectFramework : TestBase
     {
-        [Fact]
+        [Fact(Skip="Emitting this until x-targetting full support is in")]
-        public void Migrating_netcoreapp_project_Populates_TargetFrameworkIdentifier_and_TargetFrameworkVersion()
+        public void Migrating_netcoreapp_project_Does_not_populate_TargetFrameworkIdentifier_and_TargetFrameworkVersion()
+        {
+            var testDirectory = Temp.CreateDirectory().Path;
+            var testPJ = new ProjectJsonBuilder(TestAssetsManager)
+                .FromTestAssetBase("TestAppWithRuntimeOptions")
+                .WithCustomProperty("buildOptions", new Dictionary<string, string>
+                {
+                    { "emitEntryPoint", "false" }
+                })
+                .SaveToDisk(testDirectory);
+
+            var projectContext = ProjectContext.Create(testDirectory, FrameworkConstants.CommonFrameworks.NetCoreApp10);
+            var mockProj = ProjectRootElement.Create();
+
+            var migrationSettings = new MigrationSettings(testDirectory, testDirectory, "1.0.0", mockProj);
+            var migrationInputs = new MigrationRuleInputs(
+                new[] { projectContext }, 
+                mockProj, 
+                mockProj.AddItemGroup(),
+                mockProj.AddPropertyGroup());
+
+            new MigrateTFMRule().Apply(migrationSettings, migrationInputs);
+
+            mockProj.Properties.Count(p => p.Name == "TargetFrameworkIdentifier").Should().Be(0);
+            mockProj.Properties.Count(p => p.Name == "TargetFrameworkVersion").Should().Be(0);
+        }
+
+        public void Migrating_MultiTFM_project_Populates_TargetFrameworks_with_short_tfms()
+        {
+            var testDirectory = Temp.CreateDirectory().Path;
+            var testPJ = new ProjectJsonBuilder(TestAssetsManager)
+                .FromTestAssetBase("TestLibraryWithMultipleFrameworks")
+                .SaveToDisk(testDirectory);
+
+            var projectContext = ProjectContext.Create(testDirectory, FrameworkConstants.CommonFrameworks.NetCoreApp10);
+            var mockProj = ProjectRootElement.Create();
+
+            var migrationSettings = new MigrationSettings(testDirectory, testDirectory, "1.0.0", mockProj);
+            var migrationInputs = new MigrationRuleInputs(
+                new[] { projectContext }, 
+                mockProj, 
+                mockProj.AddItemGroup(), 
+                mockProj.AddPropertyGroup());
+
+            new MigrateTFMRule().Apply(migrationSettings, migrationInputs);
+
+            mockProj.Properties.Count(p => p.Name == "TargetFrameworks").Should().Be(1);
+            mockProj.Properties.First(p => p.Name == "TargetFrameworks")
+                .Value.Should().Be("net20;net35;net40;net461;netstandard1.5");
+        }
+
+        public void Migrating_Single_TFM_project_Populates_TargetFrameworks_with_short_tfm()
         {
             var testDirectory = Temp.CreateDirectory().Path;
             var testPJ = new ProjectJsonBuilder(TestAssetsManager)
@@ -31,14 +82,17 @@ namespace Microsoft.DotNet.ProjectJsonMigration.Tests
             var mockProj = ProjectRootElement.Create();
 
             // Run BuildOptionsRule
-            var testSettings = new MigrationSettings(testDirectory, testDirectory, "1.0.0", mockProj);
+            var migrationSettings = new MigrationSettings(testDirectory, testDirectory, "1.0.0", mockProj);
-            var testInputs = new MigrationRuleInputs(new[] { projectContext }, mockProj, mockProj.AddItemGroup(), mockProj.AddPropertyGroup());
+            var migrationInputs = new MigrationRuleInputs(
-            new MigrateTFMRule().Apply(testSettings, testInputs);
+                new[] { projectContext }, 
+                mockProj, 
+                mockProj.AddItemGroup(), 
+                mockProj.AddPropertyGroup());
 
-            mockProj.Properties.Count(p => p.Name == "TargetFrameworkIdentifier").Should().Be(1);
+            new MigrateTFMRule().Apply(migrationSettings, migrationInputs);
-            mockProj.Properties.Count(p => p.Name == "TargetFrameworkVersion").Should().Be(1);
+
-            mockProj.Properties.First(p => p.Name == "TargetFrameworkIdentifier").Value.Should().Be(".NETCoreApp");
+            mockProj.Properties.Count(p => p.Name == "TargetFrameworks").Should().Be(1);
-            mockProj.Properties.First(p => p.Name == "TargetFrameworkVersion").Value.Should().Be("v1.0");
+            mockProj.Properties.First(p => p.Name == "TargetFrameworks").Value.Should().Be("netcoreapp1.0");
         }
     }
 }

test/Microsoft.DotNet.ProjectJsonMigration.Tests/project.json

@@ -2,7 +2,7 @@
   "version": "1.0.0-*",
   "buildOptions": {
     "copyToOutput": ["MSBuild.exe", "MSBuild.exe.config"],
-    "keyFile": "../../tools/test_key.snk",
+    "keyFile": "../../tools/test_key.snk"
   },
   "dependencies": {
     "Microsoft.NETCore.App": {
@@ -21,9 +21,6 @@
     },
     "Microsoft.DotNet.Cli.Utils": {
       "target": "project"
-    },
-    "dotnet": {
-      "target":"project"
     }
   },
   

test/dotnet-migrate.Tests/GivenThatIWantToMigrateTestApps.cs

@@ -15,6 +15,7 @@ using Microsoft.DotNet.Tools.Common;
 using Microsoft.DotNet.Cli;
 using Microsoft.DotNet.Tools.Migrate;
 using Build3Command = Microsoft.DotNet.Tools.Test.Utilities.Build3Command;
+using BuildCommand = Microsoft.DotNet.Tools.Test.Utilities.BuildCommand;
 
 namespace Microsoft.DotNet.Migration.Tests
 {
@@ -70,6 +71,26 @@ namespace Microsoft.DotNet.Migration.Tests
             outputsIdentical.Should().BeTrue();
         }
 
+        [Theory]
+        // TODO: Enable this when X-Targeting is in
+        // [InlineData("TestLibraryWithMultipleFrameworks")]
+        public void It_migrates_projects_with_multiple_TFMs(string projectName)
+        {
+            var projectDirectory =
+                TestAssetsManager.CreateTestInstance(projectName, callingMethod: "i").WithLockFiles().Path;
+            var outputComparisonData = BuildProjectJsonMigrateBuildMSBuild(projectDirectory);
+
+            var outputsIdentical =
+                outputComparisonData.ProjectJsonBuildOutputs.SetEquals(outputComparisonData.MSBuildBuildOutputs);
+
+            if (!outputsIdentical)
+            {
+                OutputDiagnostics(outputComparisonData);
+            }
+
+            outputsIdentical.Should().BeTrue();
+        }
+
         [Theory]
         [InlineData("TestAppWithLibrary/TestLibrary")]
         [InlineData("TestLibraryWithAnalyzer")]
@@ -91,33 +112,6 @@ namespace Microsoft.DotNet.Migration.Tests
             outputsIdentical.Should().BeTrue();
         }
 
-        [Fact]
-        public void It_migrates_an_app_with_scripts_and_the_scripts_run()
-        {
-            var projectDirectory =
-                TestAssetsManager.CreateTestInstance("TestAppWithMigrateableScripts", callingMethod: "i").WithLockFiles().Path;
-
-            BuildProjectJson(projectDirectory);
-            var projectJsonBuildOutputs = new HashSet<string>(CollectBuildOutputs(projectDirectory));
-            CleanBinObj(projectDirectory);
-
-            MigrateProject(projectDirectory);
-            Restore(projectDirectory);
-            var msBuildStdOut = BuildMSBuild(projectDirectory);
-
-            var msbuildBuildOutputs = new HashSet<string>(CollectBuildOutputs(projectDirectory));
-
-            var outputsIdentical = projectJsonBuildOutputs.SetEquals(msbuildBuildOutputs);
-            outputsIdentical.Should().BeTrue();
-                VerifyAllMSBuildOutputsRunnable(projectDirectory);
-
-            var outputDir =
-                PathUtility.EnsureTrailingSlash(Path.Combine(projectDirectory, "bin", "Debug", "netcoreapp1.0"));
-
-            msBuildStdOut.Should().Contain($"precompile_output ?Debug? ?{outputDir}? ?.NETCoreApp,Version=v1.0?");
-            msBuildStdOut.Should().Contain($"postcompile_output ?Debug? ?{outputDir}? ?.NETCoreApp,Version=v1.0?");
-        }
-
         private MigratedBuildComparisonData GetDotnetNewComparisonData(string projectDirectory, string dotnetNewType)
         {
             DotnetNew(projectDirectory, dotnetNewType);