use node:test in postcss-tape (#1224)

Author: romainmenke

.github/bin/new-plugin.mjs

@@ -52,7 +52,7 @@ console.log(`- Creating new plugin ${pluginName}`);
 	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'src', 'index.ts'));
 	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'test', '_import.mjs'));
 	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'test', '_require.cjs'));
-	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, '.tape.mjs'));
+	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'test', '_tape.mjs'));
 	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'README.md'));
 	await replaceBasePluginReferencesForFilePath(path.join(pluginDir, 'INSTALL.md'));
 	await fsp.writeFile(

.github/workflows/test.yml

@@ -40,7 +40,9 @@ jobs:
 
       - name: npm ci
         run: |
-          npm ci --ignore-scripts
+          npm ci
+        env:
+          BROWSER_TESTS: true
         if: ${{ (matrix.is_base_node_version && matrix.is_base_os_version) }}
 
       - name: npm ci
@@ -85,14 +87,6 @@ jobs:
           NPM_CONFIG_ENGINE_STRICT=false npm install --ignore-scripts
           npm run test:cli --if-present $MODIFIED_WORKSPACES
 
-        # Browser Tests
-        # running "npm ci" again, but allowing scripts so that Chrome is installed
-      - name: test:browser
-        run: |
-          npm ci
-          npm run test:browser --if-present $MODIFIED_WORKSPACES
-        if: matrix.is_base_node_version && matrix.is_base_os_version
-
         # E2E Tests
       - name: test:e2e
         run: |

experimental/css-has-pseudo/package.json

@@ -56,8 +56,8 @@
 		"lint": "node ../../.github/bin/format-package-json.mjs",
 		"prepublishOnly": "npm run build && npm run test",
 		"stryker": "stryker run --logLevel error",
-		"test": "node .tape.mjs && node ./test/_import.mjs && node ./test/_require.cjs",
-		"test:rewrite-expects": "REWRITE_EXPECTS=true node .tape.mjs"
+		"test": "node --test",
+		"test:rewrite-expects": "REWRITE_EXPECTS=true node --test"
 	},
 	"homepage": "https://github.com/csstools/postcss-plugins/tree/main/experimental/css-has-pseudo#readme",
 	"repository": {

experimental/css-has-pseudo/.tape.mjs …perimental/css-has-pseudo/test/_tape.mjsexperimental/css-has-pseudo/.tape.mjs renamed to experimental/css-has-pseudo/test/_tape.mjs experimental/css-has-pseudo/.tape.mjs renamed to experimental/css-has-pseudo/test/_tape.mjs

@@ -1,9 +1,9 @@
 import { postcssTape } from '@csstools/postcss-tape';
 import plugin from '@csstools/css-has-pseudo-experimental';
 
-postcssTape(plugin)({
+postcssTape(plugin, { skipPackageNameCheck: true })({
 	'basic': {
 		message: 'supports basic usage',
-		warnings: 1
+		warnings: 1,
 	},
 });

experimental/postcss-gradient-stop-increments/package.json

@@ -63,8 +63,8 @@
 		"docs": "node ../../.github/bin/generate-docs/install.mjs && node ../../.github/bin/generate-docs/readme.mjs",
 		"lint": "node ../../.github/bin/format-package-json.mjs",
 		"prepublishOnly": "npm run build && npm run test",
-		"test": "node .tape.mjs && node ./test/_import.mjs && node ./test/_require.cjs",
-		"test:rewrite-expects": "REWRITE_EXPECTS=true node .tape.mjs"
+		"test": "node --test",
+		"test:rewrite-expects": "REWRITE_EXPECTS=true node --test"
 	},
 	"homepage": "https://github.com/csstools/postcss-plugins/tree/main/plugins/postcss-gradient-stop-increments#readme",
 	"repository": {

…stcss-gradient-stop-increments/.tape.mjs …-gradient-stop-increments/test/_tape.mjsexperimental/postcss-gradient-stop-increments/.tape.mjs renamed to experimental/postcss-gradient-stop-increments/test/_tape.mjs experimental/postcss-gradient-stop-increments/.tape.mjs renamed to experimental/postcss-gradient-stop-increments/test/_tape.mjs

@@ -3,15 +3,15 @@ import plugin from '@csstools/postcss-gradient-stop-increments-experimental';
 
 postcssTape(plugin)({
 	basic: {
-		message: "supports basic usage",
+		message: 'supports basic usage',
 	},
 	'examples/example': {
 		message: 'minimal example',
 	},
 	'examples/example:preserve-true': {
 		message: 'minimal example',
 		options: {
-			preserve: true
-		}
+			preserve: true,
+		},
 	},
 });

experimental/postcss-nesting/package.json

@@ -49,8 +49,8 @@
 		"docs": "node ../../.github/bin/generate-docs/install.mjs",
 		"lint": "node ../../.github/bin/format-package-json.mjs",
 		"prepublishOnly": "npm run build && npm run test",
-		"test": "node .tape.mjs && node ./test/_import.mjs && node ./test/_require.cjs",
-		"test:rewrite-expects": "REWRITE_EXPECTS=true node .tape.mjs"
+		"test": "node --test",
+		"test:rewrite-expects": "REWRITE_EXPECTS=true node --test"
 	},
 	"homepage": "https://github.com/csstools/postcss-plugins/tree/main/plugins/postcss-nesting#readme",
 	"repository": {

experimental/postcss-nesting/.tape.mjs …erimental/postcss-nesting/test/_tape.mjsexperimental/postcss-nesting/.tape.mjs renamed to experimental/postcss-nesting/test/_tape.mjs experimental/postcss-nesting/.tape.mjs renamed to experimental/postcss-nesting/test/_tape.mjs

@@ -9,10 +9,10 @@ const mixinPluginRule = () => {
 				node.replaceWith(postcss.parse('& .in{ &.deep { color: blue; }}', {from : 'mixin.css'}));
 			},
 		},
-	}
-}
+	};
+};
 
