Skip to content

Fix Android & iOS CI issues #1627

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.

Sign up for GitHub

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

Merged
denrase merged 5 commits into from
Sep 11, 2023
Merged

Fix Android & iOS CI issues #1627

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2569234
Use iOS 16.4 simulator to run native tests
denrase Sep 5, 2023
530a185
Update andorid emulator params
denrase Sep 5, 2023
4c175e6
change key
denrase Sep 5, 2023
4b7e80b
Run on arch x86_64
denrase Sep 5, 2023
1d825ed
fix working dir
denrase Sep 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Run on arch x86_64
  • Loading branch information
@denrase
denrase committed Sep 5, 2023
commit 4b7e80ba3a7d3680f2c1ddb2b54863c5c800c036
12 changes: 6 additions & 6 deletions .github/workflows/flutter_test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,9 @@ jobs:
working-directory: ./flutter/example
api-level: 31
profile: Nexus 6
arch: arm64-v8a
arch: x86_64
force-avd-creation: false
avd-name: macOS-avd-arm64-v8a-31
avd-name: macOS-avd-x86_64-31
emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
disable-animations: true
script: echo 'Generated AVD snapshot for caching.'
Expand All @@ -85,9 +85,9 @@ jobs:
working-directory: ./flutter/example
api-level: 31
Copy link
Contributor

@buenaflor buenaflor Sep 5, 2023

Choose a reason for hiding this comment

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

Won't it work with api-level 21 anymore?

Copy link
Collaborator Author

@denrase denrase Sep 11, 2023

Choose a reason for hiding this comment

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

We still have a test where we check build with min version. Figured the more recent emulators might give us better performance and stability.

profile: Nexus 6
arch: arm64-v8a
arch: x86_64
force-avd-creation: false
avd-name: macOS-avd-arm64-v8a-31
avd-name: macOS-avd-x86_64-31
emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
disable-animations: true
script: ./gradlew testDebugUnitTest
Expand All @@ -98,9 +98,9 @@ jobs:
working-directory: ./flutter/example
api-level: 31
profile: Nexus 6
arch: arm64-v8a
arch: x86_64
force-avd-creation: false
avd-name: macOS-avd-arm64-v8a-31
avd-name: macOS-avd-x86_64-31
emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
disable-animations: true
script: flutter test integration_test/integration_test.dart --verbose
Expand Down