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

chore(#337): fix e2e tests on CI #348

Merged
merged 30 commits into from
Mar 11, 2024
Merged

chore(#337): fix e2e tests on CI #348

merged 30 commits into from
Mar 11, 2024

Conversation

jkuester
Copy link
Contributor

@jkuester jkuester commented Feb 23, 2024

Closes #337

Upgrade our way out of the CI test failures. Uplifts all our androidTestImplementation dependencies to the latest versions. Also updates our GitHub actions jobs for running the tests to use Ubuntu and build against Android 33. This has the added benefit of making the tests run quite a bit faster for some reason...

…run (trust but verify...)"

This reverts commit 1b1b42b.
This reverts commit 7ae69bd.
Copy link
Contributor

@latin-panda latin-panda left a comment

Choose a reason for hiding this comment

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

Thanks Josh! It's good to go

@jkuester
Copy link
Contributor Author

For the record, I now believe that the main issue that was causing these tests to break was the WebView version being used in the emulated Android devices for the GH actions. The tests were originally running with Android 10. When I spin up an emulated version of Android 10 locally, it loads with webview 74.0.3729.185 which is below our minimum version for the CHT of 90. When I try to connect to a CHT instance with an emulated Android 10, I get the same infinite spinner of death as seems to be occurring on the GH actions.

So, I expect the upgrade to run with a newer version of Android was all that was needed for these tests, but I will retain the other changes here too since they are good to have as well!

@jkuester jkuester merged commit 5d7b775 into master Mar 11, 2024
6 checks passed
@jkuester jkuester deleted the 337_fix_ci branch March 11, 2024 13:18
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.

Login Test Broken
3 participants