test-inventory 0.2.0 → 0.3.0

package/dist/resolve.js

@@ -267,7 +267,12 @@ function workspacePackages(directory) {
         const workspaces = readJson(path.join(folder, 'package.json'))?.workspaces;
         const fromManifest = Array.isArray(workspaces) ? workspaces : record(workspaces)?.packages;
         const pnpm = readText(path.join(folder, 'pnpm-workspace.yaml'));
-        const patterns = (Array.isArray(fromManifest) ? fromManifest : pnpm ? pnpmWorkspacePatterns(pnpm) : null)?.filter((item) => typeof item === 'string');
+        // pnpm reads only pnpm-workspace.yaml, and npm, Yarn and Bun only package.json, so a repository that moved to
+        // pnpm may still have an old list there. Packages from both are found.
+        const lists = [Array.isArray(fromManifest) ? fromManifest : null, pnpm ? pnpmWorkspacePatterns(pnpm) : null];
+        const patterns = lists.some((list) => list !== null)
+            ? lists.flat().filter((item) => typeof item === 'string')
+            : null;
         if (patterns) {
             // Packages are inside the workspace. A pattern or a link that leads out of its folder could send the search through
             // the whole disk, so neither is followed.

package/dist/types.d.ts

@@ -49,6 +49,22 @@ export interface Inventory {
     tests: TestRecord[];
     diagnostics: Diagnostic[];
 }
+/**
+ * One line of the JSON Lines output (`--format jsonl`): the inventory with each record on its own line, marked by
+ * `type`. The `inventory` line comes first, then the suites, tests and diagnostics in the order of the JSON output,
+ * and the `summary` line last.
+ */
+export type InventoryLine = ({
+    type: 'inventory';
+} & Pick<Inventory, 'schemaVersion' | 'tool' | 'framework' | 'frameworkVersion'>) | ({
+    type: 'suite';
+} & SuiteRecord) | ({
+    type: 'test';
+} & TestRecord) | ({
+    type: 'diagnostic';
+} & Diagnostic) | ({
+    type: 'summary';
+} & Summary);
 /** @inline */
 export interface Tool {
     name: 'test-inventory';
@@ -219,7 +235,7 @@ export interface TestRecord {
     caseCount: Count | null;
     /** Declared inside a loop, `forEach` or `map`. */
     isInLoop: boolean;
-    /** Declared inside `if`, `?:`, `&&` or `switch`. */
+    /** Declared inside `if`, `?:`, `&&`, `switch` or a `catch` block. */
     isInCondition: boolean;
     /** Declared inside a plain function. */
     isInFunction: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "test-inventory",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Test metadata from source code, not from test results.",
   "keywords": [
     "testing",
@@ -71,7 +71,7 @@
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "format": "prettier --write . --ignore-path .gitignore --ignore-path .prettierignore --ignore-path .git/info/exclude",
-    "format:check": "prettier --check . --ignore-path .gitignore --ignore-path .prettierignore --ignore-path .git/info/exclude",
+    "format:check": "node scripts/format-check.ts",
     "docs:api": "typedoc",
     "docs:dev": "npm run docs:api && vitepress dev docs",
     "docs:build": "npm run docs:api && vitepress build docs",

package/schema/v1.json

@@ -730,7 +730,7 @@
         },
         "isInCondition": {
           "type": "boolean",
-          "description": "Declared inside `if`, `?:`, `&&` or `switch`."
+          "description": "Declared inside `if`, `?:`, `&&`, `switch` or a `catch` block."
         },
         "isInFunction": {
           "type": "boolean",