Skip to content
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

Stabilizing Detox Tests #3146

Merged
merged 59 commits into from
Oct 1, 2021
Merged

Stabilizing Detox Tests #3146

merged 59 commits into from
Oct 1, 2021

Conversation

cortisiko
Copy link
Member

Description
Scope:

getting Detox up and running on Bitrise

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

Issue

Resolves #???

@cortisiko cortisiko requested a review from a team as a code owner September 17, 2021 21:02
sethkfman and others added 27 commits September 17, 2021 17:02
Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽

@rickycodes rickycodes merged commit 72245ea into develop Oct 1, 2021
@rickycodes rickycodes deleted the detox/stabilization branch October 1, 2021 18:12
rickycodes pushed a commit that referenced this pull request Jan 31, 2022
* fix browser test

* added logic to provide min SDK version in gradle command

* updated e2e scripts to include minSDK being set to 26

* increased detox timeout, modified debug e2e build step to trigger yarn watch

* bumping detox version

* remove assertion for congrations view  after wallet import

* removing unwanted assertion

* updated ios to not release sentry sourcemaps on e2e tests

* enable sourcemaps for ios e2e build and added sentry properties for releaseE2E builds

* updated ios to not release sentry sourcemaps on e2e tests

* updated yarn commands to use the --pre flag for detox builds

* using release builds step for detox build command

* updated sentry prop logic and added echo's for testing

* updated logic to populate the auth token when athe sentry file is created

* changing emulator name

* increasing timeout and changing emulator name

* changing emulator name from "emulator" to "pixel"

* reverting emulator name for release build

* increasing wait time in wallet test

* added RAM increase for android emulator

* increasing emulator storage.

* attempting to fix on launch for android

* revert

* debugging wallet test on bitrise

* readding tests and adding video recording

* moving the analytics opt in test action earlier in tests

* moving the analytics opt in test action earlier in tests (wallet and start-exploring test

* updating yarn

* wallet test fix

* Merge in develop

* adding timeouts in wallet test

* adding a test step for protect your wallet modal.

* making larger test steps smaller.

* adding try catch to prevent flakiness in test.

* adding waits for bitrise.

* adding more waits

* making bitrise great.

* more changes to troubled tests

* fix wallet assertion

* add timeout to onboarding test

* add timeout for wallet test

* timeout for browser test

* remove exception

* increase wait time

* adding more waits

* fix browser test and onboarding wizard test.

* browser test fixes.

* removing test action in helper file

* readd helper test action

* increasing test timeout

* browser test was flakey because of duckduckgo ad. fixed that

* wait for onboarding test

* remove flakey test step in onboarding test

* onboarding opt out weird behavior.

Co-authored-by: sethkfman <10342624+sethkfman@users.noreply.github.com>
Co-authored-by: Cal Leung <cleun007@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants