diff options
author | Ben Straub <bs@github.com> | 2012-12-15 01:46:45 +0400 |
---|---|---|
committer | Ben Straub <bs@github.com> | 2012-12-15 01:46:45 +0400 |
commit | b9e7e2b4e148deb90119d4c2c52d8d9e889527cd (patch) | |
tree | 731eff52328710a47cd702b63430c575e1bfb999 /tests-clar/clone | |
parent | 0015b5875eda08fe1e593942710125e9db7896b5 (diff) |
Move non-options back out of options struct
Diffstat (limited to 'tests-clar/clone')
-rw-r--r-- | tests-clar/clone/network.c | 24 | ||||
-rw-r--r-- | tests-clar/clone/nonetwork.c | 29 |
2 files changed, 25 insertions, 28 deletions
diff --git a/tests-clar/clone/network.c b/tests-clar/clone/network.c index 49b57bbe5..d3009660e 100644 --- a/tests-clar/clone/network.c +++ b/tests-clar/clone/network.c @@ -9,6 +9,7 @@ CL_IN_CATEGORY("network") #define LIVE_EMPTYREPO_URL "http://github.com/libgit2/TestEmptyRepository" static git_repository *g_repo; +static git_remote *g_origin; static git_clone_options g_options; void test_clone_network__initialize(void) @@ -17,14 +18,12 @@ void test_clone_network__initialize(void) memset(&g_options, 0, sizeof(git_clone_options)); g_options.version = GIT_CLONE_OPTIONS_VERSION; - g_options.out = &g_repo; - g_options.local_path = "./foo"; - cl_git_pass(git_remote_new(&g_options.origin_remote, NULL, "origin", LIVE_REPO_URL, GIT_REMOTE_DEFAULT_FETCH)); + cl_git_pass(git_remote_new(&g_origin, NULL, "origin", LIVE_REPO_URL, GIT_REMOTE_DEFAULT_FETCH)); } void test_clone_network__cleanup(void) { - git_remote_free(g_options.origin_remote); + git_remote_free(g_origin); } static void cleanup_repository(void *path) @@ -43,7 +42,7 @@ void test_clone_network__network_full(void) cl_set_cleanup(&cleanup_repository, "./foo"); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_assert(!git_repository_is_bare(g_repo)); cl_git_pass(git_remote_load(&origin, g_repo, "origin")); @@ -58,7 +57,7 @@ void test_clone_network__network_bare(void) cl_set_cleanup(&cleanup_repository, "./foo"); g_options.bare = true; - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_assert(git_repository_is_bare(g_repo)); cl_git_pass(git_remote_load(&origin, g_repo, "origin")); @@ -70,8 +69,7 @@ void test_clone_network__cope_with_already_existing_directory(void) cl_set_cleanup(&cleanup_repository, "./foo"); p_mkdir("./foo", GIT_DIR_MODE); - cl_git_pass(git_clone(&g_options)); - git_repository_free(g_repo); g_repo = NULL; + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); } void test_clone_network__empty_repository(void) @@ -80,10 +78,10 @@ void test_clone_network__empty_repository(void) cl_set_cleanup(&cleanup_repository, "./foo"); - git_remote_free(g_options.origin_remote); - cl_git_pass(git_remote_new(&g_options.origin_remote, NULL, "origin", LIVE_EMPTYREPO_URL, GIT_REMOTE_DEFAULT_FETCH)); + git_remote_free(g_origin); + cl_git_pass(git_remote_new(&g_origin, NULL, "origin", LIVE_EMPTYREPO_URL, GIT_REMOTE_DEFAULT_FETCH)); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_assert_equal_i(true, git_repository_is_empty(g_repo)); cl_assert_equal_i(true, git_repository_head_orphan(g_repo)); @@ -100,7 +98,7 @@ void test_clone_network__can_prevent_the_checkout_of_a_standard_repo(void) git_buf path = GIT_BUF_INIT; cl_set_cleanup(&cleanup_repository, "./foo"); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_git_pass(git_buf_joinpath(&path, git_repository_workdir(g_repo), "master.txt")); cl_assert_equal_i(false, git_path_isfile(git_buf_cstr(&path))); @@ -139,7 +137,7 @@ void test_clone_network__can_checkout_a_cloned_repo(void) cl_set_cleanup(&cleanup_repository, "./foo"); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_git_pass(git_buf_joinpath(&path, git_repository_workdir(g_repo), "master.txt")); cl_assert_equal_i(true, git_path_isfile(git_buf_cstr(&path))); diff --git a/tests-clar/clone/nonetwork.c b/tests-clar/clone/nonetwork.c index 3626c544a..623a0683f 100644 --- a/tests-clar/clone/nonetwork.c +++ b/tests-clar/clone/nonetwork.c @@ -6,6 +6,7 @@ #define LIVE_REPO_URL "git://github.com/libgit2/TestGitRepository" static git_clone_options g_options; +static git_remote *g_origin; static git_repository *g_repo; void test_clone_nonetwork__initialize(void) @@ -14,14 +15,12 @@ void test_clone_nonetwork__initialize(void) memset(&g_options, 0, sizeof(git_clone_options)); g_options.version = GIT_CLONE_OPTIONS_VERSION; - g_options.out = &g_repo; - g_options.local_path = "./foo"; - cl_git_pass(git_remote_new(&g_options.origin_remote, NULL, "origin", cl_git_fixture_url("testrepo.git"), GIT_REMOTE_DEFAULT_FETCH)); + cl_git_pass(git_remote_new(&g_origin, NULL, "origin", cl_git_fixture_url("testrepo.git"), GIT_REMOTE_DEFAULT_FETCH)); } void test_clone_nonetwork__cleanup(void) { - git_remote_free(g_options.origin_remote); + git_remote_free(g_origin); } static void cleanup_repository(void *path) @@ -37,38 +36,38 @@ static void cleanup_repository(void *path) void test_clone_nonetwork__bad_url(void) { /* Clone should clean up the mess if the URL isn't a git repository */ - git_remote_free(g_options.origin_remote); - cl_git_pass(git_remote_new(&g_options.origin_remote, NULL, "origin", "not_a_repo", GIT_REMOTE_DEFAULT_FETCH)); + git_remote_free(g_origin); + cl_git_pass(git_remote_new(&g_origin, NULL, "origin", "not_a_repo", GIT_REMOTE_DEFAULT_FETCH)); - cl_git_fail(git_clone(&g_options)); + cl_git_fail(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_assert(!git_path_exists("./foo")); g_options.bare = true; - cl_git_fail(git_clone(&g_options)); + cl_git_fail(git_clone(&g_repo, g_origin, "./foo", &g_options)); cl_assert(!git_path_exists("./foo")); } void test_clone_nonetwork__local(void) { cl_set_cleanup(&cleanup_repository, "./foo"); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); } void test_clone_nonetwork__local_absolute_path(void) { const char *local_src = cl_fixture("testrepo.git"); - git_remote_free(g_options.origin_remote); - cl_git_pass(git_remote_new(&g_options.origin_remote, NULL, "origin", local_src, GIT_REMOTE_DEFAULT_FETCH)); + git_remote_free(g_origin); + cl_git_pass(git_remote_new(&g_origin, NULL, "origin", local_src, GIT_REMOTE_DEFAULT_FETCH)); cl_set_cleanup(&cleanup_repository, "./foo"); - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); } void test_clone_nonetwork__local_bare(void) { cl_set_cleanup(&cleanup_repository, "./foo"); g_options.bare = true; - cl_git_pass(git_clone(&g_options)); + cl_git_pass(git_clone(&g_repo, g_origin, "./foo", &g_options)); } void test_clone_nonetwork__fail_when_the_target_is_a_file(void) @@ -76,7 +75,7 @@ void test_clone_nonetwork__fail_when_the_target_is_a_file(void) cl_set_cleanup(&cleanup_repository, "./foo"); cl_git_mkfile("./foo", "Bar!"); - cl_git_fail(git_clone(&g_options)); + cl_git_fail(git_clone(&g_repo, g_origin, "./foo", &g_options)); } void test_clone_nonetwork__fail_with_already_existing_but_non_empty_directory(void) @@ -85,5 +84,5 @@ void test_clone_nonetwork__fail_with_already_existing_but_non_empty_directory(vo p_mkdir("./foo", GIT_DIR_MODE); cl_git_mkfile("./foo/bar", "Baz!"); - cl_git_fail(git_clone(&g_options)); + cl_git_fail(git_clone(&g_repo, g_origin, "./foo", &g_options)); } |