Merge pull request #657 from relative-ci/update-tests

vio · web-flow · commit 43e78eddd2cc · 2025-11-09T13:05:03.000+01:00
Update tests
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,7 +1,7 @@
 name: ci
-on: 
+on:
   push:
-    branches: 
+    branches:
       - master
       - next
     tags:
@@ -51,8 +51,8 @@ jobs:
           retention-days: 1
           path: dist*/
 
-  test-package:
-    name: Test package on node ${{ matrix.node }} / ${{ matrix.os }} / rollup v${{ matrix.rollup }}
+  test-package-rollup:
+    name: 'rollup v${{ matrix.rollup }} / node ${{ matrix.node }} / ${{ matrix.os }}'
     needs:
       - build
     runs-on: ubuntu-latest
@@ -98,19 +98,72 @@ jobs:
 
       - name: Install test package dependencies
         run: |
-          npm install 
+          npm install
           npm install rollup@${{ matrix.rollup }} --force
           npx rollup -v
-        working-directory: test/package
+        working-directory: test/package/rollup
 
       - name: Test package
-        run: npm run test:package
+        run: npm run test
+        working-directory: test/package/rollup
+
+  test-package-vite:
+    name: 'vite v${{ matrix.vite }} / node ${{ matrix.node }} / ${{ matrix.os }}'
+    needs:
+      - build
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node: ['22', '20', '18']
+        os: [ubuntu-latest, windows-latest]
+        vite: ['5', '6', '7']
+        exclude:
+          - os: windows-latest
+            node: 20
+          - os: windows-latest
+            node: 18
+          - vite: 7
+            node: 18
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5.0.0
+
+      - name: Setup node & npm
+        uses: relative-ci/.github/actions/setup-node-npm@v1.1.5
+        with:
+          version: ${{ matrix.node }}
+          cache: never
+          install: false
+
+      - name: Install vitest
+        run: npm install vitest
+
+      - name: Download build files
+        uses: actions/download-artifact@v6.0.0
+        with:
+          name: build-artifacts
+
+      - name: Setup local npm registry
+        run: ./scripts/setup-registry.sh
+
+      - name: Install test package dependencies
+        run: |
+          npm install
+          npm install vite@${{ matrix.vite }} --force
+          npx vite -v
+        working-directory: test/package/vite
+
+      - name: Test package
+        run: npm run test
+        working-directory: test/package/vite
 
   tests:
     needs:
       - lint
       - test-unit
-      - test-package
+      - test-package-rollup
+      - test-package-vite
     runs-on: ubuntu-latest
     steps:
       - run: exit 0
diff --git a/package.json b/package.json
@@ -50,7 +50,9 @@
     "build": "tsc && rollup -c rollup.config.mjs",
     "lint": "exit 0",
     "test:unit": "vitest test/unit",
-    "test:package": "cd test/package && vitest",
+    "test:package": "npm run test:package:rollup && npm run test:package:vite",
+    "test:package:rollup": "cd test/package/rollup && vitest",
+    "test:package:vite": "cd test/package/vite && vitest",
     "bump": "./scripts/bump.sh",
     "release": "./scripts/release.sh"
   },
