tests use stub logger from test-helper

diff --git a/package-lock.json b/package-lock.json
index e47bc80e1094e3d4ef01a3616301591a10ec39c6..9c160571492ec95eeb409b7f5eb29651e05a8e24 100644 (file)
--- a/package-lock.json
+++ b/package-lock.json
@@ -19,8 +19,9 @@
       },
       "devDependencies": {
         "@squeep/eslint-config": "^1",
+        "@squeep/test-helper": "git+https://git.squeep.com/squeep-test-helper#v3.0.1",
         "eslint": "^9",
-        "husky": "^9.1.7",
+        "husky": "^9",
         "mocha": "^11",
         "nyc": "^17",
         "sinon": "^21"
@@ -1058,6 +1059,11 @@
         "node": ">=17"
       }
     },
+    "node_modules/@squeep/test-helper": {
+      "version": "3.0.1",
+      "resolved": "git+https://git.squeep.com/squeep-test-helper#062a2d18088fc8b41a126a0d8dcc3af6723d7bca",
+      "dev": true
+    },
     "node_modules/@tootallnate/quickjs-emscripten": {
       "version": "0.23.0",
       "resolved": "https://registry.npmjs.org/@tootallnate/quickjs-emscripten/-/quickjs-emscripten-0.23.0.tgz",

diff --git a/package.json b/package.json
index bf6c526b612c72dccfdffb5ae9b0247cd8bb7d65..34fe62e899ae97f2dd9efbadc53979527a50c652 100644 (file)
--- a/package.json
+++ b/package.json
@@ -31,6 +31,7 @@
   },
   "devDependencies": {
     "@squeep/eslint-config": "^1",
+    "@squeep/test-helper": "git+https://git.squeep.com/squeep-test-helper#v3.0.1",
     "eslint": "^9",
     "husky": "^9",
     "mocha": "^11",

diff --git a/test/src/authenticator.js b/test/src/authenticator.js
index 8d7755e5ae322a47c59b06fb9d2206c964ef1118..42629deb2b4a420a9475c1972255ebc8cb865385 100644 (file)
--- a/test/src/authenticator.js
+++ b/test/src/authenticator.js
@@ -1,5 +1,6 @@
 'use strict';
 
+const { StubLogger } = require('@squeep/test-helper');
 const assert = require('node:assert');
 const sinon = require('sinon');
 
@@ -13,7 +14,7 @@ describe('Authenticator', function () {
   let authenticator, logger, db, options;
 
   beforeEach(function () {
-    logger = { debug: () => {} };
+    logger = new StubLogger(sinon);
     db = {
       context: async (fn) => fn({}),
       getAuthById: async () => {},

diff --git a/test/src/manager.js b/test/src/manager.js
index c25d4e52f8f9e94b9f23693a8325a367ced3c80d..1607a19168d419e258c87cd833b26bde50785edf 100644 (file)
--- a/test/src/manager.js
+++ b/test/src/manager.js
@@ -1,5 +1,6 @@
 'use strict';
 
+const { StubLogger } = require('@squeep/test-helper');
 const assert = require('node:assert');
 const sinon = require('sinon');
 
@@ -11,15 +12,11 @@ const { ServeStaticFile, SlugGeneratorExhausted } = require('../../src/errors');
 const noExpectedException = 'did not get expected exception';
 
 describe('Manager', function () {
-  let manager, options;
+  let manager, logger, options;
   let res, ctx;
 
   beforeEach(function () {
-    const stubLogger = {
-      debug: sinon.stub(),
-      error: sinon.stub(),
-      info: sinon.stub(),
-    };
+    logger = new StubLogger(sinon);
     const stubDb = {
       context: async (fn) => await fn({}),
       transaction: async (_dbCtx, fn) => await fn({}),
@@ -38,7 +35,7 @@ describe('Manager', function () {
     ctx = {
       params: {},
     };
-    manager = new Manager(stubLogger, stubDb, options);
+    manager = new Manager(logger, stubDb, options);
 
   });
   afterEach(function () {

diff --git a/test/src/service.js b/test/src/service.js
index c19984e6ae6e97a32dd95e7725de75dc4bba71fc..eee860f461453694f94c8be5e15da2529de20f70 100644 (file)
--- a/test/src/service.js
+++ b/test/src/service.js
@@ -1,5 +1,6 @@
 'use strict';
 
+const { StubLogger } = require('@squeep/test-helper');
 const assert = require('node:assert');
 const sinon = require('sinon');
 
@@ -14,13 +15,7 @@ describe('service', function () {
 
   beforeEach(function () {
     options = {};
-    logger = process.env['VERBOSE_TESTS'] ? console : {
-      debug: () => undefined,
-      warn: () => undefined,
-      info: () => undefined,
-      error: () => undefined,
-      log: () => undefined,
-    };
+    logger = new StubLogger(sinon);
     asyncLocalStorage = {
       getStore: () => ({}),
     };