Re: pgAdmin 4 commit: Fix the feature tests failuers. - Mailing list pgadmin-hackers

From Dave Page
Subject Re: pgAdmin 4 commit: Fix the feature tests failuers.
Date
Msg-id CA+OCxow=vrb-3AR6BN4bB5m-yQD8N1-VmVKL=5_rv81kEuKVvQ@mail.gmail.com
Whole thread Raw
In response to Re: pgAdmin 4 commit: Fix the feature tests failuers.  (Harshal Dhumal <harshal.dhumal@enterprisedb.com>)
Responses Re: pgAdmin 4 commit: Fix the feature tests failuers.
List pgadmin-hackers


On Tue, Aug 29, 2017 at 10:27 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:

On Tue, Aug 29, 2017 at 1:48 PM, Dave Page <dpage@pgadmin.org> wrote:


On Tue, Aug 29, 2017 at 8:41 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Ashesh,

I ran feature test case on mac couple times however I didn't get failure which Dave and you got.
In my case feature test cases mostly falling in xxs_debugger test case.

This was because as when we configure database sever to debug functions it adds 21 its own functions in database and when we run this feature test, it adds one more function (with name test_function) to actually debug it.
But due to its name (test_function) it appears at bottom in visible area of browser and some times
selenium fails to click on that element. To overcome this I have renamed test function name from test_function
to a_test_function so that it will appear at top position under function collection node (patch attached).

Wouldn't it be better to scroll until it's in view? That seems like it would be much less likely to break in the future.

test_function already appears in visible area (but at bottom) this patch is just additional safeguard.

Yes, I get that... but it's really not the point I was making :-)
 

Inline image 1

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

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




--
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: Harshal Dhumal
Date:
Subject: Re: pgAdmin 4 commit: Fix the feature tests failuers.
Next
From: Surinder Kumar
Date:
Subject: Re: [pgAdmin4][Patch]: RM_2596 - Query tool not working in DesktopRuntime on Mac OS X