On 12/12/18 11:27 am, Paul Gevers wrote: [...] >> Attached patch uses runuser in the pgsql methods instead of su. > > Did you run the tests that are part of dbconfig-common with your patch > applied? I am running it now and there seems to be at least one test > case failing. > > test_dbc_psql_db_installed_real > case 1: pgsqld in path > case 2: pgsqld not in path > ASSERT:pgsqld should not have been found >
Looks like this test at least fails even without a patch and code seems outdated considering the method it tests. Currently, for me all mock tests pass and real test are failing with or without the patch. -- Sunil
signature.asc
Description: OpenPGP digital signature