Skip to content

Commit 513d78e

Browse files
committed
refactor: rename method more appropriately
Signed-off-by: Chris Laprun <[email protected]>
1 parent 3dd77c1 commit 513d78e

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
@@ -112,7 +112,7 @@ public Set<DependentResourceNode> getTopLevelDependentResources() {
112112
return topLevelResources;
113113
}
114114

115-
public Set<DependentResourceNode> getBottomLevelResource() {
115+
public Set<DependentResourceNode> getBottomLevelDependentResources() {
116116
return bottomLevelResource;
117117
}
118118

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)