From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgadmin][patch] Make Feature and Unit tests Green Again |
Date: | 2018-01-26 10:47:21 |
Message-ID: | CA+OCxozf_EYdmYpB4L7axh1kBtucOmSzj2Qzn1=BcJjVdn=fdw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Thu, Jan 25, 2018 at 7:10 PM, Joao De Almeida Pereira <
jdealmeidapereira(at)pivotal(dot)io> wrote:
> Hi Hackers,
>
> Attached you can find a diff with the correction of Bug 3057.
> With the latest changes to display some information in GP some Feature
> Tests stopped working when run against a GP instance.
> Also we found out that the Feature Test(PGDataType) against 9.2 is failing
> for some time as well.
> Some Unit tests were not working when running with Python Version 3.5 due
> to missing functions in the Mocking Library.
>
> This patch solves all the problems above.
>
I notice that datatype_test.json adds a field for
"datatypes_minimum_versions" (maybe should be datatype_minimum_versions?),
whilst the code is checking the server version is greater than that value.
Should the comparison be >=, or is the name misleading?
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-01-26 11:49:42 | pgAdmin 4 commit: Include a hash of the executable path in the lock and |
Previous Message | Joao De Almeida Pereira | 2018-01-25 19:10:21 | [pgadmin][patch] Make Feature and Unit tests Green Again |