proxmox-mirrors/libgit2
1
0
Fork 0
mirror of https://git.proxmox.com/git/libgit2 synced 2025-05-04 00:55:04 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

index: refine add_from_workdir() error report

Browse Source
This commit is contained in:
nulltoken 2012-11-10 21:01:05 +01:00
parent 69c068c79f
commit 33f95a9b32
3 changed files with 48 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
src/index.c
View File

@ -348,6 +348,12 @@ void git_index_clear(git_index *index)
index->tree = NULL;
}
static int create_index_error(int error, const char *msg)
{
giterr_set(GITERR_INDEX, msg);
return error;
}
int git_index_set_caps(git_index *index, unsigned int caps)
{
int old_ignore_case;
@ -362,11 +368,8 @@ int git_index_set_caps(git_index *index, unsigned int caps)
if (INDEX_OWNER(index) == NULL ||
git_repository_config__weakptr(&cfg, INDEX_OWNER(index)) < 0)
{
giterr_set(GITERR_INDEX,
"Cannot get repository config to set index caps");
return -1;
}
return create_index_error(-1,
"Cannot get repository config to set index caps");
if (git_config_get_bool(&val, cfg, "core.ignorecase") == 0)
index->ignore_case = (val != 0);
@ -402,11 +405,9 @@ int git_index_read(git_index *index)
git_buf buffer = GIT_BUF_INIT;
git_futils_filestamp stamp;
if (!index->index_file_path) {
giterr_set(GITERR_INDEX,
if (!index->index_file_path)
return create_index_error(-1,
"Failed to read index: The index is in-memory only");
return -1;
}
if (!index->on_disk || git_path_exists(index->index_file_path) == false) {
git_index_clear(index);
@ -437,11 +438,9 @@ int git_index_write(git_index *index)
git_filebuf file = GIT_FILEBUF_INIT;
int error;
if (!index->index_file_path) {
giterr_set(GITERR_INDEX,
"Failed to write index: The index is in-memory only");
return -1;
}
if (!index->index_file_path)
return create_index_error(-1,
"Failed to read index: The index is in-memory only");
git_vector_sort(&index->entries);
git_vector_sort(&index->reuc);
@ -474,11 +473,9 @@ int git_index_write_tree(git_oid *oid, git_index *index)
repo = (git_repository *)GIT_REFCOUNT_OWNER(index);
if (repo == NULL) {
giterr_set(GITERR_INDEX, "Failed to write tree. "
if (repo == NULL)
return create_index_error(-1, "Failed to write tree. "
"The index file is not backed up by an existing repository");
return -1;
}
return git_tree__write_index(oid, index, repo);
}
@ -539,13 +536,16 @@ static int index_entry_init(git_index_entry **entry_out, git_index *index, const
git_buf full_path = GIT_BUF_INIT;
int error;
if (INDEX_OWNER(index) == NULL ||
(workdir = git_repository_workdir(INDEX_OWNER(index))) == NULL)
{
giterr_set(GITERR_INDEX,
if (INDEX_OWNER(index) == NULL)
return create_index_error(-1,
"Could not initialize index entry. "
"Index is not backed up by an existing repository.");
workdir = git_repository_workdir(INDEX_OWNER(index));
if (!workdir)
return create_index_error(GIT_EBAREREPO,
"Could not initialize index entry. Repository is bare");
return -1;
}
if ((error = git_buf_joinpath(&full_path, workdir, rel_path)) < 0)
return error;

11
tests-clar/index/inmemory.c
View File

@ -9,3 +9,14 @@ void test_index_inmemory__can_create_an_inmemory_index(void)
git_index_free(index);
}
void test_index_inmemory__cannot_add_from_workdir_to_an_inmemory_index(void)
{
git_index *index;
cl_git_pass(git_index_new(&index));
cl_assert_equal_i(GIT_ERROR, git_index_add_from_workdir(index, "test.txt"));
git_index_free(index);
}

13
tests-clar/index/tests.c
View File

@ -248,3 +248,16 @@ void test_index_tests__add(void)
git_repository_free(repo);
}
void test_index_tests__add_from_workdir_to_a_bare_repository_returns_EBAREPO(void)
{
git_repository *bare_repo;
git_index *index;
cl_git_pass(git_repository_open(&bare_repo, cl_fixture("testrepo.git")));
cl_git_pass(git_repository_index(&index, bare_repo));
cl_assert_equal_i(GIT_EBAREREPO, git_index_add_from_workdir(index, "test.txt"));
git_index_free(index);
git_repository_free(bare_repo);
}