Skip to content

fix(testing): reduce flakiness in extension e2e tests #5481

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Aug 22, 2022
Merged

fix(testing): reduce flakiness in extension e2e tests #5481

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
b14f9e0
fix(e2e): update test extension
jsjoeio Aug 19, 2022
d8ab7d7
feat(e2e): add test extension methods to model
jsjoeio Aug 19, 2022
1be591c
fix(e2e): update extension tests
jsjoeio Aug 19, 2022
d136b13
Update test/e2e/models/CodeServer.ts
jsjoeio Aug 22, 2022
4dc187a
Update test/e2e/models/CodeServer.ts
jsjoeio Aug 22, 2022
e36a876
fixup
jsjoeio Aug 22, 2022
7906ebf
try asher approach
jsjoeio Aug 22, 2022
9c71a39
fixup: refactor and delete old methods
jsjoeio Aug 22, 2022
aea4c56
try again joe
jsjoeio Aug 22, 2022
567eb83
Update test/e2e/extensions.test.ts
jsjoeio Aug 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup
  • Loading branch information
@jsjoeio
jsjoeio committed Aug 22, 2022
commit e36a8762ef1430c5fce8da81489c1a9291413c4c
1 change: 0 additions & 1 deletion test/e2e/models/CodeServer.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,6 @@ export class CodeServerPage {
const selector = "text=test extension loaded"
this.codeServer.logger.debug("Waiting for test extension to load...")

await this.page.waitForLoadState("load")
const loaded = await this.page.isVisible(selector)

this.codeServer.logger.debug(`Test extension has ${loaded ? "" : "not"} loaded`)
Expand Down