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

From Dave Page
Subject Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox
Date
Msg-id CA+OCxoyhbvCF2OnsYpL1xzOenGZp457PyR2oYP+yF2ZkU+0cyA@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi

On Tue, May 15, 2018 at 7:05 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 ?

postgresql.conf attached (though there's nothing unusual in it afaics). There are no changes in datatype_test.json.

Server is PostgreSQL 9.4.10 on x86_64-apple-darwin, compiled by i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.11.00), 64-bit

Cluster encoding is UTF-8, locale is C.
 
 

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
Attachment

pgadmin-hackers by date:

Previous
From: Aditya Toshniwal
Date:
Subject: Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch] Feature #3270 Add support for runningregression tests against Firefox