Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -399,9 +399,9 @@ if (-not $SkipValidation) {
Write-Host " ✅ $($phase.Name): Valid" -ForegroundColor Green
} else {
Write-Host " ❌ $($phase.Name): INVALID" -ForegroundColor Red
foreach ($error in $result.Errors) {
Write-Host " - $error" -ForegroundColor Red
$allValidationErrors += "$($phase.Name): $error"
foreach ($err in $result.Errors) {
Write-Host " - $err" -ForegroundColor Red
$allValidationErrors += "$($phase.Name): $err"
}
}

Expand Down Expand Up @@ -435,8 +435,8 @@ if (-not $SkipValidation) {
Write-Host "╚═══════════════════════════════════════════════════════════╝" -ForegroundColor Red
Write-Host ""
Write-Host "Found $($allValidationErrors.Count) validation error(s):" -ForegroundColor Red
foreach ($error in $allValidationErrors) {
Write-Host " - $error" -ForegroundColor Red
foreach ($err in $allValidationErrors) {
Write-Host " - $err" -ForegroundColor Red
}
Write-Host ""
Write-Host "💡 Fix these issues in the state file before posting the review comment." -ForegroundColor Cyan
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -152,8 +152,12 @@ if (-not [string]::IsNullOrWhiteSpace($ReportFile)) {

# Use report content as summary if not provided
if ([string]::IsNullOrWhiteSpace($Summary)) {
# Extract key results from report
$Summary = $reportContent
# Extract key results from report, excluding verbose "Test Results Details" section
if ($reportContent -match '(?s)^(.*?)(?=####\s*Test Results Details)') {
$Summary = $Matches[1].TrimEnd()
} else {
$Summary = $reportContent
}
}
}

Expand Down Expand Up @@ -181,19 +185,13 @@ _Run `verify-tests-fail.ps1` for full details._
}

# Status emoji
$statusEmoji = if ($Status -eq "Passed") { "✅ PASSED" } else { "❌ FAILED" }
$statusEmoji = if ($Status -eq "Passed") { "✅ passed" } else { "❌ failed" }
$modeDesc = if ($Mode -eq "FullVerification") { "Full Verification" } else { "Failure Only" }

# Build verification section content
# Build verification section content - wrapped in collapsible with status in summary
$verifyContent = @"
### 🚦 Test Verification

**Result**: $statusEmoji
**Mode**: $modeDesc
**Platform**: $Platform

<details>
<summary>Expand Details</summary>
<summary><b>🚦 Test Verification: $statusEmoji</b></summary>

$Summary

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -564,18 +564,18 @@ function Write-MarkdownReport {
$statusSymbol = if ($VerificationPassed) { "✅" } else { "❌" }

$markdown = @"
# Test Verification Report
### Test Verification Report

**Date:** $reportDate | **Platform:** $($Platform.ToUpper()) | **Status:** $status

## Summary
#### Summary

| Check | Expected | Actual | Result |
|-------|----------|--------|--------|
| Tests WITHOUT fix | FAIL | $(if ($FailedWithoutFix) { "FAIL" } else { "PASS" }) | $(if ($FailedWithoutFix) { "✅" } else { "❌" }) |
| Tests WITH fix | PASS | $(if ($PassedWithFix) { "PASS" } else { "FAIL" }) | $(if ($PassedWithFix) { "✅" } else { "❌" }) |

## $statusSymbol Final Verdict
#### $statusSymbol Final Verdict

$(if ($VerificationPassed) {
@"
Expand Down Expand Up @@ -607,7 +607,7 @@ $(if (-not $FailedWithoutFix) {

---

## Configuration
#### Configuration

**Platform:** $Platform
**Test Filter:** $TestFilter
Expand All @@ -629,7 +629,7 @@ $(($NewFiles | ForEach-Object { "- ``$_``" }) -join "`n")

---

## Test Results Details
#### Test Results Details

### Test Run 1: WITHOUT Fix

Expand Down Expand Up @@ -683,7 +683,7 @@ $(Get-Content $WithFixLog -Raw)

---

## Logs
#### Logs

- Full verification log: ``$ValidationLog``
- Test output without fix: ``$WithoutFixLog``
Expand Down