ci: add codecov and fix test

.github/workflows/main.yml

@@ -17,35 +17,42 @@ jobs:
     name: Tests on Node.js v${{ matrix.node.version }}
     steps:
       - name: Checkout repo
-        uses: actions/checkout@v5
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
 
       - name: Setup node
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0
         with:
           node-version: ${{ matrix.node.version }}
+          cache: npm
 
-      - name: Download deps
-        uses: bahmutov/npm-install@v1
+      - name: Install dependencies
+        run: npm ci
 
       - name: Build sources
         run: ${{ matrix.node.env }} npm run build
 
       - name: Run tests
-        run: ${{ matrix.node.env }} npm run test
+        run: ${{ matrix.node.env }} npm run test:coverage
+
+      - name: Codecov
+        uses: codecov/codecov-action@671740ac38dd9b0130fbe1cec585b89eea48d3de # v5.5.2
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 
   lint:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repo
-        uses: actions/checkout@v5
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
 
       - name: Setup node
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6.2.0
         with:
           node-version: "22.x"
+          cache: npm
 
-      - name: Download deps
-        uses: bahmutov/npm-install@v1
+      - name: Install dependencies
+        run: npm ci
 
       - name: Build sources
         run: npm run build

jest.config.js

@@ -8,6 +8,7 @@ module.exports = {
   testMatch: ["**/test/*.js"],
   testPathIgnorePatterns: ["<rootDir>/test/helpers.js"],
   setupFilesAfterEnv: ["<rootDir>/test/helpers.js"],
+  coveragePathIgnorePatterns: ["<rootDir>/test"],
   watchPathIgnorePatterns: [
     // Ignore the output generated by plugin tests
     // when watching for changes to avoid the test

package.json

@@ -48,6 +48,7 @@
     "fix": "npm run fix:code && npm run fmt",
     "install-test-webpack-versions": "./bin/install-test-webpack-versions.sh",
     "test": "npm run install-test-webpack-versions && NODE_OPTIONS=--openssl-legacy-provider jest --runInBand",
+    "test:coverage": "npm run test -- --coverage",
     "test-dev": "npm run install-test-webpack-versions && NODE_OPTIONS=--openssl-legacy-provider jest --watch --runInBand"
   },
   "dependencies": {

test/Logger.js

@@ -1,4 +1,4 @@
-const Logger = require("../lib/Logger");
+const Logger = require("../src/Logger");
 
 class TestLogger extends Logger {
   constructor(level) {

test/dev-server/webpack.config.js

@@ -1,7 +1,7 @@
 "use strict";
 
 const path = require("node:path");
-const BundleAnalyzerPlugin = require("../../lib/BundleAnalyzerPlugin");
+const BundleAnalyzerPlugin = require("../../src/BundleAnalyzerPlugin");
 
 module.exports = {
   mode: "development",

test/helpers.js

@@ -2,7 +2,7 @@ const { readdirSync } = require("node:fs");
 const path = require("node:path");
 const webpack = require("webpack");
 
-const BundleAnalyzerPlugin = require("../lib/BundleAnalyzerPlugin");
+const BundleAnalyzerPlugin = require("../src/BundleAnalyzerPlugin");
 
 /* global it */
 

test/parseUtils.js

@@ -1,7 +1,7 @@
 const fs = require("node:fs");
 const path = require("node:path");
 
-const { parseBundle } = require("../lib/parseUtils");
+const { parseBundle } = require("../src/parseUtils");
 
 const BUNDLES_DIR = path.resolve(__dirname, "./bundles");
 

test/plugin.js

@@ -2,7 +2,7 @@ const fs = require("node:fs");
 const path = require("node:path");
 const url = require("node:url");
 const puppeteer = require("puppeteer");
-const BundleAnalyzerPlugin = require("../lib/BundleAnalyzerPlugin");
+const BundleAnalyzerPlugin = require("../src/BundleAnalyzerPlugin");
 const { isZstdSupported } = require("../src/sizeUtils");
 const {
   forEachWebpackVersion,

test/statsUtils.js

@@ -1,7 +1,7 @@
 const { readFileSync } = require("node:fs");
 const path = require("node:path");
 const { globSync } = require("tinyglobby");
-const { StatsSerializeStream } = require("../lib/statsUtils");
+const { StatsSerializeStream } = require("../src/statsUtils");
 
 async function stringify(json) {
   return new Promise((resolve, reject) => {

test/utils.js

@@ -1,4 +1,4 @@
-const { createAssetsFilter } = require("../lib/utils");
+const { createAssetsFilter } = require("../src/utils");
 
 describe("createAssetsFilter", () => {
   it("should create a noop filter if pattern is not set", () => {

test/viewer.js

@@ -1,8 +1,8 @@
 const crypto = require("node:crypto");
 const net = require("node:net");
 
-const Logger = require("../lib/Logger");
-const { getEntrypoints, startServer } = require("../lib/viewer");
+const Logger = require("../src/Logger");
+const { getEntrypoints, startServer } = require("../src/viewer");
 
 describe("WebSocket server", () => {
   it("should not crash when an error is emitted on the websocket", (done) => {