From 9aecb5b8ab9c0d60e999007d981a3e6899ccdd99 Mon Sep 17 00:00:00 2001 From: Chris Simon Date: Sun, 17 Mar 2024 04:25:35 +0000 Subject: [PATCH] test(vscode): switch to correct assertions for hover content sequencing, as it seems to be working correctly now, albeit possibly intermittently --- .../contextive/test/single-root/Hover.test.fs | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/src/vscode/contextive/test/single-root/Hover.test.fs b/src/vscode/contextive/test/single-root/Hover.test.fs index 293645eb..85f4b214 100644 --- a/src/vscode/contextive/test/single-root/Hover.test.fs +++ b/src/vscode/contextive/test/single-root/Hover.test.fs @@ -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." + } ]