Thanks for reviewing, Ashesh. We have updated the patch. The headers are all consistent and we removed the __init__.py files in directories containing only .sql.
Thanks! Joao & Matt On Wed, Apr 26, 2017 at 11:22 AM, Joao Pedro De Almeida Pereira < jdealmeidapere...@pivotal.io> wrote: > Hello Ashesh, > > Thanks for reviewing the patch. > > We added the __init__.py files into templates to convert them into > packages so that the tests inside of them can be found by the test runner. > > Thanks! > Joao & Sarah > > On Wed, Apr 26, 2017 at 1:26 AM, Ashesh Vashi < > ashesh.va...@enterprisedb.com> wrote: > >> On Mon, Apr 24, 2017 at 4:43 PM, Dave Page <dp...@pgadmin.org> wrote: >> >>> Ashesh, can you review/commit this please? >>> >>> On Fri, Apr 21, 2017 at 8:42 PM, Joao Pedro De Almeida Pereira < >>> jdealmeidapere...@pivotal.io> wrote: >>> >>>> Hi Hackers, >>>> >>>> We found out that when you are connected to a GreenPlum database and >>>> try to get Dependents and Dependencies of an object the application was >>>> returning a SQL error. >>>> >>>> This patch splits the SQL query used to retrieve the Dependents, >>>> Dependencies, and Roles SQL file into multiple versioned files. >>>> Add Unit Tests for each file. >>>> Also added __init__.py files to other test directories to run the tests >>>> in them. >>>> >>> Hi Joao & Sarah, >> >> Why do we need to add __init__.py in the template directory? >> I didn't understand the purpose of the adding __init__.py files in the >> template directories. >> >> NOTE: The headers in those files are not consistent with the other >> project files. >> >> -- >> >> Thanks & Regards, >> >> Ashesh Vashi >> EnterpriseDB INDIA: Enterprise PostgreSQL Company >> <http://www.enterprisedb.com/> >> >> >> *http://www.linkedin.com/in/asheshvashi* >> <http://www.linkedin.com/in/asheshvashi> >> >>> Add ORDER BY into Copy Selection Feature test to ensure the results are >>>> retrieved always in the same order >>>> Renamed the Scenario of the xss_checks_pgadmin_debugger_test and skip >>>> it for versions less than 9.1 >>>> >>>> Thanks >>>> >>>> Joao & Sarah >>>> >>>> >>>> -- >>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >>>> To make changes to your subscription: >>>> http://www.postgresql.org/mailpref/pgadmin-hackers >>>> >>>> >>> >>> >>> -- >>> Dave Page >>> Blog: http://pgsnake.blogspot.com >>> Twitter: @pgsnake >>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >>> >> >> >
0001-Fix-error-while-checking-Dependents-and-Dependencies.patch
Description: Binary data
-- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers