Skip to content

Commit 529c952

Browse files
szymonrybczakfacebook-github-bot
authored andcommitted
Move script checking Appium server status to separate file (#38692)
Summary: This PR is small cleanup in `.circleci/config.yml` file. Request here: #36267 (comment) ## Changelog: [INTERNAL] [CHANGED] - move script checking Appium server status to separate file Pull Request resolved: #38692 Test Plan: CI Green ✅ Reviewed By: NickGerleman Differential Revision: D47926222 Pulled By: ryancat fbshipit-source-id: eeff338ca018549762dce46a744b80f659b4bd5d
1 parent 2856bef commit 529c952

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

.circleci/config.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -785,11 +785,7 @@ jobs:
785785
command: mv /tmp/e2e/Build/Products/Debug-iphonesimulator/RNTester.app packages/rn-tester-e2e/apps/rn-tester.app
786786
- run:
787787
name: Check Appium server status
788-
command: |
789-
if ! nc -z 127.0.0.1 4723; then
790-
echo Could not find Appium server!
791-
exit 1
792-
fi
788+
command: scripts/circleci/check_appium_server_status.sh
793789
- run:
794790
name: Run E2E tests
795791
command: |
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
#!/bin/bash
2+
# Copyright (c) Meta Platforms, Inc. and affiliates.
3+
#
4+
# This source code is licensed under the MIT license found in the
5+
# LICENSE file in the root directory of this source tree.
6+
7+
# This script checks if Appium server is running on default port (which is 4723).
8+
if ! nc -z 127.0.0.1 4723; then
9+
echo Could not find Appium server.
10+
exit 1
11+
fi

0 commit comments

Comments
 (0)