From 3658ca6c5582c983106f38596b30afccfbc375bb Mon Sep 17 00:00:00 2001 From: Dave Page Date: Tue, 7 Mar 2017 13:49:52 +0000 Subject: [PATCH] Avoid leaving chromedriver processes cluttering the landscape after running the feature tests. Per https://github.com/seleniumhq/selenium-google-code-issue-archive/issues/3378, we were doing it wrong :-( --- web/regression/feature_utils/app_starter.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/regression/feature_utils/app_starter.py b/web/regression/feature_utils/app_starter.py index f9160af34..6ef3206f2 100644 --- a/web/regression/feature_utils/app_starter.py +++ b/web/regression/feature_utils/app_starter.py @@ -33,5 +33,5 @@ class AppStarter: self.driver.get("http://" + self.app_config.DEFAULT_SERVER + ":" + random_server_port) def stop_app(self): - self.driver.close() + self.driver.quit() os.killpg(os.getpgid(self.pgadmin_process.pid), signal.SIGTERM)