ci: print a failed-test summary at the end of the Pester run

[mikemadeja]

Summary

Pester's own console output shows each failure inline as it happens, but nothing at the very end lists which tests failed without scrolling back through a (sometimes 1000+ line) log. Adds a short "FAILED TESTS" block right after the run, listing each failed test's full Describe/It path and error message - e.g.:

==================================================
 FAILED TESTS (1)
==================================================

[-] Remove-PiHoleGroup (Integration).returns the raw API response when RawOutput is set
    Response status code does not indicate success: 400 (Bad Request).

This meant switching Run.Exit from $true to $false and calling exit manually with the failure count afterward, since Invoke-Pester's own Run.Exit handling calls exit internally and would skip the summary entirely.

Also fixed a real, separately-discovered issue while touching this file: the "Ensure Pester 5 is available" step only checked -ge 5.0.0, which would silently accept a newer major version like 6.x if one happened to already be installed on the agent - which is exactly what was found installed on the shared self-hosted agent (v6.2.0, whose result-object shape can't be assumed compatible with 5.x, the version this whole pipeline's Pester-specific code actually targets, including the new summary logic). Both the install check and Import-Module now pin to the 5.x range specifically.

Test plan

• Verified both the failing and all-passing paths locally against real Pester runs: the summary prints correctly (and only) on failure, and the exit code correctly reflects the failure count in both cases

🤖 Generated with Claude Code