chore(tests): update testing dependencies (#2842)

* chore(tests): update mock implementations for consistency

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(tests): update mock implementations for consistency

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(tests): enhance type definitions for mock functions in test files

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(importV1): update coverage ignore comments for clarity

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(importV1): clarify coverage ignore comments and enhance logger tests

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(importV1): update v8 ignore comments for clarity

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(test-frontend): update docker build command to use --no-cache option

Signed-off-by: John Bair <john.bair@swirldslabs.com>

* chore(docker): update redis service to use official image and add keyspace notifications

Signed-off-by: John Bair <john.bair@swirldslabs.com>

---------

Signed-off-by: John Bair <john.bair@swirldslabs.com>

Author: jbair06

NOTICE

@@ -112,8 +112,9 @@ services:
     depends_on:
       - database
   redis:
-    build: ./redis
+    image: redis
     container_name: cache
+    command: redis-server --notify-keyspace-events Ex
     ports:
       - "6380:6379"
     volumes:

back-end/docker-compose.yaml

@@ -27,7 +27,7 @@
     "create-admin": "npx ts-node ./scripts/create-admin.ts",
     "create-users": "npx ts-node ./scripts/add-users.ts",
     "deploy:dev": "sh ./k8s/dev/deploy.sh",
-    "test:libs": "jest --testPathPattern='libs/.*\\.spec\\.ts$'",
+    "test:libs": "jest --testPathPatterns='libs/.*\\.spec\\.ts$'",
     "test:migrations": "pnpm --filter=typeorm test",
     "test:all": "INCLUDE_LIBS=0 pnpm --filter=api test && INCLUDE_LIBS=0 pnpm --filter=chain test && INCLUDE_LIBS=0 pnpm --filter=notifications test && pnpm test:libs && pnpm --filter=typeorm test",
     "test:cov:all": "pnpm --filter=api test:cov && pnpm --filter=chain test:cov && pnpm --filter=notifications test:cov && pnpm --filter=typeorm test:cov",
@@ -82,13 +82,13 @@
   "devDependencies": {
     "@eslint/eslintrc": "3.3.5",
     "@eslint/js": "10.0.1",
-    "@hashgraph/hedera-local": "2.39.4",
+    "@hashgraph/hedera-local": "2.40.1",
     "@nestjs/cli": "11.0.16",
     "@nestjs/schematics": "11.0.9",
     "@nestjs/testing": "catalog:nestjs-core",
-    "@testcontainers/postgresql": "11.13.0",
-    "@testcontainers/rabbitmq": "11.13.0",
-    "@testcontainers/redis": "11.13.0",
+    "@testcontainers/postgresql": "catalog:testcontainers",
+    "@testcontainers/rabbitmq": "catalog:testcontainers",
+    "@testcontainers/redis": "catalog:testcontainers",
     "@types/express": "5.0.6",
     "@types/jest": "catalog:",
     "@types/node": "catalog:",
@@ -104,12 +104,12 @@
     "eslint-plugin-prettier": "5.5.5",
     "globals": "17.6.0",
     "jest": "catalog:",
-    "jest-mock-extended": "4.0.0",
+    "jest-mock-extended": "4.0.1",
     "picocolors": "1.1.1",
     "prettier": "catalog:",
     "source-map-support": "0.5.21",
     "supertest": "7.2.2",
-    "testcontainers": "11.13.0",
+    "testcontainers": "catalog:testcontainers",
     "ts-jest": "catalog:",
     "ts-loader": "catalog:",
     "ts-node": "catalog:",

back-end/package.json

@@ -78,7 +78,7 @@
     "@types/semver": "catalog:",
     "@types/unzipper": "0.10.11",
     "@vitejs/plugin-vue": "5.2.4",
-    "@vitest/coverage-v8": "3.2.4",
+    "@vitest/coverage-v8": "catalog:vitest",
     "@vue/eslint-config-prettier": "10.2.0",
     "@vue/eslint-config-typescript": "14.1.3",
     "@vue/test-utils": "2.4.6",
@@ -97,7 +97,7 @@
     "vite-plugin-electron": "0.29.1",
     "vite-plugin-eslint": "1.8.1",
     "vite-plugin-vue-devtools": "7.7.9",
-    "vitest": "3.2.4",
+    "vitest": "catalog:vitest",
     "vitest-mock-extended": "3.1.1",
     "vue": "3.5.29",
     "vue-tsc": "2.2.12"

front-end/package.json

@@ -43,12 +43,11 @@ async function filterCandidate(filePath: string): Promise<V1ImportCandidate | nu
           transactionBytes: txHex,
           nodeSignatures: jsonObj,
         };
+      /* v8 ignore start */
       } else {
-        /* It's unclear how we can get a transaction without id => we disable code coverage here */
-        /* c8 ignore next */
         result = null
-        /* c8 ignore next */
       }
