testaro 78.0.0 → 78.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testaro",
-  "version": "78.0.0",
+  "version": "78.0.2",
   "description": "Run 1300 web accessibility tests from 10 tools and get a standardized report",
   "main": "index.js",
   "scripts": {

package/procs/launch.js

@@ -83,9 +83,11 @@ const deSlash = url => (url || '').replace(/\/+$/, '');
 // Close a browser context and/or its browser, if they exist.
 const browserClose = exports.browserClose = async page => {
   if (page) {
-    const browserContext = page.context;
+    // Get the context (i.e. window) of the page and the browser of the context. These are methods, not properties; referencing them as properties made this function silently fail to close anything, because a function object has no close method and the resulting TypeError was caught and discarded.
+    const browserContext = page.context();
     if (browserContext) {
-      const {browser} = browserContext;
+      // The browser is null for a context not owned by a browser, such as a persistent context.
+      const browser = browserContext.browser();
       try {
         await browserContext.close();
       }

package/tests/testaro.js

@@ -324,7 +324,10 @@ const allRules = [
     what: 'motion without user request',
     contaminates: false,
     needsAccessibleName: false,
-    timeOut: 5,
+    // The budget must cover a full-page screenshot (itself allowed 4 seconds in
+    // procs/shoot.js), decoding two full-page PNGs, and a pixel comparison; 5
+    // seconds made the rule time out whenever an initial image existed.
+    timeOut: 30,
     defaultOn: true
   },
   {