Skip to content
This repository has been archived by the owner on Jun 26, 2024. It is now read-only.

refactor(explore): expression context in handlers #149

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

skjindal93
Copy link
Contributor

No description provided.

@github-actions
Copy link

Unit Test Results

  60 files  ±0    60 suites  ±0   7s ⏱️ -1s
338 tests ±0  338 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9f5fe99. ± Comparison against base commit 876b58b.

@codecov
Copy link

codecov bot commented Oct 28, 2022

Codecov Report

Merging #149 (9f5fe99) into main (876b58b) will increase coverage by 0.09%.
The diff coverage is 97.05%.

@@             Coverage Diff              @@
##               main     #149      +/-   ##
============================================
+ Coverage     79.32%   79.42%   +0.09%     
  Complexity     1223     1223              
============================================
  Files           114      114              
  Lines          5509     5530      +21     
  Branches        453      455       +2     
============================================
+ Hits           4370     4392      +22     
+ Misses          937      936       -1     
  Partials        202      202              
Flag Coverage Δ
unit 79.42% <97.05%> (+0.09%) ⬆️

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

Impacted Files Coverage Δ
...y/service/explore/entity/EntityRequestHandler.java 90.00% <66.66%> (+0.98%) ⬆️
...gateway/service/explore/ExploreRequestContext.java 89.65% <100.00%> (-0.35%) ⬇️
...rtrace/gateway/service/explore/ExploreService.java 89.65% <100.00%> (-0.51%) ⬇️
...rtrace/gateway/service/explore/RequestHandler.java 89.20% <100.00%> (-0.08%) ⬇️
...ay/service/explore/TheRestGroupRequestHandler.java 97.54% <100.00%> (+0.21%) ⬆️
...ervice/explore/TimeAggregationsRequestHandler.java 100.00% <100.00%> (ø)
...ore/TimeAggregationsWithGroupByRequestHandler.java 93.97% <100.00%> (+1.43%) ⬆️

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant