Modify the query conversion logic for multi valued fields

[suresh-prakash]

No description provided.

[kotharironak]

lgtm

[kotharironak]

document.toJson() -> document

[codecov]

Codecov Report

Merging #236 (205d717) into main (b7762c5) will increase coverage by 0.09%.
The diff coverage is 84.90%.

@@             Coverage Diff              @@
##               main     #236      +/-   ##
============================================
+ Coverage     69.04%   69.14%   +0.09%     
- Complexity      647      650       +3     
============================================
  Files           109      110       +1     
  Lines          4545     4592      +47     
  Branches        487      493       +6     
============================================
+ Hits           3138     3175      +37     
- Misses         1151     1157       +6     
- Partials        256      260       +4     

Flag	Coverage Δ
integration	69.14% <84.90%> (+0.09%)	⬆️
unit	52.79% <24.52%> (-0.33%)	⬇️

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

Impacted Files	Coverage Δ
...r/filter/ContainsFilteringExpressionConverter.java	80.48% <80.48%> (ø)
...ce/entity/query/service/converter/ValueHelper.java	82.44% <100.00%> (+0.41%)	⬆️
...ilter/FilteringExpressionConverterFactoryImpl.java	75.00% <100.00%> (+3.57%)	⬆️
...onverter/filter/RelationalExpressionConverter.java	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[github-actions]

Test Results

  59 files  ±0    59 suites  ±0   33s ⏱️ +4s
294 tests +1  293 ✔️ +1  1 💤 ±0  0 ❌ ±0 
296 runs  +1  295 ✔️ +1  1 💤 ±0  0 ❌ ±0 

Results for commit 205d717. ± Comparison against base commit b7762c5.

This pull request removes 2 and adds 3 tests. Note that renamed tests count towards both.

org.hypertrace.entity.query.service.converter.UpdateConverterTest ‑ [1] ATTRIBUTE_UPDATE_OPERATOR_ADD_TO_LIST_IF_ABSENT
org.hypertrace.entity.query.service.converter.UpdateConverterTest ‑ [2] ATTRIBUTE_UPDATE_OPERATOR_REMOVE_FROM_LIST

org.hypertrace.entity.query.service.converter.UpdateConverterTest ‑ [1] ATTRIBUTE_UPDATE_OPERATOR_REMOVE_FROM_LIST
org.hypertrace.entity.query.service.converter.UpdateConverterTest ‑ [2] ATTRIBUTE_UPDATE_OPERATOR_ADD_TO_LIST_IF_ABSENT
org.hypertrace.entity.service.service.EntityQueryServiceTest ‑ testExecuteWithNotInArrayFilter()

♻️ This comment has been updated with latest results.