-mixinPluginRule.postcss = true
+mixinPluginRule.postcss = true;
 
 const mixinPluginDeclaration = () => {
 	return {
@@ -22,10 +22,10 @@ const mixinPluginDeclaration = () => {
 				node.replaceWith(postcss.parse('color: blue;', {from : 'mixin.css'}));
 			},
 		},
-	}
-}
+	};
+};
 
-mixinPluginDeclaration.postcss = true
+mixinPluginDeclaration.postcss = true;
 
 postcssTape(plugin)({
 	'basic': {
@@ -54,7 +54,7 @@ postcssTape(plugin)({
 	},
 	'invalid-selector': {
 		message: 'warns on invalid selectors',
-		warnings: 2
+		warnings: 2,
 	},
 	'media': {
 		message: 'supports nested @media',
@@ -80,11 +80,11 @@ postcssTape(plugin)({
 	},
 	'mixin-declaration': {
 		message: 'supports other visitors (mixin declaration)',
-		plugins: [mixinPluginDeclaration(), plugin()]
+		plugins: [mixinPluginDeclaration(), plugin()],
 	},
 	'mixin-rule': {
 		message: 'supports other visitors (mixin rule)',
-		plugins: [mixinPluginRule(), plugin()]
+		plugins: [mixinPluginRule(), plugin()],
 	},
 	'spec-examples': {
 		message: 'supports all spec examples',
@@ -98,5 +98,5 @@ postcssTape(plugin)({
 	},
 	'ampersand-everywhere': {
 		message: 'allow & to be used everywhere',
-	}
+	},
 });

knip.json

@@ -9,12 +9,9 @@
 		},
 		"packages/*": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"scripts/**/*.mjs",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs",
-				"test/test.mjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.{ts,js,mjs}",
@@ -23,10 +20,8 @@
 		},
 		"plugins/postcss-design-tokens": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.ts",
@@ -38,10 +33,8 @@
 		},
 		"plugins/postcss-global-data": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.ts",
@@ -53,15 +46,12 @@
 		},
 		"plugins/*": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"scripts/**/*.mjs",
 				"src/browser-global.js",
 				"src/browser.js",
 				"src/index.js",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs",
-				"test/test.mjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.{ts,js,mjs}",
@@ -70,15 +60,12 @@
 		},
 		"experimental/*": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"scripts/**/*.mjs",
 				"src/browser-global.js",
 				"src/browser.js",
 				"src/index.js",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs",
-				"test/test.mjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.{ts,js,mjs}",
@@ -87,11 +74,8 @@
 		},
 		"plugin-packs/postcss-bundler": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs",
-				"test/test.mjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.ts",
@@ -105,12 +89,9 @@
 		},
 		"plugin-packs/*": {
 			"entry": [
-				".tape.{cjs,mjs}",
 				"scripts/**/*.mjs",
 				"src/index.ts",
-				"test/_import.mjs",
-				"test/_require.cjs",
-				"test/test.mjs"
+				"test/**/*.{cjs,mjs}"
 			],
 			"project": [
 				"**/*.{ts,js,mjs}",