Skip to content

Conversation

@GurtejSohi
Copy link
Contributor

Description

Adding support for date constant expression for mongo. For postgres, for now, we're not supporting it but can be supported later.

Testing

Added integration tests

Checklist:

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

@github-actions
Copy link

Test Results

 71 files  ±0   71 suites  ±0   33s ⏱️ -1s
517 tests ±0  517 ✅ ±0  0 💤 ±0  0 ❌ ±0 
834 runs  +2  834 ✅ +2  0 💤 ±0  0 ❌ ±0 

Results for commit 8ec75f1. ± Comparison against base commit 35fac78.

@codecov
Copy link

codecov bot commented Dec 23, 2025

Codecov Report

❌ Patch coverage is 35.00000% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.26%. Comparing base (35fac78) to head (8ec75f1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...tore/mongo/query/parser/MongoProjectingParser.java 0.00% 6 Missing ⚠️
...ser/MongoNonProjectedSortTypeExpressionParser.java 0.00% 3 Missing ⚠️
...umentstore/expression/impl/ConstantExpression.java 83.33% 1 Missing ⚠️
...ansformer/MongoSelectionsAddingTransformation.java 0.00% 1 Missing ⚠️
...sformer/MongoSelectionsUpdatingTransformation.java 0.00% 1 Missing ⚠️
...umentstore/parser/SelectTypeExpressionVisitor.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #262      +/-   ##
============================================
- Coverage     80.41%   80.26%   -0.15%     
  Complexity     1337     1337              
============================================
  Files           231      231              
  Lines          6060     6080      +20     
  Branches        545      545              
============================================
+ Hits           4873     4880       +7     
- Misses          813      826      +13     
  Partials        374      374              
Flag Coverage Δ
integration 80.26% <35.00%> (-0.15%) ⬇️
unit 58.02% <0.00%> (-0.20%) ⬇️

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.

@GurtejSohi GurtejSohi merged commit e434dfa into main Dec 23, 2025
5 of 7 checks passed
@GurtejSohi GurtejSohi deleted the add-date-constant-expression branch December 23, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants