Обсуждение: [pgadmin][patch] Make Feature and Unit tests Green Again
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.
Thanks
Joao
Вложения
Hi
On Thu, Jan 25, 2018 at 7:10 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.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
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Hello,
We changed the name in the JSON and extracted the validation to a function to make the code more readable.
Thanks
Joao
On Fri, Jan 26, 2018 at 5:47 AM Dave Page <dpage@pgadmin.org> wrote:
HiOn Thu, Jan 25, 2018 at 7:10 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.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
Вложения
Thanks - patch applied!
On Fri, Jan 26, 2018 at 3:54 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello,We changed the name in the JSON and extracted the validation to a function to make the code more readable.ThanksJoaoOn Fri, Jan 26, 2018 at 5:47 AM Dave Page <dpage@pgadmin.org> wrote:HiOn Thu, Jan 25, 2018 at 7:10 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.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
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company