Commit 72df1cd8 by Edward Thomson

test: improve badname verification test

The name of the `add_invalid_filename` function suggests that we
_want_ to add an invalid filename.  Rename the function to show that
we expect to _fail_ to add the invalid filename.
parent f3b28604
...@@ -541,7 +541,7 @@ void test_index_tests__add_bypath_to_a_bare_repository_returns_EBAREPO(void) ...@@ -541,7 +541,7 @@ void test_index_tests__add_bypath_to_a_bare_repository_returns_EBAREPO(void)
git_repository_free(bare_repo); git_repository_free(bare_repo);
} }
static void add_invalid_filename(git_repository *repo, const char *fn) static void assert_add_bypath_fails(git_repository *repo, const char *fn)
{ {
git_index *index; git_index *index;
git_buf path = GIT_BUF_INIT; git_buf path = GIT_BUF_INIT;
...@@ -562,7 +562,7 @@ static void add_invalid_filename(git_repository *repo, const char *fn) ...@@ -562,7 +562,7 @@ static void add_invalid_filename(git_repository *repo, const char *fn)
} }
/* Test that writing an invalid filename fails */ /* Test that writing an invalid filename fails */
void test_index_tests__add_invalid_filename(void) void test_index_tests__cannot_add_invalid_filename(void)
{ {
git_repository *repo; git_repository *repo;
...@@ -577,13 +577,13 @@ void test_index_tests__add_invalid_filename(void) ...@@ -577,13 +577,13 @@ void test_index_tests__add_invalid_filename(void)
if (!git_path_exists("./invalid/.GiT")) if (!git_path_exists("./invalid/.GiT"))
cl_must_pass(p_mkdir("./invalid/.GiT", 0777)); cl_must_pass(p_mkdir("./invalid/.GiT", 0777));
add_invalid_filename(repo, ".git/hello"); assert_add_bypath_fails(repo, ".git/hello");
add_invalid_filename(repo, ".GIT/hello"); assert_add_bypath_fails(repo, ".GIT/hello");
add_invalid_filename(repo, ".GiT/hello"); assert_add_bypath_fails(repo, ".GiT/hello");
add_invalid_filename(repo, "./.git/hello"); assert_add_bypath_fails(repo, "./.git/hello");
add_invalid_filename(repo, "./foo"); assert_add_bypath_fails(repo, "./foo");
add_invalid_filename(repo, "./bar"); assert_add_bypath_fails(repo, "./bar");
add_invalid_filename(repo, "subdir/../bar"); assert_add_bypath_fails(repo, "subdir/../bar");
git_repository_free(repo); git_repository_free(repo);
......
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