Re: pgAdmin IV : Unittest modular patch
От | Dave Page |
---|---|
Тема | Re: pgAdmin IV : Unittest modular patch |
Дата | |
Msg-id | CA+OCxoxj+V-nN7NrtACczFDZzUuQsfcuttnyjd1xZEgXKtVNsg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pgAdmin IV : Unittest modular patch (Navnath Gadakh <navnath.gadakh@enterprisedb.com>) |
Список | pgadmin-hackers |
Thanks - applied with some minor wording and README tweaks. On Tue, Aug 9, 2016 at 10:19 AM, Navnath Gadakh <navnath.gadakh@enterprisedb.com> wrote: > Hi Dave, > > On Mon, Aug 8, 2016 at 9:15 PM, Dave Page <dave.page@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.gadakh@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.page@enterprisedb.com> >> > wrote: >> >> >> >> On Thu, Aug 4, 2016 at 1:02 PM, Navnath Gadakh >> >> <navnath.gadakh@enterprisedb.com> wrote: >> >> > Hi Dave, >> >> > >> >> > On Thu, Aug 4, 2016 at 4:04 PM, Dave Page >> >> > <dave.page@enterprisedb.com> >> >> > wrote: >> >> >> >> >> >> Hi >> >> >> >> >> >> On Thu, Aug 4, 2016 at 10:27 AM, Navnath Gadakh >> >> >> <navnath.gadakh@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
В списке pgadmin-hackers по дате отправления:
Следующее
От: Dave PageДата:
Сообщение: pgAdmin 4 commit: Fix query tool button/menu inconsistencies. Fixes #14