Skip to content

Commit

Permalink
Fix failing integration test on Windws with Chrome
Browse files Browse the repository at this point in the history
It fixes mozilla#18775.
  • Loading branch information
calixteman committed Oct 6, 2024
1 parent 81cf42d commit d6fee54
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions test/integration/highlight_editor_spec.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -1785,6 +1785,10 @@ describe("Highlight Editor", () => {
await page.keyboard.press("Escape");
await page.waitForSelector(`${editorSelector}:not(.selectedEditor)`);

const clickHandle = await waitForPointerUp(page);
y = rect.y - rect.height;
await page.mouse.move(x, y);

const counterHandle = await page.evaluateHandle(sel => {
const el = document.querySelector(sel);
const counter = { count: 0 };
Expand All @@ -1798,10 +1802,10 @@ describe("Highlight Editor", () => {
return counter;
}, editorSelector);

const clickHandle = await waitForPointerUp(page);
y = rect.y - rect.height;
await page.mouse.move(x, y);
await page.mouse.down();
await page.waitForSelector(
`.page[data-page-number = "1"] .annotationEditorLayer.drawing`
);
for (
const endY = rect.y + 2 * rect.height;
y <= endY;
Expand All @@ -1811,6 +1815,9 @@ describe("Highlight Editor", () => {
}
await page.mouse.up();
await awaitPromise(clickHandle);
await page.waitForSelector(
`.page[data-page-number = "1"] .annotationEditorLayer:not(.drawing)`
);

const { count } = await counterHandle.jsonValue();
expect(count).withContext(`In ${browserName}`).toEqual(0);
Expand Down

0 comments on commit d6fee54

Please sign in to comment.