Skip to content

Clean up RemoteDocumentCacheTestCase #3194

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 1 commit into from
Nov 30, 2021
Merged

Conversation

schmidt-sebastian
Copy link
Contributor

Small PR that extracts common data elements from the tests and adds a new test case to test that we omit results from subcollections.

@google-cla google-cla bot added the cla: yes Override cla label Nov 30, 2021
Small PR that extracts common data elements from the tests.
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Nov 30, 2021

Coverage Report

Affected SDKs

  • firebase-firestore

    SDK overall coverage changed from 45.27% (97bac87) to 45.27% (512cacf8) by +0.00%.

    Filename Base (97bac87) Head (512cacf8) Diff
    DeleteMutation.java 90.00% 95.00% +5.00%

Test Logs

Notes

HTML coverage reports can be produced locally with ./gradlew <product>:checkCoverage.
Report files are located at <product-build-dir>/reports/jacoco/.

Head commit (512cacf8) is created by Prow via merging commits: 97bac87 6f14b12.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Nov 30, 2021

Binary Size Report

Affected SDKs

No changes between base commit (97bac87) and head commit (512cacf8).

Test Logs

Notes

Head commit (512cacf8) is created by Prow via merging commits: 97bac87 6f14b12.

@schmidt-sebastian schmidt-sebastian merged commit 1c7dd64 into master Nov 30, 2021
@schmidt-sebastian schmidt-sebastian deleted the mrschmidt/testcleanup branch November 30, 2021 23:35
schmidt-sebastian added a commit that referenced this pull request Dec 1, 2021
Small PR that extracts common data elements from the tests.
@firebase firebase locked and limited conversation to collaborators Dec 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Override cla size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants