Skip to content

Added missing tests for Standalone fusebox shell#55460

Open
chicio wants to merge 2 commits intofacebook:mainfrom
chicio:standalone_fusebox_shell_test_improvements
Open

Added missing tests for Standalone fusebox shell#55460
chicio wants to merge 2 commits intofacebook:mainfrom
chicio:standalone_fusebox_shell_test_improvements

Conversation

@chicio
Copy link
Contributor

@chicio chicio commented Feb 7, 2026

Summary:

This PR adds the missing test coverage for the standalone Fusebox debugger shell in StandaloneFuseboxShell-test.
First, I added some missing expectation for the success case (enableStandaloneFuseboxShell=true and no errors) already present:

  • the assertion that unstable_prepareFuseboxShell is called just one time during middleware initialization
  • the assertion that launchDebuggerAppWindow is never called when the standalone shell is available

Then, I added the missing test for the case when unstable_prepareFuseboxShell returns one of the error code like eg. platform_not_supported (as reported also in the TODO that I removed).
In this way now the test is verifying the full behaviour for the new standalone mode for debugger/dev tools.

Changelog:

[GENERAL] [CHANGED] - Added missing tests for Standalone fusebox shell

Test Plan:

  • Ran StandaloneFuseboxShell-test and verify all tests cases passed
  • Ran the full React Native test suite to ensure all tests pass.
  • Verified test correctness by intentionally breaking production code:
    • removing the calls to the DefaultBrowserLauncher in openDebuggerMiddleware to verify that the tests were correctly failing

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 7, 2026
@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Feb 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants