ENGTAI-68310 : Adding logic and fixing order

[Sukhamjot-Singh]

Description

There a was a bug when using NOT_IN in the array pipeline.

Fixes two things

• Mongo - NOT_IN -> $not and $in
• Postgres - Alias -> lowercase

[suresh-prakash]

You can check-in your integration tests (in a generic way without specifying any proprietary information) so that any changes to this flow would be caught early on.

[suresh-prakash]

My bad on the naming suggestion earlier. Since we are special casing "NOT_IN", it is no longer a standard one.

Suggested change

	public class MongoStandardExprNotInRelationalFilterParser implements MongoRelationalFilterParser {
	public class MongoNotInExprRelationalFilterParser implements MongoRelationalFilterParser {

[Sukhamjot-Singh]

You can check-in your integration tests (in a generic way without specifying any proprietary information) so that any changes to this flow would be caught early on.

I have added test for Mongo provider
I couldn't debug postgres issue

[suresh-prakash]

Anything we build in document-store needs to be compatible with Postgres too to avoid digressing them too much. Could you please sync. with the respective team to get this working on Postgres too? I believe that should be a smaller fix too.

[codecov]

Codecov Report

❌ Patch coverage is 62.50000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.73%. Comparing base (4adcb5e) to head (3284d79).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...filter/MongoRelationalFilterParserFactoryImpl.java	40.00%	3 Missing and 3 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #232      +/-   ##
============================================
+ Coverage     79.71%   79.73%   +0.01%     
- Complexity     1098     1099       +1     
============================================
  Files           212      213       +1     
  Lines          5330     5344      +14     
  Branches        451      455       +4     
============================================
+ Hits           4249     4261      +12     
- Misses          762      763       +1     
- Partials        319      320       +1     

Flag	Coverage Δ
integration	79.73% <62.50%> (+0.01%)	⬆️
unit	57.33% <12.50%> (-0.14%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.