DP TESTS: Add unit tests for dp_builtin.c

Reviewed-by: Sumit Bose <sbose@redhat.com>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>

3 files changed, 221 insertions, 0 deletions

diff --git a/src/tests/cmocka/data_provider/mock_dp.c b/src/tests/cmocka/data_provider/mock_dp.c
index ee3177512..387faa9e6 100644
--- a/src/tests/cmocka/data_provider/mock_dp.c
+++ b/src/tests/cmocka/data_provider/mock_dp.c
@@ -96,3 +96,26 @@ struct dp_method *mock_dp_get_methods(struct data_provider *provider,
 
     return methods;
 }
+
+struct dp_req_params *mock_dp_req_params(TALLOC_CTX *mem_ctx,
+                                         struct tevent_context *ev,
+                                         struct be_ctx *be_ctx,
+                                         struct sss_domain_info *domain,
+                                         enum dp_targets target,
+                                         enum dp_methods method)
+{
+    struct dp_req_params *params;
+
+    params = talloc_zero(mem_ctx, struct dp_req_params);
+    if (params == NULL) {
+        return NULL;
+    }
+
+    params->ev = ev;
+    params->be_ctx = be_ctx;
+    params->domain = domain;
+    params->target = target;
+    params->method = method;
+
+    return params;
+}
diff --git a/src/tests/cmocka/data_provider/mock_dp.h b/src/tests/cmocka/data_provider/mock_dp.h
index 99e7bae8c..c58378365 100644
--- a/src/tests/cmocka/data_provider/mock_dp.h
+++ b/src/tests/cmocka/data_provider/mock_dp.h
@@ -32,4 +32,11 @@ struct data_provider *mock_dp(TALLOC_CTX *mem_ctx,
 struct dp_method *mock_dp_get_methods(struct data_provider *provider,
                                       enum dp_targets target);
 
+struct dp_req_params *mock_dp_req_params(TALLOC_CTX *mem_ctx,
+                                         struct tevent_context *ev,
+                                         struct be_ctx *be_ctx,
+                                         struct sss_domain_info *domain,
+                                         enum dp_targets target,
+                                         enum dp_methods method);
+
 #endif /* _MOCK_DP_H_ */
diff --git a/src/tests/cmocka/data_provider/test_dp_builtin.c b/src/tests/cmocka/data_provider/test_dp_builtin.c
new file mode 100644
index 000000000..b2f2a7d71
--- /dev/null
+++ b/src/tests/cmocka/data_provider/test_dp_builtin.c
@@ -0,0 +1,191 @@
+/*
+    Copyright (C) 2015 Red Hat
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 3 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#include <talloc.h>
+#include <tevent.h>
+#include <errno.h>
+#include <popt.h>
+#include <security/pam_modules.h>
+
+#include "providers/backend.h"
+#include "providers/data_provider/dp_private.h"
+#include "providers/data_provider/dp_builtin.h"
+#include "providers/data_provider/dp.h"
+#include "tests/cmocka/common_mock.h"
+#include "tests/common.h"
+#include "tests/cmocka/common_mock_be.h"
+#include "tests/cmocka/data_provider/mock_dp.h"
+
+#define TESTS_PATH "tp_" BASE_FILE_STEM
+#define TEST_CONF_DB "test_dp_request.ldb"
+#define TEST_DOM_NAME "dp_request_test"
+#define TEST_ID_PROVIDER "ldap"
+
+struct test_ctx {
+    struct sss_test_ctx *tctx;
+    struct be_ctx *be_ctx;
+    struct dp_req_params *params;
+};
+
+static int test_setup(void **state)
+{
+    struct test_ctx *test_ctx;
+
+
+    assert_true(leak_check_setup());
+
+    test_ctx = talloc_zero(global_talloc_context, struct test_ctx);
+    assert_non_null(test_ctx);
+
+    test_ctx->tctx = create_dom_test_ctx(test_ctx, TESTS_PATH, TEST_CONF_DB,
+                                         TEST_DOM_NAME, TEST_ID_PROVIDER, NULL);
+    assert_non_null(test_ctx->tctx);
+
+    test_ctx->be_ctx = mock_be_ctx(test_ctx, test_ctx->tctx);
+    assert_non_null(test_ctx->be_ctx);
+
+    test_ctx->params = mock_dp_req_params(test_ctx, test_ctx->be_ctx->ev,
+                                          test_ctx->be_ctx, NULL,
+                                          DPT_ID, DPM_ACCOUNT_HANDLER);
+    assert_non_null(test_ctx->params);
+
+    check_leaks_push(test_ctx);
+
+    *state = test_ctx;
+
+    return 0;
+}
+
+static int test_teardown(void **state)
+{
+    struct test_ctx *test_ctx;
+
+    test_ctx = talloc_get_type_abort(*state, struct test_ctx);
+    assert_true(check_leaks_pop(test_ctx));
+    talloc_zfree(test_ctx);
+
+    assert_true(leak_check_teardown());
+    return 0;
+}
+
+static void test_deny_handler(void **state)
+{
+    errno_t ret;
+    struct test_ctx *test_ctx;
+    struct tevent_req *req;
+    struct pam_data *pd;
+    struct pam_data *out_pd;
+
+    test_ctx = talloc_get_type(*state, struct test_ctx);
+
+    pd = talloc_zero(test_ctx, struct pam_data);
+    assert_non_null(pd);
+
+    req = dp_access_deny_handler_send(test_ctx, NULL, pd, test_ctx->params);
+    assert_non_null(req);
+
+    tevent_loop_wait(test_ctx->tctx->ev);
+
+    ret = dp_access_deny_handler_recv(test_ctx, req, &out_pd);
+    assert_int_equal(ret, EOK);
+    assert_ptr_equal(pd, out_pd);
+    assert_int_equal(pd->pam_status, PAM_PERM_DENIED);
+
+    talloc_free(req);
+    talloc_free(pd);
+}
+
+static void test_permit_handler(void **state)
+{
+    errno_t ret;
+    struct test_ctx *test_ctx;
+    struct tevent_req *req;
+    struct pam_data *pd;
+    struct pam_data *out_pd;
+
+    test_ctx = talloc_get_type(*state, struct test_ctx);
+
+    pd = talloc_zero(test_ctx, struct pam_data);
+    assert_non_null(pd);
+
+    req = dp_access_permit_handler_send(test_ctx, NULL, pd, test_ctx->params);
+    assert_non_null(req);
+
+    tevent_loop_wait(test_ctx->tctx->ev);
+
+    ret = dp_access_permit_handler_recv(test_ctx, req, &out_pd);
+    assert_int_equal(ret, EOK);
+    assert_ptr_equal(pd, out_pd);
+    assert_int_equal(pd->pam_status, PAM_SUCCESS);
+
+    talloc_free(req);
+    talloc_free(pd);
+}
+
+int main(int argc, const char *argv[])
+{
+    poptContext pc;
+    int opt;
+    int rv;
+    int no_cleanup = 0;
+    struct poptOption long_options[] = {
+        POPT_AUTOHELP
+        SSSD_DEBUG_OPTS
+        {"no-cleanup", 'n', POPT_ARG_NONE, &no_cleanup, 0,
+         _("Do not delete the test database after a test run"), NULL },
+        POPT_TABLEEND
+    };
+
+    const struct CMUnitTest tests[] = {
+        cmocka_unit_test_setup_teardown(test_permit_handler,
+                                        test_setup,
+                                        test_teardown),
+        cmocka_unit_test_setup_teardown(test_deny_handler,
+                                        test_setup,
+                                        test_teardown),
+    };
+
+    /* Set debug level to invalid value so we can deside if -d 0 was used. */
+    debug_level = SSSDBG_INVALID;
+
+    pc = poptGetContext(argv[0], argc, argv, long_options, 0);
+    while((opt = poptGetNextOpt(pc)) != -1) {
+        switch(opt) {
+        default:
+            fprintf(stderr, "\nInvalid option %s: %s\n\n",
+                    poptBadOption(pc, 0), poptStrerror(opt));
+            poptPrintUsage(pc, stderr, 0);
+            return 1;
+        }
+    }
+    poptFreeContext(pc);
+
+    DEBUG_CLI_INIT(debug_level);
+
+    /* Even though normally the tests should clean up after themselves
+     * they might not after a failed run. Remove the old db to be sure */
+    tests_set_cwd();
+    test_dom_suite_cleanup(TESTS_PATH, TEST_CONF_DB, TEST_DOM_NAME);
+    test_dom_suite_setup(TESTS_PATH);
+
+    rv = cmocka_run_group_tests(tests, NULL, NULL);
+    if (rv == 0 && !no_cleanup) {
+        test_dom_suite_cleanup(TESTS_PATH, TEST_CONF_DB, TEST_DOM_NAME);
+    }
+
+    return rv;
+}