Compare page and node URLs less strictly when choosing between multiple indices (#936)

This fixes an issue where the URL path in the index data may
additionally have a prefix before the /documentation segment. This
change allows the comparison to still work in that scenario where there
are multiple top-level trees of data provided in the Index JSON.

Resolves: rdar://147829785

Author: mportiz08

src/utils/navigatorData.js

@@ -193,7 +193,7 @@ function extractRootNode(data) {
   //
   // otherwise, the first provided node will be used
   return data.length === 1 ? data[0] : (data.find(node => (
-    node.path.toLowerCase() === rootPath.toLowerCase()
+    node.path.toLowerCase().endsWith(rootPath.toLowerCase())
   )) ?? data[0]);
 }
 

tests/unit/utils/navigatorData.spec.js

@@ -357,6 +357,18 @@ describe('when multiple top-level children are provided', () => {
       },
     ],
   };
+  const b2 = {
+    type: 'module',
+    title: 'b',
+    path: '/some/prefix/documentation/b',
+    children: [
+      {
+        type: 'article',
+        title: 'b1',
+        path: '/some/prefix/documentation/b/b1',
+      },
+    ],
+  };
 
   describe('flattenNavigationIndex', () => {
     it('prefers the root child with the same url path prefix', () => {
@@ -376,6 +388,9 @@ describe('when multiple top-level children are provided', () => {
       flattenedIndex = flattenNavigationIndex({ swift: [a, b] });
       expect(flattenedIndex.swift.length).toBe(1);
       expect(flattenedIndex.swift[0].title).toBe(b.children[0].title);
+      flattenedIndex = flattenNavigationIndex({ swift: [a, b2] });
+      expect(flattenedIndex.swift.length).toBe(1);
+      expect(flattenedIndex.swift[0].title).toBe(b2.children[0].title);
 
       // fallback to first root node when multiple top-level nodes are provided
       // and none of them is a "module"
@@ -406,6 +421,8 @@ describe('when multiple top-level children are provided', () => {
       // prefers root node with same url path prefix when multiple are provided
       props = extractTechnologyProps({ swift: [a, b] });
       expect(props.swift.technology).toBe(b.title);
+      props = extractTechnologyProps({ swift: [a, b2] });
+      expect(props.swift.technology).toBe(b2.title);
 
       // fallback to first root node when multiple top-level nodes are provided
       // and none of them is a "module"