This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 608b8c4 Fix test
CreateUserJobTest::test_should_disable_default_helm_hooks (#21776)
608b8c4 is described below
commit 608b8c4879c188881e057e6318a0a15f54c55c7b
Author: Demetrio Ruiz <[email protected]>
AuthorDate: Fri Feb 25 02:46:15 2022 +0100
Fix test CreateUserJobTest::test_should_disable_default_helm_hooks (#21776)
The test was asserting against a wrong attribute when disabling
Helm's hooks
These hooks are required for a proper synchronization
with Kubernetes jobs when deploying with ArgoCD
---
chart/tests/test_create_user_job.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/chart/tests/test_create_user_job.py
b/chart/tests/test_create_user_job.py
index 90dd68d..832aedd 100644
--- a/chart/tests/test_create_user_job.py
+++ b/chart/tests/test_create_user_job.py
@@ -113,7 +113,7 @@ class CreateUserJobTest(unittest.TestCase):
values={"createUserJob": {"useHelmHooks": False}},
show_only=["templates/jobs/create-user-job.yaml"],
)
- annotations = jmespath.search("spec.template.metadata.annotations",
docs[0])
+ annotations = jmespath.search("metadata.annotations", docs[0])
assert annotations is None
def test_should_set_correct_helm_hooks_weight(self):
