eslint-plugin-test-governance 0.11.0 → 0.11.1

package/CHANGELOG.md

@@ -6,6 +6,14 @@ All notable changes to this project are documented here. The format follows
 
 ## [Unreleased]
 
+## [0.11.1] - 2026-11-03
+
+### Fixed
+
+- A skip declared through `Function.prototype.call`, `apply` or `bind`, such as
+  `test.skip.call(null, 'places an order', fn)` or `test.skip.bind(null)('places an order', fn)`, is
+  checked like the plain call, so it needs its ticket.
+
 ## [0.11.0] - 2026-11-01
 
 ### Added
@@ -439,7 +447,8 @@ First release.
 - Support for ESLint 9 and 10 flat config, JavaScript and TypeScript test files, `.extend()`
   fixtures, describe blocks, runtime skips and tags in titles or the `tag` option.
 
-[Unreleased]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.11.0...HEAD
+[Unreleased]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.11.1...HEAD
+[0.11.1]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.11.0...v0.11.1
 [0.11.0]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.10.1...v0.11.0
 [0.10.1]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.10.0...v0.10.1
 [0.10.0]: https://github.com/Huldoser/eslint-plugin-test-governance/compare/v0.9.0...v0.10.0

package/dist/utils/analyze.js

@@ -312,6 +312,46 @@ function isLocalVariable(sourceCode, id) {
 }
 const STATEMENT_CONTAINERS = new Set(['Program', 'BlockStatement', 'StaticBlock', 'SwitchCase', 'TSModuleBlock']);
 /** The statement a call belongs to; its marker block sits above that statement. */
+/**
+ * The declaration a call makes through `Function.prototype.call`, `apply` or `bind`, read as if it were
+ * written plainly: `it.skip.call(null, 'title', fn)`, `it.skip.apply(null, ['title', fn])` and
+ * `it.skip.bind(null)('title', fn)` all declare the test that `it.skip('title', fn)` declares. A call that
+ * isn't read this way, such as an `apply` with a list this file can't see, comes back unchanged.
+ */
+function throughFunctionMethods(original) {
+    let call = original;
+    // `it.skip.bind(null).call(null, 'title', fn)` unwraps twice; two more steps cost nothing and end the loop.
+    for (let step = 0; step < 4; step += 1) {
+        const callee = unwrap(call.callee);
+        // `it.skip.call(null, ...)`: the first argument is the receiver the method is called on.
+        if (callee.type === 'MemberExpression' && memberName(callee) === 'call') {
+            call = { ...call, callee: callee.object, arguments: call.arguments.slice(1) };
+            continue;
+        }
+        // `it.skip.apply(null, ['title', fn])`: the arguments are the elements of an array written here.
+        if (callee.type === 'MemberExpression' && memberName(callee) === 'apply') {
+            const list = call.arguments.at(1);
+            const elements = list === undefined ? undefined : unwrap(list);
+            if (elements?.type !== 'ArrayExpression' ||
+                elements.elements.some((e) => e === null || e.type === 'SpreadElement')) {
+                return call;
+            }
+            call = { ...call, callee: callee.object, arguments: elements.elements.filter((e) => e !== null) };
+            continue;
+        }
+        // `it.skip.bind(null, 'title', fn)()` and `it.skip.bind(null)('title', fn)`: `bind` returns the test
+        // function with the arguments it was given already in place.
+        if (callee.type === 'CallExpression') {
+            const bind = unwrap(callee.callee);
+            if (bind.type !== 'MemberExpression' || memberName(bind) !== 'bind')
+                return call;
+            call = { ...call, callee: bind.object, arguments: [...callee.arguments.slice(1), ...call.arguments] };
+            continue;
+        }
+        return call;
+    }
+    return call;
+}
 function anchorOf(call) {
     let node = call;
     // A call always sits inside a statement container, so the walk stops before it runs out of parents.
@@ -1184,7 +1224,8 @@ function runAnalysis(sourceCode, options) {
             ? declare(call, declaration, bodyInfos(call.arguments, []))
             : undefined;
     };
-    const visitCall = (call) => {
+    const visitCall = (original) => {
+        const call = throughFunctionMethods(original);
         const callee = target(chainOf(call.callee, framework.factories));
         if (callee === undefined)
             return subtestOf(call) ?? pickedDeclaration(call) ?? { subject: runtimeCall(call) };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-test-governance",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "description": "ESLint rules that require a ticket comment above skipped, fixme, todo and tagged Playwright, Jest, Vitest, Mocha, Cypress, WebdriverIO, node:test and Jasmine tests.",
   "keywords": [
     "eslint",
@@ -52,7 +52,7 @@
     "build": "node -e \"require('node:fs').rmSync('dist',{recursive:true,force:true})\" && tsc -p tsconfig.build.json",
     "typecheck": "tsc -p tsconfig.json",
     "test": "node --test \"tests/**/*.test.ts\"",
-    "test:coverage": "node --test --experimental-test-coverage --test-coverage-include=\"src/**\" --test-coverage-branches=100 --test-coverage-functions=100 --test-coverage-lines=100 \"tests/**/*.test.ts\"",
+    "test:coverage": "node scripts/coverage.mjs",
     "test:pack": "node scripts/pack-smoke.mjs",
     "docs": "npm run build && eslint-doc-generator",
     "docs:check": "npm run build && eslint-doc-generator --check",