Skip to content

Commit 00be68c

Browse files
ColengmsCopilot
andcommitted
Require both hover details to toggle
Check size and alignment independently so a partial-hide regression cannot satisfy the hover integration test. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 032046d commit 00be68c

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

‎Extension/test/scenarios/MultirootDeadlockTest/tests/quickInfo.test.ts‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,11 @@ suite("[Quick info test]", function(): void {
7474
let hoverText: string = "";
7575
while (Date.now() < deadline) {
7676
hoverText = await getTypeHoverText();
77-
const hasSizeAndAlignment: boolean =
78-
hoverText.includes("**Size:**") && hoverText.includes("**Alignment:**");
79-
if (hoverText.includes("struct HoverType") && hasSizeAndAlignment === showSizeAndAlignment) {
77+
const showsSize: boolean = hoverText.includes("**Size:**");
78+
const showsAlignment: boolean = hoverText.includes("**Alignment:**");
79+
if (hoverText.includes("struct HoverType")
80+
&& showsSize === showSizeAndAlignment
81+
&& showsAlignment === showSizeAndAlignment) {
8082
return hoverText;
8183
}
8284
await testHelpers.delay(50);

0 commit comments

Comments
 (0)