Ticket #3209. Adds check by include_in_menu attribute before appendin… (

#3210)

* Ticket #3209. Adds check by include_in_menu attribute before appending children categories to the returned array.
* Add test.
* fixup test falsy case

Signed-off-by: sirugh <rugh@adobe.com>

Co-authored-by: sirugh <rugh@adobe.com>
Co-authored-by: Andy Terranova <13182778+supernova-at@users.noreply.github.com>
Co-authored-by: Tommy Wiebell <twiebell@adobe.com>

packages/peregrine/lib/talons/CategoryTree/__tests__/useCategoryTree.spec.js

@@ -266,4 +266,38 @@ describe('child categories', () => {
         const { childCategories } = log.mock.calls[0][0];
         expect(childCategories.has(1)).toEqual(false);
     });
+
+    test('Does not include child category when include_in_menu is falsy', () => {
+        const categoryId = 5;
+        useLazyQuery.mockImplementation(() => {
+            return jest.fn().mockReturnValue([
+                getNavigationMenu,
+                {
+                    data: {
+                        category: {
+                            ...result.data.category,
+                            children: [
+                                ...result.data.category.children,
+                                {
+                                    id: categoryId,
+                                    include_in_menu: 0,
+                                    children_count: 0,
+                                    name: 'Five',
+                                    parentId: 1,
+                                    position: 0,
+                                    url_suffix: '.html',
+                                    url_path: '1/5'
+                                }
+                            ]
+                        }
+                    }
+                }
+            ])();
+        });
+
+        createTestInstance(<Component {...props} />);
+
+        const { childCategories } = log.mock.calls[0][0];
+        expect(childCategories.get(categoryId)).toBeUndefined();
+    });
 });

packages/peregrine/lib/talons/CategoryTree/useCategoryTree.js

@@ -70,9 +70,10 @@ export const useCategoryTree = props => {
         }
 
         children.map(category => {
-            const isLeaf = !parseInt(category.children_count);
-
-            childCategories.set(category.id, { category, isLeaf });
+            if (category.include_in_menu) {
+                const isLeaf = !parseInt(category.children_count);
+                childCategories.set(category.id, { category, isLeaf });
+            }
         });
 
         return childCategories;