Fix AnalyzeTests strict-mode crash on mixed page scripting results

[Copilot]

❔What, Why & How

When a page scripting suite contains passed and failed tests, accessing .failure on passed testcases throws under strict mode, masking the failure report.

• Guard: Check for the XML child with $testcase.SelectSingleNode('failure') before reading failure details.
• Regression coverage: Cover mixed results—including 9 passed/2 failed—and skipped, all-passed, and all-failed cases. Assert counts, failure names, messages, and stack traces.
• Release notes: Document the corrected behavior.

✅ Checklist

• Add tests (E2E, unit tests)
• Update RELEASENOTES.md
• Update documentation (e.g. for new settings or scenarios) — Not applicable
• Add telemetry — Not applicable

• Fixes [Bug]: AnalyzeTests crashes under strict mode when page scripting results contain both passed and failed tests ("The property 'failure' cannot be found") #2390

[Maria Zhelezova (mazhelez)]

Copilot resolve the merge conflicts in this pull request

[Copilot]

Copilot resolve the merge conflicts in this pull request

Resolved in 986f319 by merging the latest main, preserving the fix, regression tests, and both release-note entries. All 14 AnalyzeTests tests pass; GitHub no longer marks the PR as conflicted.

[Maria Zhelezova (mazhelez)]

Copilot please move the release notes above the v10.0 section in a new "Issues" section.

[Copilot]

🟢 Approval recommended

The targeted fix safely resolves the reported crash and includes comprehensive regression coverage.

0 open findings

What changed in this PR

Fixes strict-mode crashes when page scripting results contain both passed and failed tests.

Changes:

• Safely checks for <failure> nodes before accessing failure details.
• Adds regression coverage for mixed, skipped, all-passed, and all-failed results.
• Documents the fix in release notes.

File	Description
Actions/​AnalyzeTests/​TestResultAnalyzer.ps1	Guards failure-node access.
Tests/​AnalyzeTests.Test.ps1	Adds strict-mode regression tests.
RELEASENOTES.md	Documents issue #2390 resolution.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot please move the release notes above the v10.0 section in a new "Issues" section.

Moved the AnalyzeTests entry into a new top-level Issues section above v10.0 in d139425.

[Maria Zhelezova (mazhelez)]

Copilot resolve the merge conflicts in this pull request

[Copilot]

Copilot resolve the merge conflicts in this pull request

Merged the latest main into the branch in 46e19e2; Git merged cleanly with no conflicts. The release-note issues are consolidated under one heading in 751ba2b.