loadPages by 'en' only (#25846)

Author: peterbe

lib/page-data.js

@@ -30,11 +30,21 @@ const TRANSLATION_DRIFT_EXCEPTIONS = [
  * first since it's the most expensive work. This gets us a nested object with pages attached that we can use
  * as the basis for the siteTree after we do some versioning. We can also use it to derive the pageList.
  */
-export async function loadUnversionedTree() {
+export async function loadUnversionedTree(languagesOnly = null) {
+  if (languagesOnly && !Array.isArray(languagesOnly)) {
+    throw new Error("'languagesOnly' has to be an array")
+  }
   const unversionedTree = {}
 
+  const languagesValues = Object.entries(languages)
+    .filter(([language]) => {
+      return !languagesOnly || languagesOnly.includes(language)
+    })
+    .map(([, data]) => {
+      return data
+    })
   await Promise.all(
-    Object.values(languages).map(async (langObj) => {
+    languagesValues.map(async (langObj) => {
       const localizedContentPath = path.posix.join(__dirname, '..', langObj.dir, 'content')
       unversionedTree[langObj.code] = await createTree(localizedContentPath, langObj)
     })
@@ -129,7 +139,10 @@ export async function versionPages(obj, version, langCode, site) {
 
 // Derive a flat array of Page objects in all languages.
 export async function loadPageList(unversionedTree, languagesOnly = null) {
-  const rawTree = unversionedTree || (await loadUnversionedTree())
+  if (languagesOnly && !Array.isArray(languagesOnly)) {
+    throw new Error("'languagesOnly' has to be an array")
+  }
+  const rawTree = unversionedTree || (await loadUnversionedTree(languagesOnly))
   const pageList = []
 
   await Promise.all(

tests/rendering/pages-with-learning-tracks.js

@@ -1,21 +1,16 @@
-import { jest, beforeAll, expect } from '@jest/globals'
+import { jest, expect } from '@jest/globals'
 
 import { getDOM } from '../helpers/supertest.js'
 import { loadPages } from '../../lib/page-data.js'
 
 describe('process learning tracks', () => {
-  let pageList
-
   // Because calling `loadPages` will trigger a warmup, this can potentially
   // be very slow in CI. So we need a timeout.
   jest.setTimeout(60 * 1000)
 
-  beforeAll(async () => {
-    // Only doing English because they're the only files we do PRs for.
-    pageList = (await loadPages()).filter((page) => page.languageCode === 'en')
-  })
-
   test('pages with learningTracks ', async () => {
+    const pageList = await loadPages(undefined, ['en'])
+
     for (const page of pageList) {
       if (page.learningTracks && page.learningTracks.length > 0) {
         for (const permalink of page.permalinks) {