Linus
---- >8 ----
From e5907b328d35eb37cfd1feb3a2431cb672beb4c8 Mon Sep 17 00:00:00 2001
From: Linus Arver <linusa@xxxxxxxxxx>
Date: Thu, 15 Jun 2023 18:25:47 -0700
Subject: [PATCH] add some unit tests for interpret-trailers
Signed-off-by: Linus Arver <linusa@xxxxxxxxxx>
---
Makefile | 5 +++
t/unit-tests/t-trailer.c | 77 ++++++++++++++++++++++++++++++++++++++++
trailer.c | 33 ++---------------
trailer.h | 32 +++++++++++++++++
4 files changed, 117 insertions(+), 30 deletions(-)
create mode 100644 t/unit-tests/t-trailer.c
diff --git a/Makefile b/Makefile
index f0ca5bae4b..9d68518168 100644
--- a/Makefile
+++ b/Makefile
@@ -2668,6 +2668,7 @@ scalar-objs: $(SCALAR_OBJS)
UNIT_TEST_PROGRAMS += t-basic
UNIT_TEST_PROGRAMS += t-strbuf
+UNIT_TEST_PROGRAMS += t-trailer
UNIT_TEST_PROGS = $(patsubst %,t/unit-tests/%$X,$(UNIT_TEST_PROGRAMS))
UNIT_TEST_OBJS = $(patsubst %,t/unit-tests/%.o,$(UNIT_TEST_PROGRAMS))
UNIT_TEST_OBJS += t/unit-tests/test-lib.o
@@ -3848,5 +3849,9 @@ t/unit-tests/t-strbuf$X: t/unit-tests/t-strbuf.o t/unit-tests/test-lib.o $(GITLI
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) \
$(filter %.o,$^) $(filter %.a,$^) $(LIBS)
+t/unit-tests/t-trailer$X: t/unit-tests/t-trailer.o t/unit-tests/test-lib.o builtin/interpret-trailers.o $(GITLIBS) GIT-LDFLAGS
+ $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) \
+ $(filter %.o,$^) $(filter %.a,$^) $(LIBS)
+
.PHONY: unit-tests
unit-tests: $(UNIT_TEST_PROGS)
diff --git a/t/unit-tests/t-trailer.c b/t/unit-tests/t-trailer.c
new file mode 100644
index 0000000000..150b5606fa
--- /dev/null
+++ b/t/unit-tests/t-trailer.c
@@ -0,0 +1,77 @@
+#include "test-lib.h"
+#include "trailer.h"
+
+static void t_token_len_without_separator(const char *token, size_t expected)
+{
+ size_t result;
+ result = token_len_without_separator(token, strlen(token));
+ check_uint(result, ==, expected);
+}
+
+static void t_after_or_end(enum trailer_where where, int expected)
+{
+ size_t result;
+ result = after_or_end(where);
+ check_int(result, ==, expected);
+}
+
+
+static void t_same_token(char *a, char *b, int expected)
+{
+ struct trailer_item trailer_item = { .token = a };
+ struct arg_item arg_item = { .token = b };
+ size_t result;
+ result = same_token(&trailer_item, &arg_item);
+ check_int(result, ==, expected);
+}
+
+int cmd_main(int argc, const char **argv)
+{
+ TEST(t_same_token("", "", 1),
+ "empty trailer_item and arg_item");
+ TEST(t_same_token("foo", "foo", 1),
+ "identical");
+ TEST(t_same_token("foo", "FOO", 1),
+ "case should not matter");
+ TEST(t_same_token("foo", "foobar", 1),
+ "arg_item is longer than trailer_item");
+ TEST(t_same_token("foobar", "foo", 1),
+ "trailer_item is longer than arg_item");
+ TEST(t_same_token("foo", "bar", 0),
+ "no similarity");
+
+ TEST(t_after_or_end(WHERE_AFTER, 1), "accept WHERE_AFTER");
+ TEST(t_after_or_end(WHERE_END, 1), "accept WHERE_END");
+ TEST(t_after_or_end(WHERE_DEFAULT, 0), "reject WHERE_END");
+
+ TEST(t_token_len_without_separator("Signed-off-by:", 13),
+ "token with trailing punctuation (colon)");
+ TEST(t_token_len_without_separator("Signed-off-by", 13),
+ "token without trailing punctuation");
+ TEST(t_token_len_without_separator("Foo bar:", 7),
+ "token with spaces with trailing punctuation (colon)");
+ TEST(t_token_len_without_separator("Foo bar", 7),
+ "token with spaces without trailing punctuation");
+ TEST(t_token_len_without_separator("-Foo bar:", 8),
+ "token with leading non-separator punctuation");
+ TEST(t_token_len_without_separator("- Foo bar:", 9),
+ "token with leading non-separator punctuation and space");
+ TEST(t_token_len_without_separator("F:", 1),
+ "one-letter token");
+ TEST(t_token_len_without_separator("abc%de#f@", 8),
+ "token with punctuation in-between");
+ TEST(t_token_len_without_separator(":;*%_.#f@", 8),
+ "token with multiple leading punctuation chars");
+
+ /*
+ * These tests fail unexpectedly. They are probably bugs, although it may be
+ * the case that these bugs never bubble up to the user because of other
+ * checks we do elsewhere up the stack.
+ */
+ TEST(t_same_token("", "foo", 0),
+ "empty trailer_item");
+ TEST(t_same_token("foo", "", 0),
+ "empty arg_item");
+
+ return test_done();
+}
diff --git a/trailer.c b/trailer.c
index a2c3ed6f28..9f59d8d7a6 100644
--- a/trailer.c
+++ b/trailer.c
@@ -13,35 +13,8 @@
* Copyright (c) 2013, 2014 Christian Couder <chriscool@xxxxxxxxxxxxx>
*/
-struct conf_info {
- char *name;
- char *key;
- char *command;
- char *cmd;
- enum trailer_where where;
- enum trailer_if_exists if_exists;
- enum trailer_if_missing if_missing;
-};
-
static struct conf_info default_conf_info;
-struct trailer_item {
- struct list_head list;
- /*
- * If this is not a trailer line, the line is stored in value
- * (excluding the terminating newline) and token is NULL.
- */
- char *token;
- char *value;
-};
-
-struct arg_item {
- struct list_head list;
- char *token;
- char *value;
- struct conf_info conf;
-};
-
static LIST_HEAD(conf_head);
static char *separators = ":";
@@ -62,7 +35,7 @@ static const char *git_generated_prefixes[] = {
pos != (head); \
pos = is_reverse ? pos->prev : pos->next)
-static int after_or_end(enum trailer_where where)
+int after_or_end(enum trailer_where where)
{
return (where == WHERE_AFTER) || (where == WHERE_END);
}
@@ -73,14 +46,14 @@ static int after_or_end(enum trailer_where where)
* 13, stripping the trailing punctuation but retaining
* internal punctuation.
*/
-static size_t token_len_without_separator(const char *token, size_t len)
+size_t token_len_without_separator(const char *token, size_t len)
{
while (len > 0 && !isalnum(token[len - 1]))
len--;
return len;
}
-static int same_token(struct trailer_item *a, struct arg_item *b)
+int same_token(struct trailer_item *a, struct arg_item *b)
{
size_t a_len, b_len, min_len;
diff --git a/trailer.h b/trailer.h
index 795d2fccfd..b2031eb305 100644
--- a/trailer.h
+++ b/trailer.h
@@ -146,4 +146,36 @@ int trailer_iterator_advance(struct trailer_iterator *iter);
*/
void trailer_iterator_release(struct trailer_iterator *iter);
+int after_or_end(enum trailer_where where);
+size_t token_len_without_separator(const char *token, size_t len);
+
+struct conf_info {
+ char *name;
+ char *key;
+ char *command;
+ char *cmd;
+ enum trailer_where where;
+ enum trailer_if_exists if_exists;
+ enum trailer_if_missing if_missing;
+};
+
+struct trailer_item {
+ struct list_head list;
+ /*
+ * If this is not a trailer line, the line is stored in value
+ * (excluding the terminating newline) and token is NULL.
+ */
+ char *token;
+ char *value;
+};
+
+struct arg_item {
+ struct list_head list;
+ char *token;
+ char *value;
+ struct conf_info conf;
+};
+
+int same_token(struct trailer_item *a, struct arg_item *b);
+
#endif /* TRAILER_H */
