Merge pull request #55 from datalust/feat/integration-tweaks

Adjust integration test strategy following TypeScript port

Author: nblumhardt

.github/workflows/ci.yml

@@ -0,0 +1,38 @@
+name: CI
+
+on:
+  push:
+    branches: [ dev ]
+  pull_request:
+    branches: [ dev ]
+
+jobs:
+  test-seq:
+    name: Build and Test
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [18.x, 20.x, 22.x]
+
+    services:
+      seq:
+        image: datalust/seq:latest
+        ports:
+          - 5341:80
+          - 5342:5342
+        env:
+          ACCEPT_EULA: Y
+          SEQ_FIRSTRUN_NOAUTHENTICATION: True
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v4
+      with:
+        node-version: ${{ matrix.node-version }}
+    - run: npm ci
+    - run: npm run build
+    - run: npm test
+    - run: npm run test:integration
+    - run: cd example && npm link && node example.js

.github/workflows/node.js.yml

@@ -1,6 +1,3 @@
-# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created
-# For more information see: https://help.github.com/actions/language-and-framework-guides/publishing-nodejs-packages
-
 name: Publish
 
 on:
@@ -31,4 +28,4 @@ jobs:
       - run: npm ci
       - run: npm publish
         env:
-          NODE_AUTH_TOKEN: ${{secrets.npm_token}}
+          NODE_AUTH_TOKEN: ${{secrets.npm_token}}

.github/workflows/npm-publish.yml

@@ -65,102 +65,6 @@ The `createStream()` function accepts a configuration object with the following
 - `batchSizeLimit` (number, optional): Maximum size of a batch
 - `onError` (function, optional): Error handler callback
 
-## Development
-
-### Running Tests
-
-The test suite includes:
-- **Integration tests** that verify round-trip logging to Seq by querying the API
-- **Example tests** that verify examples work as users would use them (via `npm link`)
-
-#### Test Modes
-
-Tests can run in two modes controlled by the `MOCK_SEQ` environment variable:
-
-**Real Seq Mode (default, `MOCK_SEQ=false`):**
-- Tests connect to actual Seq instance at `http://localhost:5341`
-- Verifies full end-to-end integration
-- Requires Seq running locally
-
-**Mock Mode (`MOCK_SEQ=true`):**
-- Tests use mock transport instead of real Seq
-- No Seq instance required
-- Verifies code paths without network calls
-- Example tests are skipped (they require real Seq)
-
-#### For Contributors (Local Development)
-
-**Option 1: Test with real Seq (recommended):**
-
-```bash
-# Start Seq container (with health check)
-npm run test:setup
-
-# Run tests with real Seq integration
-npm test
-
-# Stop Seq
-npm run test:teardown
-```
-
-**Option 2: Test with mocks (no Docker required):**
-
-```bash
-# Run tests in mock mode
-MOCK_SEQ=true npm test
-
-# Or on Windows PowerShell:
-$env:MOCK_SEQ="true"; npm test
-```
-
-**Option 3: Manage your own Seq instance:**
-
-```bash
-# Start Seq however you prefer (Docker, local install, etc.)
-# Ensure it's running on http://localhost:5341
-
-npm test
-```
-
-**Build only (no tests):**
-
-```bash
-npm run build  # Just compiles TypeScript
-```
-
-#### For CI/CD
-
-GitHub Actions runs tests in both modes:
-- **Mock mode**: Tests on Linux, Windows, macOS (9 jobs: 3 OS × 3 Node versions)
-- **Real Seq mode**: Tests on Linux only with Seq service container (3 jobs: 3 Node versions)
-
-This ensures cross-platform compatibility while maintaining full integration testing coverage.
-
-### Running the Example
-
-```bash
-# Start Seq
-npm run test:setup
-
-# Run the example
-npm start
-
-# View logs at http://localhost:5341
-
-# Stop Seq
-npm run test:teardown
-```
-
-### Building
-
-```bash
-npm run build
-```
-
-The build uses the TypeScript compiler (`tsc`) to produce ES Module output in the `dist/` directory with automatic type definitions.
-
-**Note:** This package is ESM-only (like its dependency `seq-logging`). If you need CommonJS support, please use version 2.x or earlier.
-
 ## Acknowledgements
 
 Originally by Simi Hartstein and published as `simihartstein/pino-seq`; maintainership transferred to Datalust at version 0.5.

.vscode/launch.json

@@ -19,10 +19,8 @@
   "scripts": {
     "build": "tsc",
     "prepublishOnly": "npm run build",
-    "test": "npm run build && mocha",
-    "test:setup": "docker pull datalust/seq:latest && docker compose -f test/docker-compose.yml up -d --wait",
-    "test:teardown": "docker compose -f test/docker-compose.yml down",
-    "start": "npm run build && node ./example/example.js",
+    "test": "npm run build && mocha --ignore test/integration*.js",
+    "test:integration": "npm run build && mocha --include test/integration*.js --timeout 10000",
     "dev": "tsc --watch"
   },
   "repository": {