Skip to content

e2e tests: disable flaky jump-to-definition test

Warren Gifford requested to merge mrn/disable-flaky-e2e into main

Created by: mrnugget

I ran into this multiple times over the past few weeks.

Example failure: https://buildkite.com/sourcegraph/sourcegraph/builds/164530#01825985-d71b-4e4c-9bf7-bcb87ec6c596

Important part of error message:

Expected: "http://localhost:7080/github.com/sourcegraph/go-diff@3f415a150aec0685cb81b73cc201e762e075006d/-/blob/diff/parse.go?L29:6"
Received: "http://localhost:7080/github.com/sourcegraph/go-diff@3f415a150aec0685cb81b73cc201e762e075006d/-/blob/diff/parse.go?L25:30#tab=def"

Line/col are different and the tab=def too. The "Received" contains the current line/col but the expectation is that it's already navigated away.

Maybe there needs to be a timeout to ensure that the client-side code in WebHoverOverlay that turns the token into a link has been called?

Test plan

  • N/A

App preview:

Check out the client app preview documentation to learn more.

Merge request reports