Skip to content

Treat BeforeAll module-local setup failures as catastrophic#399

Merged
Marius Storhaug (MariusStorhaug) merged 2 commits into
mainfrom
issue-388-beforeall-failure
Jul 18, 2026
Merged

Treat BeforeAll module-local setup failures as catastrophic#399
Marius Storhaug (MariusStorhaug) merged 2 commits into
mainfrom
issue-388-beforeall-failure

Conversation

@MariusStorhaug

Copy link
Copy Markdown
Member

Summary

  • treat BeforeAll-ModuleLocal failure as a hard failure root cause in test result aggregation
  • run AfterAll-ModuleLocal cleanup whenever module-local setup ran, even if setup failed and tests were skipped
  • pass BeforeAll-ModuleLocal job result into Get-TestResults and fail early with a clear root-cause message
  • document the new BeforeAllModuleLocalResult input in Get-PesterTestResults

Links

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Jul 18, 2026

Copy link
Copy Markdown

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@MariusStorhaug
Marius Storhaug (MariusStorhaug) merged commit 317bcb1 into main Jul 18, 2026
69 checks passed
@MariusStorhaug
Marius Storhaug (MariusStorhaug) deleted the issue-388-beforeall-failure branch July 18, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Treat BeforeAll module-local setup failure as catastrophic: skip tests, run AfterAll cleanup, and report root cause

1 participant