test: improve test_mutt_str_strcat()

diff --git a/mutt/string.c b/mutt/string.c
index 9f84d448003fcb8522e97f930d83b8856f748ee0..21c8cb16e961ceb2a629d2801f36826d68e64509 100644 (file)
--- a/mutt/string.c
+++ b/mutt/string.c
@@ -394,7 +394,7 @@ char *mutt_str_strdup(const char *str)
  * @param buf    Buffer containing source string
  * @param buflen Length of buffer
  * @param s      String to add
- * @retval ptr Start of joined string
+ * @retval ptr Start of the buffer
  */
 char *mutt_str_strcat(char *buf, size_t buflen, const char *s)
 {

diff --git a/test/string/mutt_str_strcat.c b/test/string/mutt_str_strcat.c
index dd5313d11c8aedef1380fa4609de9374f0c91dac..d1ce6f571979e561fac4837a822bd1287065d049 100644 (file)
--- a/test/string/mutt_str_strcat.c
+++ b/test/string/mutt_str_strcat.c
@@ -33,8 +33,38 @@ void test_mutt_str_strcat(void)
     TEST_CHECK(mutt_str_strcat(NULL, 10, "apple") == NULL);
   }
 
+  {
+    char buf[64] = { 0 };
+    TEST_CHECK(mutt_str_strcat(buf, 0, "apple") == buf);
+    TEST_CHECK(strcmp(buf, "") == 0);
+  }
+
   {
     char buf[32] = { 0 };
     TEST_CHECK(mutt_str_strcat(buf, sizeof(buf), NULL) == buf);
   }
+
+  {
+    char buf[32] = { 0 };
+    TEST_CHECK(mutt_str_strcat(buf, sizeof(buf), "") == buf);
+    TEST_CHECK(strcmp(buf, "") == 0);
+  }
+
+  {
+    char buf[32] = { 0 };
+    TEST_CHECK(mutt_str_strcat(buf, sizeof(buf), "banana") == buf);
+    TEST_CHECK(strcmp(buf, "banana") == 0);
+  }
+
+  {
+    char buf[32] = "apple";
+    TEST_CHECK(mutt_str_strcat(buf, sizeof(buf), "") == buf);
+    TEST_CHECK(strcmp(buf, "apple") == 0);
+  }
+
+  {
+    char buf[32] = "apple";
+    TEST_CHECK(mutt_str_strcat(buf, sizeof(buf), "banana") == buf);
+    TEST_CHECK(strcmp(buf, "applebanana") == 0);
+  }
 }