diff --git a/e2e/tests/stable/chainsaw-test.yaml b/e2e/tests/stable/chainsaw-test.yaml index 16511be3..527e6d84 100755 --- a/e2e/tests/stable/chainsaw-test.yaml +++ b/e2e/tests/stable/chainsaw-test.yaml @@ -190,14 +190,26 @@ spec: reason: Available status: "True" type: Ready -# TODO: These conditions are appearing in the wrong order which is causing the test to fail. -# conditions: -# - reason: Available -# status: "True" -# type: Ready -# - reason: ReconcileSuccess -# status: "True" -# type: Synced + # Extra assertion for overall Bucket conditions. + # This method of iterative assertions is necessary here because + # these conditions do not always appear in the same order. + # We perform this as a separate assertion, after the initial + # assertion because this method will result in a nil error + # if the conditions do not exist. + - assert: + resource: + apiVersion: provider-ceph.ceph.crossplane.io/v1alpha1 + kind: Bucket + metadata: + name: lc-config-bucket + status: + ~.(conditions[?reason == 'Available']): + status: "True" + type: Ready + ~.(conditions[?reason == 'ReconcileSuccess']): + status: "True" + type: Synced + # Assert auto-pause-bucket is synced with paused label. - assert: @@ -290,14 +302,16 @@ spec: reason: Available status: "True" type: Ready -# TODO: These conditions are appearing in the wrong order which is causing the test to fail. -# conditions: -# - reason: Available -# status: "True" -# type: Ready -# - reason: ReconcileSuccess -# status: "True" -# type: Synced + # This method of iterative assertions is necessary here because + # these conditions do not always appear in the same order. + # It's safe to perform this assertions because we have already + # asserted that these conditios exist. + ~.(conditions[?reason == 'Available']): + status: "True" + type: Ready + ~.(conditions[?reason == 'ReconcileSuccess']): + status: "True" + type: Synced - name: Check for buckets on Localstack backends. try: