Run prettier via GitHub Action

Author: adriaandotcom

playground/callback.html

@@ -254,7 +254,7 @@ <h2>Automated events</h2>
       });
 
       const callbackMetadataLink = document.querySelector(
-        "[data-metadata-callback]",
+        "[data-metadata-callback]"
       );
       callbackMetadataLink.addEventListener("click", (event) => {
         event.preventDefault();

playground/events.html

@@ -33,9 +33,9 @@
         },
         () => {
           console.info(
-            "Simple Analytics auto events: collected before_script_load",
+            "Simple Analytics auto events: collected before_script_load"
           );
-        },
+        }
       );
     </script>
 

src/auto-events.js

@@ -155,7 +155,7 @@
       /^https?:\/\//i.test(link.href) &&
       new RegExp(
         "\\.(" + (optionsLink.downloadsExtensions || []).join("|") + ")$",
-        "i",
+        "i"
       ).test(link.pathname)
     ) {
       collect = "download";

test/helpers/get-browsers.js

@@ -38,10 +38,10 @@ module.exports = async () => {
         browser_main_version: version(item.browser_version),
       }))
       .sort((left, right) =>
-        version(left.browser_version) > version(right.browser_version) ? -1 : 1,
+        version(left.browser_version) > version(right.browser_version) ? -1 : 1
       )
       .slice(0, 2),
-    ["os", "browser_main_version"],
+    ["os", "browser_main_version"]
   );
 
   const chrome = makeUnique(
@@ -54,18 +54,18 @@ module.exports = async () => {
         );
       })
       .sort((left, right) =>
-        version(left.browser_version) > version(right.browser_version) ? -1 : 1,
+        version(left.browser_version) > version(right.browser_version) ? -1 : 1
       ),
-    ["os", "os_version"],
+    ["os", "os_version"]
   );
 
   const ios = makeUnique(
     all
       .filter(({ browser }) => browser === "iphone")
       .sort((left, right) =>
-        version(left.browser_version) > version(right.browser_version) ? -1 : 1,
+        version(left.browser_version) > version(right.browser_version) ? -1 : 1
       ),
-    ["os", "os_version"],
+    ["os", "os_version"]
   );
 
   const android = makeUnique(
@@ -76,9 +76,9 @@ module.exports = async () => {
         os_main_version: parseInt(item.os_version.split(".")[0], 10),
       }))
       .sort((left, right) =>
-        version(left.os_version) > version(right.os_version) ? -1 : 1,
+        version(left.os_version) > version(right.os_version) ? -1 : 1
       ),
-    ["os", "os_main_version"],
+    ["os", "os_main_version"]
   );
 
   const safari = makeUnique(
@@ -96,10 +96,10 @@ module.exports = async () => {
         browser_main_version: version(item.browser_version),
       }))
       .sort((left, right) =>
-        left.browser_main_version > right.browser_main_version ? -1 : 1,
+        left.browser_main_version > right.browser_main_version ? -1 : 1
       )
       .slice(0, 4),
-    ["os", "browser_main_version"],
+    ["os", "browser_main_version"]
   ).slice(0, 5);
 
   const firefox = makeUnique(
@@ -112,9 +112,9 @@ module.exports = async () => {
         );
       })
       .sort((left, right) =>
-        version(left.browser_version) > version(right.browser_version) ? -1 : 1,
+        version(left.browser_version) > version(right.browser_version) ? -1 : 1
       ),
-    ["os"],
+    ["os"]
   );
 
   const opera = makeUnique(
@@ -128,9 +128,9 @@ module.exports = async () => {
         );
       })
       .sort((left, right) =>
-        version(left.browser_version) > version(right.browser_version) ? -1 : 1,
+        version(left.browser_version) > version(right.browser_version) ? -1 : 1
       ),
-    ["os"],
+    ["os"]
   );
 
   const ipads = makeUnique(
@@ -139,9 +139,9 @@ module.exports = async () => {
         return browser === "ipad";
       })
       .sort((left, right) =>
-        version(left.os_version) > version(right.os_version) ? -1 : 1,
+        version(left.os_version) > version(right.os_version) ? -1 : 1
       ),
