Avoid passing whole report to failBuild utility

Removes generatePostrunMessage function

Author: mxmason

src/index.js

@@ -22,26 +22,19 @@ module.exports = {
 				htmlFilePaths,
 				pa11yOpts,
 			})
-			const log = issueCount === 0 ? console.log : failWithIssues ? build.failBuild : console.warn
-			log(generatePostrunMessage(issueCount, report))
+			const reportSummary =
+				`${issueCount === 0 ? 'No' : issueCount} accessibility issues found!` +
+				(issueCount > 0 ? ' Check the logs for more information.' : '')
+
+			console.log(report)
+
+			if (failWithIssues) {
+				build.failBuild(reportSummary)
+			} else {
+				console.warn(pico.magenta(reportSummary))
+			}
 		} 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) {
-	// Only print a line between the report and the summary
-	// if the report has any contents.
-	const spacer = report.length > 0 ? '\n' : ''
-	const humanReadableCount = issueCount === 0 ? 'No' : issueCount
-	const summary = `${humanReadableCount} accessibility violations found! ${
-		issueCount > 0 ? 'Check the logs above for more information.' : ''
-	}`
-	return report + spacer + pico.magenta(summary)
-}