Skip to content

Test context hook error behavior#32

Open
ngan wants to merge 1 commit intomainfrom
np-test-context-hook-error-behavior
Open

Test context hook error behavior#32
ngan wants to merge 1 commit intomainfrom
np-test-context-hook-error-behavior

Conversation

@ngan
Copy link
Collaborator

@ngan ngan commented Feb 26, 2026

Summary

  • Add integration tests verifying RSpec context hook error behavior for fixture_kit
  • Confirms after(:context) (our finish hook) still runs when before(:context) (generate) raises
  • Confirms examples execute normally before an after(:context) error is reported

Test plan

  • after(:context) runs even when before(:context) raises — FKIT_ASSERT:AFTER_CONTEXT_RUNS_AFTER_BEFORE_ERROR marker emitted
  • Examples pass before after(:context) error — FKIT_ASSERT:EXAMPLES_PASS_BEFORE_AFTER_ERROR marker emitted
  • Both intentional errors appear in subprocess output
  • Full suite passes (158 examples, 0 failures)

🤖 Generated with Claude Code

@ngan ngan force-pushed the np-test-context-hook-error-behavior branch from 0700120 to 500c718 Compare February 26, 2026 23:15
Intentionally broken — for testing error behavior in a consuming app.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ngan ngan force-pushed the np-test-context-hook-error-behavior branch from 500c718 to f47f47b Compare February 27, 2026 00:28
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