firebase / firebase-cpp-sdk

Firebase C++ SDK
http://firebase.google.com
Apache License 2.0
278 stars 114 forks source link

Improve --auto_diff mode for when GMA is the only changed SDK. #1557

Closed jonsimantov closed 7 months ago

jonsimantov commented 8 months ago

Description

Provide details of the change, and generalize the change in the PR title above.

If Android/firebase_dependencies.gradle and ios_pod/Podfile only have changed lines referencing GMA or UMP packages, then allow auto_diff to only specify GMA.

This should greatly speed up "quick" tests when updating only GMA dependencies.


Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.


Type of Change

Place an x the applicable box:

Notes

github-actions[bot] commented 7 months ago

❌  Integration test FAILED

Requested by @jonsimantov on commit d05287ef9d39e29a317202f91eee22ecbbf2143f Last updated: Tue Mar 26 18:43 PDT 2024 View integration test log & download artifacts Failures Configs
firestore [TEST] [FAILURE] [Windows] [x64] [openssl]
(1 failed tests)  ValidationTest.QueriesCannotBeSortedByAnUncommittedServerTimestamp
[TEST] [FLAKINESS] [Android] [1/3 os: macos] [1/2 android_device: android_target]
(1 failed tests)  QueryNetworkTest.EnableDisableNetwork

Add flaky tests to go/fpl-cpp-flake-tracker