Re: [pgadmin-hackers][patch] Make feature test app teardown morereliable, and tests faster - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin-hackers][patch] Make feature test app teardown morereliable, and tests faster
Date
Msg-id CA+OCxozHWntM9-WL2JTQSA--5aLfq=U03ShjRAdUQd5TakcR4Q@mail.gmail.com
Whole thread Raw
In response to [pgadmin-hackers][patch] Make feature test app teardown morereliable, and tests faster  (Atira Odhner <aodhner@pivotal.io>)
Responses Re: [pgadmin-hackers][patch] Make feature test app teardown morereliable, and tests faster
List pgadmin-hackers
Awesome - thanks, patch applied.

On Tue, Feb 28, 2017 at 10:40 PM, Atira Odhner <aodhner@pivotal.io> wrote:
>
> Hi Hackers,
>
> We made the following improvements to the test suite:
>
> Make feature test app teardown more reliable, and tests faster
>
>     - don't spin up app and chromedriver between each test
>     - catching signals also tears down the app
>     - do layout reset between tests, but assume that tests will not leave a
> modal opened.
>
>  Use selenium built-in waiting function and fix flakiness around clicking
> the alertify OK button
>
>     - we think the OK button does not have its event bound when it is
> created.
>
> If you see more flakiness around clicking the alertify OK button, let us
> know. The element is clickable but we have to arbitrarily wait for the event
> to be bound and that timing may vary system to system.
>
> The feature tests are about 7 seconds faster now.
>
> Tira & Joao
>
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>



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

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: [pgadmin-hackers] pgAdmin 4 commit: Make feature test app teardown more reliable,and tes
Next
From: Jonas Thelemann
Date:
Subject: [pgadmin-hackers] Readme Requirements Fix