From 4f209e61eaff9fabf383cabc5f472ef7dcf041cd Mon Sep 17 00:00:00 2001
From: Justin Goshi <jgoshi@microsoft.com>
Date: Fri, 17 Feb 2017 12:28:25 -0800
Subject: [PATCH] Refactor properties out of test.targets

---
 build/InputDirectories.props         |  1 +
 build/OutputDirectories.props        |  6 ++++++
 build/Test.targets                   | 19 +------------------
 build/Version.props                  |  2 ++
 build/test/TestAssetProjects.targets |  4 ++--
 build/test/TestProjects.targets      | 14 +++++++-------
 6 files changed, 19 insertions(+), 27 deletions(-)

diff --git a/build/InputDirectories.props b/build/InputDirectories.props
index 9bfeadb3a..d322b67bc 100644
--- a/build/InputDirectories.props
+++ b/build/InputDirectories.props
@@ -4,5 +4,6 @@
     <DotnetCliBuildDirectory>$(RepoRoot)/build_projects/dotnet-cli-build</DotnetCliBuildDirectory>
     <MSBuildImportsDir>$(RepoRoot)/resources/MSBuildImports</MSBuildImportsDir>
     <SrcDirectory>$(RepoRoot)/src</SrcDirectory>
+    <TestDirectory>$(RepoRoot)/test/</TestDirectory>
   </PropertyGroup>
 </Project>
diff --git a/build/OutputDirectories.props b/build/OutputDirectories.props
index 9be5c2c97..fb79f8f21 100644
--- a/build/OutputDirectories.props
+++ b/build/OutputDirectories.props
@@ -18,5 +18,11 @@
       <PackagesDirectory>$(BaseOutputDirectory)/packages</PackagesDirectory>
       <SharedFrameworkPublishDirectory>$(IntermediateDirectory)/sharedFrameworkPublish</SharedFrameworkPublishDirectory>
       <SharedFrameworkNameVersionPath>$(StageDirectory)/shared/$(SharedFrameworkName)/$(SharedFrameworkVersion)</SharedFrameworkNameVersionPath>
+      <DotnetUnderTest>$(Stage2Directory)</DotnetUnderTest>
+      <TestOutputDir>$(RepoRoot)/artifacts/testpackages/</TestOutputDir>
+      <TestPackagesBuildDir>$(TestOutputDir)/packagesBuild/</TestPackagesBuildDir>
+      <TestPackagesDir>$(TestOutputDir)/packages/</TestPackagesDir>
+      <TestArtifactsDir>$(TestOutputDir)/artifacts/</TestArtifactsDir>
+      <TestResultXmlDir>$(TestOutputDir)/results/</TestResultXmlDir>
   </PropertyGroup>
 </Project>
diff --git a/build/Test.targets b/build/Test.targets
index 5baf180e7..e5a8df3c8 100644
--- a/build/Test.targets
+++ b/build/Test.targets
@@ -1,9 +1,4 @@
 <Project ToolsVersion="14.0" DefaultTargets="Test">
-  <PropertyGroup>
-    <PathSeparator>/</PathSeparator>
-    <PathSeparator Condition=" '$(OSName)' == 'win' ">\</PathSeparator>
-  </PropertyGroup>
-
   <Import Project="test/TestPackageProjects.targets" />
   <Import Project="test/TestProjects.targets" />
 
@@ -16,12 +11,11 @@
       <PathListSeparator Condition=" '$(OSName)' == 'win' ">%3b</PathListSeparator>
       <ExecPath>$(Stage2Directory)$(PathListSeparator)$(PATH)</ExecPath>
       <ExecPath Condition=" '$(OSName)' == 'win' ">$(Stage2Directory)</ExecPath>
-      <TestArtifactsEnv>$(TestArtifactsDir)</TestArtifactsEnv>
     </PropertyGroup>
 
     <ItemGroup>
       <TestTaskEnvironmentVariables Include="PATH=$(ExecPath)" />
-      <TestTaskEnvironmentVariables Include="TEST_ARTIFACTS=$(TestArtifactsEnv)" />
+      <TestTaskEnvironmentVariables Include="TEST_ARTIFACTS=$(TestArtifactsDir)" />
     </ItemGroup>
 
     <PropertyGroup>
@@ -55,17 +49,6 @@
   <Target Name="PrepareTests"
           DependsOnTargets="Init;
                             SetupTestProjectData">
-    <PropertyGroup>
-      <TestPackageBuildVersionSuffix>$(CommitCount)</TestPackageBuildVersionSuffix>
-      <TestOutputDir>$(RepoRoot)/artifacts/testpackages/</TestOutputDir>
-      <TestPackagesBuildDir>$(TestOutputDir)/packagesBuild/</TestPackagesBuildDir>
-      <TestPackagesDir>$(TestOutputDir)/packages/</TestPackagesDir>
-      <TestArtifactsDir>$(TestOutputDir)/artifacts/</TestArtifactsDir>
-      <TestResultXmlDir>$(TestOutputDir)/results/</TestResultXmlDir>
-      <TestDirectory>$(RepoRoot)/test/</TestDirectory>
-      <DotnetUnderTest>$(Stage2Directory)</DotnetUnderTest>
-    </PropertyGroup>
-
     <MakeDir Directories="$(TestPackagesDir)" Condition="!Exists('$(TestPackagesDir)')"/>
   </Target>
 
