Skip to content
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

feat: add Firebase test #6

Merged
merged 4 commits into from
Nov 16, 2022
Merged

feat: add Firebase test #6

merged 4 commits into from
Nov 16, 2022

Conversation

burmecia
Copy link
Member

What kind of change does this PR introduce?

This PR is to add test case for Firebase FDW. This test uses Firebase emulator as the backend and test data reading for auth users and docs.

What is the current behavior?

Currently there is no test case for Firebase FDW.

What is the new behavior?

Firebase FDW test case is added.

Additional context

N/A

@burmecia burmecia requested a review from olirice November 16, 2022 06:39
Copy link
Collaborator

@olirice olirice left a comment

Choose a reason for hiding this comment

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

looks great!

Comment on lines +14 to +17
clickhouse:
condition: service_healthy
firebase:
condition: service_healthy
Copy link
Collaborator

Choose a reason for hiding this comment

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

nice! didn't know about this

@burmecia burmecia merged commit 8aa7fc5 into main Nov 16, 2022
@burmecia burmecia deleted the feat/firebase-test branch November 16, 2022 23:52
kamyshdm pushed a commit to dymium-io/supabase-wrappers that referenced this pull request Jun 17, 2024
…tion

working Add Table

Former-commit-id: d533bc2b56c5bebaf5742cb4bc6430b8c98a8a0b [formerly 93c0d2d80603608bb894262e84c13a42be4dc847]
Former-commit-id: 2b80af2c5faf69a5d29c76fe0d9210dd8ea7b109
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants