Skip to content

Commit e01e5d6

Browse files
hbmartinhbmartin
committed
prettier test formatting
1 parent 4432ff7 commit e01e5d6

9 files changed

Lines changed: 78 additions & 57 deletions

File tree

‎package.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@
4444
"lint": "eslint .",
4545
"lint:fix": "tsc --noEmit && eslint . --fix",
4646
"typecheck": "tsc --noEmit",
47-
"format": "prettier --write \"src/**/*.{ts,tsx,js,jsx,json,css,md}\"",
48-
"format:check": "prettier --check \"src/**/*.{ts,tsx,js,jsx,json,css,md}\"",
47+
"format": "prettier --write \"./**/*.{ts,tsx,js,jsx,json,css,md}\"",
48+
"format:check": "prettier --check \"./**/*.{ts,tsx,js,jsx,json,css,md}\"",
4949
"test": "vitest --run",
5050
"test:ui": "vitest --ui",
5151
"test:coverage": "vitest --coverage",

‎tests/lib/client.test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,10 +80,10 @@ describe('client module exports', () => {
8080
'useWebviewApi',
8181
'useVscodeState',
8282
'useLogger',
83-
'isViewApiRequest'
83+
'isViewApiRequest',
8484
];
8585

86-
expectedExports.forEach(exportName => {
86+
expectedExports.forEach((exportName) => {
8787
expect(exports).toContain(exportName);
8888
});
8989
});
@@ -101,4 +101,4 @@ describe('client module exports', () => {
101101
expect(exports.length).toBeLessThanOrEqual(10);
102102
});
103103
});
104-
});
104+
});

‎tests/lib/client/WebviewLogger.test.ts‎

Lines changed: 56 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -159,22 +159,34 @@ describe('WebviewLogger', () => {
159159
logger.error('Error message');
160160

161161
expect(mockVsCodeApi.postMessage).toHaveBeenCalledTimes(4);
162-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(1, expect.objectContaining({
163-
level: LogLevel.DEBUG,
164-
message: '[TestLogger] Debug message',
165-
}));
166-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(2, expect.objectContaining({
167-
level: LogLevel.INFO,
168-
message: '[TestLogger] Info message',
169-
}));
170-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(3, expect.objectContaining({
171-
level: LogLevel.WARN,
172-
message: '[TestLogger] Warning message',
173-
}));
174-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(4, expect.objectContaining({
175-
level: LogLevel.ERROR,
176-
message: '[TestLogger] Error message',
177-
}));
162+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
163+
1,
164+
expect.objectContaining({
165+
level: LogLevel.DEBUG,
166+
message: '[TestLogger] Debug message',
167+
})
168+
);
169+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
170+
2,
171+
expect.objectContaining({
172+
level: LogLevel.INFO,
173+
message: '[TestLogger] Info message',
174+
})
175+
);
176+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
177+
3,
178+
expect.objectContaining({
179+
level: LogLevel.WARN,
180+
message: '[TestLogger] Warning message',
181+
})
182+
);
183+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
184+
4,
185+
expect.objectContaining({
186+
level: LogLevel.ERROR,
187+
message: '[TestLogger] Error message',
188+
})
189+
);
178190
});
179191

180192
it('should handle complex data objects', () => {
@@ -200,12 +212,18 @@ describe('WebviewLogger', () => {
200212
logger.warn('Message with undefined', undefined);
201213

202214
expect(mockVsCodeApi.postMessage).toHaveBeenCalledTimes(2);
203-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(1, expect.objectContaining({
204-
data: null,
205-
}));
206-
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(2, expect.objectContaining({
207-
data: undefined,
208-
}));
215+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
216+
1,
217+
expect.objectContaining({
218+
data: null,
219+
})
220+
);
221+
expect(mockVsCodeApi.postMessage).toHaveBeenNthCalledWith(
222+
2,
223+
expect.objectContaining({
224+
data: undefined,
225+
})
226+
);
209227
});
210228
});
211229

@@ -214,28 +232,34 @@ describe('WebviewLogger', () => {
214232
const specialLogger = new WebviewLogger(mockVsCodeApi, 'Test-Logger_123');
215233
specialLogger.info('Test message');
216234

217-
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(expect.objectContaining({
218-
message: '[Test-Logger_123] Test message',
219-
}));
235+
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(
236+
expect.objectContaining({
237+
message: '[Test-Logger_123] Test message',
238+
})
239+
);
220240
});
221241

