Cleanup of CI tests

* Initial plan

* Remove debugging output from GitHub Actions workflows and scripts

Co-authored-by: pardeike <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: pardeike <[email protected]>

Author: Copilot

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

@@ -55,7 +55,6 @@ runs:
         "target_frameworks=$target_frameworks" | Out-File -FilePath $env:GITHUB_OUTPUT -Append;
       shell: pwsh
 
-    # Create clean TestResults directory using simple platform-specific commands
     - name: Clean TestResults (Windows)
       if: runner.os == 'Windows'
       run: |
@@ -121,32 +120,6 @@ runs:
             & $dotnet test "HarmonyTests/bin/${{inputs.build_configuration}}/$target_framework/HarmonyTests.dll" -f $target_framework --logger:"trx;LogFileName=../TestResults/$trxFileName" --logger:"console;verbosity=normal" -- ${{steps.test-args.outputs.run_settings_args}}
           }
         shell: pwsh
-        
-    # Debug step for Windows
-    - name: List TRX files (Windows)
-      if: runner.os == 'Windows'
-      run: |
-        Write-Host "Searching for TRX files on Windows..."
-        Write-Host "TRX files in TestResults directory:"
-        if (Test-Path -Path "TestResults") {
-          Get-ChildItem -Path "TestResults" -Filter "*.trx" -Recurse | ForEach-Object { Write-Host $_.FullName }
-        } else {
-          Write-Host "TestResults directory not found"
-        }
-      shell: pwsh
-      
-    # Debug step for Unix-based systems (macOS/Linux)
-    - name: List TRX files (Unix)
-      if: runner.os != 'Windows'
-      run: |
-        echo "Searching for TRX files on Unix..."
-        echo "TRX files in TestResults directory:"
-        if [ -d "TestResults" ]; then
-          find TestResults -name "*.trx" -type f | sort
-        else
-          echo "TestResults directory not found"
-        fi
-      shell: bash
 
     - name: Upload Test Result
       uses: ./.github/actions/test-upload-result

.github/actions/test-upload-result/action.yml

@@ -24,49 +24,6 @@ inputs:
 runs:
   using: "composite"
   steps:
-    # Use github-script to check files before upload (platform-independent)
-    - name: Check TRX Files
-      id: check-files
-      uses: actions/github-script@v7
-      with:
-        script: |
-          const fs = require('fs');
-          const path = require('path');
-          
-          try {
-            if (!fs.existsSync('TestResults')) {
-              console.log('TestResults directory not found');
-              return { count: 0, files: [] };
-            }
-            
-            // Helper function to find all TRX files recursively
-            function findTrxFiles(dir) {
-              let results = [];
-              const entries = fs.readdirSync(dir, { withFileTypes: true });
-              
-              for (const entry of entries) {
-                const fullPath = path.join(dir, entry.name);
-                
-                if (entry.isDirectory()) {
-                  results = results.concat(findTrxFiles(fullPath));
-                } else if (entry.isFile() && entry.name.endsWith('.trx')) {
-                  results.push(fullPath);
-                }
-              }
-              
-              return results;
-            }
-            
-            const trxFiles = findTrxFiles('TestResults');
-            console.log(`Found ${trxFiles.length} TRX files:`);
-            trxFiles.slice(0, 10).forEach(f => console.log(f));
-            
-            return { count: trxFiles.length, files: trxFiles.slice(0, 10) };
-          } catch (error) {
-            console.log(`Error checking TRX files: ${error.message}`);
-            return { count: 0, files: [] };
-          }
-
     - name: Upload Test Results
       uses: actions/upload-artifact@v4
       if: success() || failure()

.github/workflows/test-result-upload.yml

@@ -26,44 +26,6 @@ jobs:
           path: test-results
           merge-multiple: true
 
-      - name: List Test Files
-        if: always()
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const fs = require('fs');
-            const path = require('path');
-            
-            // Count TRX files
-            let trxCount = 0;
-            let firstFiles = [];
-            
-            function countTrxFiles(dir, depth = 0) {
-              if (depth > 5) return; // Avoid infinite recursion
-              
-              const entries = fs.readdirSync(dir, { withFileTypes: true });
-              
-              for (const entry of entries) {
-                const fullPath = path.join(dir, entry.name);
-                
-                if (entry.isDirectory()) {
-                  countTrxFiles(fullPath, depth + 1);
-                } else if (entry.isFile() && entry.name.endsWith('.trx')) {
-                  trxCount++;
-                  if (firstFiles.length < 5) {
-                    firstFiles.push(entry.name);
-                  }
-                }
-              }
-            }
-            
-            countTrxFiles('test-results');
-            
-            console.log(`Found ${trxCount} TRX files`);
-            if (firstFiles.length > 0) {
-              console.log(`Sample files: ${firstFiles.join(', ')}`);
-            }
-
       - name: Publish Test Results
         uses: EnricoMi/publish-unit-test-result-action@v2
         if: always()

scripts/pack.ps1

@@ -27,6 +27,5 @@ New-Item -ItemType Directory -Path Lib.Harmony.Thin\bin | Out-Null
 dotnet clean --nologo --verbosity minimal
 
 # Build Solution 
-Write-Host "Processing configuration: $config"
 dotnet build --nologo --configuration Release --verbosity minimal
 dotnet pack --nologo --configuration Release --verbosity minimal --no-restore --no-build