Skip to content

feat: nodejs integration tests#1628

Closed
edgurgel wants to merge 2 commits into
mainfrom
feat/nodejs-integration-tests
Closed

feat: nodejs integration tests#1628
edgurgel wants to merge 2 commits into
mainfrom
feat/nodejs-integration-tests

Conversation

@edgurgel

Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

  • Add postgres changes tests to deno integration tests
  • Add broadcast integration tests to the new nodejs integration tests.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 88.081% (+0.07%) from 88.007%
when pulling 02165bc on feat/nodejs-integration-tests
into 253b6cc on main.

@filipecabaco

Copy link
Copy Markdown
Member

tackled by new e2e

@filipecabaco
filipecabaco deleted the feat/nodejs-integration-tests branch April 24, 2026 17:43
@edgurgel

Copy link
Copy Markdown
Member Author

But that's using bun right?

@filipecabaco

Copy link
Copy Markdown
Member

yes but we have integration tests for nodejs in supabase-js

@edgurgel

Copy link
Copy Markdown
Member Author

Yeah but the tests here are for the main branch not whatever the supabase cli has as stable/latest. It's a way to check if any backend change caused issues with existing libraries.

We already run deno tests this PR was to add the nodejs version of it: https://github.com/supabase/realtime/blob/main/test/integration/tests.ts

@filipecabaco

Copy link
Copy Markdown
Member

got it let's reopen then

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.

3 participants