Skip to content

test: stabilize nested plugin timeout - #363

Closed
Puppo wants to merge 1 commit into
fastify:mainfrom
Puppo:test/stabilize-nested-timeout
Closed

Puppo wants to merge 1 commit into
fastify:mainfrom
Puppo:test/stabilize-nested-timeout

Conversation

@Puppo

@Puppo Puppo commented Sep 1, 2026

Copy link
Copy Markdown

Summary

  • allow enough startup time for nested plugins when the test suite runs concurrently
  • keep the assertion focused on the deliberately unresolved innermost plugin
  • preserve the upstream 100% coverage gate without production-code exclusions

The previous 10 ms budget could expire the parent plugin before nested registration completed under concurrent load, producing a failure that named first instead of third.

Validation

  • npm test — 331 tests passed, including TypeScript validation
  • npm run test:coverage — 100% statements, branches, functions, and lines
  • npm run lint
  • isolated timeout test passed 25 consecutive runs

@Puppo Puppo closed this Sep 1, 2026
@Puppo Puppo reopened this Sep 1, 2026
@Puppo Puppo closed this Sep 1, 2026
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