Merge branch '523898-add-fallbacks-for-sentry-errors-group-select' into 'master'

Filter out null groups

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/184226



Merged-by: Alexander Turinske <aturinske@gitlab.com>
Approved-by: Artur Fedorov <afedorov@gitlab.com>

ee/app/assets/javascripts/security_orchestration/components/policy_editor/scan_result/action/group_select.vue

@@ -38,9 +38,9 @@ export default {
           const { __typename, avatarUrl, id, fullName, fullPath } = data.group;
           const rootGroupMatches = fullName.includes(this.search);
 
-          const descendantGroups = (data?.group?.descendantGroups?.nodes || []).map(
-            createGroupObject,
-          );
+          const descendantGroups = (data?.group?.descendantGroups?.nodes || [])
+            .filter((group) => group)
+            .map(createGroupObject);
 
           if (!rootGroupMatches) return descendantGroups;
 
@@ -48,7 +48,7 @@ export default {
           return uniqBy([...this.groups, rootGroup, ...descendantGroups], 'id');
         }
 
-        const groups = (data?.groups?.nodes || []).map(createGroupObject);
+        const groups = (data?.groups?.nodes || []).filter((group) => group).map(createGroupObject);
         return uniqBy([...this.groups, ...groups], 'id');
       },
       debounce: DEFAULT_DEBOUNCE_AND_THROTTLE_MS,

ee/spec/frontend/security_orchestration/components/policy_editor/scan_result/action/group_select_spec.js

@@ -45,6 +45,7 @@ const DESCENDANT_GROUP_RESPONSE = {
           {
             ...group2,
           },
+          null,
         ],
         __typename: 'GroupConnection',
       },
@@ -63,6 +64,7 @@ const NAMESPACE_GROUP_RESPONSE = {
         {
           ...group2,
         },
+        null,
       ],
       __typename: 'GroupConnection',
     },
@@ -187,7 +189,7 @@ describe('GroupSelect component', () => {
       });
     });
 
-    it('contains the root group and descendent group', async () => {
+    it('contains the root group and descendent group and filtes out null values', async () => {
       createComponent({ provide: { globalGroupApproversEnabled: false } });
       await waitForApolloAndVue();
       await waitForPromises();