+      /* v8 ignore stop */
     } else {
       result = null;
     }

front-end/src/main/services/localUser/importV1.ts

@@ -23,15 +23,18 @@ vi.mock('path', () => ({
     join: vi.fn(),
   },
 }));
-vi.mock('better-sqlite3', () => ({
-  default: vi.fn().mockReturnValue({
-    prepare: vi.fn(() => ({
-      run: vi.fn(),
-    })),
+vi.mock('better-sqlite3', () => {
+  const mockInstance = {
+    prepare: vi.fn(() => ({ run: vi.fn() })),
     exec: vi.fn(),
     inTransaction: true,
-  }),
-}));
+  };
+  return {
+    default: vi.fn(function () {
+      return mockInstance;
+    }),
+  };
+});
 vi.mock('fsp', () => ({
   readdir: vi.fn(),
   readFile: vi.fn(),

front-end/src/tests/main/db/init.spec.ts

@@ -22,14 +22,18 @@ vi.mock('path', () => ({
 }));
 
 vi.mock('@prisma/adapter-better-sqlite3', () => ({
-  PrismaBetterSqlite3: vi.fn(() => ({})),
+  PrismaBetterSqlite3: vi.fn(function () {
+    return {};
+  }),
 }));
 
 vi.mock('@prisma/client', () => ({
-  PrismaClient: vi.fn(() => ({
-    $connect: vi.fn(),
-    $disconnect: vi.fn(),
-  })),
+  PrismaClient: vi.fn(function () {
+    return {
+      $connect: vi.fn(),
+      $disconnect: vi.fn(),
+    };
+  }),
 }));
 
 describe('Database path', () => {

front-end/src/tests/main/db/prisma.spec.ts

@@ -83,13 +83,16 @@ describe('Electron entry file', async () => {
     return handler[1];
   };
 
-  vi.mocked(BrowserWindow).mockReturnValue({
+  const mockBrowserWindowInstance = {
     on: vi.fn(),
     close: vi.fn(),
     isMinimized: vi.fn(),
     restore: vi.fn(),
     focus: vi.fn(),
-  } as unknown as BrowserWindow);
+  };
+  vi.mocked(BrowserWindow).mockImplementation(function () {
+    return mockBrowserWindowInstance;
+  } as unknown as typeof BrowserWindow);
 
   test('Should initialize the main process', async () => {
     is.dev = false;

front-end/src/tests/main/index.spec.ts

@@ -452,7 +452,7 @@ describe('registerUtilsListeners', () => {
       };
 
       vi.mocked(X509Certificate).mockImplementation(
-        () => mockX509Certificate as unknown as X509Certificate,
+        function () { return mockX509Certificate; } as unknown as typeof X509Certificate,
       );
       vi.mocked(createHash).mockReturnValue({
         update: vi.fn().mockReturnThis(),
@@ -480,7 +480,7 @@ describe('registerUtilsListeners', () => {
       };
 
       vi.mocked(X509Certificate).mockImplementation(
-        () => mockX509Certificate as unknown as X509Certificate,
+        function () { return mockX509Certificate; } as unknown as typeof X509Certificate,
       );
       vi.mocked(createHash).mockReturnValue({
         update: vi.fn().mockReturnThis(),
@@ -507,7 +507,7 @@ describe('registerUtilsListeners', () => {
       };
 
       vi.mocked(X509Certificate).mockImplementation(
-        () => mockX509Certificate as unknown as X509Certificate,
+        function () { return mockX509Certificate; } as unknown as typeof X509Certificate,
       );
       vi.mocked(createHash).mockReturnValue({
         update: vi.fn().mockReturnThis(),

front-end/src/tests/main/modules/ipcHandlers/utils.spec.ts

@@ -161,6 +161,18 @@ describe('logger', () => {
     );
   });
 
+  test('log method routes through info level', () => {
+    createLogger('renderer.websocket').log('test message');
+
+    expect(rootLogger.processMessage).toHaveBeenCalledWith(
+      expect.objectContaining({
+        data: ['test message'],
+        level: 'info',
+        logId: 'renderer.websocket',
+      }),
+    );
+  });
+
   test('uses fixed component names for updater and database loggers', () => {
     getAppUpdateLogger().info('Checking for update');
     getDatabaseLogger().error('Database failed');