Re: pgAdmin 4 commit: Fix the feature tests failuers.

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>
Cc: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin 4 commit: Fix the feature tests failuers.
Date: 2017-08-29 08:18:05
Message-ID: CA+OCxowsYkX_aa=wGXha3mxoBaaj-v+eB6HdsNCgYQjyD5LHiQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Aug 29, 2017 at 8:41 AM, Harshal Dhumal <
harshal(dot)dhumal(at)enterprisedb(dot)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.

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-08-29 08:42:54 Re: pgAdmin 4 commit: Update docs to more accurately reflect the SERVER_MOD
Previous Message Harshal Dhumal 2017-08-29 07:41:26 Re: pgAdmin 4 commit: Fix the feature tests failuers.