Akshay,

On Thu, Aug 22, 2019 at 2:29 PM Akshay Joshi <akshay.jo...@enterprisedb.com>
wrote:

> Nice work!
>
It's big patch, and also improvement over last status.
Can we commit the patch, as to avoid any merging issues later?

-- Thanks, Ashesh

>
> I have run the feature test couple of times on EPAS 9.6, 11, 12 and PG
> 9.4, 10, 11, 12. Following are the review comments:
>
>    - Following test cases are failing every time on *EPAS 12:*
>       - Explain query with JIT stats... fails with error "*no such
>       element: Unable to locate element:
>       {"method":"xpath","selector":"//*[contains(string(), 'JIT')]"}*"
>    - Following test cases are failing every time on *PG 12*:
>       - History query sources and generated queries toggle...ERROR
>       - Tests to check if Debugger is vulnerable to XSS ... ERROR
>    - We have *retry_count* in "browser_tool_bar_test.py" and
>    "file_manager_test.py" file. Can we make a generic function and pass the
>    locator and retry_count it would be good.
>    - We have also import WebDriverWait in so many files can we have one
>    generic class or function for all the wait events. We will pass the
>    required timeout and the locator if possible.
>
>
>
> On Wed, Aug 21, 2019 at 4:18 PM Shubham Agarwal <
> shubham.agar...@enterprisedb.com> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch for fixing the intermittent failures encountered
>> while executing the test cases multiple times.
>>
>> Please review the patch.
>>
>> --
>> Thanks & Regards,
>> Shubham Agarwal
>> EnterpriseDB Corporation
>>
>> The Postgres Database Company
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

Reply via email to