public inbox for [email protected]
help / color / mirror / Atom feedFrom: Akshay Joshi <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgadmin4][Patch]: Feature #3273 Add ability to sort in the Select File window when opening a saved query
Date: Mon, 25 Jun 2018 16:53:25 +0530
Message-ID: <CANxoLDe93XJeA-d46_H77NJGDGcGZykqaLz5b7t5y5kUkyURZg@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoxc+qDi7W_MJnw7-oZBOOn0RmP+XAb3VAQJFzFRwmZLbg@mail.gmail.com>
References: <CANxoLDe9Yh58zEP8=ZQOAwk=RszUrBwYqxXiR+wmd4pP8i5ZJg@mail.gmail.com>
<CA+OCxoxkuv7863HAB10fxUeJUezDBSNW4awos6-eDgK9-_UZkw@mail.gmail.com>
<CANxoLDeEccXUAjFhEpzrywqYpLoOB+tzgRqr4ROA_1A4Nyhzgg@mail.gmail.com>
<CA+OCxoyLdSqBw1AiX+2A-QbOKfiSxmPh9bJ2yZwLdT37K+AkKg@mail.gmail.com>
<CANxoLDf9tRM8smheRT3trk=mHvVqwwED1sjvczw1AVAGXr9TDA@mail.gmail.com>
<CA+OCxoxc+qDi7W_MJnw7-oZBOOn0RmP+XAb3VAQJFzFRwmZLbg@mail.gmail.com>
On Mon, Jun 25, 2018 at 4:43 PM, Dave Page <[email protected]> wrote:
> Hi
>
> On Fri, Jun 22, 2018 at 6:39 AM, Akshay Joshi <
> [email protected]> wrote:
>
>>
>>
>> On Thu, Jun 21, 2018 at 11:57 PM, Dave Page <[email protected]> wrote:
>>
>>> Hi
>>>
>>> On Thu, Jun 21, 2018 at 6:27 AM, Akshay Joshi <
>>> [email protected]> wrote:
>>>
>>>>
>>>>
>>>> On Thu, Jun 21, 2018 at 6:20 AM, Dave Page <[email protected]> wrote:
>>>>
>>>>> Hi
>>>>>
>>>>> On Wed, Jun 20, 2018 at 3:13 PM, Akshay Joshi <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> Hi Hackers,
>>>>>>
>>>>>> Attached is the patch for Feature #3273 Add ability to sort in the
>>>>>> Select File window when opening a saved query.
>>>>>>
>>>>>> I haven't seen any refactoring possibility, patch contains feature
>>>>>> test with *time.sleep(0.05). *I have spend lot of time to not
>>>>>> introduce "time.sleep()" and tried couple of different ways of
>>>>>> WebDriveWait, but didn't get success.
>>>>>>
>>>>>> Please review it.
>>>>>>
>>>>>
>>>>> I get the following running the feature tests:
>>>>>
>>>>> invalid syntax (xss_checks_file_manager_test.py, line 60)
>>>>>
>>>>> make: *** [check-feature] Error 1
>>>>>
>>>>> Fixed.
>>>>
>>>>
>>>>> Also; should that file be renamed now, to reflect that it tests more
>>>>> than just XSS?
>>>>>
>>>>
>>>> Done. Attached is the modified patch.
>>>>
>>>
>>> This seems to work for the most part, however, when sorting on the
>>> Modified column, it is sorting alphabetically, not by date. This doesn't
>>> seem like it's what users would expect.
>>>
>>
>> Attached is the modified patch with the above fix.
>>
>>>
> I'm seeing the following failure pretty consistently:
>
> ======================================================================
>
> ERROR: runTest (pgadmin.feature_tests.file_manager_test.
> CheckFileManagerFeatureTest)
>
> File manager feature test
>
> ----------------------------------------------------------------------
>
> Traceback (most recent call last):
>
> File "/Users/dpage/git/pgadmin4/web/regression/feature_utils/base_feature_test.py",
> line 66, in tearDown
>
> self.after()
>
> File "/Users/dpage/git/pgadmin4/web/pgadmin/feature_tests/file_manager_test.py",
> line 48, in after
>
> self.page.close_query_tool('sql', False)
>
> File "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py",
> line 104, in close_query_tool
>
> "//li[contains(@class, 'context-menu-item')]/span[contains(text(),"
>
> File "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py",
> line 171, in find_by_xpath
>
> lambda driver: driver.find_element_by_xpath(xpath)
>
> File "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py",
> line 263, in wait_for_element
>
> return self._wait_for("element to exist", element_if_it_exists)
>
> File "/Users/dpage/git/pgadmin4/web/regression/feature_utils/pgadmin_page.py",
> line 337, in _wait_for
>
> "Timed out waiting for " + waiting_for_message
>
> File "/Users/dpage/.virtualenvs/pgadmin4/lib/python3.6/site-
> packages/selenium/webdriver/support/wait.py", line 80, in until
>
> raise TimeoutException(message, screen, stacktrace)
>
> selenium.common.exceptions.TimeoutException: Message: Timed out waiting
> for element to exist
>
>
It's a Timed out issue and I haven't faced this on my machine. As per
above traceback it is failing on "self.page.close_query_tool()" which is
not the part of this patch and not reproducible either on my machine.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
--
*Akshay Joshi*
*Sr. Software Architect *
*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
view thread (8+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: [pgadmin4][Patch]: Feature #3273 Add ability to sort in the Select File window when opening a saved query
In-Reply-To: <CANxoLDe93XJeA-d46_H77NJGDGcGZykqaLz5b7t5y5kUkyURZg@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox