jest-preset-stylelint 6.1.0 → 6.2.0

package/README.md

@@ -2,7 +2,7 @@
 
 [![NPM version](https://img.shields.io/npm/v/jest-preset-stylelint.svg)](https://www.npmjs.org/package/jest-preset-stylelint) [![Build Status](https://github.com/stylelint/jest-preset-stylelint/workflows/CI/badge.svg)](https://github.com/stylelint/jest-preset-stylelint/actions)
 
-[Jest](https://facebook.github.io/jest/) preset for [Stylelint](https://github.com/stylelint) plugins.
+[Jest](https://jestjs.io/) preset for [Stylelint](https://stylelint.io/) plugins.
 
 ## Installation
 
@@ -22,14 +22,16 @@ Add the preset to your `jest.config.js` or `jest` field in `package.json`:
 }
 ```
 
-Optionally, you can avoid specifying `plugins` in every schema by defining your own setup file to configure the `testRule` function. This is useful if you have many tests. There are two additional steps to do this:
+Optionally, you can avoid specifying `plugins` in every schema by defining your own setup file to configure the `testRule`/`testRuleConfigs` functions.
+This is useful if you have many tests. There are two additional steps to do this:
 
-1. Create `jest.setup.js` in the root of your project. Provide `plugins` option to `getTestRule()`:
+1. Create `jest.setup.js` in the root of your project. Provide `plugins` option to `getTestRule`/`getTestRuleConfigs`:
 
    ```js
    const { getTestRule } = require("jest-preset-stylelint");
 
    global.testRule = getTestRule({ plugins: ["./"] });
+   global.testRuleConfigs = getTestRuleConfigs({ plugins: ["./"] });
    ```
 
 2. Add `jest.setup.js` to your `jest.config.js` or `jest` field in `package.json`:
@@ -43,7 +45,13 @@ Optionally, you can avoid specifying `plugins` in every schema by defining your
 
 ## Usage
 
-The preset exposes a global `testRule` function that you can use to efficiently test your plugin using a schema.
+This preset exposes the following global functions as a helper.
+
+See also the [type definitions](index.d.ts) for more details.
+
+### `testRule`
+
+The `testRule` function enables you to efficiently test your plugin using a schema.
 
 For example, we can test a plugin that enforces and autofixes kebab-case class selectors:
 
@@ -104,9 +112,34 @@ testRule({
 });
 ```
 
-## Schema properties
+### `testRuleConfigs`
+
+The `testRuleConfigs` function enables you to test invalid configs for a rule.
+
+For example:
+
+```js
+testInvalidRuleConfigs({
+  plugins: ["."],
+  ruleName,
+
+  accept: [
+    {
+      config: "valid"
+    }
+  ],
 
-See the [type definitions](index.d.ts).
+  reject: [
+    {
+      config: "invalid"
+    },
+    {
+      config: [/invalid/],
+      description: "regex is not allowed"
+    }
+  ]
+});
+```
 
 ## [Changelog](CHANGELOG.md)
 

package/getTestRule.js

@@ -2,9 +2,6 @@
 
 const util = require('util');
 
-// @ts-expect-error -- TS2614: Module '"stylelint"' has no exported member 'lint'. Did you mean to use 'import lint from "stylelint"' instead?
-const { lint } = require('stylelint'); // eslint-disable-line node/no-unpublished-require -- Avoid auto-install of `stylelint` peer dependency.
-
 /**
  * @typedef {import('.').TestCase} TestCase
  * @typedef {import('.').TestSchema} TestSchema
@@ -13,6 +10,14 @@ const { lint } = require('stylelint'); // eslint-disable-line node/no-unpublishe
 /** @type {import('.').getTestRule} */
 module.exports = function getTestRule(options = {}) {
 	return function testRule(schema) {
+		/** @type {import('stylelint').lint} */
+		let lint;
+
+		beforeAll(() => {
+			// eslint-disable-next-line n/no-unpublished-require -- Avoid auto-install of `stylelint` peer dependency.
+			lint = require('stylelint').lint;
+		});
+
 		describe(`${schema.ruleName}`, () => {
 			const stylelintConfig = {
 				plugins: options.plugins || schema.plugins,

package/getTestRuleConfigs.js

@@ -0,0 +1,68 @@
+'use strict';
+
+const { inspect } = require('util');
+
+/** @type {import('.').getTestRuleConfigs} */
+module.exports = function getTestRuleConfigs(options = {}) {
+	return function testRuleConfigs({
+		ruleName,
+		accept = [],
+		reject = [],
+		only = false,
+		skip = false,
+		plugins = options.plugins,
+	}) {
+		if (accept.length === 0 && reject.length === 0) {
+			throw new TypeError('The either "accept" or "reject" property must not be empty');
+		}
+
+		/** @type {import('stylelint').lint} */
+		let lint;
+
+		beforeAll(() => {
+			// eslint-disable-next-line n/no-unpublished-require
+			lint = require('stylelint').lint;
+		});
+
+		const testGroup = only ? describe.only : skip ? describe.skip : describe;
+
+		testGroup(`${ruleName} configs`, () => {
+			/**
+			 * @param {import('.').ConfigCase} case
+			 * @param {(warnings: unknown[]) => void} comparison
+			 */
+			function testConfig({ config, description, only: onlyTest, skip: skipTest }, comparison) {
+				const testFn = onlyTest ? test.only : skipTest ? test.skip : test;
+
+				testFn(`${description || inspect(config)}`, async () => {
+					const lintConfig = {
+						plugins,
+						rules: { [ruleName]: config },
+					};
+					const { results } = await lint({ code: '', config: lintConfig });
+
+					expect(results).toHaveLength(1);
+					comparison(results[0].invalidOptionWarnings);
+				});
+			}
+
+			describe('accept', () => {
+				accept.forEach((c) => {
+					testConfig(c, (warnings) => {
+						// eslint-disable-next-line jest/no-standalone-expect
+						expect(warnings).toEqual([]);
+					});
+				});
+			});
+
+			describe('reject', () => {
+				reject.forEach((c) => {
+					testConfig(c, (warnings) => {
+						// eslint-disable-next-line jest/no-standalone-expect
+						expect(warnings).toEqual([{ text: expect.stringMatching(`"${ruleName}"`) }]);
+					});
+				});
+			});
+		});
+	};
+};

package/index.d.ts

@@ -156,6 +156,29 @@ export type TestRule = (schema: TestSchema) => void;
  */
 export function getTestRule(options?: { plugins?: TestSchema['plugins'] }): TestRule;
 
+export type ConfigCase = {
+	config: unknown;
+	description?: string;
+	only?: boolean;
+	skip?: boolean;
+};
+
+/**
+ * Test configurations for a rule.
+ */
+export type TestRuleConfigs = (
+	schema: Pick<TestSchema, 'ruleName' | 'plugins' | 'only' | 'skip'> & {
+		accept?: ConfigCase[];
+		reject?: ConfigCase[];
+	},
+) => void;
+
+/**
+ * Create a `testRuleConfigs()` function with any specified plugins.
+ */
+export function getTestRuleConfigs(options?: { plugins?: TestSchema['plugins'] }): TestRuleConfigs;
+
 declare global {
 	var testRule: TestRule;
+	var testRuleConfigs: TestRuleConfigs;
 }

package/jest-preset.js

@@ -2,5 +2,6 @@
 
 module.exports = {
 	setupFiles: [require.resolve('./jest-setup.js')],
+	setupFilesAfterEnv: [require.resolve('./jest-setup-after-env.js')],
 	testEnvironment: 'node',
 };

package/jest-setup-after-env.js

@@ -0,0 +1,18 @@
+'use strict';
+
+const os = require('os');
+
+const eolDescriptor = Object.getOwnPropertyDescriptor(os, 'EOL');
+
+if (!eolDescriptor) {
+	throw new TypeError('`os` must have an `EOL` property');
+}
+
+beforeAll(() => {
+	// NOTE: `jest.replaceProperty()` is unavailable for a read-only property.
+	Object.defineProperty(os, 'EOL', { ...eolDescriptor, value: '\n' });
+});
+
+afterAll(() => {
+	Object.defineProperty(os, 'EOL', eolDescriptor);
+});

package/jest-setup.js

@@ -1,8 +1,7 @@
 'use strict';
 
-// Mock should be before stylelint required. Even if it's required inside other modules
-jest.mock('stylelint/lib/utils/getOsEol', () => () => '\n');
-
-const getTestRule = require('./getTestRule');
+const getTestRule = require('./getTestRule.js');
+const getTestRuleConfigs = require('./getTestRuleConfigs.js');
 
 global.testRule = getTestRule();
+global.testRuleConfigs = getTestRuleConfigs();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jest-preset-stylelint",
-  "version": "6.1.0",
+  "version": "6.2.0",
   "description": "Jest preset for Stylelint plugins.",
   "keywords": [
     "stylelint",
@@ -14,9 +14,11 @@
   "main": "index.js",
   "types": "index.d.ts",
   "files": [
+    "getTestRuleConfigs.js",
     "getTestRule.js",
     "jest-preset.js",
     "jest-setup.js",
+    "jest-setup-after-env.js",
     "index.d.ts"
   ],
   "scripts": {
@@ -26,7 +28,7 @@
     "lint:js": "eslint .",
     "lint:md": "remark . --quiet --frail",
     "lint:types": "tsc",
-    "release": "np",
+    "release": "np --no-release-draft",
     "pretest": "npm run lint",
     "test": "jest",
     "watch": "jest --watch",
@@ -54,22 +56,26 @@
       "@stylelint/remark-preset"
     ]
   },
+  "jest": {
+    "preset": "./jest-preset.js",
+    "testRegex": ".*\\.test\\.js$"
+  },
   "devDependencies": {
-    "@stylelint/prettier-config": "^2.0.0",
+    "@stylelint/prettier-config": "^3.0.0",
     "@stylelint/remark-preset": "^4.0.0",
-    "@types/jest": "^29.4.0",
-    "eslint": "^8.34.0",
-    "eslint-config-stylelint": "^18.0.0",
-    "eslint-plugin-jest": "^27.2.1",
+    "@types/jest": "^29.5.3",
+    "eslint": "^8.46.0",
+    "eslint-config-stylelint": "^19.1.0",
+    "eslint-plugin-jest": "^27.2.3",
     "husky": "^8.0.3",
-    "jest": "^29.4.3",
-    "lint-staged": "^13.1.2",
-    "np": "^7.6.3",
+    "jest": "^29.6.2",
+    "lint-staged": "^13.2.3",
+    "np": "^8.0.4",
     "npm-run-all": "^4.1.5",
-    "prettier": "^2.8.4",
+    "prettier": "^3.0.1",
     "remark-cli": "^11.0.0",
-    "stylelint": "^15.1.0",
-    "typescript": "^4.9.5"
+    "stylelint": "^15.10.2",
+    "typescript": "^5.1.6"
   },
   "peerDependencies": {
     "jest": "^29.0.2"