Enhance the packrefs::create_packfile() test
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79
diff --git a/tests/t10-refs.c b/tests/t10-refs.c
index 20b05e3..fd033e5 100644
--- a/tests/t10-refs.c
+++ b/tests/t10-refs.c
@@ -295,19 +295,60 @@ BEGIN_TEST("createref", create_new_object_id_ref)
must_pass(gitfo_unlink(ref_path)); /* TODO: replace with git_reference_delete() when available */
END_TEST
-BEGIN_TEST("packrefs", create_packfile)
+BEGIN_TEST("packrefs", create_packfile_with_empty_folder)
git_repository *repo;
- char temp_repo_path[GIT_PATH_MAX];
+ git_reference *reference;
+ char temp_path[GIT_PATH_MAX];
+ int path_len = 0;
+ const int mode = 0755; /* or 0777 ? */
must_pass(copydir_recurs(REPOSITORY_FOLDER, TEMP_DIR));
- git__joinpath(temp_repo_path, TEMP_DIR, TEST_REPOSITORY_NAME);
- must_pass(git_repository_open(&repo, temp_repo_path));
+ git__joinpath(temp_path, TEMP_DIR, TEST_REPOSITORY_NAME);
+ must_pass(git_repository_open(&repo, temp_path));
+
+ git__joinpath_n(temp_path, 3, repo->path_repository, GIT_REFS_HEADS_DIR, "empty_dir");
+ must_pass(gitfo_mkdir_recurs(temp_path, mode));
+
must_pass(git_reference_packall(repo));
git_repository_free(repo);
must_pass(rmdir_recurs(TEMP_DIR));
+END_TEST
+
+BEGIN_TEST("packrefs", create_packfile)
+ git_repository *repo;
+ git_reference *reference;
+ char temp_path[GIT_PATH_MAX];
+ int path_len = 0;
+
+ must_pass(copydir_recurs(REPOSITORY_FOLDER, TEMP_DIR));
+ git__joinpath(temp_path, TEMP_DIR, TEST_REPOSITORY_NAME);
+ must_pass(git_repository_open(&repo, temp_path));
+
+ /* Ensure a known loose ref can be looked up */
+ must_pass(git_repository_lookup_ref(&reference, repo, loose_tag_ref_name));
+ must_be_true((reference->type & GIT_REF_PACKED) == 0);
+ must_be_true(strcmp(reference->name, loose_tag_ref_name) == 0);
+
+ must_pass(git_reference_packall(repo));
+
+ /* Ensure the packed-refs file exists */
+ git__joinpath(temp_path, repo->path_repository, GIT_PACKEDREFS_FILE);
+ must_pass(gitfo_exists(temp_path));
+
+ /* Ensure the known ref can still be looked up but is now packed */
+ must_pass(git_repository_lookup_ref(&reference, repo, loose_tag_ref_name));
+ must_be_true((reference->type & GIT_REF_PACKED) != 0);
+ must_be_true(strcmp(reference->name, loose_tag_ref_name) == 0);
+
+ /* Ensure the known ref has been removed from the loose folder structure */
+ git__joinpath(temp_path, repo->path_repository, loose_tag_ref_name);
+ must_pass(!gitfo_exists(temp_path));
+
+ git_repository_free(repo);
+ must_pass(rmdir_recurs(TEMP_DIR));
END_TEST
static int ensure_refname_normalized(int is_oid_ref, const char *input_refname, const char *expected_refname)
@@ -508,7 +549,8 @@ git_testsuite *libgit2_suite_refs(void)
ADD_TEST(suite, "normalizeref", normalize_object_id_ref);
ADD_TEST(suite, "normalizeref", normalize_symbolic_ref);
ADD_TEST(suite, "normalizeref", jgit_tests);
- //ADD_TEST(suite, "packrefs", create_packfile);
+ ADD_TEST(suite, "packrefs", create_packfile_with_empty_folder);
+ ADD_TEST(suite, "packrefs", create_packfile);
return suite;
}