Skip to content

Commit 6f7af0d

Browse files
metacosmcsviri
authored andcommitted
fix: properly output target dependent
Signed-off-by: Chris Laprun <claprun@redhat.com>
1 parent ab0c462 commit 6f7af0d

File tree

1 file changed

+14
-11
lines changed
  • operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/dependent/workflow

1 file changed

+14
-11
lines changed

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/dependent/workflow/ExecutionAssert.java

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,13 @@ public static ExecutionAssert assertThat(List<ReconcileRecord> actual) {
2121

2222
public ExecutionAssert reconciled(DependentResource<?, ?>... dependentResources) {
2323
for (int i = 0; i < dependentResources.length; i++) {
24-
var rr = getReconcileRecordFor(dependentResources[i]);
24+
final DependentResource<?, ?> dr = dependentResources[i];
25+
var rr = getReconcileRecordFor(dr);
2526
if (rr.isEmpty()) {
26-
failWithMessage("Resource not reconciled: %s with index %d", dependentResources, i);
27+
failWithMessage("Resource not reconciled: %s with index %d", dr, i);
2728
} else {
2829
if (rr.get().isDeleted()) {
29-
failWithMessage("Resource deleted: %s with index %d", dependentResources, i);
30+
failWithMessage("Resource deleted: %s with index %d", dr, i);
3031
}
3132
}
3233
}
@@ -35,12 +36,13 @@ public ExecutionAssert reconciled(DependentResource<?, ?>... dependentResources)
3536

3637
public ExecutionAssert deleted(DependentResource<?, ?>... dependentResources) {
3738
for (int i = 0; i < dependentResources.length; i++) {
38-
var rr = getReconcileRecordFor(dependentResources[i]);
39+
final DependentResource<?, ?> dr = dependentResources[i];
40+
var rr = getReconcileRecordFor(dr);
3941
if (rr.isEmpty()) {
40-
failWithMessage("Resource not reconciled: %s with index %d", dependentResources, i);
42+
failWithMessage("Resource not reconciled: %s with index %d", dr, i);
4143
} else {
4244
if (!rr.get().isDeleted()) {
43-
failWithMessage("Resource not deleted: %s with index %d", dependentResources, i);
45+
failWithMessage("Resource not deleted: %s with index %d", dr, i);
4446
}
4547
}
4648
}
@@ -75,17 +77,18 @@ public ExecutionAssert reconciledInOrder(DependentResource<?, ?>... dependentRes
7577

7678
public ExecutionAssert notReconciled(DependentResource<?, ?>... dependentResources) {
7779
for (int i = 0; i < dependentResources.length; i++) {
78-
if (getActualDependentResources().contains(dependentResources[i])) {
79-
failWithMessage("Resource was reconciled: %s with index %d", dependentResources, i);
80+
final DependentResource<?, ?> dr = dependentResources[i];
81+
if (getActualDependentResources().contains(dr)) {
82+
failWithMessage("Resource was reconciled: %s with index %d", dr, i);
8083
}
8184
}
8285
return this;
8386
}
8487

8588
private void checkIfReconciled(int i, DependentResource<?, ?>[] dependentResources) {
86-
if (!getActualDependentResources().contains(dependentResources[i])) {
87-
failWithMessage("Dependent resource: %s, not reconciled on place %d", dependentResources[i],
88-
i);
89+
final DependentResource<?, ?> dr = dependentResources[i];
90+
if (!getActualDependentResources().contains(dr)) {
91+
failWithMessage("Dependent resource: %s, not reconciled on place %d", dr, i);
8992
}
9093
}
9194
}

0 commit comments

Comments
 (0)