Commit a336ed18 by Edward Thomson Committed by Patrick Steinhardt

test: improve badname verification test

The name of the `write_invalid_filename` function suggests that we
_want_ to write an invalid filename.  Rename the function to show that
we expect to _fail_ to write the invalid filename.
parent f49378b6
...@@ -599,7 +599,7 @@ static void replace_char(char *str, char in, char out) ...@@ -599,7 +599,7 @@ static void replace_char(char *str, char in, char out)
*c = out; *c = out;
} }
static void write_invalid_filename(git_repository *repo, const char *fn_orig) static void assert_write_fails(git_repository *repo, const char *fn_orig)
{ {
git_index *index; git_index *index;
git_oid expected; git_oid expected;
...@@ -672,13 +672,13 @@ void test_index_tests__write_invalid_filename(void) ...@@ -672,13 +672,13 @@ void test_index_tests__write_invalid_filename(void)
cl_git_pass(git_repository_init(&repo, "./invalid", 0)); cl_git_pass(git_repository_init(&repo, "./invalid", 0));
write_invalid_filename(repo, ".git/hello"); assert_write_fails(repo, ".git/hello");
write_invalid_filename(repo, ".GIT/hello"); assert_write_fails(repo, ".GIT/hello");
write_invalid_filename(repo, ".GiT/hello"); assert_write_fails(repo, ".GiT/hello");
write_invalid_filename(repo, "./.git/hello"); assert_write_fails(repo, "./.git/hello");
write_invalid_filename(repo, "./foo"); assert_write_fails(repo, "./foo");
write_invalid_filename(repo, "./bar"); assert_write_fails(repo, "./bar");
write_invalid_filename(repo, "foo/../bar"); assert_write_fails(repo, "foo/../bar");
git_repository_free(repo); git_repository_free(repo);
...@@ -696,10 +696,10 @@ void test_index_tests__honors_protect_filesystems(void) ...@@ -696,10 +696,10 @@ void test_index_tests__honors_protect_filesystems(void)
cl_repo_set_bool(repo, "core.protectHFS", true); cl_repo_set_bool(repo, "core.protectHFS", true);
cl_repo_set_bool(repo, "core.protectNTFS", true); cl_repo_set_bool(repo, "core.protectNTFS", true);
write_invalid_filename(repo, ".git./hello"); assert_write_fails(repo, ".git./hello");
write_invalid_filename(repo, ".git\xe2\x80\xad/hello"); assert_write_fails(repo, ".git\xe2\x80\xad/hello");
write_invalid_filename(repo, "git~1/hello"); assert_write_fails(repo, "git~1/hello");
write_invalid_filename(repo, ".git\xe2\x81\xaf/hello"); assert_write_fails(repo, ".git\xe2\x81\xaf/hello");
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