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

Change max retries to retry duration, refactor settings for consistency #381

Merged
merged 5 commits into from
Jan 9, 2024

Improve test coverage

eafefd4
Select commit
Loading
Failed to load commit list.
Merged

Change max retries to retry duration, refactor settings for consistency #381

Improve test coverage
eafefd4
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 9, 2024 in 0s

71.26% of diff hit (target 70.00%)

View this Pull Request on Codecov

71.26% of diff hit (target 70.00%)

Annotations

Check warning on line 91 in src/main/java/org/opensearch/flowframework/rest/AbstractSearchWorkflowAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/rest/AbstractSearchWorkflowAction.java#L90-L91

Added lines #L90 - L91 were not covered by tests

Check warning on line 133 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L133

Added line #L133 was not covered by tests

Check warning on line 159 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L159

Added line #L159 was not covered by tests

Check warning on line 175 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L175

Added line #L175 was not covered by tests

Check warning on line 177 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L177

Added line #L177 was not covered by tests

Check warning on line 180 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L179-L180

Added lines #L179 - L180 were not covered by tests

Check warning on line 183 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L183

Added line #L183 was not covered by tests

Check warning on line 188 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L188

Added line #L188 was not covered by tests

Check warning on line 199 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L199

Added line #L199 was not covered by tests

Check warning on line 201 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L201

Added line #L201 was not covered by tests

Check warning on line 203 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L203

Added line #L203 was not covered by tests

Check warning on line 206 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L206

Added line #L206 was not covered by tests

Check warning on line 209 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L209

Added line #L209 was not covered by tests

Check warning on line 211 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L211

Added line #L211 was not covered by tests

Check warning on line 213 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L213

Added line #L213 was not covered by tests

Check warning on line 215 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L215

Added line #L215 was not covered by tests

Check warning on line 125 in src/main/java/org/opensearch/flowframework/workflow/AbstractRetryableWorkflowStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/AbstractRetryableWorkflowStep.java#L125

Added line #L125 was not covered by tests