From 5d21ba316670c32ebf81ab7be7893d6b090b8954 Mon Sep 17 00:00:00 2001
From: Jason Rasmussen <jason@rasm.me>
Date: Fri, 2 May 2025 13:34:35 -0400
Subject: [PATCH] chore: logging clean up (#18031)

---
 server/src/bin/migrations.ts      | 2 +-
 server/test/medium.factory.ts     | 9 +++------
 server/test/medium/globalSetup.ts | 2 +-
 3 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/server/src/bin/migrations.ts b/server/src/bin/migrations.ts
index 7b850f6166..69070dc0cf 100644
--- a/server/src/bin/migrations.ts
+++ b/server/src/bin/migrations.ts
@@ -67,7 +67,7 @@ const runQuery = async (query: string) => {
 
 const runMigrations = async () => {
   const configRepository = new ConfigRepository();
-  const logger = new LoggingRepository(undefined, configRepository);
+  const logger = LoggingRepository.create();
   const db = getDatabaseClient();
   const databaseRepository = new DatabaseRepository(db, logger, configRepository);
   await databaseRepository.runMigrations();
diff --git a/server/test/medium.factory.ts b/server/test/medium.factory.ts
index 388c4df96b..89b1921819 100644
--- a/server/test/medium.factory.ts
+++ b/server/test/medium.factory.ts
@@ -142,18 +142,15 @@ export const getRepository = <K extends keyof RepositoriesTypes>(key: K, db: Kys
     }
 
     case 'database': {
-      const configRepo = new ConfigRepository();
-      return new DatabaseRepository(db, new LoggingRepository(undefined, configRepo), configRepo);
+      return new DatabaseRepository(db, LoggingRepository.create(), new ConfigRepository());
     }
 
     case 'email': {
-      const logger = new LoggingRepository(undefined, new ConfigRepository());
-      return new EmailRepository(logger);
+      return new EmailRepository(LoggingRepository.create());
     }
 
     case 'logger': {
-      const configMock = { getEnv: () => ({ noColor: false }) };
-      return new LoggingRepository(undefined, configMock as ConfigRepository);
+      return LoggingRepository.create();
     }
 
     case 'memory': {
diff --git a/server/test/medium/globalSetup.ts b/server/test/medium/globalSetup.ts
index e63c9f5224..4398da5c0a 100644
--- a/server/test/medium/globalSetup.ts
+++ b/server/test/medium/globalSetup.ts
@@ -42,7 +42,7 @@ const globalSetup = async () => {
   const db = new Kysely<DB>(getKyselyConfig({ connectionType: 'url', url: postgresUrl }));
 
   const configRepository = new ConfigRepository();
-  const logger = new LoggingRepository(undefined, configRepository);
+  const logger = LoggingRepository.create();
   await new DatabaseRepository(db, logger, configRepository).runMigrations();
 
   await db.destroy();