Re: pgAdmin IV : Unittest modular patch
От | Navnath Gadakh |
---|---|
Тема | Re: pgAdmin IV : Unittest modular patch |
Дата | |
Msg-id | CAOAJCYr2RpCFPFzDq9N6R7VePPCMEriFgUNmscW5N-YGKJfENg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pgAdmin IV : Unittest modular patch (Dave Page <dave.page@enterprisedb.com>) |
Ответы |
Re: pgAdmin IV : Unittest modular patch
(Dave Page <dave.page@enterprisedb.com>)
|
Список | pgadmin-hackers |
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.TableSpaceUpdat eTestCase)
------------------------------------------------------------ ----------
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.TableSpaceDe leteTestCase)
------------------------------------------------------------ ----------
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.TablespaceGetTe stCase)
------------------------------------------------------------ ----------
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.TableSpaceAddTe stCase)
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
Вложения
В списке pgadmin-hackers по дате отправления:
Предыдущее
От: Murtuza ZabuawalaДата:
Сообщение: PATCH: To fix the issue where message panel was showing incomplete info (pgAdmin4)