Support for Aggregate Queries

[dplewis]

Support for distinct() and aggregate() queries.

Pending: parse-community/parse-server#4207

[oallouch]

That's really great. All the power of MongoDB in Parse !
Do you have prepared a little documentation update for this ?

[dplewis]

@oallouch Documentation will be coming soon!

[codecov]

Codecov Report

Merging #485 into master will increase coverage by 0.06%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master     #485      +/-   ##
==========================================
+ Coverage   84.03%   84.09%   +0.06%     
==========================================
  Files          47       47              
  Lines        3801     3830      +29     
  Branches      869      875       +6     
==========================================
+ Hits         3194     3221      +27     
- Misses        607      609       +2

Impacted Files	Coverage Δ
src/CoreManager.js	100% <100%> (ø)	⬆️
src/ParseQuery.js	94.01% <93.1%> (-0.08%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a209282...7e506c9. Read the comment docs.

[dplewis]

@flovilmart Can you check this?

[flovilmart]

code looks good, but would it be too much to ask for added Jest tests for this class?

[flovilmart]

Thanks @dplewis for this! 🙏

[dplewis]

@flovilmart Quick question. I was lazy and didn't require aggregate in the query controller https://github.com/dplewis/Parse-SDK-JS/blob/aggregate/src/CoreManager.js#L272

Is that ok? I can update all the tests if needed.

[flovilmart]

That would be better to add those methods now so we don’t have to come back to it later.

[flovilmart]

CoreManager-test is failing (it specifically validates the interface)

[flovilmart]

Perfect! Waiting for CI but it all looks great!

[dplewis]

@flovilmart How does this look?

[flovilmart]

Perfect 👌🏻