@systemfsoftware/oxlint-plugin-test-discipline 4.0.0 → 4.1.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @systemfsoftware/oxlint-plugin-test-discipline
 
+## 4.1.0
+
+### Minor Changes
+
+- The `contract` test suffix is retired. `test-suffix-outside-src` now reports a test file named with the `contract` suffix, and the behaviour-test rules (`behaviour-test-requires-gherkin`, `behaviour-one-feature-per-file`, `behaviour-exercises-use-case`, `no-pseudo-gherkin-unit-tests`) apply to the `integration` suffix alone. Rename a suite that drives a real medium to the `integration` suffix.
+
+## 4.0.1
+
+### Patch Changes
+
+- `in-source-test-targets-private`'s message now names an exported operation in the `it.prop` `subject` slot as a legal target for an in-source block, beside a private helper.
+
 ## 4.0.0
 
 ### Minor Changes

package/dist/index.mjs

@@ -81,6 +81,7 @@ const SANCTIONED_TEST_DIRS = /* @__PURE__ */ new Set(["tests"]);
 const NESTED_TEST_DIR = "__tests__";
 const TEST_TREE_DIRS = /* @__PURE__ */ new Set([...SANCTIONED_TEST_DIRS, NESTED_TEST_DIR]);
 const PROPERTY_SUFFIX = ".property.test.ts";
+const INTEGRATION_SUFFIX = ".integration.test.ts";
 const DIFFERENTIAL_SUFFIX = ".differential.test.ts";
 /**
 * Forbidden outright. A schema's laws are generated, so a hand-written
@@ -152,7 +153,7 @@ const isInTestsImportScope = (filename) => {
 };
 const isInConfiguredTestDir = (filename, dirs) => directoriesOf(filename).some((segment) => dirs.includes(segment));
 const isTestFile = (basename) => TEST_BASENAME.test(basename);
-const isBehaviourBasename = (basename) => basename.endsWith(".integration.test.ts") || basename.endsWith(".contract.test.ts");
+const isBehaviourBasename = (basename) => basename.endsWith(INTEGRATION_SUFFIX);
 const CELL_SUFFIX = /\.([^.]+)\.[cm]?tsx?$/;
 const cellOf = (basename) => CELL_SUFFIX.exec(basename)?.[1];
 //#endregion
@@ -915,12 +916,12 @@ const NOT_MODULE_LEVEL_EXPECTED = "the in-source test block as a direct statemen
 const NOT_MODULE_LEVEL_ACTUAL = "an `import.meta.vitest` block nested inside another statement";
 const NOT_MODULE_LEVEL_FIX = "move the block to module level — a nested block does not run under vitest includeSource";
 const NO_PRIVATE_TARGET_NAME = "an `import.meta.vitest` block touching no module-level binding";
-const NO_PRIVATE_TARGET_EXPECTED = "an in-source test exercising this module's own code — a private helper, or a module-level function the lawful prop's `subject` names";
+const NO_PRIVATE_TARGET_EXPECTED = "an in-source test exercising this module's own code — a private helper, or an exported operation named in the `it.prop` `subject` slot";
 const NO_PRIVATE_TARGET_ACTUAL = "an in-source block referencing only exported-call or imported names";
-const NO_PRIVATE_TARGET_FIX = "test the public surface from tests/ as *.integration.test.ts; in-source blocks exist for private helpers only — if the public behaviour you meant to cover is a pure function, delete the assertion: the type system already proves it";
+const NO_PRIVATE_TARGET_FIX = "touch this module. A private helper is referenced directly; an exported operation is named in the `it.prop` `subject` slot beside its declaration — that is the one home for its laws and the file whose mutants they run. What the block cannot hold is a behaviour binding no module-level name: test the public surface from tests/ as *.integration.test.ts instead";
 const meta$20 = {
 	type: "problem",
-	docs: { description: "In-source `if (import.meta.vitest)` blocks under src/ must be at module level and exercise at least one non-exported binding; other tests belong in tests/." },
+	docs: { description: "In-source `if (import.meta.vitest)` blocks under src/ must be at module level and exercise this module's own code — a private helper, or an exported operation named in the `it.prop` `subject` slot; a public-surface test belongs in tests/ as an integration test." },
 	schema: [],
 	messages: {
 		notModuleLevel: MESSAGE$8,
@@ -3105,9 +3106,9 @@ const testFileOutsideTestsDir = defineRule({
 });
 //#endregion
 //#region src/rules/test-suffix-outside-src.config.ts
-const UNSANCTIONED_SUFFIX_EXPECTED = "*.integration.test.ts, *.contract.test.ts, *.differential.test.ts, *.trace.test.ts, or *.conformance.test.ts";
+const UNSANCTIONED_SUFFIX_EXPECTED = "*.integration.test.ts, *.differential.test.ts, *.trace.test.ts, or *.conformance.test.ts";
 const UNSANCTIONED_SUFFIX_ACTUAL = "another test suffix outside src/";
-const UNSANCTIONED_SUFFIX_FIX = "public surface -> *.integration.test.ts; real-oracle medium -> *.contract.test.ts (contract lane); parity -> *.differential.test.ts; span graph -> *.trace.test.ts; concurrent or stateful behaviour -> *.conformance.test.ts; pure-cell property -> in-source; restated literal -> delete";
+const UNSANCTIONED_SUFFIX_FIX = "public surface or real medium -> *.integration.test.ts; parity -> *.differential.test.ts; span graph -> *.trace.test.ts; concurrent or stateful behaviour -> *.conformance.test.ts; pure-cell property -> in-source; restated literal -> delete";
 //#endregion
 //#region src/rules/test-suffix-outside-src.ts
 const testSuffixOutsideSrc = defineRule({

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@systemfsoftware/oxlint-plugin-test-discipline",
   "license": "Apache-2.0",
-  "version": "4.0.0",
+  "version": "4.1.0",
   "author": "Ryan Lee <drdgvhbh@gmail.com>",
   "repository": {
     "type": "git",
@@ -45,12 +45,12 @@
     "@microsoft/api-extractor": "^7.59.1",
     "@systemfsoftware/arethetypeswrong-cli": "^4.2.0",
     "@systemfsoftware/stryker-config": "^0.1.0",
-    "@systemfsoftware/stryker-ignorer-effect-schema-declarations": "^0.1.1",
-    "@systemfsoftware/stryker-ignorer-in-source-vitest-block": "^0.1.1",
-    "@systemfsoftware/stryker-js": "^10.1.1",
-    "@systemfsoftware/stryker-js-typescript-checker": "^7.0.5",
-    "@systemfsoftware/stryker-js-vitest-runner": "^7.1.1",
-    "@systemfsoftware/stryker-test-contribution": "^3.0.3",
+    "@systemfsoftware/stryker-ignorer-effect-schema-declarations": "latest",
+    "@systemfsoftware/stryker-ignorer-in-source-vitest-block": "latest",
+    "@systemfsoftware/stryker-js": "latest",
+    "@systemfsoftware/stryker-js-typescript-checker": "latest",
+    "@systemfsoftware/stryker-js-vitest-runner": "latest",
+    "@systemfsoftware/stryker-test-contribution": "latest",
     "@systemfsoftware/tsconfig": "^2.0.1",
     "@systemfsoftware/tsdown-config": "^0.1.0",
     "@systemfsoftware/vitest-config": "^0.1.0",
@@ -69,6 +69,7 @@
     "typescript": ">=5.0.0"
   },
   "publishConfig": {
+    "access": "public",
     "provenance": true
   },
   "scripts": {