Skip to content

addressed comments and added more tests

Codecov / codecov/project failed Nov 15, 2023 in 1s

66.03% (-2.36%) compared to 56ccb1d

View this Pull Request on Codecov

66.03% (-2.36%) compared to 56ccb1d

Details

Codecov Report

Attention: 201 lines in your changes are missing coverage. Please review.

Comparison is base (56ccb1d) 68.39% compared to head (e6c9ab4) 66.03%.
Report is 3 commits behind head on main.

❗ Current head e6c9ab4 differs from pull request most recent head 3ee4d72. Consider uploading reports for the commit 3ee4d72 to get more accurate results

Files Patch % Lines
.../opensearch/flowframework/model/WorkflowState.java 0.00% 39 Missing ⚠️
...ramework/transport/GetWorkflowTransportAction.java 0.00% 31 Missing ⚠️
...arch/flowframework/rest/RestGetWorkflowAction.java 12.90% 27 Missing ⚠️
...framework/indices/FlowFrameworkIndicesHandler.java 0.00% 18 Missing ⚠️
...h/flowframework/transport/GetWorkflowResponse.java 0.00% 16 Missing ⚠️
...ch/flowframework/transport/GetWorkflowRequest.java 0.00% 15 Missing ⚠️
...a/org/opensearch/flowframework/model/Template.java 77.04% 13 Missing and 1 partial ⚠️
...ensearch/flowframework/model/ResourcesCreated.java 73.52% 8 Missing and 1 partial ⚠️
...rk/transport/ProvisionWorkflowTransportAction.java 43.75% 9 Missing ⚠️
...ch/flowframework/workflow/CreateConnectorStep.java 65.21% 8 Missing ⚠️
... and 6 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #148      +/-   ##
============================================
- Coverage     68.39%   66.03%   -2.36%     
- Complexity      339      382      +43     
============================================
  Files            45       53       +8     
  Lines          1585     1952     +367     
  Branches        143      164      +21     
============================================
+ Hits           1084     1289     +205     
- Misses          452      606     +154     
- Partials         49       57       +8     

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