222242
it('should handle empty tag', () => {
223243
const emptyTagLogger = new WebviewLogger(mockVsCodeApi, '');
224244
emptyTagLogger.info('Test message');
225245

226-
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(expect.objectContaining({
227-
message: '[] Test message',
228-
}));
246+
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(
247+
expect.objectContaining({
248+
message: '[] Test message',
249+
})
250+
);
229251
});
230252

231253
it('should handle long tag names', () => {
232254
const longTag = 'VeryLongLoggerTagNameThatExceedsTypicalLength';
233255
const longTagLogger = new WebviewLogger(mockVsCodeApi, longTag);
234256
longTagLogger.info('Test message');
235257

236-
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(expect.objectContaining({
237-
message: `[${longTag}] Test message`,
238-
}));
258+
expect(mockVsCodeApi.postMessage).toHaveBeenCalledWith(
259+
expect.objectContaining({
260+
message: `[${longTag}] Test message`,
261+
})
262+
);
239263
});
240264
});
241-
});
265+
});

‎tests/lib/client/types.test.ts‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
2-
import { DeferredPromise, type WebviewLayout, type VsCodeApi, type StateReducer } from '../../../src/lib/client/types';
2+
import {
3+
DeferredPromise,
4+
type WebviewLayout,
5+
type VsCodeApi,
6+
type StateReducer,
7+
} from '../../../src/lib/client/types';
38

49
describe('client types', () => {
510
describe('DeferredPromise', () => {
@@ -296,4 +301,4 @@ describe('client types', () => {
296301
expect(newState.name).toBe('original');
297302
});
298303
});
299-
});
304+
});

‎tests/lib/host.test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,10 @@ describe('host module exports', () => {
9999
'disallowedLogKeys',
100100
'WebviewApiProvider',
101101
'BaseWebviewViewProvider',
102-
'isViewApiRequest'
102+
'isViewApiRequest',
103103
];
104104

105-
expectedExports.forEach(exportName => {
105+
expectedExports.forEach((exportName) => {
106106
expect(exports).toContain(exportName);
107107
});
108108
});
@@ -120,4 +120,4 @@ describe('host module exports', () => {
120120
expect(exports.length).toBeLessThanOrEqual(10);
121121
});
122122
});
123-
});
123+
});

‎tests/lib/host/WebviewApiProvider.test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,8 +122,8 @@ describe('WebviewApiProvider', () => {
122122
viewType: 'view2',
123123
webview: {
124124
...mockWebviewView.webview,
125-
postMessage: vi.fn().mockResolvedValue(true)
126-
}
125+
postMessage: vi.fn().mockResolvedValue(true),
126+
},
127127
};
128128
provider.registerView('test-view-2', view2);
129129

@@ -344,4 +344,4 @@ describe('WebviewApiProvider', () => {
344344
expect(mockWebviewView.webview.postMessage).toHaveBeenCalledTimes(views);
345345
});
346346
});
347-
});
347+
});

‎tests/lib/types/index.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,4 +99,4 @@ describe('types/index', () => {
9999
expect('isViewApiEvent' in rpcModule).toBe(true);
100100
});
101101
});
102-
});
102+
});

‎tsconfig.test.json‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,5 @@
55
"noEmit": true,
66
"types": ["vitest/globals", "node", "@testing-library/jest-dom"]
77
},
8-
"include": [
9-
"tests/**/*",
10-
"src/**/*"
11-
]
12-
}
8+
"include": ["tests/**/*", "src/**/*"]
9+
}

‎vite.config.ts‎

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,7 @@ export default defineConfig({
4242
coverage: {
4343
provider: 'v8',
4444
reporter: ['text', 'json', 'html'],
45-
exclude: [
46-
'node_modules/',
47-
'tests/',
48-
'*.config.ts',
49-
'src/**/*.stories.tsx',
50-
],
45+
exclude: ['node_modules/', 'tests/', '*.config.ts', 'src/**/*.stories.tsx'],
5146
},
5247
alias: {
5348
vscode: resolve(__dirname, 'tests/setup/__mocks__/vscode.ts'),

0 commit comments

Comments
 (0)