You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by jo...@apache.org on 2018/03/12 18:05:11 UTC
[1/2] incubator-airflow git commit: [AIRFLOW-2207] Fix flaky test
that uses app.cached_app()
Repository: incubator-airflow
Updated Branches:
refs/heads/master 64100d2a2 -> a4aa46e41
[AIRFLOW-2207] Fix flaky test that uses app.cached_app()
tests.www.test_views:TestMountPoint.test_mount changes base_url
then calls airflow.www.app.cached_app().
But if another test calls app.cached_app() first without changing
base_url, succeeding test_mount fails on Travis.
So test_mount should clear cached app for itself in its setup method
so as to remount base_url forcefully.
Project: http://git-wip-us.apache.org/repos/asf/incubator-airflow/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-airflow/commit/7ab53e6f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-airflow/tree/7ab53e6f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-airflow/diff/7ab53e6f
Branch: refs/heads/master
Commit: 7ab53e6f3d12d36a9828ad6facc25598652b1023
Parents: c6bdaf8
Author: Kengo Seki <se...@apache.org>
Authored: Mon Mar 12 09:08:44 2018 -0400
Committer: Kengo Seki <se...@apache.org>
Committed: Mon Mar 12 10:44:30 2018 -0400
----------------------------------------------------------------------
tests/www/test_views.py | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-airflow/blob/7ab53e6f/tests/www/test_views.py
----------------------------------------------------------------------
diff --git a/tests/www/test_views.py b/tests/www/test_views.py
index 2548516..9461f9f 100644
--- a/tests/www/test_views.py
+++ b/tests/www/test_views.py
@@ -453,6 +453,8 @@ class TestMountPoint(unittest.TestCase):
configuration.conf.set("webserver", "base_url", "http://localhost:8080/test")
config = dict()
config['WTF_CSRF_METHODS'] = []
+ # Clear cached app to remount base_url forcefully
+ application.app = None
app = application.cached_app(config=config, testing=True)
self.client = Client(app)
[2/2] incubator-airflow git commit: Merge pull request #3123 from
sekikn/AIRFLOW-2207
Posted by jo...@apache.org.
Merge pull request #3123 from sekikn/AIRFLOW-2207
Project: http://git-wip-us.apache.org/repos/asf/incubator-airflow/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-airflow/commit/a4aa46e4
Tree: http://git-wip-us.apache.org/repos/asf/incubator-airflow/tree/a4aa46e4
Diff: http://git-wip-us.apache.org/repos/asf/incubator-airflow/diff/a4aa46e4
Branch: refs/heads/master
Commit: a4aa46e4128c103f620b0ea2544a6bf61f5ea263
Parents: 64100d2 7ab53e6
Author: Joy Gao <Jo...@apache.org>
Authored: Mon Mar 12 11:05:06 2018 -0700
Committer: Joy Gao <Jo...@apache.org>
Committed: Mon Mar 12 11:05:06 2018 -0700
----------------------------------------------------------------------
tests/www/test_views.py | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------