On Thu, Aug 22, 2019 at 2:41 PM Ashesh Vashi <ashesh.va...@enterprisedb.com>
wrote:

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

   Sure will do that.

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

-- 
*Thanks & Regards*
*Akshay Joshi*

*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*

Reply via email to