Skip to content

Commit e988ab4

Browse files
committed
refactor: rename method more appropriately (#2554)
Signed-off-by: Chris Laprun <[email protected]>
1 parent dca745e commit e988ab4

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

Diff for: operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/workflow/DefaultWorkflow.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ public Set<DependentResourceNode> getTopLevelDependentResources() {
117117
return topLevelResources;
118118
}
119119

120-
public Set<DependentResourceNode> getBottomLevelResource() {
120+
public Set<DependentResourceNode> getBottomLevelDependentResources() {
121121
return bottomLevelResource;
122122
}
123123

Diff for: operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/workflow/WorkflowCleanupExecutor.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ class WorkflowCleanupExecutor<P extends HasMetadata> extends AbstractWorkflowExe
2121
}
2222

2323
public synchronized WorkflowCleanupResult cleanup() {
24-
for (DependentResourceNode dependentResourceNode : workflow.getBottomLevelResource()) {
24+
for (DependentResourceNode dependentResourceNode : workflow
25+
.getBottomLevelDependentResources()) {
2526
handleCleanup(dependentResourceNode);
2627
}
2728
waitForScheduledExecutionsToRun();

Diff for: operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/dependent/workflow/WorkflowTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ void calculatesBottomLevelResources() {
6969
.buildAsDefaultWorkflow();
7070

7171
Set<DependentResource> bottomResources =
72-
workflow.getBottomLevelResource().stream()
72+
workflow.getBottomLevelDependentResources().stream()
7373
.map(DependentResourceNode::getDependentResource)
7474
.collect(Collectors.toSet());
7575

0 commit comments

Comments
 (0)