Skip to content

Repair assertions and isolate test files - #15

Merged
Code0x58 merged 1 commit into
masterfrom
codex/repair-test-assertions
Oct 18, 2026
Merged

Code0x58 merged 1 commit into
masterfrom
codex/repair-test-assertions

Conversation

@Code0x58

@Code0x58 Code0x58 commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Value-assignment tests now assert their comparisons, and new-store tests inspect the file they actually create. Per-test temporary directories clean up every fixture, including files used by round-trip and commit tests.

Validation: 22 tests pass.

Stack position 2 of 9. Depends on #14; this PR targets codex/modernize-ci-packaging so its diff contains only this fix. Addresses review finding 10.

Stack overview: #14. Next: #16.

@codecov

codecov Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.83%. Comparing base (a5c8509) to head (0d1cd3d).
⚠️ Report is 3 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   95.83%   95.83%           
=======================================
  Files           1        1           
  Lines         144      144           
=======================================
  Hits          138      138           
  Misses          6        6           

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

@Code0x58
Code0x58 force-pushed the codex/repair-test-assertions branch from 42e4e8c to a5c8509 Compare October 18, 2026 15:13
@Code0x58
Code0x58 added this pull request to stack #23 October 18, 2026 15:39
Base automatically changed from codex/modernize-ci-packaging to master October 18, 2026 15:40
@Code0x58
Code0x58 force-pushed the codex/repair-test-assertions branch from a5c8509 to 0d1cd3d Compare October 18, 2026 15:40
@Code0x58
Code0x58 merged commit 067a259 into master Oct 18, 2026
17 checks passed
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.

1 participant