Skip to content

test(firestore): update integration tests to use enterprise database#8371

Merged
wu-hui merged 1 commit into
mainfrom
fix-firestore-node-tests
May 28, 2026
Merged

test(firestore): update integration tests to use enterprise database#8371
wu-hui merged 1 commit into
mainfrom
fix-firestore-node-tests

Conversation

@wu-hui
Copy link
Copy Markdown
Contributor

@wu-hui wu-hui commented May 28, 2026

This PR configures the enterprise integration tests for the Node.js
Firestore SDK to use the enterprise database instead of test-db, and
runs them with the RUN_ENTERPRISE_TESTS=yes environment variable.

@wu-hui wu-hui requested a review from a team as a code owner May 28, 2026 14:46
@product-auto-label product-auto-label Bot added the api: firestore Issues related to the Firestore API. label May 28, 2026
@wu-hui wu-hui force-pushed the fix-firestore-node-tests branch from f7d0e68 to 84bd2b3 Compare May 28, 2026 14:48
gemini-code-assist[bot]

This comment was marked as outdated.

@wu-hui wu-hui force-pushed the fix-firestore-node-tests branch from 84bd2b3 to 566fcc3 Compare May 28, 2026 14:55
@wu-hui wu-hui requested a review from MarkDuckworth May 28, 2026 14:59
@wu-hui wu-hui force-pushed the fix-firestore-node-tests branch 4 times, most recently from 7fbec62 to bc7acbb Compare May 28, 2026 18:07
@wu-hui wu-hui added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 28, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 28, 2026
Comment thread handwritten/firestore/dev/test/util/helpers.ts Outdated
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

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

lgtm, with caveat about isRest() only checking if the env variable is set, but an individual firestore instance could override this.

@wu-hui wu-hui force-pushed the fix-firestore-node-tests branch from bc7acbb to 4fe53b5 Compare May 28, 2026 20:41
@wu-hui wu-hui force-pushed the fix-firestore-node-tests branch from 4fe53b5 to b922962 Compare May 28, 2026 20:42
@wu-hui wu-hui merged commit 4a075eb into main May 28, 2026
30 checks passed
@wu-hui wu-hui deleted the fix-firestore-node-tests branch May 28, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the Firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants