diff --git a/eng/helix/content/InstallAppRuntime.ps1 b/eng/helix/content/InstallAppRuntime.ps1 index 5307a09e60fb..9d9aaffb5c80 100644 --- a/eng/helix/content/InstallAppRuntime.ps1 +++ b/eng/helix/content/InstallAppRuntime.ps1 @@ -30,6 +30,8 @@ $ProgressPreference = 'SilentlyContinue' # Workaround PowerShell/PowerShell#2138 Set-StrictMode -Version 1 +Write-Host "Extracting to $InstallDir" + $zipPackage = [io.path]::ChangeExtension($AppRuntimePath, ".zip") Write-Host "Renaming to $zipPackage" Rename-Item -Path $AppRuntimePath -NewName $zipPackage @@ -44,9 +46,8 @@ else { [System.IO.Compression.ZipFile]::ExtractToDirectory($zipPackage, ".\tmpRuntime") } -New-Item -ItemType Directory -Force -Path $InstallDir -Write-Host "Copying *.txt to $InstallDir" -Copy-Item -Path ".\tmpRuntime\*.txt" $InstallDir +Get-ChildItem -Path ".\tmpRuntime" -Recurse + Write-Host "Copying managed files to $InstallDir" Copy-Item -Path ".\tmpRuntime\runtimes\$RuntimeIdentifier\lib\$Framework\*" $InstallDir Write-Host "Copying native files to $InstallDir" diff --git a/eng/helix/content/InstallAspNetRef.ps1 b/eng/helix/content/InstallAspNetRef.ps1 deleted file mode 100644 index d5a4d2a9c66f..000000000000 --- a/eng/helix/content/InstallAspNetRef.ps1 +++ /dev/null @@ -1,40 +0,0 @@ - <# - .SYNOPSIS - Unzips an AspNetCore.App.Ref nupkg - .DESCRIPTION - This script unzips an AspNetCore.App.Ref nupkg - .PARAMETER RefPath - The path to the AspNetCore.App.Ref package to install. - .PARAMETER InstallDir - The directory to install to. - #> -param( - [Parameter(Mandatory = $true)] - $RefPath, - - [Parameter(Mandatory = $true)] - $InstallDir -) - -$ErrorActionPreference = 'Stop' -$ProgressPreference = 'SilentlyContinue' # Workaround PowerShell/PowerShell#2138 - -Set-StrictMode -Version 1 - -Write-Host "Extracting to $InstallDir" - -$zipPackage = [io.path]::ChangeExtension($RefPath, ".zip") -Write-Host "Renaming to $zipPackage" -Rename-Item -Path $RefPath -NewName $zipPackage -if (Get-Command -Name 'Microsoft.PowerShell.Archive\Expand-Archive' -ErrorAction Ignore) { - # Use built-in commands where possible as they are cross-plat compatible - Microsoft.PowerShell.Archive\Expand-Archive -Path $zipPackage -DestinationPath "$InstallDir" -Force -} -else { - Remove-Item "$InstallDir" -Recurse -ErrorAction Ignore - # Fallback to old approach for old installations of PowerShell - Add-Type -AssemblyName System.IO.Compression.FileSystem - [System.IO.Compression.ZipFile]::ExtractToDirectory($zipPackage, "$InstallDir") -} - -Get-ChildItem -Path "$InstallDir" -Recurse diff --git a/eng/helix/content/RunTests/TestRunner.cs b/eng/helix/content/RunTests/TestRunner.cs index 2a9fadfb0105..850d0b629db1 100644 --- a/eng/helix/content/RunTests/TestRunner.cs +++ b/eng/helix/content/RunTests/TestRunner.cs @@ -65,17 +65,17 @@ public bool SetupEnvironment() } } - public void DisplayContents(string path = "./") + public void DisplayContents() { try { Console.WriteLine(); - Console.WriteLine($"Displaying directory contents for {path}:"); - foreach (var file in Directory.EnumerateFiles(path)) + Console.WriteLine("Displaying directory contents:"); + foreach (var file in Directory.EnumerateFiles("./")) { Console.WriteLine(Path.GetFileName(file)); } - foreach (var file in Directory.EnumerateDirectories(path)) + foreach (var file in Directory.EnumerateDirectories("./")) { Console.WriteLine(Path.GetFileName(file)); } @@ -83,7 +83,7 @@ public void DisplayContents(string path = "./") } catch (Exception e) { - Console.WriteLine($"Exception in DisplayContents: {e.ToString()}"); + Console.WriteLine($"Exception in DisplayInitialState: {e.ToString()}"); } } @@ -95,17 +95,11 @@ public async Task InstallAspNetAppIfNeededAsync() if (Directory.Exists("Microsoft.AspNetCore.App")) { var appRuntimePath = $"{Options.DotnetRoot}/shared/Microsoft.AspNetCore.App/{Options.RuntimeVersion}"; - Console.WriteLine($"Creating directory: {appRuntimePath}"); - Directory.CreateDirectory(appRuntimePath); Console.WriteLine($"Found Microsoft.AspNetCore.App/, copying to {appRuntimePath}"); - Console.WriteLine($"Set ASPNET_RUNTIME_PATH: {appRuntimePath}"); - EnvironmentVariables.Add("ASPNET_RUNTIME_PATH", appRuntimePath); foreach (var file in Directory.EnumerateFiles("Microsoft.AspNetCore.App", "*.*", SearchOption.AllDirectories)) { - File.Copy(file, Path.Combine(appRuntimePath, Path.GetFileName(file)), overwrite: true); + File.Copy(file, Path.Combine(appRuntimePath, file), overwrite: true); } - - DisplayContents(appRuntimePath); Console.WriteLine($"Adding current directory to nuget sources: {Options.HELIX_WORKITEM_ROOT}"); diff --git a/eng/helix/content/installappruntime.sh b/eng/helix/content/installappruntime.sh index 09a82953a97e..45cb1554fab3 100755 --- a/eng/helix/content/installappruntime.sh +++ b/eng/helix/content/installappruntime.sh @@ -16,6 +16,5 @@ mkdir -p $tmpDir unzip sharedFx.zip -d $tmpDir mkdir -p $output_dir echo "Copying to $output_dir" -cp $tmpDir/*.txt $output_dir cp $tmpDir/runtimes/$rid/lib/$framework/* $output_dir cp $tmpDir/runtimes/$rid/native/* $output_dir diff --git a/eng/helix/content/installaspnetref.sh b/eng/helix/content/installaspnetref.sh deleted file mode 100644 index 6ab719a09ffa..000000000000 --- a/eng/helix/content/installaspnetref.sh +++ /dev/null @@ -1,15 +0,0 @@ -#!/usr/bin/env bash - -# Cause the script to fail if any subcommand fails -set -e - -refPath=$1 -output_dir=$2 -tmpDir=./tmpRuntime - -echo "Installing ref package from $refPath" -cp $refPath sharedFx.zip - -mkdir -p $output_dir -echo "Unzip to $output_dir" -unzip sharedFx.zip -d $output_dir diff --git a/eng/targets/Helix.targets b/eng/targets/Helix.targets index 93b8feb54f4d..154ce92ee6f2 100644 --- a/eng/targets/Helix.targets +++ b/eng/targets/Helix.targets @@ -16,16 +16,6 @@ - - - - - - - - - - diff --git a/src/Framework/test/Microsoft.AspNetCore.App.UnitTests.csproj b/src/Framework/test/Microsoft.AspNetCore.App.UnitTests.csproj index 3038b73192ba..8d9b26e7d086 100644 --- a/src/Framework/test/Microsoft.AspNetCore.App.UnitTests.csproj +++ b/src/Framework/test/Microsoft.AspNetCore.App.UnitTests.csproj @@ -3,9 +3,9 @@ $(DefaultNetCoreTargetFramework) Microsoft.AspNetCore + + false true - true - true diff --git a/src/Framework/test/SharedFxTests.cs b/src/Framework/test/SharedFxTests.cs index 2b7fae9f2957..558fc344398b 100644 --- a/src/Framework/test/SharedFxTests.cs +++ b/src/Framework/test/SharedFxTests.cs @@ -23,9 +23,7 @@ public SharedFxTests(ITestOutputHelper output) _output = output; _expectedTfm = "netcoreapp" + TestData.GetSharedFxVersion().Substring(0, 3); _expectedRid = TestData.GetSharedFxRuntimeIdentifier(); - _sharedFxRoot = string.IsNullOrEmpty(Environment.GetEnvironmentVariable("ASPNET_RUNTIME_PATH")) - ? Path.Combine(TestData.GetTestDataValue("SharedFrameworkLayoutRoot"), "shared", TestData.GetTestDataValue("RuntimePackageVersion")) - : Environment.GetEnvironmentVariable("ASPNET_RUNTIME_PATH"); + _sharedFxRoot = Path.Combine(TestData.GetTestDataValue("SharedFrameworkLayoutRoot"), "shared", "Microsoft.AspNetCore.App", TestData.GetTestDataValue("RuntimePackageVersion")); } [Fact] diff --git a/src/Framework/test/TargetingPackTests.cs b/src/Framework/test/TargetingPackTests.cs index e296884e4551..54da276513c6 100644 --- a/src/Framework/test/TargetingPackTests.cs +++ b/src/Framework/test/TargetingPackTests.cs @@ -26,9 +26,7 @@ public TargetingPackTests(ITestOutputHelper output) { _output = output; _expectedRid = TestData.GetSharedFxRuntimeIdentifier(); - _targetingPackRoot = string.IsNullOrEmpty(Environment.GetEnvironmentVariable("helix")) - ? Path.Combine(TestData.GetTestDataValue("TargetingPackLayoutRoot"), "packs", "Microsoft.AspNetCore.App.Ref", TestData.GetTestDataValue("TargetingPackVersion")) - : Path.Combine(Environment.GetEnvironmentVariable("HELIX_WORKITEM_ROOT"), "Microsoft.AspNetCore.App.Ref"); + _targetingPackRoot = Path.Combine(TestData.GetTestDataValue("TargetingPackLayoutRoot"), "packs", "Microsoft.AspNetCore.App.Ref", TestData.GetTestDataValue("TargetingPackVersion")); _isTargetingPackBuilding = bool.Parse(TestData.GetTestDataValue("IsTargetingPackBuilding")); }