diff --git a/test/package/package.json b/test/package/package.json
diff --git a/test/package/rollup/.gitignore b/test/package/rollup/.gitignore
@@ -0,0 +1,2 @@
+package-lock.json
+dist*
diff --git a/test/package/rollup/case-config.js b/test/package/rollup/case-config.js
@@ -0,0 +1,11 @@
+/* eslint-disable @typescript-eslint/no-require-imports */
+const { defineConfig } = require('rollup');
+const webpackStatsPlugin = require('rollup-plugin-webpack-stats');
+
+module.exports = defineConfig({
+  input: 'src/index.js',
+  output: {
+    dir: 'dist',
+  },
+  plugins: [webpackStatsPlugin()],
+});
diff --git a/test/package/rollup/case-config.mjs b/test/package/rollup/case-config.mjs
@@ -0,0 +1,10 @@
+import { defineConfig } from 'rollup';
+import webpackStatsPlugin from 'rollup-plugin-webpack-stats';
+
+export default defineConfig({
+  input: 'src/index.js',
+  output: {
+    dir: 'dist',
+  },
+  plugins: [webpackStatsPlugin()],
+});
diff --git a/test/package/rollup/case-config.ts b/test/package/rollup/case-config.ts
@@ -0,0 +1,10 @@
+import { defineConfig } from 'rollup';
+import webpackStatsPlugin from 'rollup-plugin-webpack-stats';
+
+export default defineConfig({
+  input: 'src/index.js',
+  output: {
+    dir: 'dist',
+  },
+  plugins: [webpackStatsPlugin({})],
+});
diff --git a/test/package/rollup/case-options.mjs b/test/package/rollup/case-options.mjs
diff --git a/test/package/rollup/configs.test.ts b/test/package/rollup/configs.test.ts
@@ -0,0 +1,22 @@
+import { promisify } from 'node:util';
+import { exec as execCallback} from 'node:child_process';
+import { describe, expect, test } from 'vitest';
+
+const exec = promisify(execCallback);
+
+const testCases = [
+  { type: 'commonjs', run: 'npm run build-commonjs' },
+  { type: 'esm', run: 'npm run build-esm' },
+  { type: 'ts', run: 'npm run build-ts' },
+];
+
+describe('package - rollup configs', () => {
+  testCases.forEach((testCase) => {
+    test(`should build successfully with rollup ${testCase.type} config`, async () => {
+      const { stderr } = await exec(`cross-env NO_COLOR=true ${testCase.run}`);
+
+      expect(stderr).toMatch('Stats saved to dist');
+      expect(stderr).toMatch('created dist in');
+    });
+  });
+});
diff --git a/test/package/rollup/options.test.ts b/test/package/rollup/options.test.ts
@@ -4,9 +4,9 @@ import { beforeEach, describe, test, expect } from 'vitest';
 import { rollup } from 'rollup';
 import { vol } from 'memfs';
 
-import rollupConfig from './rollup.config';
+import rollupConfig from './case-options.mjs';
 
