From dd02b6699e75200216a87ea87d6e93ca78c1d6ed Mon Sep 17 00:00:00 2001 From: stevenhorsman Date: Wed, 18 Dec 2024 13:41:42 +0000 Subject: [PATCH] tests: Fix qemu-coc-dev skip Fix the logic to make the test skipped on qemu-coco-dev, rather than the opposite and update the syntax to make it clearer as it incorrectly got written and reviewed by three different people in it's prior form. Signed-off-by: stevenhorsman --- tests/integration/kubernetes/k8s-job.bats | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/integration/kubernetes/k8s-job.bats b/tests/integration/kubernetes/k8s-job.bats index fecb99bdc4..c0112e4a96 100644 --- a/tests/integration/kubernetes/k8s-job.bats +++ b/tests/integration/kubernetes/k8s-job.bats @@ -9,7 +9,9 @@ load "${BATS_TEST_DIRNAME}/../../common.bash" load "${BATS_TEST_DIRNAME}/tests_common.sh" setup() { - [ "${KATA_HYPERVISOR}" = "qemu-coco-dev" ] || skip "Test not stable on qemu-coco-dev. See issue #10616" + if [ "${KATA_HYPERVISOR}" == "qemu-coco-dev" ]; then + skip "Test not stable on qemu-coco-dev. See issue #10616" + fi get_pod_config_dir job_name="job-pi-test" @@ -40,7 +42,9 @@ setup() { } teardown() { - [ "${KATA_HYPERVISOR}" = "qemu-coco-dev" ] || skip "Test not ready yet for ${KATA_HYPERVISOR}" + if [ "${KATA_HYPERVISOR}" == "qemu-coco-dev" ]; then + skip "Test not stable on qemu-coco-dev. See issue #10616" + fi # Debugging information kubectl describe pod "$pod_name"