Skip to content

test(firestore): fix broken parent expression test#16018

Merged
andrewheard merged 1 commit intomainfrom
dl/parent-test-fix
Apr 1, 2026
Merged

test(firestore): fix broken parent expression test#16018
andrewheard merged 1 commit intomainfrom
dl/parent-test-fix

Conversation

@dlarocque
Copy link
Copy Markdown
Contributor

This test is failing in CI. The pipeline in the test has no input documentats to run against, because the randomCol is empty, despite calling setData to create a document deeper in the collection. This fixes the test by adding a dummy document to the randomCol collection.

This test is failing in CI. The pipeline in the test has no input
documentats to run against, because the `randomCol` is empty, despite
calling `setData` to create a document deeper in the collection. This fixes
the test by adding a dummy document to the `randomCol` collection.
@dlarocque dlarocque requested a review from andrewheard March 31, 2026 23:27
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@dlarocque dlarocque changed the title test(firestore): fix broken parent() expression test test(firestore): fix broken parent expression test Mar 31, 2026
@danger-firebase-ios
Copy link
Copy Markdown

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

Generated by 🚫 Danger

@andrewheard andrewheard added this to the 12.12.0 - M179 milestone Apr 1, 2026
Copy link
Copy Markdown
Contributor

@andrewheard andrewheard left a comment

Choose a reason for hiding this comment

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

Thank you for resolving this so quickly, @dlarocque!

@andrewheard andrewheard merged commit dbc9149 into main Apr 1, 2026
74 checks passed
@andrewheard andrewheard deleted the dl/parent-test-fix branch April 1, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants