ci: enforce full-suite checks and align test runner modes

Author: feedbee

.github/workflows/ci-tests.yml

@@ -0,0 +1,33 @@
+name: Test Quality Gates
+
+on:
+  pull_request:
+  push:
+    branches:
+      - main
+  workflow_dispatch:
+
+concurrency:
+  group: test-quality-${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  run-all-tests:
+    name: Run Full Test Suite
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run all tests
+        run: npm test

package.json

@@ -6,8 +6,11 @@
   "scripts": {
     "start": "node src/index.js",
     "dev": "NODE_ENV=development nodemon src/index.js",
-    "test": "MONGOMS_DOWNLOAD_DIR=/tmp/mongodb-binaries node --experimental-vm-modules node_modules/jest/bin/jest.js --testTimeout=30000",
-    "test:coverage": "MONGOMS_DOWNLOAD_DIR=/tmp/mongodb-binaries node --experimental-vm-modules node_modules/jest/bin/jest.js --testTimeout=30000 --coverage",
+    "test": "MONGOMS_DOWNLOAD_DIR=/tmp/mongodb-binaries node --experimental-vm-modules node_modules/jest/bin/jest.js --testTimeout=30000 --runInBand",
+    "test:coverage": "MONGOMS_DOWNLOAD_DIR=/tmp/mongodb-binaries node --experimental-vm-modules node_modules/jest/bin/jest.js --testTimeout=30000 --runInBand --coverage",
+    "test:basic": "npm run test -- --testPathIgnorePatterns=mongodb-storage.test.js",
+    "test:mongo": "npm run test -- src/__tests__/storage/mongodb-storage.test.js",
+    "test:coverage:basic": "npm run test:coverage -- --testPathIgnorePatterns=mongodb-storage.test.js",
     "migrate": "node scripts/migrate.js"
   },
   "dependencies": {

run-tests.sh

@@ -5,6 +5,91 @@ script_dir="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 repo_root="${script_dir}"
 devcontainer_exec="${script_dir}/scripts/devcontainer-exec.sh"
 
+usage() {
+  cat <<'USAGE'
+Usage: ./run-tests.sh [--mode <basic|mongo|all>] [--coverage]
+
+Options:
+  -m, --mode      Test mode to run. Default: all
+  -c, --coverage  Enable coverage for selected mode
+  -h, --help      Show this help message
+USAGE
+}
+
+mode="all"
+coverage="false"
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    -m|--mode)
+      if [[ $# -lt 2 ]]; then
+        echo "Error: --mode requires a value" >&2
+        usage
+        exit 1
+      fi
+      mode="$2"
+      shift 2
+      ;;
+    --mode=*)
+      mode="${1#*=}"
+      shift
+      ;;
+    -c|--coverage)
+      coverage="true"
+      shift
+      ;;
+    -h|--help)
+      usage
+      exit 0
+      ;;
+    basic|mongo|all)
+      mode="$1"
+      shift
+      ;;
+    *)
+      echo "Error: unknown argument '$1'" >&2
+      usage
+      exit 1
+      ;;
+  esac
+done
+
+case "${mode}" in
+  basic|mongo|all)
+    ;;
+  *)
+    echo "Error: invalid mode '${mode}'. Use basic, mongo, or all." >&2
+    usage
+    exit 1
+    ;;
+esac
+
+if [[ "${coverage}" == "true" ]]; then
+  case "${mode}" in
+    basic)
+      test_cmd=(npm run test:coverage:basic)
+      ;;
+    mongo)
+      test_cmd=(npm run test:coverage -- --runInBand src/__tests__/storage/mongodb-storage.test.js)
+      ;;
+    all)
+      test_cmd=(npm run test:coverage)
+      ;;
+  esac
+else
+  case "${mode}" in
+    basic)
+      test_cmd=(npm run test:basic)
+      ;;
+    mongo)
+      test_cmd=(npm run test:mongo)
+      ;;
+    all)
+      test_cmd=(npm test)
+      ;;
+  esac
+fi
+
 docker_available() {
   if ! command -v docker >/dev/null 2>&1; then
     return 1
@@ -15,23 +100,24 @@ docker_available() {
 
 # 1) If already inside any container, run tests directly.
 if [[ -f "/.dockerenv" ]]; then
-  exec npm test
+  exec "${test_cmd[@]}"
 fi
 
 # 2) If Docker is unavailable, run tests locally on host.
 if ! docker_available; then
-  exec npm test
+  exec "${test_cmd[@]}"
 fi
 
 # 3) If a devcontainer for this repo is running, run tests there.
 container_id="$("${devcontainer_exec}" --find-container-id 2>/dev/null || true)"
 if [[ -n "${container_id}" ]]; then
-  exec "${devcontainer_exec}" npm test
+  exec "${devcontainer_exec}" "${test_cmd[@]}"
 fi
 
 # 4) No devcontainer: run tests in a fresh disposable container (legacy behavior).
+cmd_string="$(printf '%q ' "${test_cmd[@]}")"
 exec docker run --rm -i \
   -v "${repo_root}:/app" \
   -w /app \
   node:latest \
-  sh -c "npm install && npm test"
+  sh -c "npm install && ${cmd_string}"