Skip to content

Commit e913165

Browse files
authored
Merge pull request #1463 from github/aeisenberg/bump-timeout
2 parents 4d6076c + ef94bb3 commit e913165

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

extensions/ql-vscode/src/vscode-tests/no-workspace/telemetry.test.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -243,15 +243,16 @@ describe('telemetry reporting', function() {
243243
});
244244
});
245245

246-
it('should request permission if popup has never been seen before', async () => {
246+
it('should request permission if popup has never been seen before', async function() {
247+
this.timeout(3000);
247248
sandbox.stub(window, 'showInformationMessage').resolvesArg(3 /* "yes" item */);
248249
await ctx.globalState.update('telemetry-request-viewed', false);
249250
await enableTelemetry('codeQL.telemetry', false);
250251

251252
await telemetryListener.initialize();
252253

253-
// Wait 50 ms for user's selection to propagate in settings.
254-
await wait(50);
254+
// Wait for user's selection to propagate in settings.
255+
await wait(500);
255256

256257
// Dialog opened, user clicks "yes" and telemetry enabled
257258
expect(window.showInformationMessage).to.have.been.calledOnce;

0 commit comments

Comments
 (0)