fix: add favicon test asset (#6868)

The favicon was taken from https://github.com/mathiasbynens/small. Issue: https://bugzilla.mozilla.org/show_bug.cgi?id=1682076
puppeteer · Feb 11, 2021 · a63f53c · a63f53c
1 parent 0889b7e
commit a63f53c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/test/assets/favicon.ico b/test/assets/favicon.ico
diff --git a/test/assets/offscreenbuttons.html b/test/assets/offscreenbuttons.html
@@ -1,3 +1,4 @@
+<!DOCTYPE html>
 <style>
   button {
     position: absolute;
@@ -29,8 +30,9 @@
 <button id=btn9>9</button>
 <button id=btn10>10</button>
 <script>
-window.addEventListener('DOMContentLoaded', () => {
-  for (const button of Array.from(document.querySelectorAll('button')))
-    button.addEventListener('click', () => console.log('button #' + button.textContent + ' clicked'), false);
-}, false);
+  for (const button of document.querySelectorAll('button')) {
+    button.addEventListener('click', () => {
+      console.log(`button #${button.textContent} clicked`);
+    });
+  }
 </script>
diff --git a/test/click.spec.ts b/test/click.spec.ts
@@ -132,7 +132,7 @@ describe('Page.click', function () {
       })
     ).toBe(text);
   });
-  itFailsFirefox('should click offscreen buttons', async () => {
+  it('should click offscreen buttons', async () => {
     const { page, server } = getTestState();
 
     await page.goto(server.PREFIX + '/offscreenbuttons.html');