1 files changed, 44 insertions, 27 deletions

diff --git a/tests/mailmap/basic.c b/tests/mailmap/basic.c
index 508598440..23f447f53 100644
--- a/tests/mailmap/basic.c
+++ b/tests/mailmap/basic.c
@@ -13,43 +13,59 @@ const char TEST_MAILMAP[] =
 	"<email@foo.com> <otheremail@foo.com>\n"
 	"<email@foo.com> Other Name <yetanotheremail@foo.com>\n";
 
+struct {
+	const char *real_name;
+	const char *real_email;
+	const char *replace_name;
+	const char *replace_email;
+} expected[] = {
+	{ "Foo bar", "foo@bar.com", NULL, "foo@baz.com" },
+	{ "Foo bar", "foo@bar.com", NULL, "foo@bal.com" },
+	{ NULL, "email@foo.com", NULL, "otheremail@foo.com" },
+	{ NULL, "email@foo.com", "Other Name", "yetanotheremail@foo.com" }
+};
+
 void test_mailmap_basic__initialize(void)
 {
 	git_buf buf = GIT_BUF_INIT;
-	git_buf_attach_notowned(&buf, TEST_MAILMAP, sizeof(TEST_MAILMAP) - 1);
+	git_buf_attach_notowned(&buf, TEST_MAILMAP, strlen(TEST_MAILMAP));
 
 	cl_git_pass(git_mailmap_from_buffer(&mailmap, &buf));
 }
 
 void test_mailmap_basic__cleanup(void)
 {
-	if (mailmap) {
-		git_mailmap_free(mailmap);
-		mailmap = NULL;
-	}
+	git_mailmap_free(mailmap);
+	mailmap = NULL;
 }
 
 void test_mailmap_basic__entry(void)
 {
 	const git_mailmap_entry *entry;
 
-	cl_assert(git_mailmap_entry_count(mailmap) == 4);
+	cl_assert_equal_sz(ARRAY_SIZE(expected), git_mailmap_entry_count(mailmap));
 
-	entry = git_mailmap_entry_byindex(mailmap, 0);
-	cl_assert(entry);
-	cl_assert(!entry->replace_name);
-	cl_assert(!git__strcmp(entry->replace_email, "foo@baz.com"));
+	for (size_t i = 0; i < ARRAY_SIZE(expected); ++i) {
+		entry = git_mailmap_entry_byindex(mailmap, i);
+		cl_assert(entry);
+		cl_assert_equal_s(entry->real_name, expected[i].real_name);
+		cl_assert_equal_s(entry->real_email, expected[i].real_email);
+		cl_assert_equal_s(entry->replace_name, expected[i].replace_name);
+		cl_assert_equal_s(entry->replace_email, expected[i].replace_email);
+	}
+}
 
-	entry = git_mailmap_entry_byindex(mailmap, 10000);
+void test_mailmap_basic__entry_large_index(void)
+{
+	const git_mailmap_entry *entry =
+		git_mailmap_entry_byindex(mailmap, 10000);
 	cl_assert(!entry);
 }
 
 void test_mailmap_basic__lookup_not_found(void)
 {
 	const git_mailmap_entry *entry = git_mailmap_entry_lookup(
-		mailmap,
-		"Whoever",
-		"doesnotexist@fo.com");
+		mailmap, "Whoever", "doesnotexist@fo.com");
 	cl_assert(!entry);
 }
 
@@ -58,31 +74,32 @@ void test_mailmap_basic__lookup(void)
 	const git_mailmap_entry *entry = git_mailmap_entry_lookup(
 		mailmap, "Typoed the name once", "foo@baz.com");
 	cl_assert(entry);
-	cl_assert(!git__strcmp(entry->real_name, "Foo bar"));
+	cl_assert_equal_s(entry->real_name, "Foo bar");
 }
 
 void test_mailmap_basic__empty_email_query(void)
 {
 	const char *name;
 	const char *email;
-	git_mailmap_resolve(
-		&name, &email, mailmap, "Author name", "otheremail@foo.com");
-	cl_assert(!git__strcmp(name, "Author name"));
-	cl_assert(!git__strcmp(email, "email@foo.com"));
+	cl_git_pass(git_mailmap_resolve(
+		&name, &email, mailmap, "Author name", "otheremail@foo.com"));
+	cl_assert_equal_s(name, "Author name");
+	cl_assert_equal_s(email, "email@foo.com");
 }
 
 void test_mailmap_basic__name_matching(void)
 {
 	const char *name;
 	const char *email;
-	git_mailmap_resolve(
-		&name, &email, mailmap, "Other Name", "yetanotheremail@foo.com");
-	cl_assert(!git__strcmp(name, "Other Name"));
-	cl_assert(!git__strcmp(email, "email@foo.com"));
+	cl_git_pass(git_mailmap_resolve(
+		&name, &email, mailmap, "Other Name", "yetanotheremail@foo.com"));
+
+	cl_assert_equal_s(name, "Other Name");
+	cl_assert_equal_s(email, "email@foo.com");
 
-	git_mailmap_resolve(
+	cl_git_pass(git_mailmap_resolve(
 		&name, &email, mailmap,
-		"Other Name That Doesn't Match", "yetanotheremail@foo.com");
-	cl_assert(!git__strcmp(name, "Other Name That Doesn't Match"));
-	cl_assert(!git__strcmp(email, "yetanotheremail@foo.com"));
+		"Other Name That Doesn't Match", "yetanotheremail@foo.com"));
+	cl_assert_equal_s(name, "Other Name That Doesn't Match");
+	cl_assert_equal_s(email, "yetanotheremail@foo.com");
 }