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-30 10:13:18
Message-ID: CA+OCxoxk-d8EDvR1bo+CRAi20Vzd54rAYDVW49yZH6NHKqtCYw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Regardless, I've applied this for now to stop the tests failing. Please
look for a more appropriate (scroll-if-needed) solution when you have spare
time.

Thanks.

On Tue, Aug 29, 2017 at 10:28 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Tue, Aug 29, 2017 at 10:27 AM, Harshal Dhumal <
> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>
>>
>> On Tue, Aug 29, 2017 at 1:48 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>>
>>>
>>> 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.
>>>
>>
>> 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 :-)
>
>
>>
>> [image: 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
>

--
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 Harshal Dhumal 2017-08-30 10:17:54 Re: pgAdmin 4 commit: Fix the feature tests failuers.
Previous Message Dave Page 2017-08-30 10:12:34 pgAdmin 4 commit: Rename the debugger XSS test function to 'a_test_func