[ 
https://issues.apache.org/jira/browse/AIRFLOW-4865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16885973#comment-16885973
 ] 

ASF subversion and git services commented on AIRFLOW-4865:
----------------------------------------------------------

Commit 496d7c969528e697361511064f8ba990af405109 in airflow's branch 
refs/heads/master from Joshua Carp
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=496d7c9 ]

[AIRFLOW-4865] Add context manager to set temporary config values in tests. 
(#5569)



> Use decorator to limit scope of config changes in tests
> -------------------------------------------------------
>
>                 Key: AIRFLOW-4865
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-4865
>             Project: Apache Airflow
>          Issue Type: Sub-task
>          Components: tests
>    Affects Versions: 2.0.0
>            Reporter: Ash Berlin-Taylor
>            Priority: Minor
>
> We have a number of cases where we change a config setting in our tests - 
> right now we just do ` {{config.set()}} call.
> Most of the time these changes to config are not reverted, or are only 
> reverted due the order that the tests run in, which is a bit fragile.
> We already have a tests/test_utils/decorators.py that contains a 
> mock_conf_get decorator that we should use everywhere we need to change a 
> config value in a test.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to