From 09dc2e2b7a8398b8b6ea12d0b5c41ef754ecc0f7 Mon Sep 17 00:00:00 2001 From: Kaxil Naik Date: Mon, 29 Jun 2020 15:16:41 +0100 Subject: [PATCH] Fix failing test in DagCode (#9565) PR https://github.com/apache/airflow/pull/9554 introduced this error and because of Github issue currently (github is down / has degraded performance) the CI didn't run fully GitOrigin-RevId: ee0335315e421c8ce9e826f7ac5e8f4c82f171fe --- tests/models/test_dagcode.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/models/test_dagcode.py b/tests/models/test_dagcode.py index 728da3ad282..77a1cfc643e 100644 --- a/tests/models/test_dagcode.py +++ b/tests/models/test_dagcode.py @@ -54,6 +54,7 @@ def _write_two_example_dags(self): return [bash_dag, xcom_dag] @conf_vars({('core', 'store_dag_code'): 'True'}) + @patch("airflow.models.dag.settings.STORE_DAG_CODE", True) def _write_example_dags(self): example_dags = make_example_dags(example_dags_module) for dag in example_dags.values(): @@ -66,6 +67,7 @@ def test_sync_to_db(self): self._compare_example_dags(example_dags) + @conf_vars({('core', 'store_dag_code'): 'True'}) def test_bulk_sync_to_db(self): """Dg code can be bulk written into database.""" example_dags = make_example_dags(example_dags_module) @@ -76,6 +78,7 @@ def test_bulk_sync_to_db(self): self._compare_example_dags(example_dags) + @conf_vars({('core', 'store_dag_code'): 'True'}) def test_bulk_sync_to_db_half_files(self): """Dg code can be bulk written into database.""" example_dags = make_example_dags(example_dags_module) @@ -135,6 +138,8 @@ def _compare_example_dags(self, example_dags): self.assertEqual(result.source_code, source_code) @conf_vars({('core', 'store_dag_code'): 'True'}) + @patch("airflow.models.dag.settings.STORE_DAG_CODE", True) + @patch("airflow.models.dagcode.STORE_DAG_CODE", True) def test_code_can_be_read_when_no_access_to_file(self): """ Test that code can be retrieved from DB when you do not have access to Code file. @@ -152,6 +157,7 @@ def test_code_can_be_read_when_no_access_to_file(self): self.assertIn(test_string, dag_code) @conf_vars({('core', 'store_dag_code'): 'True'}) + @patch("airflow.models.dag.settings.STORE_DAG_CODE", True) def test_db_code_updated_on_dag_file_change(self): """Test if DagCode is updated in DB when DAG file is changed""" example_dag = make_example_dags(example_dags_module).get('example_bash_operator')