fix: CI integration tests (#1009)

Author: closetcaiman

.github/workflows/tests.yml

@@ -5,16 +5,28 @@ on:
   workflow_dispatch:
 
 jobs:
-  tests:
+  cpp-tests:
     runs-on: ubuntu-latest
     steps:
-
       - name: Checkout
         uses: actions/checkout@v4
 
       - name: Setup
         uses: ./.github/actions/setup
 
-      - name: Run tests
-        run: yarn test
+      - name: Run C++ tests
+        working-directory: packages/react-native-audio-api
+        run: yarn test:cpp
+
+  js-tests:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup
+        uses: ./.github/actions/setup
 
+      - name: Run JS integration tests
+        working-directory: packages/react-native-audio-api
+        run: yarn test:js

package.json

@@ -16,9 +16,7 @@
     "format": "yarn workspaces foreach -A -p run format",
     "clean": "del-cli packages/**/android/build apps/**/android/build apps/**/android/app/build apps/**/ios/build packages/**/lib node_modules apps/**/node_modules packages/**/node_modules",
     "typecheck": "yarn workspaces foreach -A -p run typecheck",
-    "test": "bash packages/react-native-audio-api/common/cpp/test/RunTests.sh",
-    "test:graph": "bash packages/react-native-audio-api/common/cpp/test/RunTestsGraph.sh",
-    "test:graph:docker": "bash packages/react-native-audio-api/common/cpp/test/RunTestsGraphDocker.sh",
+    "test": "yarn workspace react-native-audio-api run test",
     "check-audio-enum-sync": "bash packages/react-native-audio-api/scripts/check-audio-events-sync.sh"
   },
   "devDependencies": {

packages/react-native-audio-api/.eslintrc.js

@@ -5,6 +5,13 @@ module.exports = {
     {
       files: ['./src/**/*.{ts,tsx}'],
     },
+    {
+      files: ['./tests/**/*.{ts,tsx}'],
+      parserOptions: {
+        project: './tsconfig.test.json',
+        tsconfigRootDir: __dirname,
+      },
+    },
   ],
-  ignorePatterns: ['lib', 'src/web-core/custom/signalsmithStretch' ],
+  ignorePatterns: ['lib', 'src/web-core/custom/signalsmithStretch'],
 };

packages/react-native-audio-api/common/cpp/test/RunTests.sh

@@ -9,7 +9,7 @@ cleanup() {
 
 trap cleanup EXIT
 
-cd packages/react-native-audio-api/common/cpp/test
+cd common/cpp/test
 
 cmake -S . -B build -Wno-dev
 

packages/react-native-audio-api/package.json

@@ -45,7 +45,12 @@
     "app.plugin.js"
   ],
   "scripts": {
-    "test": "jest",
+    "test": "yarn test:js && yarn test:cpp",
+    "test:js": "jest",
+    "test:cpp": "bash common/cpp/test/RunTests.sh",
+    "test:full": "yarn test && yarn test:graph",
+    "test:graph": "bash common/cpp/test/RunTestsGraph.sh",
+    "test:graph:docker": "bash common/cpp/test/RunTestsGraphDocker.sh",
     "typecheck": "tsc --noEmit",
     "lint": "yarn lint:js && yarn lint:cpp && yarn lint:ios && yarn lint:kotlin",
     "lint:js": "eslint src && yarn prettier --check src",

packages/react-native-audio-api/tests/integration.test.ts

@@ -231,8 +231,6 @@ describe('Mock Integration Tests', () => {
         streamPath: 'https://example.com/audio-stream',
       });
 
-      expect(streamer.streamPath).toBe('https://example.com/audio-stream');
-
       // Connect to output
       streamer.connect(context.destination);
 
@@ -242,17 +240,6 @@ describe('Mock Integration Tests', () => {
       streamer.resume();
       streamer.stop();
     });
-
-    it('should handle streamer initialization errors', () => {
-      const context = new MockAPI.AudioContext();
-      const streamer = context.createStreamer();
-
-      // First initialization should succeed
-      expect(streamer.initialize('http://stream1.com')).toBe(true);
-
-      // Second initialization should fail
-      expect(() => streamer.initialize('http://stream2.com')).toThrow();
-    });
   });
 
   describe('System Integration', () => {

packages/react-native-audio-api/tests/mock.test.ts

@@ -204,22 +204,11 @@ describe('React Native Audio API Mocks', () => {
 
     describe('StreamerNode', () => {
       it('should create a StreamerNode', () => {
-        const streamer = context.createStreamer();
+        const streamer = context.createStreamer({
+          streamPath: 'http://example.com/stream',
+        });
         expect(streamer).toBeInstanceOf(MockAPI.StreamerNode);
       });
-
-      it('should support initialization with stream path', () => {
-        const streamer = context.createStreamer();
-        const result = streamer.initialize('http://example.com/stream');
-        expect(result).toBe(true);
-        expect(streamer.streamPath).toBe('http://example.com/stream');
-      });
-
-      it('should throw error on duplicate initialization', () => {
-        const streamer = context.createStreamer();
-        streamer.initialize('http://example.com/stream');
-        expect(() => streamer.initialize('http://example.com/other')).toThrow();
-      });
     });
 
     describe('WorkletNodes', () => {

packages/react-native-audio-api/tsconfig.test.json

@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "include": ["src", "tests"]
+}