Update vitest monorepo to v4.1.0 (#547)

no ref

Updated to vitest v4.1.0.

Author: 9larsons

nx.json

@@ -1,34 +1,41 @@
 {
-    "$schema": "./node_modules/nx/schemas/nx-schema.json",
-    "targetDefaults": {
-        "build": {
-            "dependsOn": ["^build"],
-            "cache": true
-        },
-        "lint": {
-            "cache": true
-        },
-        "test": {
-            "dependsOn": ["^build"],
-            "cache": true
-        }
+  "$schema": "./node_modules/nx/schemas/nx-schema.json",
+  "targetDefaults": {
+    "build": {
+      "dependsOn": [
+        "^build"
+      ],
+      "cache": true
     },
-    "release": {
-        "projects": ["packages/*"],
-        "projectsRelationship": "independent",
-        "version": {
-            "preserveMatchingDependencyRanges": false,
-            "git": {
-                "commit": true,
-                "tag": true,
-                "commitMessage": "Published new versions"
-            }
-        },
-        "changelog": {
-            "workspaceChangelog": false
-        },
-        "publish": {
-            "registry": "https://registry.npmjs.org"
-        }
+    "lint": {
+      "cache": true
+    },
+    "test": {
+      "dependsOn": [
+        "^build"
+      ],
+      "cache": true
+    }
+  },
+  "release": {
+    "projects": [
+      "packages/*"
+    ],
+    "projectsRelationship": "independent",
+    "version": {
+      "preserveMatchingDependencyRanges": false,
+      "git": {
+        "commit": true,
+        "tag": true,
+        "commitMessage": "Published new versions"
+      }
+    },
+    "changelog": {
+      "workspaceChangelog": false
+    },
+    "publish": {
+      "registry": "https://registry.npmjs.org"
     }
-}
+  },
+  "analytics": false
+}

package.json

@@ -15,7 +15,7 @@
     "presetup": "yarn",
     "setup": "yarn",
     "test": "nx run-many -t test --parallel=10 --outputStyle=dynamic-legacy",
-    "test:ci": "nx run-many -t test --outputStyle=dynamic",
+    "test:ci": "nx run-many -t test --outputStyle=static",
     "lint": "nx run-many -t lint --outputStyle=dynamic",
     "format": "oxfmt -c .oxfmtrc.json \"packages/**/*.{js,ts,json,md}\"",
     "format:check": "oxfmt -c .oxfmtrc.json --check \"packages/**/*.{js,ts,json,md}\"",
@@ -28,13 +28,13 @@
   },
   "devDependencies": {
     "@nx/js": "22.6.5",
-    "@vitest/coverage-v8": "3.2.4",
+    "@vitest/coverage-v8": "4.1.0",
     "nx": "22.6.5",
     "oxfmt": "0.45.0",
     "oxlint": "1.60.0",
     "sinon": "21.1.2",
     "ts-node": "10.9.2",
-    "vitest": "3.2.4"
+    "vitest": "4.1.0"
   },
   "resolutions": {
     "node-loggly-bulk": "^4.0.2"

packages/bookshelf-pagination/test/pagination.test.js

@@ -294,6 +294,58 @@ describe('@tryghost/bookshelf-pagination', function () {
         assert.equal(modelState.rawCalls[0], 'count(distinct posts.id) as aggregate');
     });
 
+    it('useSmartCount supports SQL fragments returned as an array', async function () {
+        const {bookshelf, modelState} = createBookshelf({countRows: [{aggregate: 1}]});
+        const model = new bookshelf.Model();
+
+        const originalQuery = model.query;
+        model.query = function () {
+            const qb = originalQuery.apply(this, arguments);
+            if (arguments.length === 0) {
+                qb.toSQL = () => [
+                    {sql: 'select *'},
+                    {},
+                    {sql: 'from `posts`, `tags` where `posts`.`id` = `tags`.`post_id`'}
+                ];
+            }
+            return qb;
+        };
+
+        await model.fetchPage({page: 1, limit: 10, useSmartCount: true});
+
+        assert.equal(modelState.rawCalls[0], 'count(distinct posts.id) as aggregate');
+    });
+
+    it('useSmartCount falls back safely when compiled SQL is missing', async function () {
+        const {bookshelf, modelState} = createBookshelf({countRows: [{aggregate: 1}]});
+        const model = new bookshelf.Model();
+
+        const originalQuery = model.query;
+        model.query = function () {
+            const qb = originalQuery.apply(this, arguments);
+            if (arguments.length === 0) {
+                qb.toSQL = () => ({});
+            }
+            return qb;
+        };
+
+        await model.fetchPage({page: 1, limit: 10, useSmartCount: true});
+
+        assert.equal(modelState.rawCalls[0], 'count(*) as aggregate');
+    });
+
+    it('handleRelation does not duplicate entries and ignores same-table properties', function () {
+        const {bookshelf} = createBookshelf();
+        const model = new bookshelf.Model();
+        model.eagerLoad = ['authors'];
+
+        paginationPlugin.paginationUtils.handleRelation(model, 'posts.id');
+        paginationPlugin.paginationUtils.handleRelation(model, 'title');
+        paginationPlugin.paginationUtils.handleRelation(model, 'authors.name');
+
+        assert.deepEqual(model.eagerLoad, ['authors']);
+    });
+
     it('falls back to zero total when aggregate row is missing', async function () {
         const { bookshelf } = createBookshelf({ countRows: [] });
         const model = new bookshelf.Model();

packages/config/lib/config.js

@@ -1,11 +1,7 @@
 const getConfig = require('./get-config');
 
-let config;
+const config = getConfig();
 function initConfig() {
-    if (!config) {
-        config = getConfig();
-    }
-
     return config;
 }
 

packages/config/test/config.test.js

@@ -75,6 +75,11 @@ describe('Config', function () {
         assert.equal(config.get('env'), 'development');
     });
 
+    it('handles missing process root gracefully', function () {
+        const config = withEnv('testing', () => withRoot('', () => loadFreshGetConfig()()));
+        assert.equal(config.get('env'), 'testing');
+    });
+
     it('index exports lib/config and only initializes config once', function () {
         const originalGetConfig = require(getConfigPath);
         const fakeConfig = { name: 'fake-config' };

packages/domain-events/lib/DomainEvents.js

@@ -38,6 +38,8 @@ class DomainEvents {
             }
             if (this.#trackingEnabled) {
                 this.#onProcessed();
+            } else {
+                // Tracking is disabled outside tests.
             }
         });
     }
@@ -61,6 +63,8 @@ class DomainEvents {
     static dispatchRaw(name, data) {
         if (this.#trackingEnabled) {
             this.#dispatchCount += DomainEvents.ee.listenerCount(name);
+        } else {
+            // Tracking is disabled outside tests.
         }
         DomainEvents.ee.emit(name, data);
     }
@@ -80,8 +84,9 @@ class DomainEvents {
                 // Resolve immediately if there are no events in the queue
                 resolve();
                 return;
+            } else {
+                this.#awaitQueue.push({ resolve });
             }
-            this.#awaitQueue.push({ resolve });
         });
     }
 
@@ -92,8 +97,27 @@ class DomainEvents {
                 item.resolve();
             }
             this.#awaitQueue = [];
+        } else {
+            // Wait for the remaining tracked listeners.
         }
     }
