Skip to content

Commit def118f

Browse files
chore(deps-dev): bump the testing group with 3 updates (#83)
* chore(deps-dev): bump the testing group with 3 updates Bumps the testing group with 3 updates: [@vitest/coverage-v8](https://github.com/vitest-dev/vitest/tree/HEAD/packages/coverage-v8), [@vitest/ui](https://github.com/vitest-dev/vitest/tree/HEAD/packages/ui) and [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest). Updates `@vitest/coverage-v8` from 3.2.4 to 4.0.7 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v4.0.7/packages/coverage-v8) Updates `@vitest/ui` from 3.2.4 to 4.0.7 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v4.0.7/packages/ui) Updates `vitest` from 3.2.4 to 4.0.7 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v4.0.7/packages/vitest) --- updated-dependencies: - dependency-name: "@vitest/coverage-v8" dependency-version: 4.0.7 dependency-type: direct:development update-type: version-update:semver-major dependency-group: testing - dependency-name: "@vitest/ui" dependency-version: 4.0.7 dependency-type: direct:development update-type: version-update:semver-major dependency-group: testing - dependency-name: vitest dependency-version: 4.0.7 dependency-type: direct:development update-type: version-update:semver-major dependency-group: testing ... Signed-off-by: dependabot[bot] <support@github.com> * fix: update package versions to remove caret (^) for specific dependencies * fix: add 'node' type to tsconfig for improved type checking * chore(tests): update mocking implementations and improve test coverage --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Mellisa Hankins <mellisa.hankins@iress.com>
1 parent d1cb7c4 commit def118f

File tree

20 files changed

+361
-234
lines changed

20 files changed

+361
-234
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
"ts-dedent": "^2.2.0",
4949
"tsx": "^4.20.6",
5050
"typescript": "^5.9.3",
51-
"vite": "^7.2.1",
51+
"vite": "7.2.1",
5252
"wait-on": "^9.0.1",
5353
"ws": "^8.18.3",
5454
"zx": "^8.8.5"

packages/components/package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@
4040
"@types/react": "^19.1.1",
4141
"@types/react-dom": "^19.1.1",
4242
"@vitejs/plugin-react": "^5.0.2",
43-
"@vitest/coverage-v8": "^3.2.4",
44-
"@vitest/ui": "^3.2.4",
43+
"@vitest/coverage-v8": "4.0.7",
44+
"@vitest/ui": "4.0.7",
4545
"concurrently": "^9.2.1",
4646
"eslint": "9.39.1",
4747
"eslint-plugin-file-progress": "^3.0.2",
@@ -63,12 +63,12 @@
6363
"rimraf": "^6.1.0",
6464
"storybook": "^9.0.0",
6565
"typescript": "5.9.3",
66-
"vite": "^7.2.1",
66+
"vite": "7.2.1",
6767
"vite-bundle-visualizer": "^1.2.1",
6868
"vite-plugin-dts": "4.5.4",
6969
"vite-plugin-static-copy": "3.1.4",
7070
"vite-tsconfig-paths": "^5.1.4",
71-
"vitest": "3.2.4"
71+
"vitest": "4.0.7"
7272
},
7373
"dependencies": {
7474
"@floating-ui/react": "0.27.16",

packages/components/src/components/Autocomplete/Autocomplete.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { act, render, screen } from '@testing-library/react';
1+
import { act, render, screen, waitFor } from '@testing-library/react';
22
import { axe } from 'jest-axe';
33
import { IressAutocomplete, IressAutocompleteProps } from '.';
44
import userEvent from '@testing-library/user-event';
@@ -323,7 +323,7 @@ describe('IressAutocomplete', () => {
323323

324324
const options = await screen.findAllByRole('option');
325325

326-
expect(options).toHaveLength(1);
326+
await waitFor(() => expect(options).toHaveLength(1));
327327
expect(options[0]).toHaveTextContent(MOCK_LABEL_VALUE_META[0].label);
328328
});
329329
});

