Skip to content

Commit

Permalink
fix android ci
Browse files Browse the repository at this point in the history
  • Loading branch information
vaind committed Aug 1, 2024
1 parent f1140fa commit 082406e
Show file tree
Hide file tree
Showing 6 changed files with 4,147 additions and 5,306 deletions.
10 changes: 3 additions & 7 deletions .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -417,14 +417,9 @@ jobs:
working-directory: test/e2e
run: yarn install

- name: Setup tmate session
uses: mxschmitt/action-tmate@v3

- name: Run tests on Android
if: ${{ matrix.platform == 'android' }}
uses: reactivecircus/android-emulator-runner@77986be26589807b8ebab3fde7bbf5c60dabec32 # [email protected]
env:
ANDROID_EMULATOR_WAIT_TIME_BEFORE_KILL: 60
with:
api-level: 30
force-avd-creation: false
Expand All @@ -443,8 +438,9 @@ jobs:
-timezone US/Pacific
script: |
./scripts/e2e.mjs ${{ matrix.platform }} --test
# https://github.com/ReactiveCircus/android-emulator-runner/issues/385
killall --signal 9 crashpad_handler
# For some reason there's another appium process running that prevents the emulator to stop.
# We need to kill it manually.
pkill -f appium
- uses: actions/cache@v4
if: ${{ matrix.platform == 'ios' }}
Expand Down
1 change: 0 additions & 1 deletion .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
"editor.formatOnSave": true,
"editor.rulers": [120],
"editor.tabSize": 2,
"files.autoSave": "onWindowChange",
"files.trimTrailingWhitespace": true,
"files.insertFinalNewline": true,
"search.exclude": {
Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -130,5 +130,6 @@
"expo": {
"optional": true
}
}
},
"packageManager": "[email protected]+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
}
2 changes: 1 addition & 1 deletion scripts/e2e.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ if (actions.includes('test')) {
processesToKill[name] = {
process: process,
complete: new Promise((resolve, _reject) => {
process.on('exit', resolve);
process.on('close', resolve);
})
};
}
Expand Down
Loading

0 comments on commit 082406e

Please sign in to comment.