Commit f6fded8f by Vicent Marti

Proper cleanup jeez

parent 600d8dbf
#include "clar_libgit2.h" #include "clar_libgit2.h"
void test_config_read__cleanup(void)
{
cl_fixture_cleanup("./empty");
}
void test_config_read__simple_read(void) void test_config_read__simple_read(void)
{ {
git_config *cfg; git_config *cfg;
...@@ -436,11 +431,18 @@ void test_config_read__simple_read_from_specific_level(void) ...@@ -436,11 +431,18 @@ void test_config_read__simple_read_from_specific_level(void)
git_config_free(cfg); git_config_free(cfg);
} }
static void clean_empty_config(void *unused)
{
GIT_UNUSED(unused);
cl_fixture_cleanup("./empty");
}
void test_config_read__can_load_and_parse_an_empty_config_file(void) void test_config_read__can_load_and_parse_an_empty_config_file(void)
{ {
git_config *cfg; git_config *cfg;
int i; int i;
cl_set_cleanup(&clean_empty_config, NULL);
cl_git_mkfile("./empty", ""); cl_git_mkfile("./empty", "");
cl_git_pass(git_config_open_ondisk(&cfg, "./empty")); cl_git_pass(git_config_open_ondisk(&cfg, "./empty"));
cl_assert_equal_i(GIT_ENOTFOUND, git_config_get_int32(&i, cfg, "nope.neither")); cl_assert_equal_i(GIT_ENOTFOUND, git_config_get_int32(&i, cfg, "nope.neither"));
......
...@@ -24,8 +24,6 @@ static struct test_entry test_entries[] = { ...@@ -24,8 +24,6 @@ static struct test_entry test_entries[] = {
{48, "src/revobject.h", 1448, 0x4C3F7FE2} {48, "src/revobject.h", 1448, 0x4C3F7FE2}
}; };
static char *path_to_cleanup = NULL;
// Helpers // Helpers
static void copy_file(const char *src, const char *dst) static void copy_file(const char *src, const char *dst)
{ {
...@@ -73,14 +71,6 @@ void test_index_tests__initialize(void) ...@@ -73,14 +71,6 @@ void test_index_tests__initialize(void)
{ {
} }
void test_index_tests__cleanup(void)
{
if (path_to_cleanup)
cl_fixture_cleanup(path_to_cleanup);
path_to_cleanup = NULL;
}
void test_index_tests__empty_index(void) void test_index_tests__empty_index(void)
{ {
git_index *index; git_index *index;
...@@ -207,6 +197,12 @@ void test_index_tests__sort1(void) ...@@ -207,6 +197,12 @@ void test_index_tests__sort1(void)
git_index_free(index); git_index_free(index);
} }
static void cleanup_myrepo(void *opaque)
{
GIT_UNUSED(opaque);
cl_fixture_cleanup("myrepo");
}
void test_index_tests__add(void) void test_index_tests__add(void)
{ {
git_index *index; git_index *index;
...@@ -215,6 +211,8 @@ void test_index_tests__add(void) ...@@ -215,6 +211,8 @@ void test_index_tests__add(void)
const git_index_entry *entry; const git_index_entry *entry;
git_oid id1; git_oid id1;
cl_set_cleanup(&cleanup_myrepo, NULL);
/* Intialize a new repository */ /* Intialize a new repository */
cl_git_pass(git_repository_init(&repo, "./myrepo", 0)); cl_git_pass(git_repository_init(&repo, "./myrepo", 0));
...@@ -250,7 +248,6 @@ void test_index_tests__add(void) ...@@ -250,7 +248,6 @@ void test_index_tests__add(void)
git_index_free(index); git_index_free(index);
git_repository_free(repo); git_repository_free(repo);
path_to_cleanup = "myrepo";
} }
void test_index_tests__add_from_workdir_to_a_bare_repository_returns_EBAREPO(void) void test_index_tests__add_from_workdir_to_a_bare_repository_returns_EBAREPO(void)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment