-
-
Notifications
You must be signed in to change notification settings - Fork 53
chore: Add integration tests for Android #1146
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
Open
tustanivsky
wants to merge
46
commits into
main
Choose a base branch
from
chore/android-tests
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 30 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
7042674
Update sample app to receive input args on Android (required for test…
tustanivsky 6e34b25
Add Android integration tests to CI
tustanivsky 588aada
Comment crash capturing test
tustanivsky c82a55e
Fix test script
tustanivsky 6b14d0a
Fix minor issues
tustanivsky a7bb5e3
Rename vars
tustanivsky 7b3ab68
Clean up
tustanivsky a47a7ea
Better comments
tustanivsky 35c488b
Refactor
tustanivsky 297cf36
Fix workflow
tustanivsky d62218d
Fix
tustanivsky 35862ce
Simplify mechanism to pass input args to Android app
tustanivsky 3bc3f33
Clean up comments
tustanivsky 281af4a
Merge branch 'main' into chore/android-tests
tustanivsky c9079bc
Test SauceLabs
tustanivsky 957c826
Add separate SauceLabs test script
tustanivsky 11d997a
Fix log retrieval
tustanivsky f87da41
Fix
tustanivsky 4ea1b22
Rename script for running Android integration tests locally
tustanivsky d09d95e
Update readme
tustanivsky 89c6014
Revert CI tweaks for faster iterations
tustanivsky b99f9a6
Clean up
tustanivsky 4c7547b
Fix
tustanivsky f0bb43d
Fix typo
tustanivsky 5fea785
Limit android e2e tests to 2 parallel jobs
tustanivsky 9eeb769
Remove Pester install step as it is redundant
tustanivsky aeb2f78
Rename test scripts
tustanivsky c6777cc
Clean up
tustanivsky d03ecbb
Rework SauceLabs API requests
tustanivsky 30d3be2
Simplify logs retrieval
tustanivsky 532fa78
Remove unnecessary quotes
tustanivsky 3e09c20
Configure new SauceLabs credentials
tustanivsky 1c27be6
Change test device
tustanivsky 3d8a9c7
Change SauceLabs test job name title
tustanivsky 7dbcb78
Fix android test results upload check
tustanivsky e4a7195
Make SauceLabs device name configurable
tustanivsky e43d13f
Fix sample app
tustanivsky c218ae8
Try using app-runner Android providers for running tests
tustanivsky 25fe615
Fix param name
tustanivsky 69f3e65
Pass device name to SauceLabs provider
tustanivsky 0b5fa5f
Connect to SauceLabs device set in env var
tustanivsky 3b20dc8
Bump app-runner
tustanivsky c22b2c6
Merge android test scripts
tustanivsky 49d01ee
Remove platform-specific test scripts
tustanivsky e4d0fb4
Add SauceLabs session name override
tustanivsky 95d2fe6
Bump app-runner
tustanivsky 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
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or 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 hidden or 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,50 @@ | ||
| on: | ||
| workflow_call: | ||
| inputs: | ||
| unreal-version: | ||
| required: true | ||
| type: string | ||
|
|
||
| jobs: | ||
| integration-test: | ||
| name: Integration Test | ||
| runs-on: ubuntu-latest | ||
|
|
||
| env: | ||
| SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }} | ||
| SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }} | ||
| GITHUB_TOKEN: ${{ github.token }} | ||
| SAUCE_REGION: eu-central-1 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| with: | ||
| submodules: recursive | ||
|
|
||
| - name: Download sample build | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: UE ${{ inputs.unreal-version }} sample build (Android) | ||
| path: sample-build | ||
|
|
||
| - name: Run integration tests | ||
| id: run-integration-tests | ||
| shell: pwsh | ||
| working-directory: integration-test | ||
| env: | ||
| SENTRY_UNREAL_TEST_DSN: ${{ secrets.SENTRY_UNREAL_TEST_DSN }} | ||
| SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_API_TOKEN }} | ||
| SENTRY_UNREAL_TEST_APP_PATH: ${{ github.workspace }}/sample-build/SentryPlayground-arm64.apk | ||
| UNREAL_VERSION: ${{ inputs.unreal-version }} | ||
| run: | | ||
| cmake -B build -S . | ||
| Invoke-Pester Integration.Android.SauceLabs.Tests.ps1 -CI | ||
|
|
||
| - name: Upload integration test output | ||
| if: ${{ always() }} | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: UE ${{ inputs.unreal-version }} integration test output (Android) | ||
| path: | | ||
| integration-test/output/ | ||
| retention-days: 14 | ||
This file contains hidden or 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 hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.