Merge pull request github#7580 from github/repo-sync

Octomerger · web-flow · commit 7845f8c1a2e1 · 2021-06-23T00:54:59.000+10:00
repo sync
diff --git a/middleware/contextualizers/generic-toc.js b/middleware/contextualizers/generic-toc.js
@@ -50,7 +50,7 @@ module.exports = async function genericToc (req, res, next) {
 async function getTocItems (pagesArray, context, isRecursive, renderIntros) {
   return (await Promise.all(pagesArray.map(async (child) => {
     // Do not include hidden child items on a TOC page
-    if (child.page.hidden) return
+    if (child.page.hidden && !context.currentPath.includes('/early-access/')) return
 
     return {
       title: child.renderedFullTitle,
diff --git a/tests/unit/early-access.js b/tests/unit/early-access.js
@@ -1,6 +1,8 @@
 const fs = require('fs').promises
 const path = require('path')
 const { testViaActionsOnly } = require('../helpers/conditional-runs')
+const { getDOM } = require('../helpers/supertest')
+const got = require('got')
 
 describe('cloning early-access', () => {
   testViaActionsOnly('the content directory exists', async () => {
@@ -18,3 +20,25 @@ describe('cloning early-access', () => {
     expect(await fs.stat(eaDir)).toBeTruthy()
   })
 })
+
+describe('rendering early-access', () => {
+  jest.setTimeout(5 * 60 * 1000)
+
+  testViaActionsOnly('the top-level TOC renders locally', async () => {
+    const $ = await getDOM('/en/early-access')
+    expect($.html().includes('Hello, local developer! This page is not visible on production.')).toBe(true)
+    expect($('ul a').length).toBeGreaterThan(5)
+  })
+
+  testViaActionsOnly('the top-level TOC does not render on production', async () => {
+    async function getEarlyAccess () {
+      return await got('https://docs.github.com/en/early-access')
+    }
+    await expect(getEarlyAccess).rejects.toThrowError('Response code 404 (Not Found)')
+  })
+
+  testViaActionsOnly('TOCs display on category pages', async () => {
+    const $ = await getDOM('/en/early-access/github/enforcing-best-practices-with-github-policies')
+    expect($('ul a').length).toBeGreaterThan(5)
+  })
+})
