run setup-deps before running tests

grdsdev · grdsdev · commit b0ec5235dec8 · 2025-06-16T08:18:36.000-03:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -92,7 +92,6 @@ jobs:
       - name: Run Deno tests
         run: |
           cd test/deno
-          npm run setup-deps
           npm test || npm test
 
       - name: Stop Supabase
diff --git a/test/deno/package.json b/test/deno/package.json
@@ -2,7 +2,7 @@
   "name": "test-deno",
   "private": true,
   "scripts": {
-    "test": "deno test --allow-all --unstable-sloppy-imports integration.test.ts",
+    "test": "npm run setup-deps && deno test --allow-all --unstable-sloppy-imports integration.test.ts",
     "setup-deps": "node setup-deps.js"
   },
   "dependencies": {
diff --git a/test/integration.browser.test.ts b/test/integration.browser.test.ts
@@ -27,8 +27,10 @@ const content = `<html>
             const [realtimeStatus, setRealtimeStatus] = React.useState(null)
             const channel = supabase.channel('realtime:public:todos')
             React.useEffect(() => {
-                if (channel.state === 'closed') {
-                    channel.subscribe((status) => { if (status === 'SUBSCRIBED') setRealtimeStatus(status) })
+                channel.subscribe((status) => { if (status === 'SUBSCRIBED') setRealtimeStatus(status) })
+
+                return () => {
+                    channel.unsubscribe()
                 }
             }, [])
             if (realtimeStatus) {
