fix: don't exclude _error page from build

Author: danielcondemarin

lib/__tests__/getNextPagesFromBuildDir.test.js

@@ -73,15 +73,16 @@ describe("getNextPagesFromBuildDir", () => {
     });
   });
 
-  it("should skip _ pages like app, document and error", () => {
-    expect.assertions(1);
+  it("should skip _app and _document pages", () => {
+    expect.assertions(2);
 
     fs.readdir.mockImplementationOnce((path, cb) =>
-      cb(null, ["_app", "_document"])
+      cb(null, ["_app.js", "_document.js", "_error.js"])
     );
 
     return getNextPagesFromBuildDir("/path/to/build").then(nextPages => {
-      expect(nextPages).toEqual([]);
+      expect(nextPages).toHaveLength(1);
+      expect(nextPages[0].pageName).toEqual("_error");
     });
   });
 

lib/getNextPagesFromBuildDir.js

@@ -11,11 +11,16 @@ const logPages = nextPages => {
   logger.log(`Found ${pageNames.length} next page(s)`);
 };
 
+const excludeBuildFiles = ["_app.js", "_document.js", "compatLayer.js"];
+
 module.exports = async (buildDir, pageConfig = {}) => {
   const buildFiles = await readdirAsync(buildDir);
 
   const nextPages = buildFiles
-    .filter(bf => !bf.startsWith("_") && bf !== "compatLayer.js")
+    .filter(bf => {
+      console.log(bf);
+      return !excludeBuildFiles.includes(bf);
+    })
     .map(fileName => {
       const pagePath = path.join(buildDir, fileName);