Improve logging (#50)

* Install picocolors

* Log final output even when no a11y issues

Author: mxmason

package-lock.json

@@ -10,6 +10,7 @@
   "dependencies": {
     "pa11y": "^6.1.0",
     "path-type": "^4.0.0",
+    "picocolors": "^1.0.0",
     "readdirp": "^3.6.0"
   },
   "scripts": {

package.json

@@ -2,6 +2,7 @@
 
 const { getConfiguration } = require('./config')
 const pluginCore = require('./pluginCore')
+const pico = require('picocolors')
 
 module.exports = {
 	async onPostBuild({ constants, inputs, utils: { build } }) {
@@ -22,17 +23,23 @@ module.exports = {
 				htmlFilePaths,
 				pa11yOpts,
 			})
-			if (issueCount > 0) {
-				const postRunMsg = `Pa11y found ${issueCount} accessibility violations on your site! Check the logs above for more information.`
-				console.log(report)
-				if (failWithIssues) {
-					build.failBuild(postRunMsg)
-				} else {
-					console.warn(postRunMsg)
-				}
-			}
+			const log = issueCount === 0 ? console.log : failWithIssues ? build.failBuild : console.warn
+			log(generatePostrunMessage(issueCount, report))
 		} catch (err) {
 			build.failBuild(err.message)
 		}
 	},
 }
+
+/**
+ * Generates the message sent to the build log after a11y checks have been peformed.
+ * @param {number} issueCount
+ * @param {string} report
+ */
+function generatePostrunMessage(issueCount, report) {
+	const humanReadableCount = issueCount === 0 ? 'No' : issueCount
+	const summary = `${humanReadableCount} accessibility violations found! ${
+		issueCount > 0 ? 'Check the logs above for more information.' : ''
+	}`
+	return report + '\n' + pico.magenta(summary)
+}