CARE-265: Adding .NET Framework build features

.github/actions/build-dotnet/Build-DotNetSolutionOrProject.ps1

@@ -74,11 +74,7 @@ if ($noErrors -and !$?)
     exit 1
 }
 
-# With node reuse, dotnet build spawns processes that while speed up build, they can cause dotnet test and other dotnet
-# tools to randomly hang. So, here we shut down those processes for later actions.
-# For details see: https://github.com/Lombiq/UI-Testing-Toolbox/issues/228.
-Write-Output "Shutting down .NET build servers."
-dotnet build-server shutdown
+Close-DotNetBuildServers
 
 if ($expectedErrorCodes)
 {

.github/actions/build-dotnet/action.yml

@@ -68,6 +68,7 @@ runs:
       shell: pwsh
       run: |
         "${{ github.action_path }}" >> $Env:GITHUB_PATH
+        (Resolve-Path "${{ github.action_path }}/../../../Scripts").Path >> $Env:GITHUB_PATH
 
     - name: Setup & Detect Cache Configuration
       id: detect-cache-configuration
@@ -125,7 +126,7 @@ runs:
         '@
 
         $switches = @{
-            Solution = (Get-ChildItem ${{ inputs.solution-or-project-path }}).FullName
+            SolutionOrProject = (Get-ChildItem ${{ inputs.solution-or-project-path }}).FullName
             Verbosity = "${{ inputs.verbosity }}"
             EnableCodeAnalysis = "${{ inputs.enable-code-analysis }}"
             Version = "${{ inputs.publish-version }}"

.github/actions/checkout/action.yml

@@ -0,0 +1,20 @@
+name: Checkout
+description: >
+  Runs actions/checkout. Only to centralize which version we use from the action. Intentionally not documented in
+  Actions.md since it's meant to be only for internal use.
+
+inputs:
+  token:
+    required: false
+    default: ${{ github.token }}
+    description: >
+      The GitHub token to authenticate checkout. Pass in a GitHub personal access token if authenticated submodules are
+      used.
+
+runs:
+  using: "composite"
+  steps:
+    - uses: actions/[email protected]
+      with:
+        submodules: 'recursive'
+        token: ${{ inputs.token }}

.github/actions/msbuild/Build-DotNetFrameworkSolutionOrProject.ps1

@@ -0,0 +1,77 @@
+param (
+    [string] $SolutionOrProject,
+    [string] $Verbosity,
+    [string] $EnableCodeAnalysis,
+    [string] $Version,
+    [string] $Switches)
+
+function ConvertTo-Array([string] $rawInput)
+{
+    $rawInput.Replace("`r", "").Split("`n") | ForEach-Object { $_.Trim() } | Where-Object { $_ }
+}
+
+Write-Output ".NET version number: $Version"
+
+if (Test-Path src/Utilities/Lombiq.Gulp.Extensions/Lombiq.Gulp.Extensions.csproj)
+{
+    Write-Output "::group::Gulp Extensions found. It needs to be explicitly built before the solution."
+
+    # These need to be different than that of msbuild.
+    $gulpBuildSwitches = ConvertTo-Array @"
+    --configuration:Release
+    --nologo
+    --verbosity:$Verbosity
+    --warnaserror
+    --warnAsMessage:MSB3026
+    --consoleLoggerParameters:NoSummary
+    -p:TreatWarningsAsErrors=true
+    -p:RunAnalyzersDuringBuild=$EnableCodeAnalysis
+    -p:Retries=4
+    -p:RetryDelayMilliseconds=1000
+    -p:Version=$Version
+"@
+
+    $startTime = [DateTime]::Now
+    dotnet build src/Utilities/Lombiq.Gulp.Extensions/Lombiq.Gulp.Extensions.csproj @gulpBuildSwitches
+    $endTime = [DateTime]::Now
+
+    Write-Output ("Gulp Extensions build took {0:0.###} seconds." -f ($endTime - $startTime).TotalSeconds)
+    Write-Output "::endgroup::"
+}
+
+# - -p:Retries and -p:RetryDelayMilliseconds are to retry builds if it fails the first time due to random locks.
+
+$buildSwitches = ConvertTo-Array @"
+    -p:Configuration=Release
+    -restore
+    --verbosity:$Verbosity
+    --warnaserror
+    -p:TreatWarningsAsErrors=true
+    -p:RunAnalyzersDuringBuild=$EnableCodeAnalysis
+    -p:Retries=4
+    -p:RetryDelayMilliseconds=1000
+    -p:Version=$Version
+    $Switches
+"@
+
+Write-Output "Building solution or project with ``msbuild $SolutionOrProject $($buildSwitches -join " ")``."
+
+msbuild $SolutionOrProject @buildSwitches
+
+# Without this, if the msbuild command fails with certain MSB error codes (not build errors), then still the wouldn't
+# cause this script to fail.
+
+# error MSB3644: The reference assemblies for .NETFramework,Version=v4.6.1 were not found. To resolve this, install the
+# Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET
+# Framework Developer Packs at https://aka.ms/msbuild/developerpacks 
+
+if ($?)
+{
+    Write-Output "Build successful."
+    continue
+}
+
+Write-Output "::error::Build failed. See the errors above in the build log."
+exit 1
+
+Close-DotNetBuildServers

.github/actions/msbuild/action.yml

@@ -0,0 +1,73 @@
+name: MSBuild .NET Framework Build with Static Code Analysis
+description: Builds the .NET Framework solution in the given directory with static code analysis.
+
+inputs:
+  directory:
+    required: false
+    default: .
+    description: Path to the directory where a solution file can be found.
+  verbosity:
+    required: false
+    default: quiet
+    description: Verbosity parameter for msbuild.
+  enable-code-analysis:
+    required: false
+    default: "true"
+    description: If set to "true", static code analysis is enabled during the build.
+  publish-version:
+    required: false
+    default: "1.${{ github.run_number }}.${{ github.run_attempt }}-${{ github.run_id }}"
+    description: The value given to `msbuild`'s `-Version` switch.
+  dotnet-build-switches:
+    required: false
+    default: ""
+    description: >
+      Additional command line switches given to `msbuild`. You must split these into multiple lines, even the
+      parameter value.
+  solution-or-project-path:
+    required: false
+    default: "*.sln"
+    description: >
+      The path of the solution or project file to be built. If you have exactly one .sln file in the current directory
+      then this can be omitted. Otherwise make sure to specify it to avoid an "MSB1008: Only one project can be
+      specified." error that is thrown when the `*.sln` wildcard results in multiple files. To build a project file you
+      must specify a path that ends with `.csproj`. An example: "./src/*Web/*.Web.csproj". The value is given to
+      PowerShell's `Get-ChildItem` cmdlet as-is, so grepping is still possible but a name with spaces must be escaped
+      separately.
+
+runs:
+  using: "composite"
+  steps:
+    - name: Setup Scripts
+      shell: pwsh
+      run: |
+        "${{ github.action_path }}" >> $Env:GITHUB_PATH
+        (Resolve-Path "${{ github.action_path }}/../../../Scripts").Path >> $Env:GITHUB_PATH
+
+    - name: Add MSBuild to PATH
+      # v1.1.3
+      uses: microsoft/setup-msbuild@34cfbaee7f672c76950673338facd8a73f637506
+      with:
+        vs-version: "[17.3,]"
+
+    - name: Build and Static Code Analysis
+      shell: pwsh
+      working-directory: ${{ inputs.directory }}
+      run: |
+        $buildSwitches = @'
+            ${{ inputs.dotnet-build-switches }}
+        '@
+
+        $switches = @{
+            SolutionOrProject = (Get-ChildItem ${{ inputs.solution-or-project-path }}).FullName
+            Verbosity = "${{ inputs.verbosity }}"
+            EnableCodeAnalysis = "${{ inputs.enable-code-analysis }}"
+            Version = "${{ inputs.publish-version }}"
+            Switches = $buildSwitches
+        }
+
+        $startTime = [DateTime]::Now
+        Build-DotNetFrameworkSolutionOrProject @switches
+        $endTime = [DateTime]::Now
+
+        Write-Output ("Solution or project build took {0:0.###} seconds." -f ($endTime - $startTime).TotalSeconds)

.github/actions/test-dotnet/Invoke-SolutionTests.ps1

@@ -1,4 +1,4 @@
-param ($Verbosity, $Filter)
+param ($Verbosity, $Filter, $Configuration)
 
 # Note that this script will only find tests if they were previously build in Release mode.
 
@@ -26,13 +26,18 @@ else
 
 $Env:Lombiq_Tests_UI__BrowserConfiguration__Headless = "true"
 
+# Usually tests should be built with the Debug configuration. If that's selected, then most possibly the whole solution
+# was built with Release, so we need to build but not restore. Otherwise, the Release tests are already built, so we
+# don't need to build either.
+$optOut = $Configuration -eq "Debug" ? "--no-restore" : "--no-build"
+
 $tests = dotnet sln list |
     Select-Object -Skip 2 |
     Select-String "\.Tests\." |
     Select-String -NotMatch "Lombiq.Tests.UI.csproj" |
     Select-String -NotMatch "Lombiq.Tests.csproj" |
     Where-Object {
-        $result = dotnet test --no-restore --list-tests --verbosity $Verbosity $_ 2>&1 | Out-String -Width 9999
+        $result = dotnet test $optOut --configuration $Configuration --list-tests --verbosity $Verbosity $_ 2>&1 | Out-String -Width 9999
         -not [string]::IsNullOrEmpty($result) -and $result.Contains("The following Tests are available")
     }
 

.github/actions/test-dotnet/action.yml

@@ -17,6 +17,10 @@ inputs:
     required: false
     description: >
       Filter expression for dotnet test. See: https://learn.microsoft.com/en-us/dotnet/core/testing/selective-unit-tests.
+  test-configuration:
+    required: false
+    description: Value passed to the --configuration parameter of dotnet test.
+    default: "Debug"
   ui-test-parallelism:
     required: false
     default: "-1"
@@ -54,7 +58,14 @@ runs:
     - name: Run Tests
       shell: pwsh
       working-directory: ${{ inputs.build-directory }}
-      run: Invoke-SolutionTests -Verbosity ${{ inputs.test-verbosity }} -Filter "${{ inputs.test-filter }}"
+      run: |
+        $switches = @{
+          Verbosity = "${{ inputs.test-verbosity }}"
+          Filter = "${{ inputs.test-filter }}"
+          Configuration = "${{ inputs.test-configuration }}"
+        }
+
+        Invoke-SolutionTests @switches
 
     # Note that uploading the failure dumps would fail under Windows if the path were too long, regardless of
     # LongPathsEnabled, see: https://github.com/actions/upload-artifact/issues/309. To get around that we merge all

.github/workflows/build-and-test-orchard-core.yml

@@ -114,9 +114,8 @@ jobs:
         - ${{ inputs.timeout-minutes }}
     timeout-minutes: ${{ matrix.timeout-minutes }}
     steps:
-    - uses: actions/[email protected]
-      with:
-        submodules: 'recursive'
+    - name: Checkout
+      uses: Lombiq/GitHub-Actions/.github/actions/checkout@issue/CARE-265
 
     - name: Set up .NET
       uses: Lombiq/GitHub-Actions/.github/actions/setup-dotnet@dev
@@ -133,7 +132,7 @@ jobs:
       uses: Lombiq/GitHub-Actions/.github/actions/enable-corepack@dev
 
     - name: Build and Static Code Analysis
-      uses: Lombiq/GitHub-Actions/.github/actions/build-dotnet@dev
+      uses: Lombiq/GitHub-Actions/.github/actions/build-dotnet@issue/CARE-265
       with:
         directory: ${{ inputs.build-directory }}
         verbosity: ${{ inputs.build-verbosity }}
@@ -153,7 +152,7 @@ jobs:
         location: ${{ inputs.build-directory}}
 
     - name: Tests
-      uses: Lombiq/GitHub-Actions/.github/actions/test-dotnet@dev
+      uses: Lombiq/GitHub-Actions/.github/actions/test-dotnet@issue/CARE-265
       with:
         build-directory: ${{ inputs.build-directory }}
         test-verbosity: ${{ inputs.build-verbosity }}

.github/workflows/build-dotnet.yml

@@ -88,9 +88,8 @@ jobs:
         - ${{ inputs.timeout-minutes }}
     timeout-minutes: ${{ matrix.timeout-minutes }}
     steps:
-    - uses: actions/[email protected]
-      with:
-        submodules: 'recursive'
+    - name: Checkout
+      uses: Lombiq/GitHub-Actions/.github/actions/checkout@issue/CARE-265
 
     - name: Set up .NET
       uses: Lombiq/GitHub-Actions/.github/actions/setup-dotnet@dev

.github/workflows/deploy-to-azure-app-service.yml

@@ -116,9 +116,8 @@ jobs:
         shell: pwsh
     timeout-minutes: ${{ inputs.timeout-minutes }}
     steps:
-    - uses: actions/[email protected]
-      with:
-        submodules: 'recursive'
+    - name: Checkout
+      uses: Lombiq/GitHub-Actions/.github/actions/checkout@issue/CARE-265
 
     - name: Set up .NET
       uses: Lombiq/GitHub-Actions/.github/actions/setup-dotnet@dev