Skip to content

Commit 562274f

Browse files
Update backend mock and test suite
1 parent 7a92843 commit 562274f

2 files changed

Lines changed: 11 additions & 2 deletions

File tree

__mocks__/papi-backend.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ const mockSelectProject = jest.fn();
1010
const mockGetOpenWebViewDefinition = jest.fn();
1111
const mockOnDidOpenWebView = jest.fn();
1212
const mockOnDidCloseWebView = jest.fn();
13+
const mockRegisterValidator = jest.fn();
1314
const mockLogger = {
1415
debug: jest.fn(),
1516
error: jest.fn(),
@@ -24,6 +25,9 @@ const papi = {
2425
dialogs: {
2526
selectProject: mockSelectProject,
2627
},
28+
projectSettings: {
29+
registerValidator: mockRegisterValidator,
30+
},
2731
webViewProviders: {
2832
registerWebViewProvider: mockRegisterWebViewProvider,
2933
},
@@ -44,6 +48,7 @@ const defaultExport = {
4448
__mockGetOpenWebViewDefinition: mockGetOpenWebViewDefinition,
4549
__mockOnDidOpenWebView: mockOnDidOpenWebView,
4650
__mockOnDidCloseWebView: mockOnDidCloseWebView,
51+
__mockRegisterValidator: mockRegisterValidator,
4752
__mockLogger: mockLogger,
4853
};
4954

src/__tests__/main.test.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ interface PapiBackendTestMock {
1616
__mockGetOpenWebViewDefinition: jest.Mock;
1717
__mockOnDidOpenWebView: jest.Mock;
1818
__mockOnDidCloseWebView: jest.Mock;
19+
__mockRegisterValidator: jest.Mock;
1920
__mockLogger: { debug: jest.Mock; error: jest.Mock; info: jest.Mock; warn: jest.Mock };
2021
}
2122

@@ -34,6 +35,7 @@ function isPapiBackendTestMock(m: unknown): m is PapiBackendTestMock {
3435
'__mockGetOpenWebViewDefinition' in m &&
3536
'__mockOnDidOpenWebView' in m &&
3637
'__mockOnDidCloseWebView' in m &&
38+
'__mockRegisterValidator' in m &&
3739
'__mockLogger' in m
3840
);
3941
}
@@ -47,6 +49,7 @@ const {
4749
__mockGetOpenWebViewDefinition,
4850
__mockOnDidOpenWebView,
4951
__mockOnDidCloseWebView,
52+
__mockRegisterValidator,
5053
__mockLogger,
5154
} = papiBackendMock;
5255

@@ -117,6 +120,7 @@ describe('main', () => {
117120
beforeEach(() => {
118121
__mockRegisterWebViewProvider.mockResolvedValue({ dispose: jest.fn() });
119122
__mockRegisterCommand.mockResolvedValue({ dispose: jest.fn() });
123+
__mockRegisterValidator.mockResolvedValue({ dispose: jest.fn() });
120124
__mockOpenWebView.mockResolvedValue('mock-webview-id');
121125
__mockSelectProject.mockResolvedValue(undefined);
122126
__mockGetOpenWebViewDefinition.mockResolvedValue(undefined);
@@ -151,12 +155,12 @@ describe('main', () => {
151155
);
152156
});
153157

154-
it('adds all four registrations to the activation context', async () => {
158+
it('adds all five registrations to the activation context', async () => {
155159
const context = createTestActivationContext();
156160

157161
await activate(context);
158162

159-
expect(context.registrations.unsubscribers.size).toBe(4);
163+
expect(context.registrations.unsubscribers.size).toBe(5);
160164
});
161165

162166
it('logs activation start and finish', async () => {

0 commit comments

Comments
 (0)