diff options
| author | Sumit Bose <sbose@redhat.com> | 2017-05-08 17:30:06 +0200 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2017-06-01 16:49:59 +0200 |
| commit | ee7e72a65d323636600ffda271d5b5c4ddbc78b1 (patch) | |
| tree | 80ec58ac4f2d1eb8ccf54f502567a7ec556c134c /src/tests | |
| parent | 32474fa2f0a6dc09386bab405fc3461cb3dd12ac (diff) | |
| download | sssd-ee7e72a65d323636600ffda271d5b5c4ddbc78b1.tar.gz sssd-ee7e72a65d323636600ffda271d5b5c4ddbc78b1.tar.xz sssd-ee7e72a65d323636600ffda271d5b5c4ddbc78b1.zip | |
sysdb: sysdb_get_certmap() allow empty certmap
Since sysdb_get_certmap() returns the user name hint information as well
it should return a result even if there are no certmaps.
Related to https://pagure.io/SSSD/sssd/issue/3395
Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
Diffstat (limited to 'src/tests')
| -rw-r--r-- | src/tests/cmocka/test_sysdb_certmap.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/tests/cmocka/test_sysdb_certmap.c b/src/tests/cmocka/test_sysdb_certmap.c index fb0716556..72edf5f53 100644 --- a/src/tests/cmocka/test_sysdb_certmap.c +++ b/src/tests/cmocka/test_sysdb_certmap.c @@ -88,8 +88,8 @@ static void test_sysdb_get_certmap_not_exists(void **state) ret = sysdb_get_certmap(ctctx, ctctx->tctx->sysdb, &certmap, &user_name_hint); - assert_int_equal(ret, ENOENT); - + assert_int_equal(ret, EOK); + assert_null(certmap); } static void check_certmap(struct certmap_info *m, struct certmap_info *r, @@ -134,7 +134,7 @@ static void test_sysdb_update_certmap(void **state) int ret; const char *domains[] = { "dom1.test", "dom2.test", "dom3.test", NULL }; struct certmap_info map_a = { discard_const("map_a"), 11, discard_const("abc"), discard_const("def"), NULL }; - struct certmap_info map_b = { discard_const("map_b"), 22, discard_const("abc"), NULL, domains }; + struct certmap_info map_b = { discard_const("map_b"), UINT_MAX, discard_const("abc"), NULL, domains }; struct certmap_info *certmap_empty[] = { NULL }; struct certmap_info *certmap_a[] = { &map_a, NULL }; struct certmap_info *certmap_b[] = { &map_b, NULL }; @@ -152,7 +152,8 @@ static void test_sysdb_update_certmap(void **state) ret = sysdb_get_certmap(ctctx, ctctx->tctx->sysdb, &certmap, &user_name_hint); - assert_int_equal(ret, ENOENT); + assert_int_equal(ret, EOK); + assert_null(certmap); ret = sysdb_update_certmap(ctctx->tctx->sysdb, certmap_a, false); assert_int_equal(ret, EOK); |