Skip to content

test(cli): clean up multi-output test files in afterAll#653

Merged
hildjj merged 1 commit intopeggyjs:mainfrom
knoan:fix/gh652
Dec 15, 2025
Merged

test(cli): clean up multi-output test files in afterAll#653
hildjj merged 1 commit intopeggyjs:mainfrom
knoan:fix/gh652

Conversation

@knoan
Copy link
Contributor

@knoan knoan commented Dec 15, 2025

Summary

  • Add afterAll hook to clean up multi1.peggy, multi2.peggy, and the multi-out directory after the --multi-output test suite completes

Fixes #652

Test plan

  • Tests pass with npm test -- -t "multi-output"
  • Verified no leftover files remain after test execution

🤖 Generated with Claude Code

Add afterAll hook to remove multi1.peggy, multi2.peggy, and
the multi-out directory after the --multi-output test suite.

Fixes peggyjs#652

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.48%. Comparing base (ddcb829) to head (849bcc4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #653   +/-   ##
=======================================
  Coverage   99.48%   99.48%           
=======================================
  Files          34       34           
  Lines        4313     4313           
=======================================
  Hits         4291     4291           
  Misses         22       22           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hildjj hildjj merged commit cba4f9f into peggyjs:main Dec 15, 2025
10 checks passed
@hildjj
Copy link
Contributor

hildjj commented Dec 15, 2025

Thanks!

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.

tests leave files behind

3 participants