diff options
Diffstat (limited to 'tests/t0401-parse.c')
| -rw-r--r-- | tests/t0401-parse.c | 243 |
1 files changed, 164 insertions, 79 deletions
diff --git a/tests/t0401-parse.c b/tests/t0401-parse.c index d01f7573a..3c6a4c09f 100644 --- a/tests/t0401-parse.c +++ b/tests/t0401-parse.c @@ -7,13 +7,13 @@ static char *test_commits_broken[] = { - /* empty commit */ +/* empty commit */ "", - /* random garbage */ +/* random garbage */ "asd97sa9du902e9a0jdsuusad09as9du098709aweu8987sd\n", - /* broken endlines 1 */ +/* broken endlines 1 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\r\n\ parent 05452d6349abcd67aa396dfb28660d765d8b2a36\r\n\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\r\n\ @@ -21,7 +21,7 @@ committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\r\n\ \r\n\ a test commit with broken endlines\r\n", - /* broken endlines 2 */ +/* broken endlines 2 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\ parent 05452d6349abcd67aa396dfb28660d765d8b2a36\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\ @@ -29,7 +29,7 @@ committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\ \ another test commit with broken endlines", - /* starting endlines */ +/* starting endlines */ "\ntree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ @@ -37,53 +37,40 @@ committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ \n\ a test commit with a starting endline\n", - /* corrupted commit 1 */ +/* corrupted commit 1 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ parent 05452d6349abcd67aa396df", - /* corrupted commit 2 */ +/* corrupted commit 2 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ parent ", - /* corrupted commit 3 */ +/* corrupted commit 3 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ parent ", - /* corrupted commit 4 */ +/* corrupted commit 4 */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ par", - /* - * FIXME: duplicated parents? - * It this supposed to pass? - */ -/* -"tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ -parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n\ -parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n\ -author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ -committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ -\n\ -duplicated parent", -*/ }; static char *test_commits_working[] = { - /* simple commit with no message */ +/* simple commit with no message */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ \n", - /* simple commit, no parent */ +/* simple commit, no parent */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ committer Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ \n\ a simple commit which works\n", - /* simple commit, 1 parents */ +/* simple commit, 1 parents */ "tree f6c0dad3c7b3481caa9d73db21f91964894a945b\n\ parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n\ author Vicent Marti <tanoku@gmail.com> 1273848544 +0200\n\ @@ -94,81 +81,179 @@ a simple commit which works\n", BEGIN_TEST(parse_oid_test) - git_oid oid; + git_oid oid; #define TEST_OID_PASS(string, header) { \ - char *ptr = string;\ - char *ptr_original = ptr;\ - size_t len = strlen(ptr);\ - must_pass(git_commit__parse_oid(&oid, &ptr, ptr + len, header));\ - must_be_true(ptr == ptr_original + len);\ + char *ptr = string;\ + char *ptr_original = ptr;\ + size_t len = strlen(ptr);\ + must_pass(git_commit__parse_oid(&oid, &ptr, ptr + len, header));\ + must_be_true(ptr == ptr_original + len);\ } #define TEST_OID_FAIL(string, header) { \ - char *ptr = string;\ - size_t len = strlen(ptr);\ - must_fail(git_commit__parse_oid(&oid, &ptr, ptr + len, header));\ + char *ptr = string;\ + size_t len = strlen(ptr);\ + must_fail(git_commit__parse_oid(&oid, &ptr, ptr + len, header));\ } - TEST_OID_PASS("parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "parent "); - TEST_OID_PASS("tree 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); - TEST_OID_PASS("random_heading 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "random_heading "); - TEST_OID_PASS("stuck_heading05452d6349abcd67aa396dfb28660d765d8b2a36\n", "stuck_heading"); - TEST_OID_PASS("tree 5F4BEFFC0759261D015AA63A3A85613FF2F235DE\n", "tree "); - TEST_OID_PASS("tree 1A669B8AB81B5EB7D9DB69562D34952A38A9B504\n", "tree "); - TEST_OID_PASS("tree 5B20DCC6110FCC75D31C6CEDEBD7F43ECA65B503\n", "tree "); - TEST_OID_PASS("tree 173E7BF00EA5C33447E99E6C1255954A13026BE4\n", "tree "); - - TEST_OID_FAIL("parent 05452d6349abcd67aa396dfb28660d765d8b2a36", "parent "); - TEST_OID_FAIL("05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); - TEST_OID_FAIL("parent05452d6349abcd67aa396dfb28660d765d8b2a6a\n", "parent "); - TEST_OID_FAIL("parent 05452d6349abcd67aa396dfb280d765d8b2a6\n", "parent "); - TEST_OID_FAIL("tree 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); - TEST_OID_FAIL("parent 0545xd6349abcd67aa396dfb28660d765d8b2a36\n", "parent "); - TEST_OID_FAIL("parent 0545xd6349abcd67aa396dfb28660d765d8b2a36FF\n", "parent "); - TEST_OID_FAIL("", "tree "); - TEST_OID_FAIL("", ""); + TEST_OID_PASS("parent 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "parent "); + TEST_OID_PASS("tree 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); + TEST_OID_PASS("random_heading 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "random_heading "); + TEST_OID_PASS("stuck_heading05452d6349abcd67aa396dfb28660d765d8b2a36\n", "stuck_heading"); + TEST_OID_PASS("tree 5F4BEFFC0759261D015AA63A3A85613FF2F235DE\n", "tree "); + TEST_OID_PASS("tree 1A669B8AB81B5EB7D9DB69562D34952A38A9B504\n", "tree "); + TEST_OID_PASS("tree 5B20DCC6110FCC75D31C6CEDEBD7F43ECA65B503\n", "tree "); + TEST_OID_PASS("tree 173E7BF00EA5C33447E99E6C1255954A13026BE4\n", "tree "); + + TEST_OID_FAIL("parent 05452d6349abcd67aa396dfb28660d765d8b2a36", "parent "); + TEST_OID_FAIL("05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); + TEST_OID_FAIL("parent05452d6349abcd67aa396dfb28660d765d8b2a6a\n", "parent "); + TEST_OID_FAIL("parent 05452d6349abcd67aa396dfb280d765d8b2a6\n", "parent "); + TEST_OID_FAIL("tree 05452d6349abcd67aa396dfb28660d765d8b2a36\n", "tree "); + TEST_OID_FAIL("parent 0545xd6349abcd67aa396dfb28660d765d8b2a36\n", "parent "); + TEST_OID_FAIL("parent 0545xd6349abcd67aa396dfb28660d765d8b2a36FF\n", "parent "); + TEST_OID_FAIL("", "tree "); + TEST_OID_FAIL("", ""); #undef TEST_OID_PASS #undef TEST_OID_FAIL END_TEST +BEGIN_TEST(parse_person_test) + +#define TEST_PERSON_PASS(_string, _header, _name, _email, _time) { \ + char *ptr = _string; \ + size_t len = strlen(_string);\ + git_commit_person person; \ + must_pass(git_commit__parse_person(&person, &ptr, ptr + len, _header));\ + must_be_true(strncmp(_name, person.name, 63) == 0);\ + must_be_true(strncmp(_email, person.email, 63) == 0);\ + must_be_true(_time == person.time);\ +} + +#define TEST_PERSON_FAIL(_string, _header) { \ + char *ptr = _string; \ + size_t len = strlen(_string);\ + git_commit_person person; \ + must_fail(git_commit__parse_person(&person, &ptr, ptr + len, _header));\ +} + + TEST_PERSON_PASS( + "author Vicent Marti <tanoku@gmail.com> 12345 \n", + "author ", + "Vicent Marti", + "tanoku@gmail.com", + 12345); + + TEST_PERSON_PASS( + "author Vicent Marti <> 12345 \n", + "author ", + "Vicent Marti", + "", + 12345); + + TEST_PERSON_PASS( + "author Vicent Marti <tanoku@gmail.com> 231301 +2020\n", + "author ", + "Vicent Marti", + "tanoku@gmail.com", + 231301); + + TEST_PERSON_PASS( + "author Vicent Marti with an outrageously long name \ + which will probably overflow the buffer <tanoku@gmail.com> 12345 \n", + "author ", + "Vicent Marti with an outrageously long name \ + which will probably overflow the buffer", + "tanoku@gmail.com", + 12345); + + TEST_PERSON_PASS( + "author Vicent Marti <tanokuwithaveryveryverylongemail\ + whichwillprobablyvoverflowtheemailbuffer@gmail.com> 12345 \n", + "author ", + "Vicent Marti", + "tanokuwithaveryveryverylongemail\ + whichwillprobablyvoverflowtheemailbuffer@gmail.com", + 12345); + + TEST_PERSON_FAIL( + "author Vicent Marti <tanoku@gmail.com> 12345 \n", + "author "); + + TEST_PERSON_FAIL( + "author Vicent Marti <tanoku@gmail.com> 12345 \n", + "committer "); + + TEST_PERSON_FAIL( + "author Vicent Marti 12345 \n", + "author "); + + TEST_PERSON_FAIL( + "author Vicent Marti <broken@email 12345 \n", + "author "); + + TEST_PERSON_FAIL( + "author Vicent Marti <tanoku@gmail.com> notime \n", + "author "); + + TEST_PERSON_FAIL( + "author Vicent Marti <tanoku@gmail.com>\n", + "author "); + + TEST_PERSON_FAIL( + "author ", + "author "); + +#undef TEST_PERSON_PASS +#undef TESt_PERSON_FAIL + +END_TEST + BEGIN_TEST(parse_buffer_test) - const int broken_commit_count = sizeof(test_commits_broken) / sizeof(*test_commits_broken); - const int working_commit_count = sizeof(test_commits_working) / sizeof(*test_commits_working); - int i; + const int broken_commit_count = sizeof(test_commits_broken) / sizeof(*test_commits_broken); + const int working_commit_count = sizeof(test_commits_working) / sizeof(*test_commits_working); + + const unsigned int default_flags = 0xFF; /* parse everything */ + + int i; - git_revpool *pool = gitrp_alloc(NULL); + git_revpool *pool = gitrp_alloc(NULL); - for (i = 0; i < broken_commit_count; ++i) { - git_commit commit; - memset(&commit, 0x0, sizeof(git_commit)); - commit.object.pool = pool; + for (i = 0; i < broken_commit_count; ++i) { + git_commit *commit; + commit = git__malloc(sizeof(git_commit)); + memset(commit, 0x0, sizeof(git_commit)); + commit->object.pool = pool; - must_fail(git_commit__parse_buffer( - &commit, - test_commits_broken[i], - strlen(test_commits_broken[i])) - ); + must_fail(git_commit__parse_buffer( + commit, + test_commits_broken[i], + strlen(test_commits_broken[i]), + default_flags) + ); - git_commit_list_clear(&commit.parents, 0); - } + git_commit__free(commit); + } - for (i = 0; i < working_commit_count; ++i) { - git_commit commit; - memset(&commit, 0x0, sizeof(git_commit)); - commit.object.pool = pool; + for (i = 0; i < working_commit_count; ++i) { + git_commit *commit; + commit = git__malloc(sizeof(git_commit)); + memset(commit, 0x0, sizeof(git_commit)); + commit->object.pool = pool; - must_pass(git_commit__parse_buffer( - &commit, - test_commits_working[i], - strlen(test_commits_working[i])) - ); + must_pass(git_commit__parse_buffer( + commit, + test_commits_working[i], + strlen(test_commits_working[i]), + default_flags) + ); - git_commit_list_clear(&commit.parents, 0); - } + git_commit__free(commit); + } - gitrp_free(pool); + gitrp_free(pool); END_TEST |