test: improve test_mutt_buffer_alloc()

diff --git a/mutt/buffer.c b/mutt/buffer.c
index 60166c3c1ee2b17e531e4352e5493b576a5efa3f..4ecb1f51038105c8ad28e2f008d6d112fd9f4690 100644 (file)
--- a/mutt/buffer.c
+++ b/mutt/buffer.c
@@ -298,13 +298,13 @@ bool mutt_buffer_is_empty(const struct Buffer *buf)
  */
 struct Buffer *mutt_buffer_alloc(size_t size)
 {
-  struct Buffer *b = mutt_mem_calloc(1, sizeof(struct Buffer));
+  struct Buffer *buf = mutt_mem_calloc(1, sizeof(struct Buffer));
 
-  b->data = mutt_mem_calloc(1, size);
-  b->dptr = b->data;
-  b->dsize = size;
+  buf->data = mutt_mem_calloc(1, size);
+  buf->dptr = buf->data;
+  buf->dsize = size;
 
-  return b;
+  return buf;
 }
 
 /**

diff --git a/test/buffer/mutt_buffer_alloc.c b/test/buffer/mutt_buffer_alloc.c
index 576a649b0d0a9744bc6c3da4135457f7833d91c5..96c6fb9e05e89eff3b841bb140769d5cf1f6b7ab 100644 (file)
--- a/test/buffer/mutt_buffer_alloc.c
+++ b/test/buffer/mutt_buffer_alloc.c
@@ -23,9 +23,21 @@
 #define TEST_NO_MAIN
 #include "acutest.h"
 #include "config.h"
+#include <stdbool.h>
 #include "mutt/mutt.h"
 
 void test_mutt_buffer_alloc(void)
 {
   // struct Buffer *mutt_buffer_alloc(size_t size);
+
+  size_t alloc_tests[] = { 0, 42, 1048576 };
+
+  for (size_t i = 0; i < mutt_array_size(alloc_tests); i++)
+  {
+    struct Buffer *buf = NULL;
+    TEST_CASE_("%lu", alloc_tests[i]);
+    buf = mutt_buffer_alloc(alloc_tests[i]);
+    TEST_CHECK(buf != NULL);
+    mutt_buffer_free(&buf);
+  }
 }