|
 |
52797d |
From 551a83052ff92fa975f1015e2fc25b174f85b0e6 Mon Sep 17 00:00:00 2001
|
|
|
52797d |
From: Aleš Matěj <amatej@redhat.com>
|
|
|
52797d |
Date: Tue, 26 Oct 2021 09:33:51 +0200
|
|
|
52797d |
Subject: [PATCH] Add unittest for setting up repo with empty keyfile (RhBug:1994614)
|
|
|
52797d |
|
|
|
52797d |
The test ensures we don't crash when the keyfile is empty/not loaded.
|
|
|
52797d |
Test for: https://github.com/rpm-software-management/libdnf/commit/be66d0908d939ddaf3022c5489012bd1dbdb0075
|
|
|
52797d |
|
|
|
52797d |
RHEL bug:
|
|
|
52797d |
https://bugzilla.redhat.com/show_bug.cgi?id=1994614
|
|
|
52797d |
Fedora bug:
|
|
|
52797d |
https://bugzilla.redhat.com/show_bug.cgi?id=1946024
|
|
|
52797d |
---
|
|
|
52797d |
tests/libdnf/dnf-self-test.c | 21 +++++++++++++++++++++
|
|
|
52797d |
1 file changed, 21 insertions(+)
|
|
|
52797d |
|
|
|
52797d |
diff --git a/tests/libdnf/dnf-self-test.c b/tests/libdnf/dnf-self-test.c
|
|
|
52797d |
index 32a21e6..5295837 100644
|
|
|
52797d |
--- a/tests/libdnf/dnf-self-test.c
|
|
|
52797d |
+++ b/tests/libdnf/dnf-self-test.c
|
|
|
52797d |
@@ -199,6 +199,26 @@ ch_test_repo_func(void)
|
|
|
52797d |
g_object_unref(context);
|
|
|
52797d |
}
|
|
|
52797d |
|
|
|
52797d |
+static void
|
|
|
52797d |
+dnf_repo_setup_with_empty_keyfile(void)
|
|
|
52797d |
+{
|
|
|
52797d |
+ DnfContext *context = dnf_context_new();
|
|
|
52797d |
+ dnf_context_set_release_ver(context, "33");
|
|
|
52797d |
+ DnfRepo *repo = dnf_repo_new(context);
|
|
|
52797d |
+ GKeyFile *empty_key_file = g_key_file_new();
|
|
|
52797d |
+
|
|
|
52797d |
+ dnf_repo_set_keyfile(repo, empty_key_file);
|
|
|
52797d |
+
|
|
|
52797d |
+ GError *error = NULL;
|
|
|
52797d |
+ // Empty key file is not an errror, there is just no configuration to be loaded.
|
|
|
52797d |
+ g_assert(dnf_repo_setup(repo, &error));
|
|
|
52797d |
+ g_assert_no_error(error);
|
|
|
52797d |
+
|
|
|
52797d |
+ g_object_unref(repo);
|
|
|
52797d |
+ g_object_unref(context);
|
|
|
52797d |
+ g_key_file_free(empty_key_file);
|
|
|
52797d |
+}
|
|
|
52797d |
+
|
|
|
52797d |
static guint _allow_cancel_updates = 0;
|
|
|
52797d |
static guint _action_updates = 0;
|
|
|
52797d |
static guint _package_progress_updates = 0;
|
|
|
52797d |
@@ -1217,6 +1237,7 @@ main(int argc, char **argv)
|
|
|
52797d |
g_test_add_func("/libdnf/lock", dnf_lock_func);
|
|
|
52797d |
g_test_add_func("/libdnf/lock[threads]", dnf_lock_threads_func);
|
|
|
52797d |
g_test_add_func("/libdnf/repo", ch_test_repo_func);
|
|
|
52797d |
+ g_test_add_func("/libdnf/repo_empty_keyfile", dnf_repo_setup_with_empty_keyfile);
|
|
|
52797d |
g_test_add_func("/libdnf/state", dnf_state_func);
|
|
|
52797d |
g_test_add_func("/libdnf/state[child]", dnf_state_child_func);
|
|
|
52797d |
g_test_add_func("/libdnf/state[parent-1-step]", dnf_state_parent_one_step_proxy_func);
|
|
|
52797d |
--
|
|
|
52797d |
libgit2 1.1.0
|
|
|
52797d |
|