fix(firestore, iOS): account for filters in getAggregateFromServer with collection groups

[SelaseKay]

Related issues

Fixes #8253

Release Summary

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • Yes
• My change supports the following platforms;
  • Android
  • iOS
• My change includes tests;
  • e2e tests added or updated in packages/\*\*/e2e
  • jest tests added or updated in packages/\*\*/__tests__
• I have updated TypeScript types that are affected by my change.
• This is a breaking change;
  • Yes
  • No

Test Plan

---

Think react-native-firebase is great? Please consider supporting the project with any of the below:

• 👉 Star this repo on GitHub ⭐️
• 👉 Follow React Native Firebase and Invertase on Twitter

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
react-native-firebase	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 31, 2025 5:48pm

[CLAassistant]

All committers have signed the CLA.

[russellwheatley]

Nice one! Worth creating a couple of e2e tests specifically for collectionGroup here: https://github.com/invertase/react-native-firebase/blob/vertexai/packages/firestore/e2e/Aggregate/AggregateQuery.e2e.js#L103

Create a separate describe('collectionGroup') , and test count(), average() and sum() to ensure they all work 😄

[SelaseKay]

@russellwheatley, added test.

[russellwheatley]

@SelaseKay - Could you format the file? Might be able to do it in the file with keyboard shortcut. Alternatively, run yarn lint:js. It's a touch unreadable. also - merge with main. Cheers 😄

[russellwheatley]

See comment

[SelaseKay]

@russellwheatley , resolved

[russellwheatley]

@SelaseKay - did you merge with main? I'm seeing code that is already in main branch.

[SelaseKay]

@russellwheatley, merged main into firestore_8253 and pushed

[mikehardy]

This looks great to me @SelaseKay - nice

I left a confirmation comment for the original issue reporter to make certain the fix was only needed for iOS. By my analysis this is the complete fix because I think android was already working correctly but best to ask... #8253 (comment)