Skip to content

Commit a5c8211

Browse files
CoderCococlaude
andcommitted
test(desktop-main): remove empty beforeEach from logger singleton suite
The block had a stale comment claiming it reset module state but its body was empty. `initialLogger` is captured at static import time (before any test runs) so isolation is already guaranteed; no reset hook is needed. Refs #149. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent a1133ec commit a5c8211

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

app/packages/desktop-main/src/logger.test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { describe, it, expect, beforeEach } from 'vitest';
1+
import { describe, it, expect } from 'vitest';
22
import winston from 'winston';
33
import DailyRotateFile from 'winston-daily-rotate-file';
44

@@ -51,10 +51,6 @@ describe('createLogger', () => {
5151
});
5252

5353
describe('logger singleton', () => {
54-
beforeEach(() => {
55-
// Reset to console-only by re-creating; each test starts fresh.
56-
});
57-
5854
it('should reassign the exported logger binding when createLogger is called', async () => {
5955
// Capture the initial (console-only) reference.
6056
const beforeCall = initialLogger;

0 commit comments

Comments
 (0)