-    ["os_version"],
+    ["os_version"]
   );
 
   const browsers = [

test/helpers/index.js

@@ -27,7 +27,7 @@ module.exports.makeUnique = (array = [], keys = []) => {
 
   return array.reduce((list, item) => {
     const has = list.find((listItem) =>
-      keys.every((key) => listItem[key] === item[key]),
+      keys.every((key) => listItem[key] === item[key])
     );
     if (!has) list.push(item);
     return list;
@@ -92,7 +92,7 @@ module.exports.navigate = async ({ browser, os, name, driver, commands }) => {
         `${amount || 1}x`,
         typeof exceeded === "number"
           ? `(found request in ${exceeded}ms)`
-          : `(exceeded timeout)`,
+          : `(exceeded timeout)`
       );
     } else if (script) {
       log(`script (${name})`, `${localhost}/?${searchParams}`);

test/helpers/server.js

@@ -66,7 +66,7 @@ const route = async (req, res) => {
       <html>
         <head><title>Simple Analytics Test</title></head>
         <body><h1>Path: ${pathname}</h1></body>
-      <html>`,
+      <html>`
     );
     return res.end();
   }

test/index.js

@@ -144,7 +144,7 @@ const log = (...messages) => DEBUG && console.log("    => Test:", ...messages);
 
 if (!BROWSERSTACK_USERNAME || !BROWSERSTACK_ACCESS_KEY) {
   console.error(
-    "BROWSERSTACK_USERNAME nor BROWSERSTACK_ACCESS_KEY are not defined.",
+    "BROWSERSTACK_USERNAME nor BROWSERSTACK_ACCESS_KEY are not defined."
   );
   process.exit(1);
 }
@@ -213,7 +213,7 @@ const getDeviceName = ({
 
   const suiteInstance = Mocha.Suite.create(
     mochaInstance.suite,
-    "Public Script Test Suite",
+    "Public Script Test Suite"
   );
 
   if (CI)
@@ -223,16 +223,16 @@ const getDeviceName = ({
         async function () {
           expect(
             browsers,
-            "Should have more than 20 browsers",
+            "Should have more than 20 browsers"
           ).to.have.lengthOf.at.least(20);
-        },
-      ),
+        }
+      )
     );
 
   suiteInstance.addTest(
     new Mocha.Test(`Test Node.js environment`, async function () {
       expect(process.version, "Should use Node.js 16.16").to.match(/^v16\.16/);
-    }),
+    })
   );
 
   const appendBrowserData = (browser) => {
@@ -409,7 +409,7 @@ const getDeviceName = ({
     const testName = `Testing ${browser.name} (${index + 1}/${total})`;
     const nextBrowser = browsers[index + 1];
     suiteInstance.addTest(
-      new Mocha.Test(testName, () => test(browser, nextBrowser)),
+      new Mocha.Test(testName, () => test(browser, nextBrowser))
     );
   }
 

test/test-beacon.js

@@ -9,48 +9,48 @@ module.exports = async () => {
 
   expect(
     beaconRequests,
-    "There are not enough beacon requests found",
+    "There are not enough beacon requests found"
   ).to.have.lengthOf(2);
 
   expect(
     beaconRequests[0].pathname,
-    "First beacon request should be send via GIF",
+    "First beacon request should be send via GIF"
   ).to.equal("/simple.gif");
 
   expect(
     beaconRequests[1].pathname,
-    "Second beacon request should be send via /append",
+    "Second beacon request should be send via /append"
   ).to.equal("/append");
 
   beaconRequests.map((request) => {
     expect(
       request,
-      "There are no page view requests with body found",
+      "There are no page view requests with body found"
     ).to.have.property("body");
 
     expect(
       request.body,
-      "All required keys should be present",
+      "All required keys should be present"
     ).to.include.all.keys(["duration", "scrolled", "original_id", "type"]);
 
     expect(
       UUIDvalidate(request.body.original_id, 4),
-      "original_id should be a valid UUIDv4",
+      "original_id should be a valid UUIDv4"
     ).to.be.true;
 
     expect(
       parseInt(request.body.scrolled, 10),
-      "Scrolled should be close to 35 percent",
+      "Scrolled should be close to 35 percent"
     ).to.be.closeTo(35, 5);
 
     expect(
       parseInt(request.body.duration, 10),
-      "Duration should be a valid number",
+      "Duration should be a valid number"
     ).to.be.a("number");
 
     expect(
       parseInt(request.body.duration, 10),
-      "Duration should be close to 0 or 1 second",
+      "Duration should be close to 0 or 1 second"
     ).to.be.closeTo(2, 2);
   });
 };

test/test-events.js

@@ -10,31 +10,31 @@ module.exports = async ({ os, browser }) => {
   expect(requests, "There should be two event requests").to.have.lengthOf(3);
 
   expect(requests[0].body.event, "Event should be 'event_123'").to.equal(
-    "event_123",
+    "event_123"
   );
 
   expect(requests[1].body.event, "Event should be 'functionoutput'").to.equal(
-    "functionoutput",
+    "functionoutput"
   );
 
   expect(requests[2].body.metadata, "Event should have metadata").to.contain(
-    '","bool":false,"int":20301,"string":"hi\'/301%20uas@#*0"}',
+    '","bool":false,"int":20301,"string":"hi\'/301%20uas@#*0"}'
   );
 
   expect(
     new Date(JSON.parse(requests[2].body.metadata).date),
-    "Event should have a date in the last 5 minutes",
+    "Event should have a date in the last 5 minutes"
   ).to.greaterThan(new Date(Date.now() - 300000));
 
   requests.map((request) => {
     expect(
       request,
-      "There are no event requests with body found",
+      "There are no event requests with body found"
     ).to.have.property("body");
 
     expect(
       request.body,
-      "All required keys should be present",
+      "All required keys should be present"
     ).to.include.all.keys([
       "version",
       "hostname",
@@ -47,34 +47,34 @@ module.exports = async ({ os, browser }) => {
 
     expect(
       UUIDvalidate(request.body.id, 4),
-      "event id should be a valid UUIDv4",
+      "event id should be a valid UUIDv4"
     ).to.be.true;
 
     expect(
       UUIDvalidate(request.body.session_id, 4),
-      "session_id should be a valid UUIDv4",
+      "session_id should be a valid UUIDv4"
     ).to.be.true;
 
     expect(
       UUIDvalidate(request.body.page_id, 4),
-      "page_id should be a valid UUIDv4",
+      "page_id should be a valid UUIDv4"
     ).to.be.true;
 
     if (os === "ios" || browser === "safari") {
       expect(
         request.body.hostname,
-        "Hostname should be bs-local.com on OS X Safari",
+        "Hostname should be bs-local.com on OS X Safari"
       ).to.equal("bs-local.com:3000");
     } else {
       expect(
         request.body.hostname,
-        "Hostname should be localhost on non iOS",
+        "Hostname should be localhost on non iOS"
       ).to.equal("localhost:3000");
     }
 
     expect(
       parseInt(request.body.version, 10),
-      "Version should be a valid number",
+      "Version should be a valid number"
     ).to.be.a("number");
   });
 };

test/test-no-pushstate.js

@@ -11,28 +11,28 @@ module.exports = async () => {
 
   expect(
     pageViewRequests,
-    "There are not enough page views requests found",
+    "There are not enough page views requests found"
   ).to.have.lengthOf(2);
 
   expect(
     pageViewRequests[0].body.unique,
-    "The first visit should be unique",
+    "The first visit should be unique"
   ).to.equal("true");
 
   expect(
     pageViewRequests[1].body.unique,
-    "The second visit should not be unique",
+    "The second visit should not be unique"
   ).to.equal("false");
 
   pageViewRequests.map((request) => {
     expect(
       request,
-      "There are no page view requests with body found",
+      "There are no page view requests with body found"
     ).to.have.property("body");
 
     expect(
       request.body,
-      "All required keys should be present",
+      "All required keys should be present"
     ).to.include.all.keys([
       "hostname",
       "https",
@@ -53,7 +53,7 @@ module.exports = async () => {
 
     expect(
       parseInt(request.body.version, 10),
-      "Version should be a valid number",
+      "Version should be a valid number"
     ).to.be.a("number");
 
     // We replace "https:" with "http:" string on CI