chore: migrate test runner from mocha to jest

.eslintrc

@@ -1,4 +1,7 @@
 {
+  "env": {
+    "jest": true,
+  },
   "extends": "@dabh/eslint-config-populist",
   "rules": {
     "one-var": ["error", { "var": "never", "let": "never", "const": "never" }],

.github/workflows/ci.yml

@@ -21,6 +21,7 @@ jobs:
 
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         node:
           - 16
@@ -31,18 +32,24 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
+
       - name: Install Dependencies
         run: npm clean-install
+
       - name: Lint
         run: npm run lint
+
+      - name: Unit Tests (with coverage)
+        run: npm run test:unit
+
       - name: Integration Tests
         run: npm run test:integration
-      - name: Test Coverage
-        run: npm run test:coverage
+
       - name: Report test coverage to Coveralls.io
         if: matrix.node == '20'
         uses: coverallsapp/github-action@master
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
+
       - name: TypeScript Test
-        run: npx --package typescript tsc --project test
+        run: npm run test:typescript

README.md

@@ -1234,15 +1234,17 @@ yarn add winston
 
 ## Run Tests
 
-All of the winston tests are written with [`mocha`][mocha], [`nyc`][nyc], and
-[`assume`][assume].  They can be run with `npm`.
-
 ``` bash
-npm test
+npm test # Runs all tests
+npm run test:unit # Runs all Unit tests with coverage
+npm run test:integration # Runs all integration tests
+npm run test:typescript # Runs tests verifying Typescript types
 ```
 
+All of the winston tests are written with [jest]. Assertions use a mix of [assume] and the built-in jest assertion library.
+
 #### Author: [Charlie Robbins]
-#### Contributors: [Jarrett Cruger], [David Hyde], [Chris Alderson]
+#### Contributors: [Jarrett Cruger], [David Hyde], [Chris Alderson], [Jonathon Terry]
 
 [Transports]: #transports
 [Logging levels]: #logging-levels
@@ -1254,11 +1256,10 @@ npm test
 
 [RFC5424]: https://tools.ietf.org/html/rfc5424
 [util.format]: https://nodejs.org/dist/latest/docs/api/util.html#util_util_format_format_args
-[mocha]: https://mochajs.org
-[nyc]: https://github.com/istanbuljs/nyc
 [assume]: https://github.com/bigpipe/assume
 [logform]: https://github.com/winstonjs/logform#readme
 [winston-transport]: https://github.com/winstonjs/winston-transport
+[jest]: https://jestjs.io/
 
 [Read the `winston@2.x` documentation]: https://github.com/winstonjs/winston/tree/2.x
 
@@ -1268,4 +1269,5 @@ npm test
 [Charlie Robbins]: http://github.com/indexzero
 [Jarrett Cruger]: https://github.com/jcrugzz
 [David Hyde]: https://github.com/dabh
-[Chris Alderson]: https://github.com/chrisalderson
+[Chris Alderson]: https://github.com/chrisalderson
+[Jonathon Terry]: https://github.com/maverick1872

jest.config.js

@@ -0,0 +1,26 @@
+/**
+ * @type {import('@jest/types').Config.InitialOptions}
+ */
+module.exports = {
+  collectCoverage: false,
+  collectCoverageFrom: [
+    '<rootDir>/lib/**/*.js'
+  ],
+  coverageDirectory: 'coverage',
+  testEnvironment: 'node',
+  testMatch: [
+    '<rootDir>/test/**/*.test.js'
+  ],
+  globalSetup: '<rootDir>/test/globalSetup.js',
+  silent: true,
+  verbose: true,
+  coverageThreshold: {
+    global: {
+      functions: 74.54,
+      lines: 72.48,
+      statements: 72.25,
+      branches: 64.04
+    }
+  }
+};
+

lib/winston/profiler.js

@@ -48,6 +48,6 @@ class Profiler {
 
     return this.logger.write(info);
   }
-};
+}
 
 module.exports = Profiler;