Skip to content

Setup CI for named db tests #5206

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

Merged
merged 9 commits into from
Jul 31, 2023
Merged

Setup CI for named db tests #5206

merged 9 commits into from
Jul 31, 2023

Conversation

wu-hui
Copy link
Contributor

@wu-hui wu-hui commented Jul 27, 2023

Example run of the new job: https://github.com/firebase/firebase-android-sdk/actions/runs/5684374125/job/15407013872?pr=5206

This PR has two parts:

  • Setup a new job to run Firestore tests against a named database: test-db as opposed to the default one.
  • Fix tests that fail with new test-db tests due to asserts that are too restrictive.

@github-actions
Copy link
Contributor

Release note changes

No release note changes were detected. If you made changes that should be
present in the next release, ensure you've added an entry in the appropriate
CHANGELOG.md file(s).

@google-oss-bot
Copy link
Contributor

1 Warning
⚠️ Did you forget to add a changelog entry? (Add the 'no-changelog' label to the PR to silence this warning.)

Generated by 🚫 Danger

@github-actions
Copy link
Contributor

github-actions bot commented Jul 27, 2023

Unit Test Results

   820 files  +   656     820 suites  +656   40m 31s ⏱️ + 37m 50s
4 995 tests +3 793  4 974 ✔️ +3 788  21 💤 +  5  0 ±0 
9 899 runs  +7 495  9 857 ✔️ +7 485  42 💤 +10  0 ±0 

Results for commit 8b5c050. ± Comparison against base commit 622d65b.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jul 27, 2023

Coverage Report 1

Affected Products

  • firebase-database

    Overall coverage changed from 50.22% (622d65b) to 50.15% (cfba03d) by -0.07%.

    FilenameBase (622d65b)Merge (cfba03d)Diff
    ChildChangeAccumulator.java96.67%83.33%-13.33%
    DoubleNode.java100.00%88.24%-11.76%
  • firebase-firestore

    Overall coverage changed from 44.63% (622d65b) to 44.62% (cfba03d) by -0.01%.

    FilenameBase (622d65b)Merge (cfba03d)Diff
    DeleteMutation.java95.24%90.48%-4.76%
    PatchMutation.java100.00%98.39%-1.61%
  • firebase-installations

    Overall coverage changed from ? (622d65b) to 58.61% (cfba03d) by ?.

    22 individual files with coverage change

    FilenameBase (622d65b)Merge (cfba03d)Diff
    AutoValue_InstallationResponse.java?58.93%?
    AutoValue_PersistedInstallationEntry.java?61.18%?
    AutoValue_TokenResult.java?45.65%?
    AwaitListener.java?0.00%?
    Clock.java?0.00%?
    CrossProcessLock.java?51.85%?
    FirebaseInstallations.java?96.62%?
    FirebaseInstallationServiceClient.java?4.80%?
    FirebaseInstallationsException.java?78.57%?
    FirebaseInstallationsRegistrar.java?100.00%?
    GetAuthTokenListener.java?100.00%?
    GetIdListener.java?70.00%?
    IidStore.java?22.39%?
    InstallationResponse.java?100.00%?
    PersistedInstallation.java?97.01%?
    PersistedInstallationEntry.java?100.00%?
    RandomFidGenerator.java?20.00%?
    RequestLimiter.java?100.00%?
    StateListener.java?0.00%?
    SystemClock.java?100.00%?
    TokenResult.java?100.00%?
    Utils.java?95.00%?

  • firebase-messaging

    Overall coverage changed from 84.29% (622d65b) to 84.18% (cfba03d) by -0.11%.

    FilenameBase (622d65b)Merge (cfba03d)Diff
    Metadata.java41.27%36.51%-4.76%
  • firebase-perf

    Overall coverage changed from 71.20% (622d65b) to 71.19% (cfba03d) by -0.02%.

    FilenameBase (622d65b)Merge (cfba03d)Diff
    RateLimiter.java91.54%90.77%-0.77%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/V2OtZhJwqs.html

@wu-hui wu-hui force-pushed the wuandy/TestNamedDb branch from 7443794 to 9704f53 Compare July 27, 2023 18:13
@wu-hui wu-hui force-pushed the wuandy/TestNamedDb branch from 85bd6ba to 776d97c Compare July 27, 2023 18:52
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jul 27, 2023

Size Report 1

Affected Products

  • firebase-firestore

    TypeBase (622d65b)Merge (cfba03d)Diff
    aar1.36 MB1.36 MB+32 B (+0.0%)
    apk (release)3.95 MB3.95 MB-68 B (-0.0%)
  • firebase-firestore-ktx

    TypeBase (622d65b)Merge (cfba03d)Diff
    apk (release)4.34 MB4.34 MB-140 B (-0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/dOTnUSHak1.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jul 27, 2023

Startup Time Report 1

The report is too large (122,628 chars) to be displayed on GitHub. Please check this report on GCS.

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/YnbO3hLmtK/index.html

@@ -70,6 +70,12 @@ android {
def targetBackend = findProperty("targetBackend") ?: "emulator"
buildConfigField("String", "TARGET_BACKEND", "\"$targetBackend\"")

if (findProperty('targetDatabaseId')) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Alternatively

def targetDatabaseId = findProperty('targetDatabaseId') ?: "(default)"
buildConfigField("String", "TARGET_DATABASE_ID", "\"$targetDatabaseId\"")

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@rlazo rlazo merged commit 8ce2dfc into master Jul 31, 2023
@rlazo rlazo deleted the wuandy/TestNamedDb branch July 31, 2023 20:01
davidmotson pushed a commit that referenced this pull request Aug 3, 2023
* Initial setup

* fix gradle file

* separate job

* Control

* Address Feedback.
@firebase firebase locked and limited conversation to collaborators Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants