conditionally clone docs-early-access by branchname (#24674)

* conditionally clone docs-early-access by branchname

* fix path

* right api

* the correct use of Octokit

* try that

* destructuring total_count

* test early access with fs.existsSync

* debugging

* bad lookup

* better lookup technique

* trying something

* always check it out

* with debugging

* with token

* with path

* better

* go for it

* only on docs-internal

* better name

* formatting

* tidying up

* correct octokit

* feedbacked

* correct hopefully

* final version

* disable test that checks for perfect translation-to-English file parity

* Accidentally committed in the wrong branch.

This reverts commit adee780718ea7414934e2257d17b0c004bd804af.

* match the azure-preview-env-deploy.yml

Author: peterbe

.github/workflows/test.yml

@@ -51,6 +51,52 @@ jobs:
           # Enables cloning the Early Access repo later with the relevant PAT
           persist-credentials: 'false'
 
+      - name: Figure out which docs-early-access branch to checkout, if internal repo
+        if: ${{ github.repository == 'github/docs-internal' }}
+        id: check-early-access
+        uses: actions/github-script@2b34a689ec86a68d8ab9478298f91d5401337b7d
+        env:
+          BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
+        with:
+          github-token: ${{ secrets.DOCUBOT_REPO_PAT }}
+          result-encoding: string
+          script: |
+            // If being run from a PR, this becomes 'my-cool-branch'.
+            // If run on main, with the `workflow_dispatch` action for
+            // example, the value becomes 'main'.
+            const { BRANCH_NAME } = process.env
+            try {
+              const response = await github.repos.getBranch({
+                owner: 'github',
+                repo: 'docs-early-access',
+                BRANCH_NAME,
+              })
+              console.log(`Using docs-early-access branch called '${BRANCH_NAME}'.`)
+              return BRANCH_NAME
+            } catch (err) {
+              if (err.status === 404) {
+                console.log(`There is no docs-early-access branch called '${BRANCH_NAME}' so checking out 'main' instead.`)
+                return 'main'
+              }
+              throw err
+            }
+
+      - name: Check out docs-early-access too, if internal repo
+        if: ${{ github.repository == 'github/docs-internal' }}
+        uses: actions/checkout@ec3a7ce113134d7a93b817d10a8272cb61118579
+        with:
+          repository: github/docs-early-access
+          token: ${{ secrets.DOCUBOT_REPO_PAT }}
+          path: docs-early-access
+          ref: ${{ steps.check-early-access.outputs.result }}
+
+      - name: Merge docs-early-access repo's folders
+        run: |
+          mv docs-early-access/assets assets/images/early-access
+          mv docs-early-access/content content/early-access
+          mv docs-early-access/data data/early-access
+          rm -r docs-early-access
+
       - name: Checkout LFS objects
         run: git lfs checkout
 
@@ -78,13 +124,6 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      - name: Clone early access
-        if: ${{ github.repository == 'github/docs-internal' }}
-        run: script/early-access/clone-for-build.js
-        env:
-          DOCUBOT_REPO_PAT: ${{ secrets.DOCUBOT_REPO_PAT }}
-          GIT_BRANCH: ${{ github.head_ref || github.ref }}
-
       - name: Cache nextjs build
         uses: actions/cache@937d24475381cd9c75ae6db12cb4e79714b926ed
         with:

tests/rendering/breadcrumbs.js

@@ -1,6 +1,7 @@
 import { getDOM, getJSON } from '../helpers/supertest.js'
 import { jest } from '@jest/globals'
 
+// TODO: Use `describeViaActionsOnly` instead. See tests/rendering/server.js
 const describeInternalOnly =
   process.env.GITHUB_REPOSITORY === 'github/docs-internal' ? describe : describe.skip
 // Breadcrumbs were moved to the Header and in the Menu for mobile, so there are now double the Breadcrumbs