diff --git a/build/Version.props b/build/Version.props
index 11e41049f..0bdb360e0 100644
--- a/build/Version.props
+++ b/build/Version.props
@@ -12,5 +12,7 @@
     
     <SdkVersion>$(NugetVersion)</SdkVersion>
     <SdkNugetVersion>$(SdkVersion)</SdkNugetVersion>
+
+    <TestPackageBuildVersionSuffix>$(CommitCount)</TestPackageBuildVersionSuffix>
   </PropertyGroup>
 </Project>
diff --git a/build/test/TestAssetProjects.targets b/build/test/TestAssetProjects.targets
index c6d913ca3..fc73bd17a 100644
--- a/build/test/TestAssetProjects.targets
+++ b/build/test/TestAssetProjects.targets
@@ -31,10 +31,10 @@
 
   <Target Name="SetupBuildTestAssetProjectInputs">
     <ItemGroup>
-      <NoAutoBuildTestAssets Include="TestAssets$(PathSeparator)TestProjects$(PathSeparator)**$(PathSeparator).noautobuild" />
+      <NoAutoBuildTestAssets Include="TestAssets/TestProjects/**/.noautobuild" />
       <NoAutoBuildTestAssetProjects Include="%(NoAutoBuildTestAssets.RelativeDir)project.json" />
 
-      <PreTestAssetProjects Include="TestAssets$(PathSeparator)TestProjects$(PathSeparator)**$(PathSeparator)project.json"/>
+      <PreTestAssetProjects Include="TestAssets/TestProjects/**/project.json"/>
                          
       <TestAssetProjects Include="%(PreTestAssetProjects.RelativeDir)project.json"
                          Exclude="@(NoAutoBuildTestAssetProjects)"/>
diff --git a/build/test/TestProjects.targets b/build/test/TestProjects.targets
index 846c7be22..20045c923 100644
--- a/build/test/TestProjects.targets
+++ b/build/test/TestProjects.targets
@@ -10,20 +10,20 @@
       <PreTestProjectsToExclude Condition=" '$(IsDesktopAvailable)' != 'True' " 
                              Include="test/binding-redirects.Tests/binding-redirects.Tests.csproj;" />
       -->
-      <PreTestProjectsToExclude Include="test$(PathSeparator)binding-redirects.Tests$(PathSeparator)binding-redirects.Tests.csproj;" />
+      <PreTestProjectsToExclude Include="test/binding-redirects.Tests/binding-redirects.Tests.csproj;" />
 
       <PreTestProjectsToExclude Condition=" 'Non-test projects in test directory' != 'consider moving elsewhere' "
-                             Include="test$(PathSeparator)ArgumentsReflector$(PathSeparator)ArgumentsReflector.csproj;
-                                      test$(PathSeparator)Microsoft.DotNet.Tools.Tests.Utilities$(PathSeparator)Microsoft.DotNet.Tools.Tests.Utilities.csproj;
-                                      test$(PathSeparator)Msbuild.Tests.Utilities$(PathSeparator)Msbuild.Tests.Utilities.csproj;
-                                      test$(PathSeparator)Performance$(PathSeparator)Performance.csproj" />
+                             Include="test/ArgumentsReflector/ArgumentsReflector.csproj;
+                                      test/Microsoft.DotNet.Tools.Tests.Utilities/Microsoft.DotNet.Tools.Tests.Utilities.csproj;
+                                      test/Msbuild.Tests.Utilities/Msbuild.Tests.Utilities.csproj;
+                                      test/Performance/Performance.csproj" />
 
       <PreTestProjectsToExclude Condition=" 'Executed after primary test phase.' != ' Consider moving.' "
-                             Include="test$(PathSeparator)Installer$(PathSeparator)Microsoft.DotNet.Cli.Msi.Tests$(PathSeparator)Microsoft.DotNet.Cli.Msi.Tests.csproj;" />
+                             Include="test/Installer/Microsoft.DotNet.Cli.Msi.Tests/Microsoft.DotNet.Cli.Msi.Tests.csproj;" />
 
       <TestProjectsToExclude Include="%(PreTestProjectsToExclude.Fullpath)" />
 
-      <PreTestProjects Include="test$(PathSeparator)*$(PathSeparator)*.csproj;" />
+      <PreTestProjects Include="test/*/*.csproj;" />
 
       <TestProjects Include="%(PreTestProjects.Fullpath)"
                     Exclude="@(TestProjectsToExclude)" />