feat(firestore): add firestore recursive delete sample#10263
feat(firestore): add firestore recursive delete sample#10263alarconesparza wants to merge 4 commits intoGoogleCloudPlatform:mainfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a new Java sample for recursively deleting Firestore collections, including the necessary Maven configuration, an integration test, and a utility class. Feedback focuses on improving resource management by using try-with-resources for Firestore instances, ensuring test stability by correctly restoring System.out, and maintaining assertion consistency by using JUnit 5 throughout the integration test.
|
|
||
| package com.example.firebase; | ||
|
|
||
| import static org.junit.Assert.assertTrue; |
There was a problem hiding this comment.
|
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
Description
Fixes #
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
pom.xmlparent set to latestshared-configurationmvn clean verifyrequiredmvn -P lint checkstyle:checkrequiredmvn -P lint clean compile pmd:cpd-check spotbugs:checkadvisory only