Skip to content

Commit 383894e

Browse files
committed
Ran prettier on ts/js files
1 parent 1409e36 commit 383894e

6 files changed

Lines changed: 78 additions & 46 deletions

File tree

src/mcp/tools/addRemoteFileSources.spec.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,12 @@ describe('addRemoteFileSources', () => {
8383
},
8484
])(
8585
'should handle $scenario',
86-
async ({ folderUris, languageId, expectedParsedUris, expectedLanguageId }) => {
86+
async ({
87+
folderUris,
88+
languageId,
89+
expectedParsedUris,
90+
expectedLanguageId,
91+
}) => {
8792
const tool = createAddRemoteFileSourcesTool();
8893
const result = await tool.handler({ folderUris, languageId });
8994

src/services/ConfigService.spec.ts

Lines changed: 33 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,11 @@ describe('getImportPrefixes', () => {
359359
['not configured', undefined, undefined],
360360
['empty array', [], []],
361361
['single valid prefix', ['controller'], ['controller']],
362-
['multiple valid prefixes', ['controller', '_my_prefix'], ['controller', '_my_prefix']],
362+
[
363+
'multiple valid prefixes',
364+
['controller', '_my_prefix'],
365+
['controller', '_my_prefix'],
366+
],
363367
])('should return without errors when %s', (_label, given, expected) => {
364368
configMap.set(CONFIG_KEY.importPrefixes, given);
365369

@@ -371,21 +375,38 @@ describe('getImportPrefixes', () => {
371375

372376
it.each([
373377
['one invalid entry (empty string)', [''], [], ['']],
374-
['one invalid entry (starts with digit)', ['1controller'], [], ['1controller']],
378+
[
379+
'one invalid entry (starts with digit)',
380+
['1controller'],
381+
[],
382+
['1controller'],
383+
],
375384
['one invalid entry (contains hyphen)', ['my-prefix'], [], ['my-prefix']],
376385
['one invalid entry (dotted name)', ['my.prefix'], [], ['my.prefix']],
377-
['mixed valid and invalid entries', ['valid', '1bad', 'also_valid', 'my-prefix'], ['valid', 'also_valid'], ['1bad', 'my-prefix']],
378-
])('should filter invalid entries and show errors when %s', (_label, given, expectedResult, expectedInvalid) => {
379-
configMap.set(CONFIG_KEY.importPrefixes, given);
386+
[
387+
'mixed valid and invalid entries',
388+
['valid', '1bad', 'also_valid', 'my-prefix'],
389+
['valid', 'also_valid'],
390+
['1bad', 'my-prefix'],
391+
],
392+
])(
393+
'should filter invalid entries and show errors when %s',
394+
(_label, given, expectedResult, expectedInvalid) => {
395+
configMap.set(CONFIG_KEY.importPrefixes, given);
380396

381-
const result = ConfigService.getImportPrefixes();
397+
const result = ConfigService.getImportPrefixes();
382398

383-
expect(result).toEqual(expectedResult);
384-
expect(vscode.window.showErrorMessage).toHaveBeenCalledTimes(expectedInvalid.length);
385-
for (const invalid of expectedInvalid) {
386-
expect(vscode.window.showErrorMessage).toHaveBeenCalledWith(
387-
expect.stringContaining(`'${invalid}' is not a valid import prefix name`)
399+
expect(result).toEqual(expectedResult);
400+
expect(vscode.window.showErrorMessage).toHaveBeenCalledTimes(
401+
expectedInvalid.length
388402
);
403+
for (const invalid of expectedInvalid) {
404+
expect(vscode.window.showErrorMessage).toHaveBeenCalledWith(
405+
expect.stringContaining(
406+
`'${invalid}' is not a valid import prefix name`
407+
)
408+
);
409+
}
389410
}
390-
});
411+
);
391412
});

src/services/RemoteFileSourceService.spec.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,10 @@ import type { GroovyPackageName, PythonModuleFullname } from '../types';
1313
vi.mock('vscode');
1414
vi.mock('../util', async () => {
1515
const actual = await vi.importActual<typeof import('../util')>('../util');
16-
return { ...actual, getClearControllerPrefixesScript: vi.fn().mockReturnValue('') };
16+
return {
17+
...actual,
18+
getClearControllerPrefixesScript: vi.fn().mockReturnValue(''),
19+
};
1720
});
1821

1922
// ── setup ────────────────────────────────────────────────────────────────────

src/shared/msg/vscodePostMsg.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,4 +116,3 @@ export type VscodeSessionDetailsResponseMsg = {
116116
export type VscodeVariablePanelMsg =
117117
| VscodeLoginOptionsResponseMsg
118118
| VscodeSessionDetailsResponseMsg;
119-

src/util/pythonUtils.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ describe('extractControllerImportPrefixes', () => {
2020
])('%s', (_, make: (args: string) => string) => {
2121
it.each([
2222
['double-quoted prefix', '"my_prefix"', new Set(['my_prefix'])],
23-
["single-quoted prefix", `'my_prefix'`, new Set(['my_prefix'])],
23+
['single-quoted prefix', `'my_prefix'`, new Set(['my_prefix'])],
2424
['no arg defaults to "controller"', '', new Set(['controller'])],
2525
['whitespace inside parens', ' "spaced" ', new Set(['spaced'])],
2626
])('%s', (_label, args, expected) => {

src/webViews/variablePanel/main.ts

Lines changed: 34 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -16,36 +16,40 @@ const vscode = acquireVsCodeApi();
1616

1717
window.addEventListener(
1818
'message',
19-
({ data, origin }: MessageEvent<DhVariablePanelMsg | VscodeVariablePanelMsg>) => {
20-
logger.info('Received message:', JSON.stringify(data), origin);
21-
22-
// From DH iframe -> VS Code
23-
if (isLoginOptionsRequestFromDh(data)) {
24-
logger.info('Received login options request from iframe');
25-
vscode.postMessage({ data });
26-
return;
19+
({
20+
data,
21+
origin,
22+
}: MessageEvent<DhVariablePanelMsg | VscodeVariablePanelMsg>) => {
23+
logger.info('Received message:', JSON.stringify(data), origin);
24+
25+
// From DH iframe -> VS Code
26+
if (isLoginOptionsRequestFromDh(data)) {
27+
logger.info('Received login options request from iframe');
28+
vscode.postMessage({ data });
29+
return;
30+
}
31+
32+
if (isSessionDetailsRequestFromDh(data)) {
33+
logger.info('Received session details request from iframe');
34+
vscode.postMessage({ data });
35+
return;
36+
}
37+
38+
// From VS Code -> DH iframe
39+
if (isLoginOptionsResponseFromVscode(data)) {
40+
logger.info('Received login response from ext');
41+
const iframeWindow = getIframeContentWindow();
42+
iframeWindow.postMessage(data.payload, data.targetOrigin);
43+
return;
44+
}
45+
46+
if (isSessionDetailsResponseFromVscode(data)) {
47+
logger.info('Received session details from ext');
48+
const iframeWindow = getIframeContentWindow();
49+
iframeWindow.postMessage(data.payload, data.targetOrigin);
50+
return;
51+
}
2752
}
28-
29-
if (isSessionDetailsRequestFromDh(data)) {
30-
logger.info('Received session details request from iframe');
31-
vscode.postMessage({ data });
32-
return;
33-
}
34-
35-
// From VS Code -> DH iframe
36-
if (isLoginOptionsResponseFromVscode(data)) {
37-
logger.info('Received login response from ext');
38-
const iframeWindow = getIframeContentWindow();
39-
iframeWindow.postMessage(data.payload, data.targetOrigin);
40-
return;
41-
}
42-
43-
if (isSessionDetailsResponseFromVscode(data)) {
44-
logger.info('Received session details from ext');
45-
const iframeWindow = getIframeContentWindow();
46-
iframeWindow.postMessage(data.payload, data.targetOrigin);
47-
return;
48-
}
49-
});
53+
);
5054

5155
createDhIframe(vscode);

0 commit comments

Comments
 (0)