-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[Android] Introduces API 34 emulators for subset of tests #4820
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
Closed
Closed
Changes from 1 commit
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
dec3d69
Merge remote-tracking branch 'upstream/main' into camx_occ
camsim99 0e0333b
Merge remote-tracking branch 'upstream/main'
camsim99 bd7ac99
Merge remote-tracking branch 'upstream/main'
camsim99 5c3363b
Merge remote-tracking branch 'upstream/main'
camsim99 fed9621
Undo changes
camsim99 5aabe34
Merge remote-tracking branch 'upstream/main'
camsim99 2b9a352
Merge remote-tracking branch 'upstream/main'
camsim99 a1173da
Merge remote-tracking branch 'upstream/main'
camsim99 cbc3d6b
Merge remote-tracking branch 'upstream/main'
camsim99 cae5a4c
Merge remote-tracking branch 'upstream/main'
camsim99 72283db
Merge remote-tracking branch 'upstream/main'
camsim99 166a77c
Merge remote-tracking branch 'upstream/main'
camsim99 399780e
Merge remote-tracking branch 'upstream/main'
camsim99 8d5d0e7
Merge remote-tracking branch 'upstream/main'
camsim99 084d960
Merge remote-tracking branch 'upstream/main'
camsim99 d2a59ac
Merge remote-tracking branch 'upstream/main'
camsim99 a1422bf
Merge remote-tracking branch 'upstream/main'
camsim99 bdd87a6
Merge remote-tracking branch 'upstream/main'
camsim99 137a28b
Merge remote-tracking branch 'upstream/main'
camsim99 bc0db5a
Merge remote-tracking branch 'upstream/main'
camsim99 d04b466
Merge remote-tracking branch 'upstream/main'
camsim99 a9cfe87
Merge remote-tracking branch 'upstream/main'
camsim99 a32def1
Merge remote-tracking branch 'upstream/main'
camsim99 4785148
Merge remote-tracking branch 'upstream/main'
camsim99 7a8fc69
Merge remote-tracking branch 'upstream/main'
camsim99 b02e15f
Merge remote-tracking branch 'upstream/main'
camsim99 c6e5868
Merge remote-tracking branch 'upstream/main'
camsim99 0c0065a
Merge remote-tracking branch 'upstream/main'
camsim99 9dfe259
Merge remote-tracking branch 'upstream/main'
camsim99 bfcc0df
Merge remote-tracking branch 'upstream/main'
camsim99 7ee21ea
Bump tests to 34
camsim99 cc2957d
Bump linux android
camsim99 0026d0e
Try filtering packages
camsim99 0bc3706
Merge remote-tracking branch 'upstream/main' into test_34_1
camsim99 791dd78
Add filter script
camsim99 c687491
Add quick actions test fix
camsim99 d91007d
Add 6 back in
camsim99 4c628c7
Remove quick actions fix attempt, partially address review
camsim99 83afe66
Correct exclusions/inclusions
camsim99 89a968a
Remove exlude filter
camsim99 4430919
Path provider fails with 33, so put back to 34
camsim99 f39b0b0
Exclude webview
camsim99 cd977fe
Merge remote-tracking branch 'upstream/main' into test_34_1
camsim99 70c7aeb
Merge branch 'main' into test_34_1
camsim99 57117d8
Update still_requires_api_33_avd_drive_examples.yaml
camsim99 844c8cb
Add new tasks, reduce config files to one
camsim99 3861779
Merge branch 'test_34_1' of github.com:camsim99/packages into test_34_1
camsim99 a42b3e6
Fix comments
camsim99 ea10dde
Mark bringup true
camsim99 f4b839e
Merge remote-tracking branch 'upstream/main' into test_34_1
camsim99 685f810
Get rid of duplicate tasks
camsim99 5272d12
Update exclusions, delte unused files
camsim99 a764265
Fix integration test, try removing line that may be failing due to pe…
camsim99 c8883bc
Try using permissions, alternatively
camsim99 3cfd2b3
Remove experimental changes
camsim99 106db42
Merge remote-tracking branch 'upstream/main' into test_34_1
camsim99 7dab380
Undo my changes
camsim99 378c808
Add audio permission for music
camsim99 e3229a8
Add permissions and bump version
camsim99 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
Remove exlude filter
- Loading branch information
commit 89a968aeacb8e69f612ab0a85985fd5e77190f2b
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
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.
I would suggest having just one config file covering everything that needs an exclusion for any reason, and then passing that config file to every task. That way there's a clear and consistent separation between 33 and 34. Without that, it'll be tricky to avoid double-running some tests.