Clean up config, final message

Author: mxmason

plugin/config.js

@@ -21,16 +21,16 @@ const getConfiguration = ({
   runners = runners || DEFAULT_RUNNERS;
 
   if (isInvalidRunner(runners)) {
-    throw new Error(`Invalid value for \`runners\` input. Runners must be ${AXE} or ${HTMLCS}`);
+    throw new Error(`Invalid value for \`runners\` input. Runners must be \`${AXE}\` or \`${HTMLCS}\`.`);
   }
   return {
     absolutePublishDir: PUBLISH_DIR || process.env.PUBLISH_DIR,
     checkPaths: checkPaths || DEFAULT_CHECK_PATHS,
     debugMode: debugMode || false,
     ignoreDirectories: ignoreDirectories || DEFAULT_IGNORE_DIRECTORIES,
+    resultMode: resultMode || DEFAULT_RESULT_MODE,
     pa11yOpts: {
       runners,
-      resultMode: resultMode || DEFAULT_RESULT_MODE,
       userAgent: PA11Y_USER_AGENT,
     }
   }

plugin/index.js

@@ -14,7 +14,8 @@ module.exports = {
       checkPaths,
       debugMode,
       ignoreDirectories,
-      pa11yOpts
+      pa11yOpts,
+      resultMode,
     } = getConfiguration({ constants, inputs })
     const htmlFilePaths = await pluginCore.generateFilePaths({
       absolutePublishDir,
@@ -32,8 +33,13 @@ module.exports = {
       pa11yOpts,
     });
     if (issueCount > 0) {
+      const postRunMsg = `Pa11y found ${issueCount} accessibility issues with your site! Check the logs above for more information.`
       console.log(results);
-      build.failBuild(`Pa11y found ${issueCount} accessibility issues with your site! Check the logs above for more information`);
+      if (resultMode === 'error') {
+        build.failBuild(postRunMsg)
+      } else {
+        console.warn(postRunMsg)
+      }
     }
 
   } catch(err) {