packages/components/src/components/Popover/mocks/TestPopoverProvider.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ export const MOCK_FLOATING_UI_INTERACTIONS: PopoverHookReturn['interactions'] =
6464

6565
export const getMockPopoverContext = (
6666
contextProps: Partial<PopoverHookReturn> = {},
67-
) =>
67+
): PopoverHookReturn =>
6868
({
6969
...contextProps,
7070
activeIndex: contextProps.activeIndex ?? 0,

packages/mcp-server/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,15 +45,15 @@
4545
"@types/minimist": "1.2.5",
4646
"@types/node": "^24.10.0",
4747
"@types/turndown": "5.0.6",
48-
"@vitest/coverage-v8": "3.2.4",
48+
"@vitest/coverage-v8": "4.0.7",
4949
"minimist": "1.2.8",
5050
"playwright": "1.56.1",
5151
"rimraf": "6.1.0",
5252
"tsx": "4.20.6",
5353
"turndown": "7.2.2",
5454
"turndown-plugin-gfm": "1.0.2",
5555
"typescript": "5.9.3",
56-
"vitest": "3.2.4"
56+
"vitest": "4.0.7"
5757
},
5858
"repository": {
5959
"type": "git",

packages/mcp-server/scripts/generate.test.ts

Lines changed: 30 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import fs from 'fs';
77
// Mock external dependencies before importing
88
vi.mock('fs');
99
vi.mock('path');
10-
vi.mock('turndown');
10+
vi.mock('turndown', () => ({
11+
default: vi.fn(),
12+
}));
1113
vi.mock('turndown-plugin-gfm');
1214
vi.mock('playwright');
1315
vi.mock('chromium');
@@ -48,6 +50,10 @@ import { gfm } from 'turndown-plugin-gfm';
4850
import { Browser, chromium, Page } from 'playwright';
4951
import config from './config.js';
5052
import path from 'path';
53+
import type { Mock } from 'vitest';
54+
55+
// Cast mocked TurndownService as Mock
56+
const MockedTurndownService = TurndownService as unknown as Mock;
5157

5258
describe('generate.ts', () => {
5359
describe('checkDocItemsFileExists', () => {
@@ -186,12 +192,14 @@ describe('generate.ts', () => {
186192
use: mockUse,
187193
} as unknown as TurndownService;
188194

189-
// Make the constructor return our mock instance
190-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance);
195+
// Make the constructor return our mock instance (using function for constructor)
196+
MockedTurndownService.mockImplementation(function (this: unknown) {
197+
return mockTurndownInstance;
198+
});
191199

192200
const result = setupTurndownService();
193201

194-
expect(TurndownService).toHaveBeenCalled();
202+
expect(MockedTurndownService).toHaveBeenCalled();
195203
expect(mockUse).toHaveBeenCalledWith(gfm);
196204
expect(result).toBe(mockTurndownInstance);
197205
});
@@ -828,7 +836,9 @@ describe('generate.ts', () => {
828836
use: vi.fn(),
829837
turndown: vi.fn().mockReturnValue('# Test content'),
830838
};
831-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance as never);
839+
MockedTurndownService.mockImplementation(function (this: unknown) {
840+
return mockTurndownInstance as never;
841+
});
832842

833843
const result = await generateDocumentation({ force: true });
834844

@@ -884,7 +894,9 @@ describe('generate.ts', () => {
884894
use: vi.fn(),
885895
turndown: vi.fn().mockReturnValue('# Test content'),
886896
};
887-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance as never);
897+
MockedTurndownService.mockImplementation(function (this: unknown) {
898+
return mockTurndownInstance as never;
899+
});
888900

889901
await generateDocumentation({ force: false });
890902

@@ -915,9 +927,9 @@ describe('generate.ts', () => {
915927
use: vi.fn(),
916928
turndown: vi.fn().mockReturnValue('# Test content'),
917929
};
918-
vi.mocked(TurndownService).mockReturnValue(
919-
mockTurndownInstance as unknown as TurndownService,
920-
);
930+
MockedTurndownService.mockImplementation(function (this: unknown) {
931+
return mockTurndownInstance as unknown as TurndownService;
932+
});
921933

922934
const result = await generateDocumentation({ force: false });
923935

@@ -952,7 +964,9 @@ describe('generate.ts', () => {
952964
use: vi.fn(),
953965
turndown: vi.fn().mockReturnValue('# Test content'),
954966
};
955-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance as never);
967+
MockedTurndownService.mockImplementation(function (this: unknown) {
968+
return mockTurndownInstance as never;
969+
});
956970

957971
const result = await generateDocumentation({ force: false });
958972

@@ -1069,7 +1083,9 @@ describe('generate.ts', () => {
10691083
use: vi.fn(),
10701084
turndown: vi.fn().mockReturnValue('# Test content'),
10711085
};
1072-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance as never);
1086+
MockedTurndownService.mockImplementation(function (this: unknown) {
1087+
return mockTurndownInstance as never;
1088+
});
10731089

10741090
const result = await generateDocumentation({ force: false });
10751091

@@ -1165,7 +1181,9 @@ describe('generate.ts', () => {
11651181
use: vi.fn(),
11661182
turndown: vi.fn().mockReturnValue('# Test content'),
11671183
};
1168-
vi.mocked(TurndownService).mockReturnValue(mockTurndownInstance as never);
1184+
MockedTurndownService.mockImplementation(function (this: unknown) {
1185+
return mockTurndownInstance as never;
1186+
});
11691187

11701188
await generateDocumentation({ force: false }, customConfig);
11711189

packages/mcp-server/scripts/guidelines.test.ts

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,18 @@ vi.mock('fs', () => ({
4646
},
4747
}));
4848

49-
// Mock path
50-
vi.mock('path', async () => {
51-
const actual = await vi.importActual('path');
52-
return {
53-
...actual,
54-
join: vi.fn(),
55-
dirname: vi.fn(),
56-
};
57-
});
49+
// Mock path - we need to create mock functions that will be referenced
50+
const mockJoin = vi.fn();
51+
const mockDirname = vi.fn();
52+
53+
vi.mock('path', () => ({
54+
default: {
55+
join: mockJoin,
56+
dirname: mockDirname,
57+
},
58+
join: mockJoin,
59+
dirname: mockDirname,
60+
}));
5861

5962
describe('guidelines.ts', () => {
6063
let mockFs: {
@@ -73,18 +76,17 @@ describe('guidelines.ts', () => {
7376

7477
// Get mocked modules
7578
const fs = await import('fs');
76-
const path = await import('path');
7779

7880
mockFs = {
79-
existsSync: vi.mocked(fs.default.existsSync),
80-
readFileSync: vi.mocked(fs.default.readFileSync),
81-
writeFileSync: vi.mocked(fs.default.writeFileSync),
82-
mkdirSync: vi.mocked(fs.default.mkdirSync),
81+
existsSync: fs.default.existsSync as Mock,
82+
readFileSync: fs.default.readFileSync as Mock,
83+
writeFileSync: fs.default.writeFileSync as Mock,
84+
mkdirSync: fs.default.mkdirSync as Mock,
8385
};
8486

8587
mockPath = {
86-
join: vi.mocked(path.join),
87-
dirname: vi.mocked(path.dirname),
88+
join: mockJoin,
89+
dirname: mockDirname,
8890
};
8991

9092
// Mock console

packages/mcp-server/src/searchHandlers.test.ts

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,15 @@
11
/**
22
* Tests for search handlers functionality
33
*/
4-
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
4+
import {
5+
describe,
6+
it,
7+
expect,
8+
vi,
9+
beforeEach,
10+
afterEach,
11+
type Mock,
12+
} from 'vitest';
513
import {
614
handleGetUsageExamples,
715
handleSearchIdsDocs,
@@ -17,7 +25,13 @@ vi.mock('./config.js', () => ({
1725
DOCS_DIR: '/mock/docs/dir',
1826
}));
1927

20-
const mockUtils = vi.mocked(utils);
28+
const mockUtils = {
29+
getMarkdownFiles: utils.getMarkdownFiles as Mock,
30+
readFileContent: utils.readFileContent as Mock,
31+
fileExists: utils.fileExists as Mock,
32+
mapIressComponentToFile: utils.mapIressComponentToFile as Mock,
33+
extractIressComponents: utils.extractIressComponents as Mock,
34+
};
2135

2236
describe('searchHandlers', () => {
2337
beforeEach(() => {

packages/mcp-server/src/toolHandler.test.ts

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/**
22
* Tests for toolHandler.ts
33
*/
4-
import { describe, it, expect, vi, beforeEach } from 'vitest';
4+
import { describe, it, expect, vi, beforeEach, type Mock } from 'vitest';
55
import { handleToolCall } from './toolHandler.js';
66
import { type ToolResponse } from './types.js';
77
import {
@@ -40,19 +40,16 @@ vi.mock('./iressHandlers.js', () => ({
4040
}));
4141

4242
// Type the mocked functions
43-
const mockedHandleFindComponent = vi.mocked(handleFindComponent);
44-
const mockedHandleGetComponentProps = vi.mocked(handleGetComponentProps);
45-
const mockedHandleListComponents = vi.mocked(handleListComponents);
46-
const mockedHandleGetUsageExamples = vi.mocked(handleGetUsageExamples);
47-
const mockedHandleSearchIdsDocs = vi.mocked(handleSearchIdsDocs);
48-
const mockedHandleGetDesignTokens = vi.mocked(handleGetDesignTokens);
49-
const mockedHandleGetDesignGuidelines = vi.mocked(handleGetDesignGuidelines);
50-
const mockedHandleGetIressComponentInfo = vi.mocked(
51-
handleGetIressComponentInfo,
52-
);
53-
const mockedHandleAnalyzeComponentMentions = vi.mocked(
54-
handleAnalyzeComponentMentions,
55-
);
43+
const mockedHandleFindComponent = handleFindComponent as Mock;
44+
const mockedHandleGetComponentProps = handleGetComponentProps as Mock;
45+
const mockedHandleListComponents = handleListComponents as Mock;
46+
const mockedHandleGetUsageExamples = handleGetUsageExamples as Mock;
47+
const mockedHandleSearchIdsDocs = handleSearchIdsDocs as Mock;
48+
const mockedHandleGetDesignTokens = handleGetDesignTokens as Mock;
49+
const mockedHandleGetDesignGuidelines = handleGetDesignGuidelines as Mock;
50+
const mockedHandleGetIressComponentInfo = handleGetIressComponentInfo as Mock;
51+
const mockedHandleAnalyzeComponentMentions =
52+
handleAnalyzeComponentMentions as Mock;
5653

5754
describe('handleToolCall', () => {
5855
beforeEach(() => {

packages/mcp-server/src/utils.test.ts

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,15 @@
11
/**
22
* Tests for utility functions
33
*/
4-
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
4+
import {
5+
describe,
6+
it,
7+
expect,
8+
vi,
9+
beforeEach,
10+
afterEach,
11+
type Mock,
12+
} from 'vitest';
513
import * as fs from 'fs';
614
import type { ComponentMapping } from './types.js';
715
import {
@@ -18,7 +26,11 @@ vi.mock('./config.js', () => ({
1826
DOCS_DIR: '/mock/docs/dir',
1927
}));
2028

21-
const mockedFs = vi.mocked(fs);
29+
const mockedFs = {
30+
existsSync: fs.existsSync as Mock,
31+
readdirSync: fs.readdirSync as Mock,
32+
readFileSync: fs.readFileSync as Mock,
33+
};
2234

2335
// Test helper functions and types
2436
const createComponentMapping = (

0 commit comments

Comments
 (0)