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
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 |