@jrpool/kilotest 46.0.8 → 46.0.14

package/test/fixtures/db/reports/260101T0009-brd.json

@@ -0,0 +1,147 @@
+{
+  "id": "260101T0009-brd",
+  "what": "Branch Coverage Page",
+  "strict": false,
+  "standard": "only",
+  "device": {
+    "id": "default"
+  },
+  "browserID": "chromium",
+  "creationTimeStamp": "260101T0009",
+  "executionTimeStamp": "260101T0009",
+  "target": {
+    "what": "Branch Coverage Page",
+    "url": "https://example.com/branch"
+  },
+  "sources": {
+    "worker": "test-worker"
+  },
+  "acts": [
+    {
+      "type": "test",
+      "which": "axe",
+      "startTime": "26-01-01T00:00",
+      "endTime": "26-01-01T00:01",
+      "result": {
+        "standardResult": {
+          "instances": [
+            {
+              "ruleID": "r99",
+              "what": "r99",
+              "ordinalSeverity": 2,
+              "outcome": "failed",
+              "count": 1,
+              "catalogIndex": "0",
+              "checkpoint": 0,
+              "issueID": "duplicateID"
+            },
+            {
+              "ruleID": "r100",
+              "what": "Element has no role",
+              "ordinalSeverity": 1,
+              "outcome": "failed",
+              "catalogIndex": "1",
+              "checkpoint": 0,
+              "issueID": "duplicateID"
+            },
+            {
+              "ruleID": "r101",
+              "what": "Duplicate ID found",
+              "ordinalSeverity": 2,
+              "outcome": "failed",
+              "count": 1,
+              "catalogIndex": "2",
+              "checkpoint": 0,
+              "issueID": "duplicateID"
+            },
+            {
+              "ruleID": "r102",
+              "what": "Duplicate ID in orphan",
+              "ordinalSeverity": 2,
+              "outcome": "failed",
+              "count": 1,
+              "catalogIndex": "3",
+              "checkpoint": 0,
+              "issueID": "duplicateID"
+            }
+          ],
+          "outcomeTotals": {
+            "failed": 4,
+            "cantTell": 0
+          }
+        }
+      }
+    },
+    {
+      "type": "test",
+      "which": "alfa",
+      "startTime": "26-01-01T00:00",
+      "endTime": "26-01-01T00:01",
+      "result": {
+        "standardResult": {}
+      }
+    },
+    {
+      "type": "test",
+      "which": "ibm",
+      "startTime": "26-01-01T00:00",
+      "endTime": "26-01-01T00:01",
+      "result": {
+        "standardResult": {
+          "instances": [
+            {
+              "ruleID": "r50",
+              "what": "Link has no text",
+              "ordinalSeverity": 2,
+              "outcome": "failed",
+              "catalogIndex": "0",
+              "checkpoint": 0
+            }
+          ],
+          "outcomeTotals": {
+            "failed": 1,
+            "cantTell": 0
+          }
+        }
+      }
+    }
+  ],
+  "jobData": {
+    "startTime": "26-01-01T00:00",
+    "endTime": "26-01-01T00:10",
+    "elapsedSeconds": 600,
+    "issuelessRules": []
+  },
+  "catalog": {
+    "0": {
+      "tagName": "DIV",
+      "id": "",
+      "startTag": "<div>",
+      "text": "",
+      "textLinkable": false,
+      "boxID": "0:0:100:50",
+      "pathID": "/html/body/div[1]",
+      "headingIndex": "",
+      "checkpoint": 0
+    },
+    "1": {
+      "id": "",
+      "startTag": "<div>",
+      "text": null,
+      "textLinkable": false,
+      "boxID": "5:5:90:40",
+      "pathID": "/html/body/div[2]",
+      "headingIndex": "",
+      "checkpoint": 0
+    },
+    "2": {
+      "id": "",
+      "text": "No tags",
+      "textLinkable": false,
+      "headingIndex": "",
+      "checkpoint": 0
+    }
+  },
+  "images": {},
+  "checkpoints": []
+}

package/test/setup.cjs

@@ -0,0 +1,14 @@
+/*
+  test/setup.cjs
+  Clears alert configuration env vars before any modules are required,
+  so that tests calling sendAlert indirectly never trigger real email alerts.
+  This file is loaded via --require before any test file runs.
+
+  The env vars are set to empty strings rather than deleted, because
+  dotenv does not override existing env vars. This prevents the .env
+  file loaded by index.js from re-arming the alert configuration.
+*/
+
+for (const key of ['MANAGER_EMAIL', 'ALERT_API_HOST', 'ALERT_API_PATH', 'ALERT_API_KEY', 'ALERT_FROM']) {
+  process.env[key] = '';
+}

package/util.js

@@ -142,11 +142,18 @@ const getJobNames = exports.getJobNames = async () => {
   const jobNames = {};
   let fileNames;
   for (const category of ['queue', 'claimed', 'failed']) {
+    const categoryPath = path.join(jobsPath(), category);
     try {
-      fileNames = await fs.readdir(path.join(jobsPath(), category));
+      fileNames = await fs.readdir(categoryPath);
     }
     catch(error) {
-      return `ERROR: Job directory ${category} not readable (${error.message})`;
+      if (error.code === 'ENOENT') {
+        await fs.mkdir(categoryPath, {recursive: true});
+        fileNames = [];
+      }
+      else {
+        return `ERROR: Job directory ${category} not readable (${error.message})`;
+      }
     }
     jobNames[category] = fileNames;
   }
@@ -220,6 +227,9 @@ exports.getPOSTData = request => new Promise(resolve => {
       const query = querystring.parse(body);
       resolve(query);
     }
+    else {
+      resolve(null);
+    }
   });
 });
 // Returns the waiting test and retest recommendations.
@@ -604,9 +614,9 @@ const getPageData = exports.getPageData = async (timeStamp, jobID) => {
     return report;
   }
   const {url, what} = report.target;
-  const reportStats = await getReportStats(timeStamp, jobID)
-  // Otherwise, i.e. if it succeeded, get the elapsed time in days since the report was created.
-  const daysAgo = getAgoDays(reportStats.reportTime);
+  // Get the elapsed time in days since the report was completed, using the
+  // report content rather than the file system birth time.
+  const daysAgo = getAgoDays(new Date(`20${report.jobData.endTime}Z`));
   // Return the data.
   return {
     what,
@@ -640,7 +650,7 @@ exports.getPageDataStrings = async (timeStamp, jobID, pageData) => {
   };
 };
 // Returns the creation time and size of a report.
-const getReportStats = exports.getReportStats = async (timeStamp, jobID) => {
+exports.getReportStats = async (timeStamp, jobID) => {
   let reportStat;
   try {
     reportStat = await fs.stat(path.join(reportsPath(), `${timeStamp}-${jobID}.json`));