On (01/12/14 13:49), Pavel Reichl wrote: > START_TEST (test_sysdb_attrs_to_list) > { > struct sysdb_attrs *attrs_list[3]; >@@ -4907,10 +4959,9 @@ START_TEST(test_sysdb_has_enumerated) > fail_if(ret != EOK, "Could not set up the test"); > > ret = sysdb_has_enumerated(test_ctx->domain, &enumerated); >- fail_if(ret != EOK, "Error [%d][%s] checking enumeration", >- ret, strerror(ret)); >- >- fail_if(enumerated, "Enumeration should default to false"); >+ fail_if(ret != ENOENT, >+ "Error [%d][%s] checking enumeration ENOENT is expected", >+ ret, strerror(ret)); > > ret = sysdb_set_enumerated(test_ctx->domain, true); > fail_if(ret != EOK, "Error [%d][%s] setting enumeration", >@@ -6231,6 +6282,9 @@ Suite *create_sysdb_suite(void) > tcase_add_loop_test(tc_memberof, test_sysdb_remove_local_group_by_gid, > MBO_GROUP_BASE , MBO_GROUP_BASE + 10); > >+ /* Misc */ >+ tcase_add_test(tc_sysdb, test_sysdb_set_get_bool); >+ Almost ACK.
Could you move this line among other test from test case sysdb (tc_sysdb) The test test_sysdb_set_get_bool is added to test case tc_sysdb, but us between two tests from test case tc_memberof > /* Ghost users tests */ > tcase_add_loop_test(tc_memberof, > test_sysdb_memberof_store_group_with_ghosts, > MBO_GROUP_BASE , MBO_GROUP_BASE + 10); LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel