Hi

On Fri, Mar 3, 2017 at 6:41 PM, Atira Odhner <aodh...@pivotal.io> wrote:
> Hey Dave,
>
> We're not able to reproduce that issue over here. Any chance you could debug
> it?

Per the discussion at
https://github.com/seleniumhq/selenium-google-code-issue-archive/issues/3378,
it looks like we're not closing the driver in the correct way (one
would think that close() means exit and cleanup whilst quit() means
bail out without regard for everyone else, but apparently not...).

Anyway, the attached patch seems to fix the issue for me on my laptop.
I'll go commit it - please yelp if you object.

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
diff --git a/web/regression/feature_utils/app_starter.py 
b/web/regression/feature_utils/app_starter.py
index f9160af3..6ef3206f 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)
-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to