testilo 33.0.1 → 33.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testilo",
-  "version": "33.0.1",
+  "version": "33.0.3",
   "description": "Prepares and processes Testaro reports",
   "main": "call.js",
   "scripts": {

package/rescore.js

@@ -17,6 +17,9 @@ const {getNowStamp} = require('./procs/util');
 
 // Rescores a report.
 exports.rescore = (scorer, report, restrictionType, includedIDs) => {
+  const result = {
+    success: true
+  };
   // If tools are restricted:
   const {acts, id, score} = report;
   if (restrictionType === 'tools') {
@@ -30,8 +33,11 @@ exports.rescore = (scorer, report, restrictionType, includedIDs) => {
     }
     // Otherwise, i.e. if any tool included by the restriction is not in the report:
     else {
-      // Report this.
-      console.log(`ERROR: Report includes only tools ${Array.from(reportToolIDs).join(', ')}`);
+      // Report this and abort rescoring.
+      const message = `Report includes only tools ${Array.from(reportToolIDs).join(', ')}`;
+      result.success = false;
+      result.error = message;
+      console.log(`ERROR: ${message}`);
     }
   }
   // Otherwise, if issues are restricted:
@@ -93,16 +99,23 @@ exports.rescore = (scorer, report, restrictionType, includedIDs) => {
   }
   // Otherwise, i.e. if neither tools nor issues are restricted:
   else {
-    // Report this.
-    console.log('ERROR: Neither tools nor issues are restricted');
+    // Report this and abort rescoring.
+    const message = 'Neither tools nor issues are restricted';
+    result.success = false;
+    result.error = message;
+    console.log(`ERROR: ${message}`);
   }
   // Add rescoring data to the report.
   report.rescore = {
     originalID: id,
     timeStamp: getNowStamp(),
     restrictionType,
-    includedIDs
+    includedIDs,
+    result
+  }
+  // If rescoring was not aborted:
+  if (result.success) {
+    // Rescore the revised report.
+    scorer(report);
   }
-  // Score the revised report.
-  scorer(report);
 }