Skip to content

Commit 04c9276

Browse files
author
Joel Griffith
committed
Adds a test case for the browser exiting during the diag check
1 parent 9200c04 commit 04c9276

File tree

1 file changed

+44
-1
lines changed

1 file changed

+44
-1
lines changed

x-pack/plugins/reporting/server/routes/diagnostic/browser.test.ts

Lines changed: 44 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ describe('POST /diagnose/browser', () => {
129129
});
130130
});
131131

132-
it('logs a message when the browser starts, but then crashes', async () => {
132+
it('logs a message when the browser starts, but then has problems later', async () => {
133133
registerDiagnoseBrowser(core, mockLogger);
134134

135135
await server.start();
@@ -169,6 +169,49 @@ describe('POST /diagnose/browser', () => {
169169
});
170170
});
171171

172+
it('logs a message when the browser starts, but then crashes', async () => {
173+
registerDiagnoseBrowser(core, mockLogger);
174+
175+
await server.start();
176+
177+
mockedCreateInterface.mockImplementation(() => ({
178+
addEventListener: (e: string, cb: any) => {
179+
setTimeout(() => cb(fontNotFoundMessage), 0);
180+
},
181+
removeEventListener: jest.fn(),
182+
removeAllListeners: jest.fn(),
183+
close: jest.fn(),
184+
}));
185+
186+
mockedSpawn.mockImplementation(() => ({
187+
removeAllListeners: jest.fn(),
188+
kill: jest.fn(),
189+
addEventListener: (e: string, cb: any) => {
190+
if (e === 'exit') {
191+
setTimeout(() => cb(), 5);
192+
}
193+
},
194+
removeEventListener: jest.fn(),
195+
}));
196+
197+
return supertest(httpSetup.server.listener)
198+
.post('/api/reporting/diagnose/browser')
199+
.expect(200)
200+
.then(({ body }) => {
201+
expect(body).toMatchInlineSnapshot(`
202+
Object {
203+
"help": Array [
204+
"The browser couldn't locate a default font. Please see https://www.elastic.co/guide/en/kibana/current/reporting-troubleshooting.html#reporting-troubleshooting-system-dependencies to fix this issue.",
205+
],
206+
"logs": "Could not find the default font
207+
Browser exited abnormally during startup
208+
",
209+
"success": false,
210+
}
211+
`);
212+
});
213+
});
214+
172215
it('cleans up process and subscribers', async () => {
173216
registerDiagnoseBrowser(core, mockLogger);
174217

0 commit comments

Comments
 (0)