Skip to content

test: reproduce hanging RequestManager requests and missing cleanup#5752

Open
ayushi-forge wants to merge 1 commit intosugarlabs:masterfrom
ayushi-forge:test-requestmanager-timeout-clean
Open

test: reproduce hanging RequestManager requests and missing cleanup#5752
ayushi-forge wants to merge 1 commit intosugarlabs:masterfrom
ayushi-forge:test-requestmanager-timeout-clean

Conversation

@ayushi-forge
Copy link

Adds regression tests that expose hanging requests in RequestManager
when callbacks never resolve and pendingRequests is not cleaned up.

This PR intentionally contains tests only.

@ayushi-forge
Copy link
Author

ayushi-forge commented Feb 15, 2026

Closing this PR in favor of a clean follow-up PR that contains
only the RequestManager regression tests.

New PR: #5752

@github-actions
Copy link
Contributor

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

RequestManager.test.js

@ayushi-forge
Copy link
Author

Note: The failing Jest tests are expected here.
They intentionally reproduce the current RequestManager behavior
(hanging requests and missing cleanup).

A follow-up PR will address the fix once this approach is approved.

@ayushi-forge ayushi-forge force-pushed the test-requestmanager-timeout-clean branch from 32a1811 to a395025 Compare February 16, 2026 04:26
@github-actions
Copy link
Contributor

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

RequestManager.test.js

@ayushi-forge
Copy link
Author

Reiterating for clarity: the failing Jest tests in this PR are intentional.
They document the current RequestManager bug (hanging requests + missing cleanup).

Once this approach is approved, I’ll open a follow-up PR that implements
the timeout + rejection + cleanup fix.

@omsuneri
Copy link
Member

@ayushi-forge please tag the issue i m not getting whats the issue ????

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.

2 participants