feat(ci): add CI configuration and coverage tests for ArtoolkitPlugin

Author: kalwalt

.github/workflows/build.yml

@@ -0,0 +1,31 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - '**'
+  pull_request:
+    branches:
+      - '**'
+
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version-file: .nvmrc
+          cache: npm
+
+      - name: Install
+        run: npm ci
+
+      - name: Build
+        run: npm run build
+
+      - name: Test with coverage
+        run: npm run coverage

tests/plugin.more.spec.ts

@@ -0,0 +1,66 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { ArtoolkitPlugin } from '../src/plugin.js';
+import { createEventBus } from './setupTests';
+
+describe('ArtoolkitPlugin (more coverage)', () => {
+    let core: { eventBus: ReturnType<typeof createEventBus> };
+
+    beforeEach(() => {
+        core = { eventBus: createEventBus() };
+    });
+
+    it('disable() removes handlers and terminates worker', async () => {
+        const plugin = new ArtoolkitPlugin({ worker: true });
+        await plugin.init(core);
+
+        // Fake a browser worker with spies
+        const addEventListener = vi.fn();
+        const removeEventListener = vi.fn();
+        const terminate = vi.fn();
+        // @ts-ignore
+        plugin._worker = { addEventListener, removeEventListener, terminate, postMessage: vi.fn() };
+
+        await plugin.enable();
+        // Simulate that we added a message listener during start
+        expect(typeof plugin.enabled).toBe('boolean');
+
+        await plugin.disable();
+
+        expect(removeEventListener).toHaveBeenCalledWith('message', expect.any(Function));
+        expect(terminate).toHaveBeenCalledTimes(1);
+    });
+
+    it('engine:update falls back when postMessage throws', async () => {
+        const plugin = new ArtoolkitPlugin({ worker: true });
+        await plugin.init(core);
+
+        const postMessage = vi.fn(() => { throw new Error('boom'); });
+        // @ts-ignore
+        plugin._worker = { postMessage };
+
+        // No throw should propagate
+        // @ts-ignore call private
+        plugin._onEngineUpdate({ id: 99, imageBitmap: {} as ImageBitmap, width: 2, height: 2 });
+
+        // Fallback tries a second post without ImageBitmap, so we expect at least one call
+        expect(postMessage).toHaveBeenCalled();
+    });
+
+    it('getMarkerState returns null when marker not tracked', async () => {
+        const plugin = new ArtoolkitPlugin({ worker: false });
+        await plugin.init(core);
+        expect(plugin.getMarkerState(12345)).toBeNull();
+    });
+
+    it('detectionResult with no detections is safely ignored', async () => {
+        const plugin = new ArtoolkitPlugin({ worker: false });
+        await plugin.init(core);
+        await plugin.enable();
+
+        // @ts-ignore
+        plugin._onWorkerMessage({ data: { type: 'detectionResult', payload: {} } });
+
+        // No exception; no markers added
+        expect(plugin.getMarkerState(1)).toBeNull();
+    });
+});

vitest.config.ts

@@ -8,14 +8,12 @@ export default defineConfig({
             provider: 'v8',
             all: true,
             include: ['src/**/*.js'],
-            // Exclude files that are either pure re-exports or not yet unit-testable (worker)
             exclude: ['src/worker/**', 'src/index.js'],
             thresholds: {
-                // Slightly relaxed thresholds while we build out more tests
-                lines: 50,
-                statements: 50,
-                branches: 40,
-                functions: 50,
+                lines: 65,
+                statements: 65,
+                branches: 50,
+                functions: 65,
             },
         },
     },