On (05/03/14 22:33), Jakub Hrozek wrote:
On Wed, Mar 05, 2014 at 04:24:19PM +0545, Pallavi Jha wrote:
> Hi,
>
>
> Few changes mentioned in earlier mail are added and the required comment is
> also added for mock_rctx(), as was discussed with Jakub. Kindly review the
> patch.
>
>
> Thanks!
> Pallavi
The patch now looks good and the test passes, so I'll ACK it.
Please note that when I applied the patch, the test failed, complaining
about a failure on line 603, which would mean create_dom_test_ctx()
failed. But I haven't been able to reproduce the error again, subsequent
runs went OK for me. If someone else sees the failure, please holler.
I don't know why, but I am able to reproduce it :-(
(Thu Mar 6 09:23:04:051345 2014) [sssd] [ldb] (0x0020): Unable to open tdb
'tests_nss/test_nss_conf.ldb'
(Thu Mar 6 09:23:04:051412 2014) [sssd] [ldb] (0x0020): Failed to connect to
'tests_nss/test_nss_conf.ldb' with backend 'tdb': Unable to open tdb
'tests_nss/test_nss_conf.ldb'
(Thu Mar 6 09:23:04:051425 2014) [sssd] [confdb_init] (0x0010): Unable to open config
database [tests_nss/test_nss_conf.ldb]
(Thu Mar 6 09:23:04:051446 2014) [sssd] [create_dom_test_ctx] (0x0020): confdb_init
failed: 5
tc
../sssd/src/tests/cmocka/test_negcache.c:603: error: Failure!
[ FAILED ] test_sss_ncache_prepopulate
BTW The test will not fail if I run "make rpms"
LS