ReviewOS

home-lang/home

Port Bun done-callback reporter matrices

#110
Closed chris opened this 10 days ago ·
10 days ago

Tracks a focused tranche of #66.

  • Run the byte-identical upstream done-async.test.ts instead of rewriting it to a native TODO.
  • Run the byte-identical upstream test-error-code-done-callback.test.ts with scoped path normalization.
  • Surface Bun.version_with_sha with native reconstruction and a Home fallback.
  • Preserve all nine done(error) code frames and all seven async done/error timing failures.
  • Add a permanent two-file exact-count regression.

Validation before publication:

  • done-async.test.ts: outer test passed; all 7 intended child failures reported
  • test-error-code-done-callback.test.ts: outer test passed; all 9 intended child failures/code frames matched
  • full home_test: 1,084 tests, 1 expected skip, 0 failures
10 days ago

Completed in 2dad763 (feat(test): port done callback reporters). The unchanged upstream done-async.test.ts and test-error-code-done-callback.test.ts now run through Home: both outer reporter assertions pass (2 total), covering all 16 intentionally failing child cases with Bun-compatible output and code frames. Added permanent corpus coverage. Final validation: zig fmt --check, git diff --check, exact corpus runs, and the full home_test gate all pass (1084 tests, 1 expected skip, 0 failures).

Sign in to comment on this issue.