-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support parallel test execution #609
Merged
Merged
Changes from 1 commit
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
ae9c471
switch to e2e tests to jest
57ef873
downgrade test project to RN51
c54109e
wait for device based on bootstatus
f5cd1b9
introduce findDevicesUDID and use in findDeviceUDID
3c26649
remove redundant check
bdf4eb8
support multiple workers. for now set to 1
fb96349
device registry
bf31acb
device registry
5c5d923
fix device registry race condition
f18c34c
move maxTestWorkers param to detox config
5004898
clean up
74849ae
-forceExit
4251edc
error when no runtime is available
3bc6518
use dot notation
f839c3d
move lock file to ~/Library/Detox
fa268cd
adjust lock file retry options
d5e7dab
use functions instead of consts when declaring functions
06b917e
use const for detox library root path
d3dcdfb
Reset EarlGrey submodule
rotemmiz caa6fc8
dummy commit
a26e200
verbose postinstall
rotemmiz 08abe3f
Merge branch 'master' into test-parallelization
rotemmiz 70cb9d5
simplified root path to fix Android build
4c72215
Revert "simplified root path to fix Android build"
406740e
simplified root path to fix Android build
04b6243
dummy commit
dc7f70a
Merge branch 'master' into test-parallelization
ddbdd06
run e2e w/ parallelization in ci
7d12cd9
Merge branch 'master' into test-parallelization
712e6de
dummy commit
9b0c50a
temp - added troubleshooting logs
bc79553
Revert "temp - added troubleshooting logs"
a080a4b
Merge branch 'master' into test-parallelization
7de97ac
merge
00c40d3
make sure single works
3630f5a
fix clear lock file path
646168c
improve closed socket error message
73c27e1
Merge branch 'master' into test-parallelization
rotemmiz f40373f
WIP
rotemmiz 5dd5d27
WIP
rotemmiz 56f208d
WIP
rotemmiz 9a388b2
Merge branch 'master' into test-parallelization
rotemmiz c48d49d
WIP
rotemmiz 320113c
trigger build
rotemmiz ffc4e20
Trigger build
yershalom 688a31b
Trigger build
yershalom 1f21ca4
waitFor timeout increased
rotemmiz 0f9007f
maxWorkers=2
rotemmiz 52d0f86
Add line for trigger build
yershalom 5b11bc9
Revert "Add line for trigger build"
yershalom d088827
last trigger build
yershalom 8eea0a7
Revert "last trigger build"
yershalom cdd18b0
Update ci.ios.sh
yershalom 920faba
Add line to ci ios
yershalom 6931a0e
Fix platfrom for jest
yershalom 7547c99
Fix jest invert platform
yershalom 0f64ffa
Trigger build
yershalom 6267fe2
use detox-test to cleanup lockfile
rotemmiz a4484dd
fix waitFor screen, should not be flaky anymore
rotemmiz 57a7b38
added unit test for a new edge case
rotemmiz efb39a6
better visiblity on messages passing on a closed ws
rotemmiz a3c9e1e
WAT
rotemmiz c9c4089
only call currentStatus if ws is open
rotemmiz 9b51f72
increase verbosity for debug purposes
rotemmiz 3a082bc
per platfrom app data path
rotemmiz d76a43a
ensure file exists, create including path if needed
rotemmiz 3f14a38
always query applesimutils byOS
rotemmiz f1d8b48
Triggering PR build
yershalom c0393b0
print stderrs of after all retries failed
rotemmiz 86b1b8a
print stderrs of after all retries failed
rotemmiz 1084c20
Revert "Triggering PR build"
rotemmiz 87c357f
Trigger PR after fixed jenkins
yershalom e6743da
Revert "Trigger PR after fixed jenkins"
yershalom 0c73826
Last try trigger PR
yershalom 00cd14b
Revert "Last try trigger PR"
yershalom dd79249
Revert "Revert "Last try trigger PR""
rotemmiz d91bd9c
Add empty line to package.json for triggering build
yershalom f7923af
Merge branch 'master' into test-parallelization
rotemmiz c6c90f4
update xcode
rotemmiz 5830c96
Merge branch 'master' into test-parallelization
rotemmiz c6357ae
MOAR retries
rotemmiz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
{ | ||
"setupTestFrameworkScriptFile" : "./helpers/init.js", | ||
"bail": true, | ||
"verbose": true, | ||
"forceExit": true | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should validate Detox doesn't leave anything open, and that everything is being closed correctly. |
||
} |
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
const detox = require('detox'); | ||
const config = require('../../package.json').detox; | ||
jest.setTimeout(480000); | ||
|
||
beforeAll(async () => { | ||
await detox.init(config); | ||
}); |
File renamed without changes.
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should validate Detox doesn't leave anything open, and that everything is being closed correctly.
Please remove
forceExit