Commit 05ac7051 by Carlos Martín Nieto

remote: test for supported URLs in a single place

Instead of using ifdefs to run the tests, use them to set when we expect
to support a particular scheme and always have the tests in the code.
parent ba67c075
......@@ -91,31 +91,24 @@ void test_network_remote_remotes__error_when_no_push_available(void)
git_remote_free(r);
}
void test_network_remote_remotes__supported_transport_methods_are_supported(void)
void test_network_remote_remotes__supported_urls(void)
{
cl_assert(git_remote_supported_url("git://github.com/libgit2/libgit2"));
cl_assert(git_remote_supported_url("http://github.com/libgit2/libgit2"));
int ssh_supported = 0, https_supported = 0;
#ifdef GIT_SSH
cl_assert(git_remote_supported_url("git@github.com:libgit2/libgit2.git"));
cl_assert(git_remote_supported_url("ssh://git@github.com/libgit2/libgit2.git"));
ssh_supported = 1;
#endif
#if defined(GIT_SSL) || defined(GIT_WINHTTP)
cl_assert(git_remote_supported_url("https://git@github.com/libgit2/libgit2.git"));
https_supported = 1;
#endif
}
void test_network_remote_remotes__unsupported_transport_methods_are_unsupported(void)
{
#ifndef GIT_SSH
cl_assert(!git_remote_supported_url("git@github.com:libgit2/libgit2.git"));
cl_assert(!git_remote_supported_url("ssh://git@github.com/libgit2/libgit2.git"));
#endif
cl_assert(git_remote_supported_url("git://github.com/libgit2/libgit2"));
cl_assert(git_remote_supported_url("http://github.com/libgit2/libgit2"));
#if !defined(GIT_SSL) && !defined(GIT_WINHTTP)
cl_assert(!git_remote_supported_url("https://git@github.com/libgit2/libgit2.git"));
#endif
cl_assert_equal_i(ssh_supported, git_remote_supported_url("git@github.com:libgit2/libgit2.git"));
cl_assert_equal_i(ssh_supported, git_remote_supported_url("ssh://git@github.com/libgit2/libgit2.git"));
cl_assert_equal_i(https_supported, git_remote_supported_url("https://github.com/libgit2/libgit2.git"));
}
void test_network_remote_remotes__refspec_parsing(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