name: Unit Tests

on:
  push:
    branches: [ main ]
  pull_request:
    branches: [ main ]

jobs:
  # ── Build module artifacts (only job that needs .NET SDK) ──────────
  build:
    runs-on: ${{ matrix.os }}
    timeout-minutes: 10
    strategy:
      fail-fast: false
      matrix:
        include:
          - os: windows-latest
            framework: netstandard2.0
          - os: ubuntu-latest
            framework: netstandard2.0

    steps:
      - uses: actions/checkout@v6

      - name: Setup .NET
        uses: actions/setup-dotnet@v5
        with:
          dotnet-version: '10.0.x'

      - name: Build module
        run: dotnet publish src/PSProxmoxVE/PSProxmoxVE.csproj --configuration Release --framework ${{ matrix.framework }} --output ./publish/${{ matrix.framework }}

      - name: Clean publish output for PS module loading
        shell: bash
        run: rm -f ./publish/${{ matrix.framework }}/*.deps.json ./publish/${{ matrix.framework }}/*.runtimeconfig.json

      - name: Upload module artifact
        uses: actions/upload-artifact@v7
        with:
          name: module-${{ matrix.framework }}-${{ matrix.os }}
          path: ./publish/${{ matrix.framework }}/

  # ── Pester tests (no .NET SDK — just PowerShell + Pester) ─────────
  pester-tests:
    needs: build
    timeout-minutes: 15
    strategy:
      fail-fast: false
      matrix:
        include:
          - os: windows-latest
            ps_version: '5.1'
            framework: netstandard2.0
            artifact_os: windows-latest
            shell: powershell
          - os: windows-latest
            ps_version: '7.5'
            framework: netstandard2.0
            artifact_os: windows-latest
            shell: pwsh
          - os: ubuntu-latest
            ps_version: '7.5'
            framework: netstandard2.0
            artifact_os: ubuntu-latest
            shell: pwsh
          - os: macos-latest
            ps_version: '7.5'
            framework: netstandard2.0
            artifact_os: ubuntu-latest
            shell: pwsh

    runs-on: ${{ matrix.os }}

    steps:
      - uses: actions/checkout@v6

      - name: Download module artifact
        uses: actions/download-artifact@v8
        with:
          name: module-${{ matrix.framework }}-${{ matrix.artifact_os }}
          path: ./publish/${{ matrix.framework }}/

      - name: Install Pester 5 (PS 5.1)
        if: matrix.ps_version == '5.1'
        shell: powershell
        run: |
          Install-Module -Name Pester -MinimumVersion 5.0 -Force -Scope CurrentUser -SkipPublisherCheck

      - name: Install Pester 5 (PS 7.x)
        if: matrix.ps_version != '5.1'
        shell: pwsh
        run: |
          Install-Module -Name Pester -MinimumVersion 5.0 -Force -Scope CurrentUser

      - name: Copy module to module path (PS 5.1)
        if: matrix.ps_version == '5.1'
        shell: powershell
        run: |
          $modulePath = "$env:USERPROFILE\Documents\WindowsPowerShell\Modules\PSProxmoxVE"
          New-Item -ItemType Directory -Path $modulePath -Force | Out-Null
          Copy-Item -Path .\publish\${{ matrix.framework }}\* -Destination $modulePath -Recurse -Force

      - name: Copy module to module path (PS 7.x, Windows)
        if: matrix.ps_version != '5.1' && matrix.os == 'windows-latest'
        shell: pwsh
        run: |
          $modulePath = "$env:USERPROFILE\Documents\PowerShell\Modules\PSProxmoxVE"
          New-Item -ItemType Directory -Path $modulePath -Force | Out-Null
          Copy-Item -Path .\publish\${{ matrix.framework }}\* -Destination $modulePath -Recurse -Force

      - name: Copy module to module path (PS 7.x, non-Windows)
        if: matrix.ps_version != '5.1' && matrix.os != 'windows-latest'
        shell: pwsh
        run: |
          $modulePath = "$HOME/.local/share/powershell/Modules/PSProxmoxVE"
          New-Item -ItemType Directory -Path $modulePath -Force | Out-Null
          Copy-Item -Path ./publish/${{ matrix.framework }}/* -Destination $modulePath -Recurse -Force

      - name: Run Pester tests (PS 5.1)
        if: matrix.ps_version == '5.1'
        shell: powershell
        run: |
          Import-Module Pester -MinimumVersion 5.0
          $config = New-PesterConfiguration
          $config.Run.Path = "tests/PSProxmoxVE.Tests"
          $config.Run.Exit = $true
          $config.Filter.ExcludeTag = @("Integration")
          $config.Output.Verbosity = "Detailed"
          $config.TestResult.Enabled = $true
          $config.TestResult.OutputFormat = "NUnitXml"
          $config.TestResult.OutputPath = "TestResults/pester-results.xml"
          Invoke-Pester -Configuration $config

      - name: Run Pester tests (PS 7.x)
        if: matrix.ps_version != '5.1'
        shell: pwsh
        run: |
          Import-Module Pester -MinimumVersion 5.0
          $config = New-PesterConfiguration
          $config.Run.Path = "tests/PSProxmoxVE.Tests"
          $config.Run.Exit = $true
          $config.Filter.ExcludeTag = @("Integration")
          $config.Output.Verbosity = "Detailed"
          $config.TestResult.Enabled = $true
          $config.TestResult.OutputFormat = "NUnitXml"
          $config.TestResult.OutputPath = "TestResults/pester-results.xml"
          Invoke-Pester -Configuration $config

      - name: Upload test results
        if: always()
        uses: actions/upload-artifact@v7
        with:
          name: pester-results-${{ matrix.os }}-ps${{ matrix.ps_version }}
          path: TestResults/