Skip to content

Enable testSendAndWaitThrowsOnTimeout in CI#9

Merged
brunoborges merged 1 commit intomainfrom
merge-upstream-20260203
Feb 3, 2026
Merged

Enable testSendAndWaitThrowsOnTimeout in CI#9
brunoborges merged 1 commit intomainfrom
merge-upstream-20260203

Conversation

@brunoborges
Copy link
Contributor

  • Enhanced CapiProxy.isAlive() to include HTTP health check
  • When proxy crashes (e.g., due to no cached response), the HTTP check detects it and triggers automatic restart
  • Removed @DisabledIfEnvironmentVariable annotation from timeout test
  • All 80 tests now pass in CI mode

The upstream Node.js harness has a bug where it crashes on 'no cached response' due to missing 'return' after exitWithNoMatchingRequestError. This fix works around it by detecting the crashed proxy and restarting.

- Enhanced CapiProxy.isAlive() to include HTTP health check
- When proxy crashes (e.g., due to no cached response), the HTTP check
  detects it and triggers automatic restart
- Removed @DisabledIfEnvironmentVariable annotation from timeout test
- All 80 tests now pass in CI mode

The upstream Node.js harness has a bug where it crashes on 'no cached
response' due to missing 'return' after exitWithNoMatchingRequestError.
This fix works around it by detecting the crashed proxy and restarting.
@brunoborges brunoborges merged commit 121a31b into main Feb 3, 2026
1 of 2 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