fix: liveQuery with containedIn not working when object field is an array

[stewones]

New Pull Request Checklist

• I am not disclosing a vulnerability.
• I am creating this PR in reference to issue LiveQuery not triggering event callbacks when using query.containedIn. #2088

Issue Description

previously it was not possible to use livequery with containedIn if the object field is an array
Related issue: #2088

Approach

I figured out that if notCointainedIn is supported, then there must be a way to make the other way around work.

TODOs before merging

• Add tests
• Add changes to documentation (guides, repository pages, in-code descriptions)
• Add security check
• Add new Parse Error codes to Parse JS SDK
• A changelog entry is created automatically using the pull request title (do not manually add a changelog entry)

[parse-github-assistant]

Thanks for opening this pull request!

• ❌ Please edit your post and use the provided template when creating a new pull request. This helps everyone to understand your post better and asks for essential information to quicker review the pull request.

[stewones]

you guys.

the issue is that containedIn only works currently for queries where the targeted field isn't an array. ie:

1st case

    const message = {
      id: new Id('Message', 'O1'),
      profile: pointer('Profile', 'abc'),
    };
    let q = new Parse.Query('Message');
    q.containedIn('profile', [
      Parse.Object.fromJSON({ className: 'Profile', objectId: 'abc' }),
      Parse.Object.fromJSON({ className: 'Profile', objectId: 'def' }),
    ]);
 expect(matchesQuery(message, q)).toBe(true);

ok works

2nd case

   const message = {
      id: new Id('Message', 'O2'),
      profiles: [pointer('Profile', 'yeahaw'), pointer('Profile', 'yes')],
    };

    let q = new Parse.Query('Message');
    q.containedIn('profiles', [
      Parse.Object.fromJSON({ className: 'Profile', objectId: 'no' }),
      Parse.Object.fromJSON({ className: 'Profile', objectId: 'yes' }),
    ]);

 expect(matchesQuery(message, q)).toBe(true);

not ok because profiles here is an array of pointers

so my PR aims to add support for the 2nd case above

thanks for your time reviewing this

[parse-github-assistant]

I will reformat the title to use the proper commit message syntax.

[codecov]

Codecov Report

Base: 94.20% // Head: 94.21% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (262ae8c) compared to base (3c75c2b).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##            alpha    #8128   +/-   ##
=======================================
  Coverage   94.20%   94.21%           
=======================================
  Files         182      182           
  Lines       13720    13724    +4     
=======================================
+ Hits        12925    12930    +5     
+ Misses        795      794    -1     

Impacted Files	Coverage Δ
src/LiveQuery/QueryTools.js	94.84% <100.00%> (+0.10%)	⬆️
src/Controllers/DatabaseController.js	93.89% <0.00%> (+0.14%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[stewones]

Heyo @mtrezza is there anything I can help with to get this merged?

[mtrezza]

This is just awaiting review, sorry for the long wait.

cc @parse-community/server-review

[dblythy]

Looks good!

[parseplatformorg]

🎉 This change has been released in version 5.3.0-alpha.23

[parseplatformorg]

🎉 This change has been released in version 5.4.0-beta.1

[parseplatformorg]

🎉 This change has been released in version 5.4.0-alpha.1

[parseplatformorg]

🎉 This change has been released in version 5.4.0

[parseplatformorg]

🎉 This change has been released in version 5.4.0