feat(build): migrate to vitest

Author: daker

.eslintrc.js

@@ -1,12 +1,9 @@
-const path = require('path');
 const prettierConf = require('./prettier.config.js');
 
 module.exports = {
-  parser: '@babel/eslint-parser',
   parserOptions: {
-    babelOptions: {
-      configFile: path.resolve(__dirname, '.babelrc.json'),
-    },
+    ecmaVersion: 2022,
+    sourceType: 'module',
   },
   extends: ['airbnb/base', 'prettier'],
   rules: {
@@ -36,27 +33,22 @@ module.exports = {
   plugins: ['prettier'],
   globals: {
     __BASE_PATH__: false,
+    __VTK_TEST_NO_WEBGL__: false,
+    __VTK_TEST_WEBGPU__: false,
     VRFrameData: true,
   },
   settings: {
     'import/resolver': {
-      webpack: {
-        config: {
-          resolve: {
-            alias: {
-              // Since vtk.js examples are written as if the vtk.js package is a dependency,
-              // we need to resolve example imports as if they were referencing vtk.js/Sources.
-              // the Examples/Utilities hack allows for imports from those folders, since our
-              // last alias overrides vtk.js/* paths to point to vtk.js/Sources/*.
-              'vtk.js/Data': path.resolve(__dirname, 'Data'),
-              'vtk.js/Examples': path.resolve(__dirname, 'Examples'),
-              'vtk.js/Utilities': path.resolve(__dirname, 'Utilities'),
-              'vtk.js/Sources': path.resolve(__dirname, 'Sources'),
-              'vtk.js': path.resolve(__dirname, 'Sources'),
-              '@kitware/vtk.js': path.resolve(__dirname, 'Sources'),
-            },
-          },
-        },
+      alias: {
+        map: [
+          ['vtk.js/Data', './Data'],
+          ['vtk.js/Examples', './Examples'],
+          ['vtk.js/Utilities', './Utilities'],
+          ['vtk.js/Sources', './Sources'],
+          ['vtk.js', './Sources'],
+          ['@kitware/vtk.js', './Sources'],
+        ],
+        extensions: ['.js', '.json'],
       },
     },
   },
@@ -71,8 +63,8 @@ module.exports = {
     // ignore js files in utilities
     'Utilities/**/*.js',
     // ignore configs
-    'karma.conf.js',
-    'webpack.*.js',
+    'vite.config.js',
+    'vitest.config.js',
     '.eslintrc.js',
   ],
 };

.github/workflows/build-test.yml

@@ -12,35 +12,34 @@ jobs:
         node: [22]
     name: ${{ matrix.os }} and node ${{ matrix.node }}
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Setup node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
       - name: Install dependencies
-        run: |
-          npm ci
-          sudo apt-get install xvfb
+        run: npm ci
+      - name: Install Playwright browsers
+        run: npx playwright install --with-deps chromium firefox
       - name: Build
         run: npm run build:release
       - name: Archive build output
         if: github.event_name != 'merge_group'
         uses: actions/upload-artifact@v4
         with:
-          name: build-results-${{ matrix.runs_on }}-node_${{ matrix.node }}
+          name: build-results-${{ matrix.os }}-node_${{ matrix.node }}
           path: dist
           retention-days: 15
       - name: Validate generated typescript definitions
         run: |
           npx tsc -p tsconfig.esm-check.json
-          npx tsc -p tsconfig.umd-check.json
       - name: Chrome and Firefox tests
-        run: xvfb-run --auto-servernum npm run test -- --browsers ChromeSwiftShader,Firefox
+        run: xvfb-run --auto-servernum npm run test:reporter
       - name: Archive test results
         if: github.event_name != 'merge_group' && (success() || failure())
         uses: actions/upload-artifact@v4
         continue-on-error: true
         with:
-          name: test-results-${{ matrix.runs_on }}-node_${{ matrix.node }}
-          path: Utilities/TestResults/Test-Report.html
+          name: test-results-${{ matrix.os }}-node_${{ matrix.node }}
+          path: Utilities/TestResults/
           retention-days: 15

.github/workflows/pr-checks.yml

@@ -8,13 +8,13 @@ jobs:
     runs-on: ubuntu-24.04
     name: Check and lint PR
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
         with:
           fetch-depth: 0 # needed so commitlint can lint the commits
       - name: Setup node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
       - run: npm ci
       - name: Enforce code style
         run: npm run validate

.github/workflows/publish.yml

@@ -22,33 +22,32 @@ jobs:
       name: npm
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
       - name: Setup node
         uses: actions/setup-node@v4
         with:
           node-version: 22
       - name: Install dependencies
-        run: |
-          npm ci
-          sudo apt-get install xvfb
+        run: npm ci
+      - name: Install Playwright browsers
+        run: npx playwright install --with-deps chromium firefox
       - name: Build
         run: npm run build:release
       - name: Validate code style
         run: npm run validate
       - name: Validate generated typescript definitions
         run: |
           npx tsc -p tsconfig.esm-check.json
-          npx tsc -p tsconfig.umd-check.json
       - name: Chrome and Firefox tests
-        run: xvfb-run --auto-servernum npm run test -- --browsers ChromeSwiftShader,Firefox
+        run: xvfb-run --auto-servernum npm run test:reporter
       - name: Archive test results
         if: always()
         uses: actions/upload-artifact@v4
         with:
           name: test-results
-          path: Utilities/TestResults/Test-Report.html
+          path: Utilities/TestResults/
           retention-days: 15
       - name: Release
         env:
@@ -70,11 +69,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
         with:
           fetch-depth: 1
       - name: Setup node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
           node-version: 22
       - name: Install dependencies

Sources/Common/Core/CellArray/test/testCellArray.js

@@ -1,67 +1,37 @@
-import test from 'tape';
-
+import { it, expect } from 'vitest';
 import vtkCellArray from 'vtk.js/Sources/Common/Core/CellArray';
 import { VtkDataTypes } from 'vtk.js/Sources/Common/Core/DataArray/Constants';
 
-test('Test cell array constructor', (t) => {
+it('Test cell array constructor', () => {
   // Empty cell arrays are allowed (empty=true in DEFAULT_VALUES)
   const emptyCellArray = vtkCellArray.newInstance();
-  t.equal(
-    emptyCellArray.getDataType(),
-    VtkDataTypes.UNSIGNED_INT,
-    'empty init data type'
-  );
-  t.equal(emptyCellArray.getNumberOfCells(), 0, 'empty init number of cells');
+  expect(emptyCellArray.getDataType()).toBe(VtkDataTypes.UNSIGNED_INT);
+  expect(emptyCellArray.getNumberOfCells()).toBe(0);
 
   const uintCellArray = vtkCellArray.newInstance({ values: [3, 0, 1, 2] });
-  t.equal(
-    uintCellArray.getDataType(),
-    VtkDataTypes.UNSIGNED_INT,
-    'uint init data type'
-  );
-  t.equal(uintCellArray.getNumberOfCells(), 1, 'uint init number of cells');
+  expect(uintCellArray.getDataType()).toBe(VtkDataTypes.UNSIGNED_INT);
+  expect(uintCellArray.getNumberOfCells()).toBe(1);
 
   const charCellArray = vtkCellArray.newInstance({
     dataType: VtkDataTypes.CHAR,
     values: [3, 0, 1, 2],
   });
-  t.equal(
-    charCellArray.getDataType(),
-    VtkDataTypes.CHAR,
-    'char init data type'
-  );
-  t.equal(charCellArray.getNumberOfCells(), 1, 'char init number of cells');
-
-  t.end();
+  expect(charCellArray.getDataType()).toBe(VtkDataTypes.CHAR);
+  expect(charCellArray.getNumberOfCells()).toBe(1);
 });
 
-test('Test vtkCellArray insertNextCell', (t) => {
+it('Test vtkCellArray insertNextCell', () => {
   const cellArray = vtkCellArray.newInstance({
     dataType: 'Uint16Array',
     empty: true,
     numberOfComponents: 1,
   });
   cellArray.insertNextCell([0, 1, 2]);
-  t.equal(
-    cellArray.getNumberOfCells(),
-    1,
-    'number of cells after first insertNextCell'
-  );
-  t.deepEqual(
-    cellArray.getData(),
-    Uint16Array.from([3, 0, 1, 2]),
-    'getData after first insertNextCell'
-  );
+  expect(cellArray.getNumberOfCells()).toBe(1);
+  expect(cellArray.getData()).toEqual(Uint16Array.from([3, 0, 1, 2]));
   cellArray.insertNextCell([3, 4, 5, 6]);
-  t.equal(
-    cellArray.getNumberOfCells(),
-    2,
-    'number of cells after second insertNextCell'
-  );
-  t.deepEqual(
-    cellArray.getData(),
-    Uint16Array.from([3, 0, 1, 2, 4, 3, 4, 5, 6]),
-    'getData after second insertNextCell'
+  expect(cellArray.getNumberOfCells()).toBe(2);
+  expect(cellArray.getData()).toEqual(
+    Uint16Array.from([3, 0, 1, 2, 4, 3, 4, 5, 6])
   );
-  t.end();
 });