Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1gqt2D-0007Tg-R5 for pgadmin-hackers@arkaria.postgresql.org; Tue, 05 Feb 2019 05:12:38 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1gqt2C-0004fj-J7 for pgadmin-hackers@arkaria.postgresql.org; Tue, 05 Feb 2019 05:12:36 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1gqt2C-0004fZ-4y for pgadmin-hackers@lists.postgresql.org; Tue, 05 Feb 2019 05:12:36 +0000 Received: from mail-lj1-x244.google.com ([2a00:1450:4864:20::244]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1gqt28-0002Kq-FT for pgadmin-hackers@postgresql.org; Tue, 05 Feb 2019 05:12:34 +0000 Received: by mail-lj1-x244.google.com with SMTP id g11-v6so1838211ljk.3 for ; Mon, 04 Feb 2019 21:12:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=y9oMyO3FqSOYYzlOuSU84/VjSHcQjielfHGrjTUzvDU=; b=kMdQNd9hYvofF1UpnqLjB0DqZ1wftYF8YuCZkD7RWoQ1isJG8UnouzYcENk+huBen6 bVA6qdkuV+OuHmf6XA/qeJTpDExFsCnRDbgHxUg2TySdTfR+q0Dx533B/dIpuZs+KbXy PGhXqdiY3QhI6CCPcMUvn/L9y+/k7gIo8uQLqvxWsoVr2sdVNZBPOZCK/o6c4BauMpYJ cd9HPa38ukGhV4/JSizDAX6nlJx4KZ4v6+KD2r6KY5ttqkY74A/t29vF0lyvegiBFI1D DAS64LUt9l5+y/heQaeSdzt1XradQzGBwt/QmuuA1j1JJkoXBcUrF4JoIG07s7V8q1ey OkFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=y9oMyO3FqSOYYzlOuSU84/VjSHcQjielfHGrjTUzvDU=; b=nfAqeNHiXEb1rXXFpYRsQ5cSrL7zacSK7vIpMyIZQKiqAY9SqCwhYsLMOmaO3K+xzk 3A485bTbJCmRs6BFE5n6/RuUiJSoJe2qlg6p2D17t+gxnMjFPsUaNFQNGb8D+Hc/hFkS TiYxEdNZ+WdDV70d5FoO+4eMLKqAX5RhyX7tk7c2FwzVBSdJ5cT7aG4wWcPPqyyqCs9L /q1yQWIswdi0uJ0zoijp4k3yOrQiGPraDfVz4L6jTMQgz6DJEAaJqwJ9WBu3aFr+vEDn rEesD+A+UoMubQ2mvdRRyJ3oNSEP/Fp2lS0t2mKJWn+6kkQS2z4CJ3g4x02lOhzNmTS9 dEQQ== X-Gm-Message-State: AHQUAubjaameg2VbArx8CNjdlhtnqzMqmVQSlx/Tbd80X0ckOPPycddj OQcdhRYynnuzraTTolTaxeD3ahZZhLJ1J1nj/d3dFg== X-Google-Smtp-Source: AHgI3IZBytqajp82iVFXVllBxPZT2rAXBk//EJfz2112/tt1KlXVjcztnVXUvZl3zx2CZFmGgjbbL0ut22KJBy0qcjE= X-Received: by 2002:a2e:4784:: with SMTP id u126-v6mr1683989lja.124.1549343549952; Mon, 04 Feb 2019 21:12:29 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Aditya Toshniwal Date: Tue, 5 Feb 2019 10:42:18 +0530 Message-ID: Subject: Re: [pgAdmin4][RM3936] Feature tests failing intermittently for SQL editor related test cases To: Akshay Joshi Cc: pgadmin-hackers Content-Type: multipart/alternative; boundary="000000000000ee311e05811ea701" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --000000000000ee311e05811ea701 Content-Type: text/plain; charset="UTF-8" Urgh. Will try to reproduce and explore possible causes of failure. On Tue, Feb 5, 2019 at 10:40 AM Akshay Joshi wrote: > Hi Aditya > > On Mon, Feb 4, 2019 at 2:18 PM Aditya Toshniwal < > aditya.toshniwal@enterprisedb.com> wrote: > >> Hi Akshay, >> >> I tried to simulate, but no luck. It works on my system :( >> > > I have tried and Feature test are failing randomly for each run. I have > verified it on OSX and Ubuntu both. > >> >> On Wed, Jan 30, 2019 at 7:15 PM Akshay Joshi < >> akshay.joshi@enterprisedb.com> wrote: >> >>> Hi Aditya >>> >>> I have applied your patch but following test cases are failing >>> consistently: >>> >>> - CopySelectedQueryResultsFeatureTest >>> - QueryToolAutoCompleteFeatureTest >>> >>> I have verified your patch on Mac OSX and Ubuntu. Chrome Version >>> 71.0.3578.98 >>> >>> On Tue, Jan 29, 2019 at 7:38 PM Akshay Joshi < >>> akshay.joshi@enterprisedb.com> wrote: >>> >>>> >>>> >>>> On Tue, 29 Jan 2019, 19:09 Dave Page >>> >>>>> Akshay, can you look at this please? >>>>> >>>> >>>> Sure. >>>> >>>>> >>>>> On Tue, Jan 29, 2019 at 1:36 PM Aditya Toshniwal >>>>> wrote: >>>>> > >>>>> > Hi Hackers, >>>>> > >>>>> > Attached is the updated patch. I have changed >>>>> fill_input_by_field_name function to use javascript to set values of text >>>>> boxes and the selector is changed from xpath to CSS with a :not(:disabled) >>>>> clause. With this, test case will fail if tried to set the value of a >>>>> disabled input. >>>>> > >>>>> > Kindly review. >>>>> > >>>>> > >>>>> > On Mon, Jan 28, 2019 at 5:44 PM Aditya Toshniwal < >>>>> aditya.toshniwal@enterprisedb.com> wrote: >>>>> >> >>>>> >> >>>>> >> >>>>> >> On Mon, Jan 28, 2019 at 5:39 PM Dave Page >>>>> wrote: >>>>> >>> >>>>> >>> On Mon, Jan 28, 2019 at 12:05 PM Aditya Toshniwal >>>>> >>> wrote: >>>>> >>> > >>>>> >>> > Hi Dave, >>>>> >>> > >>>>> >>> > On Mon, Jan 28, 2019 at 5:01 PM Dave Page >>>>> wrote: >>>>> >>> >> >>>>> >>> >> Hi >>>>> >>> >> >>>>> >>> >> On Mon, Jan 28, 2019 at 7:00 AM Aditya Toshniwal >>>>> >>> >> wrote: >>>>> >>> >> > >>>>> >>> >> > Hi Hackers, >>>>> >>> >> > >>>>> >>> >> > Attached is the patch to fix failing SQL editor related >>>>> feature tests. >>>>> >>> >> > The failure is occurring mostly because of send_keys function >>>>> of selenium which sometimes does not input the complete text. The change >>>>> will use Codemirror javascript object to set the text. >>>>> >>> >> > >>>>> >>> >> > I have also added new args flag "--modules" which can be used >>>>> after --pkg where you can test a specific module file under the pkg. This >>>>> is not so useful for build but definitely helpful for the developers. >>>>> >>> >> >>>>> >>> >> One of the most common failures I see is failure to fill in the >>>>> >>> >> details on the Add Server dialogue (it might type half of an >>>>> entry, >>>>> >>> >> then just hang). Should we get rid of all uses of send_keys? >>>>> >>> > >>>>> >>> > Setting the text using javascript executor can be done. >>>>> send_keys simulate an actual person, so if you try to set a disabled text >>>>> box using send_keys, it won't. Places where disabled fields doesn't matter, >>>>> send_keys can be removed. >>>>> >>> >>>>> >>> Can't we just do something like: >>>>> >>> >>>>> >>> if input.disabled == false: >>>>> >>> input.value = 'Foo'; >>>>> >>> else: >>>>> >>> raise exception .... >>>>> >> >>>>> >> I'll check if this can be done. Will get back. >>>>> >>> >>>>> >>> >>>>> >>> -- >>>>> >>> Dave Page >>>>> >>> Blog: http://pgsnake.blogspot.com >>>>> >>> Twitter: @pgsnake >>>>> >>> >>>>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>>>> >>> The Enterprise PostgreSQL Company >>>>> >> >>>>> >> >>>>> >> >>>>> >> -- >>>>> >> Thanks and Regards, >>>>> >> Aditya Toshniwal >>>>> >> Software Engineer | EnterpriseDB Software Solutions | Pune >>>>> >> "Don't Complain about Heat, Plant a tree" >>>>> > >>>>> > >>>>> > >>>>> > -- >>>>> > Thanks and Regards, >>>>> > Aditya Toshniwal >>>>> > Software Engineer | EnterpriseDB Software Solutions | Pune >>>>> > "Don't Complain about Heat, Plant a tree" >>>>> >>>>> >>>>> >>>>> -- >>>>> 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* >>> >> >> >> -- >> Thanks and Regards, >> Aditya Toshniwal >> Software Engineer | EnterpriseDB Software Solutions | Pune >> "Don't Complain about Heat, Plant a tree" >> > > > -- > *Akshay Joshi* > > *Sr. Software Architect * > > > > *Phone: +91 20-3058-9517Mobile: +91 976-788-8246* > -- Thanks and Regards, Aditya Toshniwal Software Engineer | EnterpriseDB Software Solutions | Pune "Don't Complain about Heat, Plant a tree" --000000000000ee311e05811ea701 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Urgh. Will try to reproduce and explore possible causes of fail= ure.

On Tue, Feb 5, 2019 at 10:40 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrot= e:
Hi=C2=A0Aditya

On Mon, Feb 4, 2019 at 2:18 PM Aditya To= shniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Akshay,

I t= ried to simulate, but no luck. It works on my system :(

=C2=A0 =C2=A0I have tried and Feature test are fai= ling randomly for each run. I have verified it on OSX and Ubuntu both.=C2= =A0=C2=A0

On Wed, Jan 30, = 2019 at 7:15 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi=C2=A0Aditya

I have appl= ied your patch but following test cases are failing consistently:
  • CopySelectedQueryResultsFeatureTest
  • QueryToolAutoCompleteF= eatureTest
I have verified your patch on Mac OSX and Ubuntu. C= hrome=C2=A0Version 71.0.3578.98

On= Tue, Jan 29, 2019 at 7:38 PM Akshay Joshi <akshay.joshi@enterprisedb.com>= ; wrote:


On Tue,= 29 Jan 2019, 19:09 Dave Page <dpage@pgadmin.org wrote:
Akshay, can you look at this please?

=C2=A0 =C2= =A0 Sure.

On Tue, Jan 29, 2019 at 1:36 PM Aditya Toshniwal
<aditya.toshniwal@enterprisedb.com> wrote:
>
> Hi Hackers,
>
> Attached is the updated patch. I have changed fill_input_by_field_name= function to use javascript to set values of text boxes and the selector is= changed from xpath to CSS with a :not(:disabled) clause. With this, test c= ase will fail if tried to set the value of a disabled input.
>
> Kindly review.
>
>
> On Mon, Jan 28, 2019 at 5:44 PM Aditya Toshniwal <ad= itya.toshniwal@enterprisedb.com> wrote:
>>
>>
>>
>> On Mon, Jan 28, 2019 at 5:39 PM Dave Page <dpage@pgadmin.org= > wrote:
>>>
>>> On Mon, Jan 28, 2019 at 12:05 PM Aditya Toshniwal
>>> <aditya.toshniwal@enterprisedb.com>= wrote:
>>> >
>>> > Hi Dave,
>>> >
>>> > On Mon, Jan 28, 2019 at 5:01 PM Dave Page <dpage@pgadmi= n.org> wrote:
>>> >>
>>> >> Hi
>>> >>
>>> >> On Mon, Jan 28, 2019 at 7:00 AM Aditya Toshniwal
>>> >> <aditya.toshniwal@enterprisedb.com<= /a>> wrote:
>>> >> >
>>> >> > Hi Hackers,
>>> >> >
>>> >> > Attached is the patch to fix failing SQL editor = related feature tests.
>>> >> > The failure is occurring mostly because of send_= keys function of selenium which sometimes does not input the complete text.= The change will use Codemirror javascript object to set the text.
>>> >> >
>>> >> > I have also added new args flag "--modules&= quot; which can be used after --pkg where you can test a specific module fi= le under the pkg. This is not so useful for build but definitely helpful fo= r the developers.
>>> >>
>>> >> One of the most common failures I see is failure to f= ill in the
>>> >> details on the Add Server dialogue (it might type hal= f of an entry,
>>> >> then just hang). Should we get rid of all uses of sen= d_keys?
>>> >
>>> > Setting the text using javascript executor can be done. s= end_keys simulate an actual person,=C2=A0 so if you try to set a disabled t= ext box using send_keys, it won't. Places where disabled fields doesn&#= 39;t matter, send_keys can be removed.
>>>
>>> Can't we just do something like:
>>>
>>> if input.disabled =3D=3D false:
>>>=C2=A0 =C2=A0input.value =3D 'Foo';
>>> else:
>>>=C2=A0 =C2=A0raise exception ....
>>
>> I'll check if this can be done. Will get back.
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>
>>
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Software Engineer | EnterpriseDB Software Solutions | Pune
>> "Don't Complain about Heat, Plant a tree"
>
>
>
> --
> Thanks and Regards,
> Aditya Toshniwal
> Software Engineer | EnterpriseDB Software Solutions | Pune
> "Don't Complain about Heat, Plant a tree"



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


--
<= div>Akshay Joshi
Sr. Software Architect


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246=


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer |=C2=A0= EnterpriseDB Software Solutions |=C2=A0= Pune
"Don't Compla= in about Heat, Plant a tree"


--
Sr. Software Architect


Phone: +91 20-3058-9517
Mobil= e: +91 976-788-8246


--
Thanks and Rega= rds,
Aditya Toshniwa= l
Softw= are Engineer |=C2=A0EnterpriseDB Software Solutions |=C2=A0Pune
&q= uot;Don't Complain about Heat, Plant a tree"
--000000000000ee311e05811ea701--