diff options
| author | Carlos Martín Nieto <cmn@dwim.me> | 2014-03-05 20:28:49 +0100 |
|---|---|---|
| committer | Carlos Martín Nieto <cmn@dwim.me> | 2014-03-07 16:03:15 +0100 |
| commit | ae32c54e5806cf8517beeb057cf640ed346508eb (patch) | |
| tree | 7ac45d3d7598966459e5fcb57e05fe790564cef5 /tests | |
| parent | a07b169834a11f60a2baa98f9735c2dfe7e4a1ee (diff) | |
| download | libgit2-ae32c54e5806cf8517beeb057cf640ed346508eb.tar.gz | |
Plug a few leaks in the tests
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/clone/empty.c | 2 | ||||
| -rw-r--r-- | tests/odb/backend/nobackend.c | 5 | ||||
| -rw-r--r-- | tests/reset/hard.c | 4 | ||||
| -rw-r--r-- | tests/reset/mixed.c | 6 |
4 files changed, 17 insertions, 0 deletions
diff --git a/tests/clone/empty.c b/tests/clone/empty.c index 78aef7dae..8f6071096 100644 --- a/tests/clone/empty.c +++ b/tests/clone/empty.c @@ -53,11 +53,13 @@ void test_clone_empty__can_clone_an_empty_local_repo_barely(void) cl_git_pass(git_branch_upstream_name(&buf, g_repo_cloned, local_name)); cl_assert_equal_s(expected_tracked_branch_name, buf.ptr); + git_buf_free(&buf); /* ...and the name of the remote... */ cl_git_pass(git_branch_remote_name(&buf, g_repo_cloned, expected_tracked_branch_name)); cl_assert_equal_s(expected_remote_name, buf.ptr); + git_buf_free(&buf); /* ...even when the remote HEAD is unborn as well */ cl_assert_equal_i(GIT_ENOTFOUND, git_reference_lookup(&ref, g_repo_cloned, diff --git a/tests/odb/backend/nobackend.c b/tests/odb/backend/nobackend.c index 7ed5acced..783641e8f 100644 --- a/tests/odb/backend/nobackend.c +++ b/tests/odb/backend/nobackend.c @@ -18,6 +18,11 @@ void test_odb_backend_nobackend__initialize(void) git_repository_set_config(_repo, config); git_repository_set_odb(_repo, odb); git_repository_set_refdb(_repo, refdb); + + /* The set increases the refcount and we don't want them anymore */ + git_config_free(config); + git_odb_free(odb); + git_refdb_free(refdb); } void test_odb_backend_nobackend__cleanup(void) diff --git a/tests/reset/hard.c b/tests/reset/hard.c index d4c7db45f..36120ee63 100644 --- a/tests/reset/hard.c +++ b/tests/reset/hard.c @@ -212,12 +212,16 @@ void test_reset_hard__reflog_is_correct(void) reflog_check(repo, "HEAD", 3, "emeric.fermas@gmail.com", exp_msg); reflog_check(repo, "refs/heads/master", 3, "emeric.fermas@gmail.com", exp_msg); + git_object_free(target); + /* Moved branch, expect default message */ cl_git_pass(git_revparse_single(&target, repo, "HEAD~^{commit}")); cl_git_pass(git_reset(repo, target, GIT_RESET_HARD, NULL, NULL)); reflog_check(repo, "HEAD", 3, "emeric.fermas@gmail.com", exp_msg); reflog_check(repo, "refs/heads/master", 4, NULL, "reset: moving"); + git_object_free(target); + /* Moved branch, expect custom message */ cl_git_pass(git_revparse_single(&target, repo, "HEAD~^{commit}")); cl_git_pass(git_reset(repo, target, GIT_RESET_HARD, NULL, "message1")); diff --git a/tests/reset/mixed.c b/tests/reset/mixed.c index 25272a75c..5d8ff63b4 100644 --- a/tests/reset/mixed.c +++ b/tests/reset/mixed.c @@ -61,12 +61,18 @@ void test_reset_mixed__reflog_is_correct(void) reflog_check(repo, "HEAD", 9, "yoram.harmelin@gmail.com", exp_msg); reflog_check(repo, "refs/heads/master", 9, "yoram.harmelin@gmail.com", exp_msg); + git_object_free(target); + target = NULL; + /* Moved branch, expect default message */ cl_git_pass(git_revparse_single(&target, repo, "HEAD~^{commit}")); cl_git_pass(git_reset(repo, target, GIT_RESET_MIXED, NULL, NULL)); reflog_check(repo, "HEAD", 9, "yoram.harmelin@gmail.com", exp_msg); reflog_check(repo, "refs/heads/master", 10, NULL, "reset: moving"); + git_object_free(target); + target = NULL; + /* Moved branch, expect custom message */ cl_git_pass(git_revparse_single(&target, repo, "HEAD~^{commit}")); cl_git_pass(git_reset(repo, target, GIT_RESET_MIXED, NULL, "message1")); |