diff options
author | Doug Bunting <6431421+dougbu@users.noreply.github.com> | 2022-04-29 04:20:15 +0300 |
---|---|---|
committer | Doug Bunting <6431421+dougbu@users.noreply.github.com> | 2022-04-29 04:22:07 +0300 |
commit | 5b14ca685958b71f6fb4c10b1744d6bebb893e9f (patch) | |
tree | 44f898c0053f092d04774286e38e566b98969af3 | |
parent | 9961ccbbd38971cdc0799eb964ac256cc40a79e3 (diff) |
Rename `RunTests` project to `HelixTestRunner`taparik/fixHelixRestoreIssue
- less confusing given `RunTests` target and runtests.sh et cetera
-rw-r--r-- | AspNetCore.sln | 2 | ||||
-rw-r--r-- | docs/UpdatingMajorVersionAndTFM.md | 4 | ||||
-rw-r--r-- | eng/Build.props | 2 | ||||
-rw-r--r-- | eng/helix/content/runtests.cmd | 4 | ||||
-rw-r--r-- | eng/helix/content/runtests.sh | 4 | ||||
-rw-r--r-- | eng/helix/helix.proj | 8 | ||||
-rw-r--r-- | eng/targets/Helix.targets | 6 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/HelixTestRunner.csproj (renamed from eng/tools/RunTests/RunTests.csproj) | 0 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/HelixTestRunnerOptions.cs (renamed from eng/tools/RunTests/RunTestsOptions.cs) | 8 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/ProcessResult.cs (renamed from eng/tools/RunTests/ProcessResult.cs) | 2 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/ProcessUtil.cs (renamed from eng/tools/RunTests/ProcessUtil.cs) | 2 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/Program.cs (renamed from eng/tools/RunTests/Program.cs) | 6 | ||||
-rw-r--r-- | eng/tools/HelixTestRunner/TestRunner.cs (renamed from eng/tools/RunTests/TestRunner.cs) | 8 | ||||
-rw-r--r-- | eng/tools/tools.slnf | 4 |
14 files changed, 30 insertions, 30 deletions
diff --git a/AspNetCore.sln b/AspNetCore.sln index 570c59e03a..8f1e7bf485 100644 --- a/AspNetCore.sln +++ b/AspNetCore.sln @@ -1704,7 +1704,7 @@ Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Microsoft.AspNetCore.RateLi EndProject Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Microsoft.AspNetCore.RateLimiting.Tests", "src\Middleware\RateLimiting\test\Microsoft.AspNetCore.RateLimiting.Tests.csproj", "{41FF4F96-98D2-4482-A2A7-4B179E80D285}" EndProject -Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "RunTests", "eng\tools\RunTests\RunTests.csproj", "{51D07AA9-6297-4F66-A7BD-71CE7E3F4A3F}" +Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "HelixTestRunner", "eng\tools\HelixTestRunner\HelixTestRunner.csproj", "{51D07AA9-6297-4F66-A7BD-71CE7E3F4A3F}" EndProject Global GlobalSection(SolutionConfigurationPlatforms) = preSolution diff --git a/docs/UpdatingMajorVersionAndTFM.md b/docs/UpdatingMajorVersionAndTFM.md index 9be461a58d..ec0f304d65 100644 --- a/docs/UpdatingMajorVersionAndTFM.md +++ b/docs/UpdatingMajorVersionAndTFM.md @@ -33,7 +33,7 @@ Once dotnet/runtime has updated their TFM, we update ours in the dependency upda * In [eng/Versions.props](/eng/Versions.props), increment `DefaultNetCoreTargetFramework` by 1. * Do a global repo search for the current version string, and update almost everything by 1 (e.g. find `net7`, replace with `net8`). See the PR linked above for examples - this shouldn't be done blindly, but on a case-by-case basis. Most things should be updated, and most choices should be obvious. - * Exceptions to this are [eng/tools/RepoTasks/RepoTasks.csproj](/eng/tools/RepoTasks/RepoTasks.csproj), [eng/tools/RepoTasks/RepoTasks.tasks](/eng/tools/RepoTasks/RepoTasks.tasks), and [eng/tools/RunTests/RunTests.csproj](/eng/tools/RunTests/RunTests.csproj). These build without the workarounds from [eng/tools/GenerateFiles/Directory.Build.targets.in](/eng/tools/GenerateFiles/Directory.Build.targets.in), and need to be kept at the previous TFM until we get an SDK containing a runtime with the new TFM. Generally this means we have to hard-code the previous TFM for these files, rather than using `DefaultNetCoreTargetFramework`. + * Exceptions to this are [eng/tools/RepoTasks/RepoTasks.csproj](/eng/tools/RepoTasks/RepoTasks.csproj), [eng/tools/RepoTasks/RepoTasks.tasks](/eng/tools/RepoTasks/RepoTasks.tasks), and [eng/tools/HelixTestRunner/HelixTestRunner.csproj](/eng/tools/HelixTestRunner/HelixTestRunner.csproj). These build without the workarounds from [eng/tools/GenerateFiles/Directory.Build.targets.in](/eng/tools/GenerateFiles/Directory.Build.targets.in), and need to be kept at the previous TFM until we get an SDK containing a runtime with the new TFM. Generally this means we have to hard-code the previous TFM for these files, rather than using `DefaultNetCoreTargetFramework`. * Add a reference to the new `SiteExtensions` package for the previous Major Version. 1. Add references to [src/SiteExtensions/LoggingAggregate/src/Microsoft.AspNetCore.AzureAppServices.SiteExtension/Microsoft.AspNetCore.AzureAppServices.SiteExtension.csproj](/src/SiteExtensions/LoggingAggregate/src/Microsoft.AspNetCore.AzureAppServices.SiteExtension/Microsoft.AspNetCore.AzureAppServices.SiteExtension.csproj) to `Microsoft.AspNetCore.AzureAppServices.SiteExtension.{PreviousMajorVersion}.0.x64` and `Microsoft.AspNetCore.AzureAppServices.SiteExtension.{PreviousMajorVersion}.0.x86`. 2. Add entries in [eng/Versions.props](/eng/Versions.props) similar to [these](https://github.com/dotnet/aspnetcore/blob/216c92b78bce31d5e81a70b589707ec2ae5ab21a/eng/Versions.props#L224-L226) - the version should be from the latest released build of .Net. @@ -56,4 +56,4 @@ Once dotnet/runtime has updated their TFM, we update ours in the dependency upda ## Ingesting an SDK with the new TFM -Typically we update the SDK we use in `main` every Monday. Once we have one that contains `Microsoft.Netcore.App` entries with the new TFM, we can update [eng/tools/RepoTasks/RepoTasks.csproj](/eng/tools/RepoTasks/RepoTasks.csproj), [eng/tools/RepoTasks/RepoTasks.tasks](/eng/tools/RepoTasks/RepoTasks.tasks), and [eng/tools/RunTests/RunTests.csproj](/eng/helix/tools/RunTests.csproj) to use `DefaultNetCoreTargetFramework` again rather than hard-coding the previous TFM. +Typically we update the SDK we use in `main` every Monday. Once we have one that contains `Microsoft.Netcore.App` entries with the new TFM, we can update [eng/tools/RepoTasks/RepoTasks.csproj](/eng/tools/RepoTasks/RepoTasks.csproj), [eng/tools/RepoTasks/RepoTasks.tasks](/eng/tools/RepoTasks/RepoTasks.tasks), and [eng/tools/HelixTestRunner/HelixTestRunner.csproj](/eng/helix/tools/HelixTestRunner.csproj) to use `DefaultNetCoreTargetFramework` again rather than hard-coding the previous TFM. diff --git a/eng/Build.props b/eng/Build.props index c2566a81e5..76464fbdd8 100644 --- a/eng/Build.props +++ b/eng/Build.props @@ -199,7 +199,7 @@ $(RepoRoot)src\Extensions\**\*.csproj; $(RepoRoot)src\BuildAfterTargetingPack\*.csproj; $(RepoRoot)src\OpenApi\**\*.csproj; - $(RepoRoot)eng\tools\RunTests\RunTests.csproj; + $(RepoRoot)eng\tools\HelixTestRunner\HelixTestRunner.csproj; " Exclude=" @(ProjectToBuild); diff --git a/eng/helix/content/runtests.cmd b/eng/helix/content/runtests.cmd index ea9383a5d5..379597f02f 100644 --- a/eng/helix/content/runtests.cmd +++ b/eng/helix/content/runtests.cmd @@ -21,8 +21,8 @@ echo. set exit_code=0 -echo "Running tests: $HELIX_CORRELATION_PAYLOAD/RunTests/RunTests.exe --target %$target% --runtime %$aspRuntimeVersion% --queue %$queue% --arch %$arch% --quarantined %$quarantined% --helixTimeout %$helixTimeout% --playwright $installPlaywright" -$HELIX_CORRELATION_PAYLOAD/RunTests/RunTests.exe --target %$target% --runtime %$aspRuntimeVersion% --queue %$queue% --arch %$arch% --quarantined %$quarantined% --helixTimeout %$helixTimeout% --playwright $installPlaywright +echo "Running tests: $HELIX_CORRELATION_PAYLOAD/HelixTestRunner/HelixTestRunner.exe --target %$target% --runtime %$aspRuntimeVersion% --queue %$queue% --arch %$arch% --quarantined %$quarantined% --helixTimeout %$helixTimeout% --playwright $installPlaywright" +$HELIX_CORRELATION_PAYLOAD/HelixTestRunner/HelixTestRunner.exe --target %$target% --runtime %$aspRuntimeVersion% --queue %$queue% --arch %$arch% --quarantined %$quarantined% --helixTimeout %$helixTimeout% --playwright $installPlaywright if not errorlevel 0 ( set exit_code=%errorlevel% ) diff --git a/eng/helix/content/runtests.sh b/eng/helix/content/runtests.sh index 629336006d..d8ca1ecd78 100644 --- a/eng/helix/content/runtests.sh +++ b/eng/helix/content/runtests.sh @@ -75,8 +75,8 @@ sync exit_code=0 -echo "Running tests: dotnet run $HELIX_CORRELATION_PAYLOAD/RunTests/RunTests.dll -- --target $1 --runtime $2 --queue $helixQueue --arch $4 --quarantined $5 --helixTimeout $6 --playwright $installPlaywright" -dotnet run $HELIX_CORRELATION_PAYLOAD/RunTests/RunTests.dll -- --target $1 --runtime $2 --queue $helixQueue --arch $4 --quarantined $5 --helixTimeout $6 --playwright $installPlaywright +echo "Running tests: dotnet run $HELIX_CORRELATION_PAYLOAD/HelixTestRunner/HelixTestRunner.dll -- --target $1 --runtime $2 --queue $helixQueue --arch $4 --quarantined $5 --helixTimeout $6 --playwright $installPlaywright" +dotnet run $HELIX_CORRELATION_PAYLOAD/HelixTestRunner/HelixTestRunner.dll -- --target $1 --runtime $2 --queue $helixQueue --arch $4 --quarantined $5 --helixTimeout $6 --playwright $installPlaywright exit_code=$? echo "Finished tests...exit_code=$exit_code" diff --git a/eng/helix/helix.proj b/eng/helix/helix.proj index ab79917aef..9c1625bcdc 100644 --- a/eng/helix/helix.proj +++ b/eng/helix/helix.proj @@ -48,14 +48,14 @@ <PackageType>runtime</PackageType> </AdditionalDotNetPackage> - <!-- Grab the tool packages for what RunTests installs. --> + <!-- Grab the tool packages for what HelixTestRunner installs. --> <HelixCorrelationPayload Include="$(NUGET_PACKAGES)\dotnet-dump\$(DotnetDumpVersion)\dotnet-dump.$(DotnetDumpVersion).nupkg" /> <HelixCorrelationPayload Include="$(NUGET_PACKAGES)\dotnet-ef\$(dotnetefVersion)\dotnet-ef.$(dotnetefVersion).nupkg" /> <HelixCorrelationPayload Include="$(NUGET_PACKAGES)\dotnet-serve\$(DotnetServeVersion)\dotnet-serve.$(DotnetServeVersion).nupkg" /> - <!-- Grab published `RunTests` project output. --> - <HelixCorrelationPayload Include="$(ArtifactsBinDir)RunTests\Debug\net7.0\publish\" - Destination="$([MSBuild]::EnsureTrailingSlash('RunTests')" /> + <!-- Grab published `HelixTestRunner` project output. --> + <HelixCorrelationPayload Include="$(ArtifactsBinDir)HelixTestRunner\Debug\net7.0\publish\" + Destination="$([MSBuild]::EnsureTrailingSlash('HelixTestRunner')" /> </ItemGroup> <Choose> diff --git a/eng/targets/Helix.targets b/eng/targets/Helix.targets index 582eead9d6..c290255176 100644 --- a/eng/targets/Helix.targets +++ b/eng/targets/Helix.targets @@ -188,9 +188,9 @@ SkipUnchangedFiles="$(SkipCopyUnchangedFiles)" /> <!-- - Use the shared framework and ref/ assemblies copied to the Helix agents and expanded by RunTests. Unfortunately, - tests of Microsoft.NET.Sdk.Web templates will not reference the just-built Microsoft.NET.Sdk.Razor. Adding a - package reference for that would confuse tests of Microsoft.NET.Sdk templates. + Use the shared framework and ref/ assemblies copied to the Helix agents and expanded by HelixTestRunner. + Unfortunately, tests of Microsoft.NET.Sdk.Web templates will not reference the just-built + Microsoft.NET.Sdk.Razor. Adding a package reference for that would confuse tests of Microsoft.NET.Sdk templates. --> <Copy SourceFiles="$(ArtifactsBinDir)GenerateFiles\Directory.Build.props" DestinationFiles="$(PublishDir)Directory.Build.props" diff --git a/eng/tools/RunTests/RunTests.csproj b/eng/tools/HelixTestRunner/HelixTestRunner.csproj index 52fd7178f0..52fd7178f0 100644 --- a/eng/tools/RunTests/RunTests.csproj +++ b/eng/tools/HelixTestRunner/HelixTestRunner.csproj diff --git a/eng/tools/RunTests/RunTestsOptions.cs b/eng/tools/HelixTestRunner/HelixTestRunnerOptions.cs index b653926c66..13d483d64c 100644 --- a/eng/tools/RunTests/RunTestsOptions.cs +++ b/eng/tools/HelixTestRunner/HelixTestRunnerOptions.cs @@ -9,11 +9,11 @@ using System.IO; using System.Runtime.InteropServices; using System.Threading.Tasks; -namespace RunTests; +namespace HelixTestRunner; -public class RunTestsOptions +public class HelixTestRunnerOptions { - public static RunTestsOptions Parse(string[] args) + public static HelixTestRunnerOptions Parse(string[] args) { var command = new RootCommand() { @@ -60,7 +60,7 @@ public class RunTestsOptions var parseResult = command.Parse(args); var sharedFxVersion = parseResult.ValueForOption<string>("--runtime"); - var options = new RunTestsOptions + var options = new HelixTestRunnerOptions { Architecture = parseResult.ValueForOption<string>("--arch"), HelixQueue = parseResult.ValueForOption<string>("--queue"), diff --git a/eng/tools/RunTests/ProcessResult.cs b/eng/tools/HelixTestRunner/ProcessResult.cs index 7f2fc0a297..29a8b3bc1e 100644 --- a/eng/tools/RunTests/ProcessResult.cs +++ b/eng/tools/HelixTestRunner/ProcessResult.cs @@ -1,7 +1,7 @@ // Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. -namespace RunTests; +namespace HelixTestRunner; public class ProcessResult { diff --git a/eng/tools/RunTests/ProcessUtil.cs b/eng/tools/HelixTestRunner/ProcessUtil.cs index 40ea7a9df1..1dc275987c 100644 --- a/eng/tools/RunTests/ProcessUtil.cs +++ b/eng/tools/HelixTestRunner/ProcessUtil.cs @@ -12,7 +12,7 @@ using System.Threading.Tasks; #nullable enable -namespace RunTests; +namespace HelixTestRunner; public static class ProcessUtil { diff --git a/eng/tools/RunTests/Program.cs b/eng/tools/HelixTestRunner/Program.cs index 41463de308..f388f76851 100644 --- a/eng/tools/RunTests/Program.cs +++ b/eng/tools/HelixTestRunner/Program.cs @@ -4,7 +4,7 @@ using System; using System.Threading.Tasks; -namespace RunTests; +namespace HelixTestRunner; class Program { @@ -12,7 +12,7 @@ class Program { try { - var runner = new TestRunner(RunTestsOptions.Parse(args)); + var runner = new TestRunner(HelixTestRunnerOptions.Parse(args)); var keepGoing = runner.SetupEnvironment(); if (keepGoing) @@ -54,7 +54,7 @@ class Program } catch (Exception e) { - Console.WriteLine($"RunTests uncaught exception: {e.ToString()}"); + Console.WriteLine($"HelixTestRunner uncaught exception: {e.ToString()}"); Environment.Exit(1); } } diff --git a/eng/tools/RunTests/TestRunner.cs b/eng/tools/HelixTestRunner/TestRunner.cs index f21f3036e4..17cbf5b0eb 100644 --- a/eng/tools/RunTests/TestRunner.cs +++ b/eng/tools/HelixTestRunner/TestRunner.cs @@ -1,17 +1,17 @@ // Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. -namespace RunTests; +namespace HelixTestRunner; public class TestRunner { - public TestRunner(RunTestsOptions options) + public TestRunner(HelixTestRunnerOptions options) { Options = options; EnvironmentVariables = new Dictionary<string, string>(); } - public RunTestsOptions Options { get; set; } + public HelixTestRunnerOptions Options { get; set; } public Dictionary<string, string> EnvironmentVariables { get; set; } public bool SetupEnvironment() @@ -262,7 +262,7 @@ public class TestRunner } catch (Exception e) { - Console.WriteLine($"Exception in RunTests: {e}"); + Console.WriteLine($"Exception in HelixTestRunner: {e}"); exitCode = 1; } return exitCode; diff --git a/eng/tools/tools.slnf b/eng/tools/tools.slnf index d5cd80951d..bdd4896e94 100644 --- a/eng/tools/tools.slnf +++ b/eng/tools/tools.slnf @@ -4,7 +4,7 @@ "projects": [ "eng\\tools\\BaselineGenerator\\BaselineGenerator.csproj", "eng\\tools\\RepoTasks\\RepoTasks.csproj", - "eng\\tools\\RunTests\\RunTests.csproj" + "eng\\tools\\HelixTestRunner\\HelixTestRunner.csproj" ] } -}
\ No newline at end of file +} |