Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: tests/utils.py

Issue 29912588: Issue 7019 - [CMS] Refactor `test_server.py` (Closed)
Patch Set: Created Oct. 16, 2018, 11:42 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tests/utils.py
diff --git a/tests/utils.py b/tests/utils.py
index ed21e24c650e59651cd2a6edcf9d1cd915c9a38e..751736a825838f5d0c44c6062d4eec8acac82fcc 100644
--- a/tests/utils.py
+++ b/tests/utils.py
@@ -47,6 +47,7 @@ def run_test_server(site_path):
# however there is an issue with Werkzeug that prevents it from properly
# handling the SIGTERM sent by p.kill() or terminate()
# Issue: https://github.com/pallets/werkzeug/issues/58
+
Vasily Kuznetsov 2018/10/16 13:18:23 This looks kind of unrelated. Probably better remo
Tudor Avram 2018/10/18 13:44:05 Done.
p = subprocess.Popen(args, stdout=subprocess.PIPE, preexec_fn=os.setsid)
time.sleep(0.5)
yield 'http://localhost:5000/'

Powered by Google App Engine
This is Rietveld