Don't show fault coverage report if there are no mutantadvice provided

Author: jon-bell

dist/index.js

@@ -88,4 +88,4 @@
   "optionalDependencies": {
     "@rollup/rollup-linux-x64-gnu": "*"
   }
-}
+}

dist/index.js.map

@@ -12,10 +12,7 @@
           "type": "number"
         }
       },
-      "required": [
-        "minimumMutantsDetected",
-        "pointsToAward"
-      ],
+      "required": ["minimumMutantsDetected", "pointsToAward"],
       "type": "object"
     },
     "BuildConfig": {
@@ -34,30 +31,19 @@
           "additionalProperties": false,
           "properties": {
             "policy": {
-              "enum": [
-                "fail",
-                "warn",
-                "ignore"
-              ],
+              "enum": ["fail", "warn", "ignore"],
               "type": "string"
             },
             "preset": {
               "const": "checkstyle",
               "type": "string"
             }
           },
-          "required": [
-            "preset",
-            "policy"
-          ],
+          "required": ["preset", "policy"],
           "type": "object"
         },
         "preset": {
-          "enum": [
-            "java-gradle",
-            "python-script",
-            "none"
-          ],
+          "enum": ["java-gradle", "python-script", "none"],
           "type": "string"
         },
         "script_info": {
@@ -118,9 +104,7 @@
           "$ref": "#/definitions/VenvInfo"
         }
       },
-      "required": [
-        "preset"
-      ],
+      "required": ["preset"],
       "type": "object"
     },
     "GradedPart": {
@@ -139,10 +123,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "name",
-        "gradedUnits"
-      ],
+      "required": ["name", "gradedUnits"],
       "type": "object"
     },
     "GradedUnit": {
@@ -168,10 +149,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "name",
-        "path"
-      ],
+      "required": ["name", "path"],
       "type": "object"
     },
     "MutationTestUnit": {
@@ -193,11 +171,7 @@
           "type": "string"
         }
       },
-      "required": [
-        "name",
-        "locations",
-        "breakPoints"
-      ],
+      "required": ["name", "locations", "breakPoints"],
       "type": "object"
     },
     "OverlayPawtograderConfig": {
@@ -233,12 +207,7 @@
                 "type": "string"
               }
             },
-            "required": [
-              "sourceClass",
-              "targetClass",
-              "name",
-              "prompt"
-            ],
+            "required": ["sourceClass", "targetClass", "name", "prompt"],
             "type": "object"
           },
           "type": "array"
@@ -259,18 +228,11 @@
               "type": "array"
             }
           },
-          "required": [
-            "files",
-            "testFiles"
-          ],
+          "required": ["files", "testFiles"],
           "type": "object"
         }
       },
-      "required": [
-        "grader",
-        "build",
-        "submissionFiles"
-      ],
+      "required": ["grader", "build", "submissionFiles"],
       "type": "object"
     },
     "PawtograderConfig": {
@@ -305,12 +267,7 @@
           ]
         }
       },
-      "required": [
-        "name",
-        "tests",
-        "points",
-        "testCount"
-      ],
+      "required": ["name", "tests", "points", "testCount"],
       "type": "object"
     },
     "ScriptInfo": {
@@ -363,11 +320,8 @@
           "type": "string"
         }
       },
-      "required": [
-        "cache_key",
-        "dir_name"
-      ],
+      "required": ["cache_key", "dir_name"],
       "type": "object"
     }
   }
-}
+}

package.json

@@ -587,13 +587,15 @@ export class OverlayGrader extends Grader<OverlayPawtograderConfig> {
         studentMutationOutput += `${mutantsNotDetected.join('\n')}`
       }
       this.logger.log('hidden', studentMutationOutput)
-      testFeedbacks.push({
-        name: 'Fault Coverage Report',
-        output: studentMutationOutput,
-        output_format: 'markdown',
-        score: 0,
-        max_score: 0
-      })
+      if (this.config.mutantAdvice) {
+        testFeedbacks.push({
+          name: 'Fault Coverage Report',
+          output: studentMutationOutput,
+          output_format: 'markdown',
+          score: 0,
+          max_score: 0
+        })
+      }
     }
     if (this.config.build.student_tests?.student_impl?.report_branch_coverage) {
       const passingTestCount = studentTestResults?.filter(