test: migrate to built-in Node.js test runner (#1611)

* test: migrate to built-in Node.js test runner * test: drop separate helpers testing step Since minimum requirements are Node.js 18, helpers should be able to run together.
cssnano · Apr 27, 2024 · 6f134fc · 6f134fc
1 parent f5d6f33
commit 6f134fc
Show file tree

Hide file tree

Showing 100 changed files with 316 additions and 474 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -29,26 +29,6 @@ jobs:
       - name: Run lint
         run: pnpm lint
 
-  test_helpers:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4.1.3
-      - name: Install pnpm
-        uses: pnpm/action-setup@v3.0.0
-        with:
-          version: 9.0.6
-      - name: Install Node.js 20.x
-        uses: actions/setup-node@v4.0.2
-        with:
-          node-version: '20'
-          cache: 'pnpm'
-
-      - name: Install dependencies
-        run: pnpm install --frozen-lockfile
-
-      - name: Test helpers
-        run: pnpm test:helpers
-
   test:
     needs: lint
     runs-on: ${{ matrix.os }}

diff --git a/package.json b/package.json
@@ -6,10 +6,9 @@
     "fixlint": "prettier --write . && pnpm lint -- --fix",
     "build:integration": "node ./util/buildFrameworks.mjs",
     "pretest": "pnpm lint",
-    "test:only": "uvu packages \"test.*\\.js$\"",
+    "test:only": "node --test",
     "test:coverage": "c8 pnpm test:only",
     "test": "pnpm test:coverage",
-    "test:helpers": "uvu packages \"test.*\\.mjs$\"",
     "types": "tsc -b",
     "all-publish": "pnpm changeset publish"
   },
@@ -29,8 +28,7 @@
     "postcss": "^8.4.38",
     "postcss-font-magician": "^4.0.0",
     "prettier": "^3.2.5",
