Skip to content

Commit 4dfe2c2

Browse files
committed
making use of the global mocks file
1 parent 2ae4c88 commit 4dfe2c2

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

__mocks__/vscode.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ const MockTreeItemCollapsibleState = {
3636
Expanded: 2
3737
};
3838

39+
const registerTreeDataProviderMock = jest.fn(() => ({ dispose: jest.fn() }));
40+
3941
class MockTreeItem {
4042
label;
4143
description;
@@ -71,6 +73,7 @@ module.exports = {
7173
warn: jest.fn(),
7274
error: jest.fn(),
7375
})),
76+
registerTreeDataProvider: registerTreeDataProviderMock,
7477
showWarningMessage: jest.fn(),
7578
createStatusBarItem: jest.fn(),
7679
showQuickPick: jest.fn(),

src/views/live-watch/live-watch.test.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ import { GDBTargetDebugSession, GDBTargetDebugTracker } from '../../debug-sessio
2222
import { gdbTargetConfiguration } from '../../debug-configuration/debug-configuration.factory';
2323
import { GDBTargetConfiguration } from '../../debug-configuration';
2424

25-
// Inline mock for registerTreeDataProvider specific to these tests
26-
const registerTreeDataProviderMock = jest.fn(() => ({ dispose: jest.fn() }));
2725

2826
describe('LiveWatchTreeDataProvider', () => {
2927
let liveWatchTreeDataProvider: LiveWatchTreeDataProvider;
@@ -50,19 +48,15 @@ describe('LiveWatchTreeDataProvider', () => {
5048

5149
describe('session management and connection tests', () => {
5250
it('should activate the live watch tree data provider', () => {
53-
(vscode.window).registerTreeDataProvider = registerTreeDataProviderMock;
5451
liveWatchTreeDataProvider.activate(tracker);
5552
});
5653

5754
it('registers the live watch tree data provider', async () => {
58-
(vscode.window).registerTreeDataProvider = registerTreeDataProviderMock;
59-
registerTreeDataProviderMock.mockClear();
6055
liveWatchTreeDataProvider.activate(tracker);
61-
expect(registerTreeDataProviderMock).toHaveBeenCalledWith('cmsis-debugger.liveWatch', liveWatchTreeDataProvider);
56+
expect(vscode.window.registerTreeDataProvider).toHaveBeenCalledWith('cmsis-debugger.liveWatch', liveWatchTreeDataProvider);
6257
});
6358

6459
it('manages session lifecycles correctly', async () => {
65-
(vscode.window).registerTreeDataProvider = registerTreeDataProviderMock;
6660
liveWatchTreeDataProvider.activate(tracker);
6761
// No active session yet
6862
expect((liveWatchTreeDataProvider as any).activeSession).toBeUndefined();
@@ -221,7 +215,6 @@ describe('LiveWatchTreeDataProvider', () => {
221215
describe('command registration', () => {
222216
beforeEach(() => {
223217
(vscode.commands as any).registerCommand?.mockClear?.();
224-
(vscode.window as any).registerTreeDataProvider = registerTreeDataProviderMock;
225218
});
226219

227220
function getRegisteredHandler(commandId: string) {

0 commit comments

Comments
 (0)