-describe('package test', () => {
+describe('rollup options', () => {
   beforeEach(() => {
     vol.reset();
   });
diff --git a/test/package/rollup/package.json b/test/package/rollup/package.json
@@ -0,0 +1,21 @@
+{
+  "name": "rollup-plugin-webpack-stats-test",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "scripts": {
+    "build-commonjs": "rollup -c case-config.js",
+    "build-esm": "rollup -c case-config.mjs",
+    "build-ts": "rollup --config case-config.ts --configPlugin typescript",
+    "test": "vitest"
+  },
+  "keywords": [],
+  "license": "ISC",
+  "devDependencies": {
+    "@rollup/plugin-typescript": "^12.1.4",
+    "cross-env": "^7.0.0",
+    "rollup": "^4.9.1",
+    "rollup-plugin-webpack-stats": "*",
+    "typescript": "^5.9.2"
+  }
+}
diff --git a/test/package/rollup/src/index.js b/test/package/rollup/src/index.js
diff --git a/test/package/vite/.gitignore b/test/package/vite/.gitignore
@@ -0,0 +1,2 @@
+package-lock.json
+dist*
diff --git a/test/package/vite/case-config.js b/test/package/vite/case-config.js
@@ -0,0 +1,16 @@
+/* eslint-disable @typescript-eslint/no-require-imports */
+const { defineConfig } = require('vite');
+const webpackStatsPlugin = require('rollup-plugin-webpack-stats');
+
+module.exports = defineConfig({
+  build: {
+    rollupOptions: {
+      output: {
+        assetFileNames: 'assets/[name][extname]',
+        chunkFileNames: 'assets/[name].js',
+        entryFileNames: 'assets/[name].js',
+      },
+    },
+  },
+  plugins: [webpackStatsPlugin()],
+});
diff --git a/test/package/vite/case-config.mjs b/test/package/vite/case-config.mjs
@@ -0,0 +1,15 @@
+import { defineConfig } from 'vite';
+import webpackStatsPlugin from 'rollup-plugin-webpack-stats';
+
+export default defineConfig({
+  build: {
+    rollupOptions: {
+      output: {
+        assetFileNames: 'assets/[name][extname]',
+        chunkFileNames: 'assets/[name].js',
+        entryFileNames: 'assets/[name].js',
+      },
+    },
+  },
+  plugins: [webpackStatsPlugin()],
+});
diff --git a/test/package/vite/case-config.ts b/test/package/vite/case-config.ts
@@ -0,0 +1,15 @@
+import { defineConfig } from 'vite';
+import webpackStatsPlugin from 'rollup-plugin-webpack-stats';
+
+export default defineConfig({
+  build: {
+    rollupOptions: {
+      output: {
+        assetFileNames: 'assets/[name][extname]',
+        chunkFileNames: 'assets/[name].js',
+        entryFileNames: 'assets/[name].js',
+      },
+    },
+  },
+  plugins: [webpackStatsPlugin()],
+});
diff --git a/test/package/vite/case-options.mjs b/test/package/vite/case-options.mjs
@@ -0,0 +1,38 @@
+import { basename } from 'path';
+import { defineConfig } from 'vite';
+import webpackStats from 'rollup-plugin-webpack-stats';
+
+const baseConfig = {
+  build: {
+    outDir: 'dist',
+    rollupOptions: {
+      output: {
+        assetFileNames: 'assets/[name][extname]',
+        chunkFileNames: 'assets/[name].js',
+        entryFileNames: 'assets/[name].js',
+      },
+    },
+  },
+};
+
+export default defineConfig([
+  {
+    ...baseConfig,
+    plugins: [webpackStats()],
+  },
+  {
+    ...baseConfig,
+    build: {
+      ...baseConfig.build,
+      outDir: 'dist2',
+    },
+    plugins: [
+      // A contrived demo to show that plugin options can access vite outputOptions
+      webpackStats((outputOptions) => {
+        return {
+          fileName: `stats-${basename(outputOptions.dir)}.json`,
+        };
+      }),
+    ],
+  },
+]);
diff --git a/test/package/vite/configs.test.ts b/test/package/vite/configs.test.ts
@@ -0,0 +1,22 @@
+import { promisify } from 'node:util';
+import { exec as execCallback } from 'node:child_process';
+import { describe, expect, test } from 'vitest';
+
+const exec = promisify(execCallback);
+
+const testCases = [
+  { type: 'commonjs', run: 'npm run build-commonjs' },
+  { type: 'esm', run: 'npm run build-esm' },
+  { type: 'ts', run: 'npm run build-ts' },
+];
+
+describe('package - vite configs', () => {
+  testCases.forEach((testCase) => {
+    test(`should build successfully with vite ${testCase.type} config`, async () => {
+      const { stdout } = await exec(`cross-env NO_COLOR=true ${testCase.run}`);
+
+      expect(stdout).toMatch('Stats saved to');
+      expect(stdout).toMatch('built in');
+    });
+  });
+});
diff --git a/test/package/vite/index.html b/test/package/vite/index.html
@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8">
+    <title>rollup-plugin-stats</title>
+  </head>
+  <body>
+    <script type="module" src="./src/index.js"></script>
+  </body>
+</html>
diff --git a/test/package/vite/options.test.ts b/test/package/vite/options.test.ts
@@ -0,0 +1,45 @@
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { beforeEach, describe, test, expect } from 'vitest';
+import { build as vite } from 'vite';
+import { vol } from 'memfs';
+
+import viteConfigs from './case-options.mjs';
+
+describe('package - vite options', () => {
+  beforeEach(() => {
+    vol.reset();
+  });
+
+  test('should output bundle stats JSON file when options is an object', async () => {
+    const config = viteConfigs[0];
+    await vite(config);
+
+    const actual = await fs.readFile(path.join(config.build.outDir, 'webpack-stats.json'), 'utf8');
+    const stats = JSON.parse(actual);
+    expect(stats).toMatchObject({
+      assets: [
+        {
+          name: 'assets/index.js',
+          size: 739,
+        },
+      ],
+    })
+  });
+
+  test('should output bundle stats JSON file when options is a builder function', async () => {
+    const config = viteConfigs[1];
+    await vite(config);
+
+    const actual = await fs.readFile(path.join(config.build.outDir, 'stats-dist2.json'), 'utf8');
+    const stats = JSON.parse(actual);
+    expect(stats).toMatchObject({
+      assets: [
+        {
+          name: 'assets/index.js',
+          size: 739,
+        },
+      ],
+    })
+  });
+});
diff --git a/test/package/vite/package.json b/test/package/vite/package.json
diff --git a/test/package/vite/src/index.js b/test/package/vite/src/index.js
