summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCarlos Martín Nieto <carlosmn@github.com>2016-05-06 18:44:37 +0200
committerCarlos Martín Nieto <carlosmn@github.com>2016-05-06 18:44:37 +0200
commit17a93afa782ac8aebb37ecb98b928215f36e2008 (patch)
tree04eb0610dcb6c1c9d8da657c0ba21df3e02f4935
parent2e43a37bcb14d324ab09de4801cb37cf2f31b9bb (diff)
parent5785ae9b5e315a2aca64ee4bac1b31bdab84c657 (diff)
downloadlibgit2-17a93afa782ac8aebb37ecb98b928215f36e2008.tar.gz
Merge pull request #3757 from johnhaley81/jh/fix-create-initial-commit
Fix `git_commit_create` for an initial commit
-rw-r--r--src/commit.c2
-rw-r--r--tests/commit/commit.c60
2 files changed, 61 insertions, 1 deletions
diff --git a/src/commit.c b/src/commit.c
index 5456751fe..79ddf2a63 100644
--- a/src/commit.c
+++ b/src/commit.c
@@ -104,7 +104,7 @@ static int validate_tree_and_parents(git_array_oid_t *parents, git_repository *r
i++;
}
- if (current_id && git_oid_cmp(current_id, git_array_get(*parents, 0))) {
+ if (current_id && (parents->size == 0 || git_oid_cmp(current_id, git_array_get(*parents, 0)))) {
giterr_set(GITERR_OBJECT, "failed to create commit: current tip is not the first parent");
error = GIT_EMODIFIED;
goto on_error;
diff --git a/tests/commit/commit.c b/tests/commit/commit.c
index c052cd568..b03169bc8 100644
--- a/tests/commit/commit.c
+++ b/tests/commit/commit.c
@@ -51,6 +51,66 @@ void test_commit_commit__create_unexisting_update_ref(void)
git_reference_free(ref);
}
+void test_commit_commit__create_initial_commit(void)
+{
+ git_oid oid;
+ git_tree *tree;
+ git_commit *commit;
+ git_signature *s;
+ git_reference *ref;
+
+ git_oid_fromstr(&oid, "a65fedf39aefe402d3bb6e24df4d4f5fe4547750");
+ cl_git_pass(git_commit_lookup(&commit, _repo, &oid));
+
+ git_oid_fromstr(&oid, "944c0f6e4dfa41595e6eb3ceecdb14f50fe18162");
+ cl_git_pass(git_tree_lookup(&tree, _repo, &oid));
+
+ cl_git_pass(git_signature_now(&s, "alice", "alice@example.com"));
+
+ cl_git_fail(git_reference_lookup(&ref, _repo, "refs/heads/foo/bar"));
+ cl_git_pass(git_commit_create(&oid, _repo, "refs/heads/foo/bar", s, s,
+ NULL, "initial commit", tree, 0, NULL));
+
+ cl_git_pass(git_reference_lookup(&ref, _repo, "refs/heads/foo/bar"));
+
+ cl_assert_equal_oid(&oid, git_reference_target(ref));
+
+ git_tree_free(tree);
+ git_commit_free(commit);
+ git_signature_free(s);
+ git_reference_free(ref);
+}
+
+void test_commit_commit__create_initial_commit_parent_not_current(void)
+{
+ git_oid oid;
+ git_oid original_oid;
+ git_tree *tree;
+ git_commit *commit;
+ git_signature *s;
+
+ git_oid_fromstr(&oid, "a65fedf39aefe402d3bb6e24df4d4f5fe4547750");
+ cl_git_pass(git_commit_lookup(&commit, _repo, &oid));
+
+ git_oid_fromstr(&oid, "944c0f6e4dfa41595e6eb3ceecdb14f50fe18162");
+ cl_git_pass(git_tree_lookup(&tree, _repo, &oid));
+
+ cl_git_pass(git_signature_now(&s, "alice", "alice@example.com"));
+
+ cl_git_pass(git_reference_name_to_id(&original_oid, _repo, "HEAD"));
+
+ cl_git_fail(git_commit_create(&oid, _repo, "HEAD", s, s,
+ NULL, "initial commit", tree, 0, NULL));
+
+ cl_git_pass(git_reference_name_to_id(&oid, _repo, "HEAD"));
+
+ cl_assert_equal_oid(&oid, &original_oid);
+
+ git_tree_free(tree);
+ git_commit_free(commit);
+ git_signature_free(s);
+}
+
void assert_commit_summary(const char *expected, const char *given)
{
git_commit *dummy;