Skip to content

Commit

Permalink
test(vscode): switch to correct assertions for hover content sequenci…
Browse files Browse the repository at this point in the history
…ng, as it seems to be working correctly now, albeit possibly intermittently
  • Loading branch information
chrissimon-au committed Mar 17, 2024
1 parent ec0898a commit 9aecb5b
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions src/vscode/contextive/test/single-root/Hover.test.fs
Original file line number Diff line number Diff line change
Expand Up @@ -83,13 +83,6 @@ let tests =
let firstHoverContent = Seq.tryHead hoverContents
let secondHoverContent = hoverContents |> Seq.tail |> Seq.tryHead

// The commented lines are what is really expected. The uncommented lines are what is happening.
// See https://github.com/microsoft/vscode/issues/178184 for why
// Notwithstanding inability to test, this is working properly at runtime.
// Leaving the incorrect assertions in place so that we find out when the bug is fixed.

// expectHoverContent firstHoverContent "class Page"
// expectHoverContent secondHoverContent "All the content displayed in a browser when a user visits a url."
expectHoverContent secondHoverContent "class Page"
expectHoverContent firstHoverContent "All the content displayed in a browser when a user visits a url."
} ]
expectHoverContent firstHoverContent "class Page"
expectHoverContent secondHoverContent "All the content displayed in a browser when a user visits a url."
} ]

0 comments on commit 9aecb5b

Please sign in to comment.