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 <32929872+druiz...@users.noreply.github.com> 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):