-    "typescript": "~5.4.5",
-    "uvu": "^0.5.6"
+    "typescript": "~5.4.5"
   },
   "browserslist": {
     "production": [

diff --git a/packages/cssnano-preset-advanced/test/autoprefixer.js b/packages/cssnano-preset-advanced/test/autoprefixer.js
@@ -1,5 +1,5 @@
 'use strict';
-const { test } = require('uvu');
+const { test } = require('node:test');
 const {
   processCSSWithPresetFactory,
 } = require('../../../util/integrationTestHelpers.js');
@@ -38,4 +38,3 @@ test(
   'should not remove outdated vendor prefixes if excluded',
   exclude('h1{-webkit-box-sizing:content-box;box-sizing:content-box}')
 );
-test.run();
diff --git a/packages/cssnano-preset-advanced/test/integrations.js b/packages/cssnano-preset-advanced/test/integrations.js
@@ -1,7 +1,7 @@
 'use strict';
 const { join } = require('path');
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const {
   integrationTests,
   loadPreset,
@@ -51,11 +51,10 @@ function excludeProcessor(options) {
     loadPreset(preset(options))
       .process(input, { from: undefined })
       .then(({ css }) => {
-        assert.is(css, input);
+        assert.strictEqual(css, input);
       });
 }
 
 test('exclude zindex', excludeProcessor({ zindex: false }));
 
 test('exclude zindex #1', excludeProcessor({ zindex: { exclude: true } }));
-test.run();
diff --git a/packages/cssnano-preset-default/test/css-declaration-sorter.js b/packages/cssnano-preset-default/test/css-declaration-sorter.js
@@ -1,5 +1,5 @@
 'use strict';
-const { test } = require('uvu');
+const { test } = require('node:test');
 const {
   processCSSWithPresetFactory,
 } = require('../../../util/integrationTestHelpers.js');
@@ -149,4 +149,3 @@ test(
     'a{border:0;border-top:3px solid}'
   )
 );
-test.run();
diff --git a/packages/cssnano-preset-default/test/integrations.js b/packages/cssnano-preset-default/test/integrations.js
@@ -1,7 +1,7 @@
 'use strict';
 const { join } = require('path');
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const {
   integrationTests,
   loadPreset,
@@ -45,11 +45,10 @@ function excludeProcessor(options) {
     loadPreset(preset(options))
       .process(input, { from: undefined })
       .then(({ css }) => {
-        assert.is(css, input);
+        assert.strictEqual(css, input);
       });
 }
 
 test('exclude colormin', excludeProcessor({ colormin: false }));
 
 test('exclude colormin #1', excludeProcessor({ colormin: { exclude: true } }));
-test.run();
diff --git a/packages/cssnano-preset-lite/test/index.js b/packages/cssnano-preset-lite/test/index.js
@@ -1,5 +1,5 @@
 'use strict';
-const { test } = require('uvu');
+const { test } = require('node:test');
 const {
   processCSSWithPresetFactory,
 } = require('../../../util/integrationTestHelpers.js');
@@ -64,4 +64,3 @@ test(
     ''
   )
 );
-test.run();
diff --git a/packages/cssnano-preset-lite/test/integrations.js b/packages/cssnano-preset-lite/test/integrations.js
@@ -1,10 +1,9 @@
 'use strict';
-const { test } = require('uvu');
+const { test } = require('node:test');
 const { integrationTests } = require('../../../util/integrationTestHelpers.js');
 const preset = require('..');
 
 test(
   'should correctly handle the framework tests',
   integrationTests(preset, `${__dirname}/integrations`)
 );
-test.run();
diff --git a/packages/cssnano-utils/test/getArguments.test.js b/packages/cssnano-utils/test/getArguments.test.js
@@ -1,13 +1,13 @@
 'use strict';
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const valueParser = require('postcss-value-parser');
 const getArguments = require('../src/getArguments.js');
 
 test('should get arguments', () => {
   const parsed = valueParser('linear-gradient(to bottom left, red, blue)');
 
-  assert.equal(getArguments(parsed.nodes[0]), [
+  assert.deepStrictEqual(getArguments(parsed.nodes[0]), [
     [
       {
         type: 'word',
@@ -58,4 +58,3 @@ test('should get arguments', () => {
     ],
   ]);
 });
-test.run();
diff --git a/packages/cssnano-utils/test/sameParent.test.js b/packages/cssnano-utils/test/sameParent.test.js
@@ -1,6 +1,6 @@
 'use strict';
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const sameParent = require('../src/sameParent.js');
 
@@ -11,7 +11,7 @@ test('should calculate same parent', () => {
       const h1 = result.root.nodes[0];
       const h2 = result.root.nodes[1];
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -25,7 +25,7 @@ test('should calculate same parent (detached nodes)', () => {
       h1.remove();
       h2.remove();
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -39,7 +39,7 @@ test('should calculate same parent (at rules)', () => {
       const h1 = result.root.nodes[0].nodes[0];
       const h2 = result.root.nodes[0].nodes[1];
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -53,7 +53,7 @@ test('should calculate same parent (multiple at rules)', () => {
       const h1 = result.root.nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0];
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -67,7 +67,7 @@ test('should calculate same parent (multiple at rules (uppercase))', () => {
       const h1 = result.root.nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0];
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -92,7 +92,7 @@ test('should calculate same parent (nested at rules)', () => {
       const h1 = result.root.nodes[0].nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0].nodes[0];
 
-      assert.is(sameParent(h1, h2), true);
+      assert.strictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -117,7 +117,7 @@ test('should calculate not same parent (nested at rules)', () => {
       const h1 = result.root.nodes[0].nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0].nodes[0];
 
-      assert.is.not(sameParent(h1, h2), true);
+      assert.notStrictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -142,7 +142,7 @@ test('should calculate not same parent (nested at rules) (2)', () => {
       const h1 = result.root.nodes[0].nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0].nodes[0];
 
-      assert.is.not(sameParent(h1, h2), true);
+      assert.notStrictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -165,7 +165,7 @@ test('should calculate not same parent (nested at rules) (3)', () => {
       const h1 = result.root.nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0].nodes[0];
 
-      assert.is.not(sameParent(h1, h2), true);
+      assert.notStrictEqual(sameParent(h1, h2), true);
     });
 });
 
@@ -188,7 +188,6 @@ test('should calculate not same parent (nested at rules) (4)', () => {
       const h1 = result.root.nodes[0].nodes[0];
       const h2 = result.root.nodes[1].nodes[0].nodes[0];
 
-      assert.is.not(sameParent(h1, h2), true);
+      assert.notStrictEqual(sameParent(h1, h2), true);
     });
 });
-test.run();
diff --git a/packages/cssnano/test/_processCss.js b/packages/cssnano/test/_processCss.js
@@ -1,5 +1,5 @@
 'use strict';
-const assert = require('uvu/assert');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const cssnano = require('..');
 
@@ -8,7 +8,7 @@ function processCss(fixture, expected, options = { from: undefined }) {
     postcss([cssnano()])
       .process(fixture, options)
       .then(({ css }) => {
-        assert.is(css, expected);
+        assert.strictEqual(css, expected);
       });
 }
 module.exports = processCss;

diff --git a/packages/cssnano/test/api.js b/packages/cssnano/test/api.js
@@ -1,6 +1,6 @@
 'use strict';
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const nano = require('..');
 
@@ -10,7 +10,7 @@ function pluginMacro(instance) {
 
   return () =>
     instance.process(css, { from: undefined }).then((result) => {
-      assert.is(result.css, min);
+      assert.strictEqual(result.css, min);
     });
 }
 
@@ -28,10 +28,9 @@ test('should work with sourcemaps', () => {
   return postcss([nano])
     .process('h1{z-index:1}', { from: undefined, map: { inline: true } })
     .then(({ css }) => {
-      assert.is(
+      assert.strictEqual(
         /sourceMappingURL=data:application\/json;base64/.test(css),
         true
       );
     });
 });
-test.run();
diff --git a/packages/cssnano/test/config_loading/config-loading.js b/packages/cssnano/test/config_loading/config-loading.js
@@ -1,7 +1,7 @@
 'use strict';
 const process = require('process');
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const litePreset = require('cssnano-preset-lite');
 const defaultPreset = require('cssnano-preset-default');
@@ -23,11 +23,10 @@ test.after(() => {
 
 test('should read the cssnano configuration file', () => {
   const processor = postcss([cssnano]);
-  assert.is(processor.plugins.length, litePreset().plugins.length);
+  assert.strictEqual(processor.plugins.length, litePreset().plugins.length);
 });
 
 test('PostCSS config should override the cssnano config', () => {
   const processor = postcss([cssnano({ preset: 'default' })]);
-  assert.is(processor.plugins.length, defaultPreset().plugins.length);
+  assert.strictEqual(processor.plugins.length, defaultPreset().plugins.length);
 });
-test.run();
diff --git a/packages/cssnano/test/fixtures.js b/packages/cssnano/test/fixtures.js
@@ -1,5 +1,5 @@
 'use strict';
-const { test } = require('uvu');
+const { test } = require('node:test');
 const processCss = require('./_processCss');
 
 test(
@@ -358,4 +358,3 @@ test(
   'should remove leading zeroes from reduced calc values',
   processCss(`.box { margin: calc(-.5 * 1rem); }`, `.box{margin:-.5rem}`)
 );
-test.run();
diff --git a/packages/cssnano/test/issue26.js b/packages/cssnano/test/issue26.js
@@ -1,6 +1,6 @@
 'use strict';
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const nano = require('..');
 
@@ -44,6 +44,5 @@ test('it should compress whitespace after node.clone()', () => {
 
   return processor
     .process(fixture, { from: undefined })
-    .then((r) => assert.is(r.css, expected));
+    .then((r) => assert.strictEqual(r.css, expected));
 });
-test.run();
diff --git a/packages/cssnano/test/issue315.js b/packages/cssnano/test/issue315.js
@@ -1,6 +1,6 @@
 'use strict';
-const { test } = require('uvu');
-const assert = require('uvu/assert');
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
 const postcss = require('postcss');
 const fontMagician = require('postcss-font-magician');
 const cssnano = require('..');
@@ -14,10 +14,9 @@ test('should work with postcss-font-magician', () => {
   return postcss([fontMagician({}), cssnano()])
     .process(css, { from: undefined })
     .then((result) => {
-      assert.snapshot(
+      assert.strictEqual(
         result.css,
         `@font-face{font-family:Alice;font-style:normal;font-weight:400;src:url(//fonts.gstatic.com/s/alice/v20/OpNCnoEEmtHa6GcArgo.eot?#) format("eot"),url(//fonts.gstatic.com/s/alice/v20/OpNCnoEEmtHa6GcOrg4.woff2) format("woff2"),url(//fonts.gstatic.com/s/alice/v20/OpNCnoEEmtHa6GcArgg.woff) format("woff")}body{font-family:Alice}`
       );
     });
 });
-test.run();