Re: pgAdmin IV : Unittest modular patch(Schemas,Collations,Trigger Functions)

From: Navnath Gadakh <navnath(dot)gadakh(at)enterprisedb(dot)com>
To: Dave Page <dave(dot)page(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Kanchan Mohitey <kanchan(dot)mohitey(at)enterprisedb(dot)com>
Subject: Re: pgAdmin IV : Unittest modular patch(Schemas,Collations,Trigger Functions)
Date: 2016-08-16 12:38:28
Message-ID: CAOAJCYqPFe29tLuYFy9wL6m8V=eskNe3iOwewwOTSH+vAqCCpQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,
Currently I am working on that. Will send it in the next patch.

Thanks!

On Tue, Aug 16, 2016 at 6:05 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com>
wrote:

> Thanks - applied!
>
> Can you please work on the following ASAP?
>
> - Removal of encoded credentials from the SMTP test output.
> - Summary display as discussed.
>
> Thanks again.
>
> On Fri, Aug 12, 2016 at 10:40 AM, Navnath Gadakh
> <navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> > Hi Dave,
> > Please find the patch for API unit test cases for Schema and its
> > sub-nodes (Collations and Trigger functions).
> > You can run testsuite using the following commands
> > For schemas
> > python regression/runtests.py --pkg
> > browser.server_groups.servers.databases.schemas
> > For Collations
> > python regression/runtests.py --pkg
> > browser.server_groups.servers.databases.schemas.collations
> > For Trigger functions
> > python regression/runtests.py --pkg
> > browser.server_groups.servers.databases.schemas.functions
> > For all nodes
> > python regression/runtests.py
> >
> > Thanks!
> >
> >
> >
> > On Tue, Aug 9, 2016 at 8:36 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com>
> > wrote:
> >>
> >> Thanks - applied with some minor wording and README tweaks.
> >>
> >> On Tue, Aug 9, 2016 at 10:19 AM, Navnath Gadakh
> >> <navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> >> > Hi Dave,
> >> >
> >> > On Mon, Aug 8, 2016 at 9:15 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com
> >
> >> > wrote:
> >> >>
> >> >> Hi
> >> >>
> >> >> I guess you need to test if tablespace_path is omitted, not just
> empty?
> >> >
> >> > Yes, thats right.. Please find the updated patch.
> >> >>
> >> >>
> >> >> This function test the add tablespace scenario (Check Tablespace
> Node)
> >> >> ...
> >> >> ERROR
> >> >>
> >> >> ============================================================
> ==========
> >> >> ERROR: setUpClass
> >> >>
> >> >>
> >> >> (pgadmin.browser.server_groups.servers.tablespaces.
> tests.test_tbspc_put.TableSpaceUpdateTestCase)
> >> >> ------------------------------------------------------------
> ----------
> >> >> Traceback (most recent call last):
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/test_tbspc_put.py",
> >> >> line 48, in setUpClass
> >> >> cls.server_group, cls.server_ids)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 122, in add_table_space
> >> >> data = get_tablespace_data(server_connect)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 41, in get_tablespace_data
> >> >> config_test_data['spc_location'] = server_config['tablespace_
> path']
> >> >> KeyError: 'tablespace_path'
> >> >>
> >> >> ============================================================
> ==========
> >> >> ERROR: setUpClass
> >> >>
> >> >>
> >> >> (pgadmin.browser.server_groups.servers.tablespaces.
> tests.test_tbspc_delete.TableSpaceDeleteTestCase)
> >> >> ------------------------------------------------------------
> ----------
> >> >> Traceback (most recent call last):
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/test_tbspc_delete.py",
> >> >> line 46, in setUpClass
> >> >> cls.server_group, cls.server_ids)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 122, in add_table_space
> >> >> data = get_tablespace_data(server_connect)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 41, in get_tablespace_data
> >> >> config_test_data['spc_location'] = server_config['tablespace_
> path']
> >> >> KeyError: 'tablespace_path'
> >> >>
> >> >> ============================================================
> ==========
> >> >> ERROR: setUpClass
> >> >>
> >> >>
> >> >> (pgadmin.browser.server_groups.servers.tablespaces.
> tests.test_tbspc_get.TablespaceGetTestCase)
> >> >> ------------------------------------------------------------
> ----------
> >> >> Traceback (most recent call last):
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/test_tbspc_get.py",
> >> >> line 45, in setUpClass
> >> >> cls.server_group, cls.server_ids)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 122, in add_table_space
> >> >> data = get_tablespace_data(server_connect)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 41, in get_tablespace_data
> >> >> config_test_data['spc_location'] = server_config['tablespace_
> path']
> >> >> KeyError: 'tablespace_path'
> >> >>
> >> >> ============================================================
> ==========
> >> >> ERROR: runTest
> >> >>
> >> >> (pgadmin.browser.server_groups.servers.tablespaces.
> tests.test_tbspc_add.TableSpaceAddTestCase)
> >> >> This function test the add tablespace scenario (Check Tablespace
> Node)
> >> >> ------------------------------------------------------------
> ----------
> >> >> Traceback (most recent call last):
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/test_tbspc_add.py",
> >> >> line 47, in runTest
> >> >> self.server_ids)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 122, in add_table_space
> >> >> data = get_tablespace_data(server_connect)
> >> >> File
> >> >>
> >> >> "/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_
> groups/servers/tablespaces/tests/utils.py",
> >> >> line 41, in get_tablespace_data
> >> >> config_test_data['spc_location'] = server_config['tablespace_
> path']
> >> >> KeyError: 'tablespace_path'
> >> >>
> >> >> ------------------------------------------------------------
> ----------
> >> >> Ran 14 tests in 9.326s
> >> >>
> >> >> FAILED (errors=4)
> >> >>
> >> >> On Mon, Aug 8, 2016 at 3:07 PM, Navnath Gadakh
> >> >> <navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> >> >> > Hi Dave,
> >> >> > Please find the updated patch for API unit test cases for Roles
> >> >> > and
> >> >> > Tablespaces nodes
> >> >> > As we finalized
> >> >> > Now,
> >> >> > 1. The user will specify the tablespace path in
> >> >> > test_config.json.in
> >> >> > 2. If tablespace path not found, skip the test cases for that
> >> >> > server(Only tablespace test cases)
> >> >> > 3. Add the skipped test summary in the test result. (Now it's
> >> >> > showing on
> >> >> > console + in log file, but need to update in a final enhanced test
> >> >> > summary
> >> >> > report. Which is research point we will work on that after
> finishing
> >> >> > all
> >> >> > nodes API test cases)
> >> >> > 4. Removed the test_ prefix from the values in the config
> files.
> >> >> >
> >> >> > Thanks!
> >> >> >
> >> >> >
> >> >> > On Thu, Aug 4, 2016 at 5:55 PM, Dave Page
> >> >> > <dave(dot)page(at)enterprisedb(dot)com>
> >> >> > wrote:
> >> >> >>
> >> >> >> On Thu, Aug 4, 2016 at 1:02 PM, Navnath Gadakh
> >> >> >> <navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> >> >> >> > Hi Dave,
> >> >> >> >
> >> >> >> > On Thu, Aug 4, 2016 at 4:04 PM, Dave Page
> >> >> >> > <dave(dot)page(at)enterprisedb(dot)com>
> >> >> >> > wrote:
> >> >> >> >>
> >> >> >> >> Hi
> >> >> >> >>
> >> >> >> >> On Thu, Aug 4, 2016 at 10:27 AM, Navnath Gadakh
> >> >> >> >> <navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> >> >> >> >>
> >> >> >> >> >> Oh - does the per-server config override the main config?
> >> >> >> >> >> That's
> >> >> >> >> >> useful. So anything that's in test_advanced_config.py can be
> >> >> >> >> >> overridden on a per-server basis in test_config.py?
> >> >> >> >> >
> >> >> >> >> > No.
> >> >> >> >> > per-server i.e advance config(test_advanced_config.
> json.in)
> >> >> >> >> > and
> >> >> >> >> > main
> >> >> >> >> > config(test_config.json) both are different files. In main
> >> >> >> >> > config
> >> >> >> >> > we
> >> >> >> >> > just
> >> >> >> >> > mention the server’s credentials.(We can also mention per
> >> >> >> >> > server
> >> >> >> >> > credentails) and in test_advanced_config.json.in(here we say
> >> >> >> >> > per-server
> >> >> >> >> > config)
> >> >> >> >> > we mention the advanced configurations i.e. test data for
> each
> >> >> >> >> > node.
> >> >> >> >>
> >> >> >> >> OK.
> >> >> >> >>
> >> >> >> >> > So, let's summarize the discussion:
> >> >> >> >> > - Let user specify the tablespace path in
> >> >> >> >> > test_advanced_config.json.in
> >> >> >> >>
> >> >> >> >> No - test_config.py. It should be per-server.
> >> >> >> >
> >> >> >> > We don't have a file named 'test_config.py' but we have a
> >> >> >> > server's
> >> >> >> > credential file named 'test_config.json'.
> >> >> >> > So, you are saying the tablespace path to be added in
> >> >> >> > 'test_config.json'.
> >> >> >> > Am I correct?
> >> >> >>
> >> >> >> Yeah, sorry. It should be a property of the server configuration.
> >> >> >>
> >> >> >>
> >> >> >> --
> >> >> >> Dave Page
> >> >> >> VP, Chief Architect, Tools & Installers
> >> >> >> EnterpriseDB: http://www.enterprisedb.com
> >> >> >> The Enterprise PostgreSQL Company
> >> >> >>
> >> >> >> Blog: http://pgsnake.blogspot.com
> >> >> >> Twitter: @pgsnake
> >> >> >
> >> >> >
> >> >> >
> >> >> >
> >> >> > --
> >> >> > Thanks,
> >> >> > Navnath Gadakh
> >> >> > Software Engineer
> >> >> > EnterpriseDB Corporation
> >> >> > Mobile: +91 9975389878
> >> >> >
> >> >> >
> >> >>
> >> >>
> >> >>
> >> >> --
> >> >> Dave Page
> >> >> VP, Chief Architect, Tools & Installers
> >> >> EnterpriseDB: http://www.enterprisedb.com
> >> >> The Enterprise PostgreSQL Company
> >> >>
> >> >> Blog: http://pgsnake.blogspot.com
> >> >> Twitter: @pgsnake
> >> >
> >> >
> >> >
> >> >
> >> > --
> >> > Thanks,
> >> > Navnath Gadakh
> >> > Software Engineer
> >> > EnterpriseDB Corporation
> >> > Mobile: +91 9975389878
> >> >
> >> >
> >>
> >>
> >>
> >> --
> >> Dave Page
> >> VP, Chief Architect, Tools & Installers
> >> EnterpriseDB: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >>
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >
> >
> >
> >
> > --
> > Thanks,
> > Navnath Gadakh
> > Software Engineer
> > EnterpriseDB Corporation
> > Mobile: +91 9975389878
> >
> >
>
>
>
> --
> Dave Page
> VP, Chief Architect, Tools & Installers
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>

--
Thanks,
Navnath Gadakh
Software Engineer
EnterpriseDB Corporation
Mobile: +91 9975389878

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Harshal Dhumal 2016-08-16 14:13:42 patch for RM1538: show db default acls [pgadmin4]
Previous Message Dave Page 2016-08-16 12:35:52 Re: pgAdmin IV : Unittest modular patch(Schemas,Collations,Trigger Functions)