From 54d79ba9983051d8d0ce0f30e433f1ba06a0013d Mon Sep 17 00:00:00 2001
From: Armin Samii <armin.samii@gmail.com>
Date: Thu, 4 Feb 2021 13:51:42 -0500
Subject: [PATCH] Bugfix: pa11y requires absolute paths for local files

See also: https://github.com/pa11y/pa11y
PUBLISH_DIR is a relative path. This changes it to use the config to get
the absolute path.
---
 plugin/index.js      | 6 +++---
 plugin/pluginCore.js | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/plugin/index.js b/plugin/index.js
index dac2629..efa1145 100644
--- a/plugin/index.js
+++ b/plugin/index.js
@@ -10,13 +10,13 @@ const pluginCore = require('./pluginCore');
 module.exports = {
     async onPostBuild({
       inputs: { checkPaths, ignoreDirectories, resultMode, debugMode },
-      constants: { PUBLISH_DIR },
-      utils: { build }
+      utils: { build },
+      netlifyConfig
     }) {
       const htmlFilePaths = await pluginCore.generateFilePaths({
         fileAndDirPaths: checkPaths,
         ignoreDirectories: ignoreDirectories || [],
-        PUBLISH_DIR
+        absolutePublishDir: netlifyConfig.build.publish
       });
       if (debugMode) {
         console.log({ htmlFilePaths });
diff --git a/plugin/pluginCore.js b/plugin/pluginCore.js
index e6ee4c9..5363cd4 100644
--- a/plugin/pluginCore.js
+++ b/plugin/pluginCore.js
@@ -36,7 +36,7 @@ const runPa11yOnFile = async function(htmlFilePath, build) {
 exports.generateFilePaths = async function({
   fileAndDirPaths, // array, mix of html and directories
   ignoreDirectories = [],
-  PUBLISH_DIR,
+  absolutePublishDir,
   testMode,
   debugMode
 }) {
@@ -46,7 +46,7 @@ exports.generateFilePaths = async function({
   );
   const htmlFilePaths = await Promise.all(
     fileAndDirPaths.map(fileAndDirPath =>
-      findHtmlFiles(`${PUBLISH_DIR}${fileAndDirPath}`, excludeDirGlobs)
+      findHtmlFiles(`${absolutePublishDir}${fileAndDirPath}`, excludeDirGlobs)
     )
   )
   return [].concat(...htmlFilePaths)