Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox
Date
Msg-id CAM9w-_=cW0wX=_OTuziztQk6VOb8gTHj3sx+ypXEH4TCK7_OSg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox
List pgadmin-hackers

On Tue, May 15, 2018 at 6:05 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Tue, May 15, 2018 at 1:24 PM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Dave,

I suggest check which keyboard you are using, US or UK. If UK, try with US keyboard.(might work)

How would that make any difference? (and yes, I am using a UK keyboard).

There are few cases reported in Selenium github where they are facing issues for non-US keyboard layouts. Refer below link,

It is possible the issues still exist and so we can try with US keyboard once.
 

Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

On Tue, May 15, 2018 at 11:35 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave

On Mon, May 14, 2018 at 6:33 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, May 14, 2018 at 2:03 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave 

On which database server you are running this test cases, I have tested it on PG 9.6  

PG 9.4.

   I have tested it with PG 9.4 on Mac OSX and Ubuntu 16, working absolutely fine. Can you please provide the postgresql.conf is there any parameter changed? OR is there any change in "datatype_test.json" file ?
 

On Mon, May 14, 2018 at 5:45 PM, Dave Page <dpage@pgadmin.org> wrote:

Hi

On Mon, May 14, 2018 at 7:29 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Anthony 

Thanks for updated patch, it is working fine on my machine. I have tested it couple of times.


I've tested it a couple of times and still see the pgDataType failure. I've attached the screenshot; I have to wonder why that test is throwing a syntax error at all as it's only supposed to test that we display different datatypes correctly. Looking further, it seems that it's trying to execute a badly corrupted query. How did it get that way? Do we need to clear the CodeMirror box before trying to replace the query?

--
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-9517
Mobile: +91 976-788-8246



--
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-9517
Mobile: +91 976-788-8246




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

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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox
Next
From: Joao De Almeida Pereira
Date:
Subject: Re: [pgAdmin4][Patch]: Minor JS Fix