GQL-52: Allow querying for Acls from within a group

[abbottry]

Overview

What is the feature?

Adds the ability to query for Acls that belong to a group, from within a group query.

Checklist

• I have added automated tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3e61d39) to head (505d858).

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #127   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          105       105           
  Lines         2323      2327    +4     
  Branches       257       258    +1     
=========================================
+ Hits          2323      2327    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dmistry1]

Here I think we can just return the dataSource, don't think a const is needed.

return dataSources.aclSourceFetch(
  handlePagingParams({
    ...args,
    permittedGroup: groupId
  }),
  context,
  parseResolveInfo(info)
) 

[abbottry]

Oh man, good catch, I was testing something.

[macrouch]

Do you have to worry about null here?

[abbottry]

No sir, we're good.