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.p...@enterprisedb.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.gad...@enterprisedb.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.p...@enterprisedb.com> > > wrote: > >> > >> Thanks - applied with some minor wording and README tweaks. > >> > >> On Tue, Aug 9, 2016 at 10:19 AM, Navnath Gadakh > >> <navnath.gad...@enterprisedb.com> wrote: > >> > Hi Dave, > >> > > >> > On Mon, Aug 8, 2016 at 9:15 PM, Dave Page <dave.p...@enterprisedb.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.gad...@enterprisedb.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.p...@enterprisedb.com> > >> >> > wrote: > >> >> >> > >> >> >> On Thu, Aug 4, 2016 at 1:02 PM, Navnath Gadakh > >> >> >> <navnath.gad...@enterprisedb.com> wrote: > >> >> >> > Hi Dave, > >> >> >> > > >> >> >> > On Thu, Aug 4, 2016 at 4:04 PM, Dave Page > >> >> >> > <dave.p...@enterprisedb.com> > >> >> >> > wrote: > >> >> >> >> > >> >> >> >> Hi > >> >> >> >> > >> >> >> >> On Thu, Aug 4, 2016 at 10:27 AM, Navnath Gadakh > >> >> >> >> <navnath.gad...@enterprisedb.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