Skip to content

Stop launching the removed WebSocket test server in objc-test.sh - #59952

Open
lazerg wants to merge 1 commit into
react:mainfrom
lazerg:fix/issue-59939-objc-test-websocket
Open

lazerg wants to merge 1 commit into
react:mainfrom
lazerg:fix/issue-59939-objc-test-websocket

Conversation

@lazerg

@lazerg lazerg commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Summary:

yarn test-ios exits before it builds RNTester. #59433 deleted packages/rn-tester/IntegrationTests/, including launchWebSocketServer.sh, but objc-test.sh still launches it and then waits a minute for something to answer on port 5555 before giving up. No test uses that server anymore, so this removes the launch, waitForWebSocketServer and the port 5555 cleanup.

Fixes #59939

Changelog:

[INTERNAL] [FIXED] - Stop launching the removed WebSocket test server in objc-test.sh

Test Plan:

I ran scripts/objc-test.sh test with lsof, yarn, curl, bundle and xcodebuild stubbed on PATH. Before the change it prints sh: ./IntegrationTests/launchWebSocketServer.sh: No such file or directory and exits 1 after WebSocket Server did not respond in time. After it, the script goes straight to the packager, pod install and xcodebuild build-for-testing. shellcheck scripts/objc-test.sh is clean.

@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 Nov 12, 2026
@facebook-github-tools facebook-github-tools Bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Nov 12, 2026

This branch has not been deployed

No deployments
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.

yarn test-ios exits before building RNTester: objc-test.sh launches the WebSocket server script #59433 deleted

1 participant