Merge pull request #1 from devalbo/cicd-dev

messing with test coverage

Author: devalbo

.github/workflows/pages.yml

@@ -29,7 +29,12 @@ jobs:
         run: npm run lint
 
       - name: Run unit tests
-        run: npm run test:run
+        run: npm run coverage:check
+
+      - name: CLI smoke test (Node)
+        env:
+          TB_SOLID_POD_DATA_PATH: ${{ runner.temp }}/tb-solid-pod-store.json
+        run: npm run cli:smoke
 
       - name: Generate JSON Schemas
         run: npm run generate:schemas

.gitignore

@@ -15,6 +15,8 @@ dist-ssr
 # Editor directories and files
 .vscode/*
 !.vscode/extensions.json
+!.vscode/settings.json
+!.vscode/tasks.json
 .idea
 .DS_Store
 *.suo
@@ -31,3 +33,6 @@ storybook-static
 test-results
 playwright-report
 playwright/.cache
+
+# Vitest coverage output
+coverage

.vscode/settings.json

@@ -0,0 +1,16 @@
+{
+  // Make editor diagnostics match CI scripts:
+  // - Lint: ESLint (npm run lint)
+  // - Types: TypeScript language service (same tsconfig as npm run typecheck)
+  "eslint.useFlatConfig": true,
+  "eslint.validate": ["javascript", "javascriptreact", "typescript", "typescriptreact"],
+  "eslint.workingDirectories": [{ "mode": "auto" }],
+  "eslint.run": "onType",
+  "editor.codeActionsOnSave": {
+    "source.fixAll.eslint": "never"
+  },
+  // Ensure VS Code uses the workspace TypeScript (same as tsc in scripts)
+  "typescript.tsdk": "node_modules/typescript/lib",
+  "typescript.enablePromptUseWorkspaceTsdk": true
+}
+

.vscode/tasks.json

@@ -0,0 +1,13 @@
+{
+  "version": "2.0.0",
+  "tasks": [
+    {
+      "label": "tb-solid-pod: check (lint + typecheck + tests)",
+      "type": "shell",
+      "command": "npm run check",
+      "group": "build",
+      "problemMatcher": []
+    }
+  ]
+}
+

README.md

@@ -230,6 +230,10 @@ exit                          Exit the CLI (Node terminal only)
 ## Testing
 
 - **Unit tests (Vitest):** `npm test` or `npm run test:run`; coverage: `npm run test:coverage`
+- **Coverage report:** `npm run test:coverage` generates:
+  - terminal summary
+  - HTML report at `coverage/index.html`
+  - lcov at `coverage/lcov.info` (for CI tooling)
 - **Storybook:** `npm run storybook` → http://localhost:6006 (component development). See [docs/testing/](docs/testing/README.md).
 - **BDD / E2E (Playwright):** Generate specs from Gherkin, then run Playwright:
   ```bash

eslint.config.js

@@ -8,7 +8,7 @@ import reactRefresh from 'eslint-plugin-react-refresh';
 import { defineConfig, globalIgnores } from 'eslint/config';
 
 export default defineConfig([
-  globalIgnores(['dist', '.features-gen']),
+  globalIgnores(['dist', '.features-gen', 'coverage']),
   // Node/Config files: process etc.
   {
     files: ['vite.config.js', 'vitest.config.ts', 'playwright.config.ts'],

package.json

@@ -40,14 +40,16 @@
     "test": "vitest",
     "test:run": "vitest run",
     "test:ui": "vitest --ui",
-    "test:coverage": "vitest --coverage",
+    "test:coverage": "vitest run --coverage",
+    "coverage:check": "vitest run --coverage --config vitest.coverage-check.config.ts",
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
     "test:e2e": "npx bddgen && playwright test",
     "test:e2e:headed": "npx bddgen && playwright test --headed",
     "test:e2e:terminal": "vitest run --config vitest.terminal-e2e.config.ts",
     "test:bdd": "npx bddgen && playwright test",
-    "cli": "tsx src/cli/run-node.tsx"
+    "cli": "tsx src/cli/run-node.tsx",
+    "cli:smoke": "node --import tsx src/cli/run-node.tsx help && node --import tsx src/cli/run-node.tsx pwd && node --import tsx src/cli/run-node.tsx ls"
   },
   "dependencies": {
     "@inrupt/vocab-common-rdf": "^1.0.5",

vitest.config.ts

@@ -11,20 +11,19 @@ export default defineConfig({
     exclude: ['tests/terminal/**', 'tests/e2e-cli-terminal.spec.ts'],
     coverage: {
       provider: 'v8',
-      reporter: ['text', 'json', 'html'],
+      reportsDirectory: './coverage',
+      reporter: ['text', 'json', 'json-summary', 'html', 'lcov'],
       include: ['src/**/*.{ts,tsx}'],
       exclude: [
+        // Demo app & UI components (coverage focuses on the library/CLI layers)
+        'src/App.tsx',
+        'src/components/**/*.{ts,tsx}',
+        'src/examples/**/*.{ts,tsx}',
         'src/**/*.stories.{ts,tsx}',
         'src/main.tsx',
         'src/index.ts',
         'src/**/*.d.ts',
       ],
-      thresholds: {
-        lines: 80,
-        functions: 80,
-        branches: 80,
-        statements: 80,
-      },
     },
   },
 })

vitest.coverage-check.config.ts

@@ -0,0 +1,22 @@
+import { defineConfig } from 'vitest/config';
+import base from './vitest.config';
+
+// CI/quality gate: enforce minimum coverage thresholds (global).
+export default defineConfig({
+  ...(base as unknown as Record<string, unknown>),
+  test: {
+    // @ts-expect-error - base config shape is compatible at runtime
+    ...(base.test ?? {}),
+    coverage: {
+      // @ts-expect-error - base config shape is compatible at runtime
+      ...(base.test?.coverage ?? {}),
+      thresholds: {
+        lines: 20,
+        functions: 20,
+        branches: 20,
+        statements: 20,
+      },
+    },
+  },
+});
+