Skip to content

Commit

Permalink
Enable disabled integration tests for Firefox
Browse files Browse the repository at this point in the history
  • Loading branch information
calixteman committed Aug 15, 2024
1 parent 8446d3a commit 75e5236
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
15 changes: 0 additions & 15 deletions test/integration/stamp_editor_spec.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -106,11 +106,6 @@ describe("Stamp Editor", () => {
it("must load a PNG which is bigger than a page", async () => {
await Promise.all(
pages.map(async ([browserName, page]) => {
if (browserName === "firefox") {
// Disabled in Firefox, because of https://bugzilla.mozilla.org/1553847.
return;
}

await switchToStamp(page);
await page.click("#editorStampAddImage");

Expand Down Expand Up @@ -138,11 +133,6 @@ describe("Stamp Editor", () => {
it("must load a SVG", async () => {
await Promise.all(
pages.map(async ([browserName, page]) => {
if (browserName === "firefox") {
// Disabled in Firefox, because of https://bugzilla.mozilla.org/1553847.
return;
}

await page.click("#editorStampAddImage");
const input = await page.$("#stampEditorFileInput");
await input.uploadFile(
Expand Down Expand Up @@ -183,11 +173,6 @@ describe("Stamp Editor", () => {
it("must check that an added image stay within the page", async () => {
await Promise.all(
pages.map(async ([browserName, page]) => {
if (browserName === "firefox") {
// Disabled in Firefox, because of https://bugzilla.mozilla.org/1553847.
return;
}

await switchToStamp(page);
const names = ["bottomLeft", "bottomRight", "topRight", "topLeft"];

Expand Down
6 changes: 0 additions & 6 deletions test/integration/viewer_spec.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -115,12 +115,6 @@ describe("PDF viewer", () => {
it("must check that we can zoom with the mouse wheel and pressed control key", async () => {
await Promise.all(
pages.map(async ([browserName, page]) => {
if (browserName === "firefox") {
// Skip this test for Firefox, as it's not working correctly.
// See https://github.com/puppeteer/puppeteer/issues/12093.
// TODO: Remove this check once the issue is resolved.
return;
}
await page.keyboard.down("Control");
let zoom = 10;
const zoomGetter = () =>
Expand Down

0 comments on commit 75e5236

Please sign in to comment.