+
+    /**
+     * Enable or disable dispatch/processed tracking. Test-only.
+     * @param {boolean} enabled
+     */
+    static setTrackingEnabledForTest(enabled) {
+        this.#trackingEnabled = enabled;
+    }
+
+    /**
+     * Clear the await queue and reset dispatch/processed counts. Test-only.
+     */
+    static resetTrackingStateForTest() {
+        this.#awaitQueue = [];
+        this.#dispatchCount = 0;
+        this.#processedCount = 0;
+    }
 }
 
 module.exports = DomainEvents;

packages/domain-events/test/DomainEvents.test.js

@@ -24,6 +24,8 @@ describe('DomainEvents', function () {
     afterEach(function () {
         sinon.restore();
         DomainEvents.ee.removeAllListeners();
+        DomainEvents.resetTrackingStateForTest();
+        DomainEvents.setTrackingEnabledForTest(process.env.NODE_ENV?.startsWith('test'));
     });
 
     it('Will call multiple subscribers with the event when it is dispatched', async function () {
@@ -78,6 +80,24 @@ describe('DomainEvents', function () {
         assert.equal(stub.calledTwice, true);
     });
 
+    it('works when tracking is disabled', async function () {
+        let handled = false;
+
+        DomainEvents.setTrackingEnabledForTest(false);
+
+        DomainEvents.subscribe(TestEvent, () => {
+            handled = true;
+        });
+
+        DomainEvents.dispatch(new TestEvent('No tracking'));
+        // Yield once so the async-wrapped handler can run before allSettled()
+        // resolves immediately with tracking disabled.
+        await sleep(0);
+        await DomainEvents.allSettled();
+
+        assert.equal(handled, true);
+    });
+
     describe('allSettled', function () {
         it('Resolves when there are no events', async function () {
             await DomainEvents.allSettled();
@@ -99,5 +119,39 @@ describe('DomainEvents', function () {
             await DomainEvents.allSettled();
             assert.equal(counter, 2);
         });
+
+        it('waits for every tracked listener before resolving', async function () {
+            let resolveFirst;
+            let resolveSecond;
+
+            DomainEvents.subscribe(TestEvent, () => {
+                return new Promise((resolve) => {
+                    resolveFirst = resolve;
+                });
+            });
+            DomainEvents.subscribe(TestEvent, () => {
+                return new Promise((resolve) => {
+                    resolveSecond = resolve;
+                });
+            });
+
+            DomainEvents.dispatch(new TestEvent('Hello, world!'));
+
+            let settled = false;
+            const allSettled = DomainEvents.allSettled().then(() => {
+                settled = true;
+            });
+
+            await sleep(0);
+            assert.equal(settled, false);
+
+            resolveFirst();
+            await sleep(0);
+            assert.equal(settled, false);
+
+            resolveSecond();
+            await allSettled;
+            assert.equal(settled, true);
+        });
     });
 });

…t/__snapshots__/example-app.test.js.snap …est_snapshots__/example-app.test.js.snappackages/express-test/test/__snapshots__/example-app.test.js.snap renamed to packages/express-test/test/__jest_snapshots__/example-app.test.js.snap packages/express-test/test/__snapshots__/example-app.test.js.snap renamed to packages/express-test/test/__jest_snapshots__/example-app.test.js.snap

@@ -1,4 +1,4 @@
-// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`Example App Set & Expect check body using snapshot matching errors correctly for random data 1: [body] 1`] = `
 Object {

packages/express-test/test/utils/overrides.js

@@ -1,6 +1,8 @@
 const { snapshotManager } = require('@tryghost/jest-snapshot');
-
+const SNAPSHOT_ROOT = '__jest_snapshots__';
 beforeAll(() => {
+    // Keep custom jest-snapshot files out of Vitest's native __snapshots__ discovery.
+    snapshotManager.defaultSnapshotRoot = SNAPSHOT_ROOT;
     snapshotManager.resetRegistry();
 });
 

packages/job-manager/test/job-manager.test.js

@@ -496,7 +496,9 @@ describe('Job Manager', function () {
                 assert.equal(JobModel.edit.args[1][0].status, 'failed');
 
                 // simulate process restart and "fresh" slate to add the job
-                jobManager.removeJob('failed-oneoff');
+                await jobManager.removeJob('failed-oneoff').catch((error) => {
+                    assert.match(error.message, /does not exist/i);
+                });
                 const completion2 = jobManager.awaitCompletion('failed-oneoff');
 
                 await jobManager.addOneOffJob({
@@ -704,6 +706,7 @@ describe('Job Manager', function () {
                         get: () => status,
                     }),
                 add: sinon.stub().resolves(),
+                edit: sinon.stub().resolves(),
             };
 
             jobManager = new JobManager({ JobModel, config: stubConfig });