Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] Adding guardrails to default use case params #659

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 9d28045 from #658.

* Adding guardrails to default use case params

Signed-off-by: Joshua Palis <[email protected]>

* Updating changelog and adding javadocs

Signed-off-by: Joshua Palis <[email protected]>

* Fixing test

Signed-off-by: Joshua Palis <[email protected]>

* Fixing integration tests, covering case in which no content is passed at all for default cases

Signed-off-by: Joshua Palis <[email protected]>

* Fixing tests

Signed-off-by: Joshua Palis <[email protected]>

* Fixing rest create workflow action

Signed-off-by: Joshua Palis <[email protected]>

* addressing PR comments

Signed-off-by: Joshua Palis <[email protected]>

* fixing test

Signed-off-by: Joshua Palis <[email protected]>

---------

Signed-off-by: Joshua Palis <[email protected]>
(cherry picked from commit 9d28045)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Apr 11, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 72.12%. Comparing base (0b119b4) to head (491886e).

Files Patch % Lines
...h/flowframework/rest/RestCreateWorkflowAction.java 12.50% 12 Missing and 2 partials ⚠️
...ensearch/flowframework/common/DefaultUseCases.java 84.61% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x     #659      +/-   ##
============================================
- Coverage     72.30%   72.12%   -0.19%     
- Complexity      680      682       +2     
============================================
  Files            82       82              
  Lines          3528     3562      +34     
  Branches        285      290       +5     
============================================
+ Hits           2551     2569      +18     
- Misses          850      863      +13     
- Partials        127      130       +3     

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

@joshpalis joshpalis merged commit 7a5ffdd into 2.x Apr 11, 2024
33 of 34 checks passed
@joshpalis joshpalis deleted the backport/backport-658-to-2.x branch April 11, 2024 23:51
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.

1 participant