libsframe: testsuite: make test names unique

Fix PR libsframe/33437 - libsframe test names are not unique

The TEST () macro definition originally in plt-findfre-2.c, was being
used to differentiate between multiple runs of the testcases.  Adapt
that definition a bit to allow for a variable number of arguments following
the test condition: A test name format string may be used by macro
users, such that the name of the tests are unique.

Move the new variadic TEST macro definition in the testsuite's common
header sframe-test.h, and use it throughout the testsuite.

Reviewed-by: Jens Remus <jremus@linux.ibm.com>

libsframe/testsuite/
	PR libsframe/33437
	* libsframe.decode/be-flipping.c: Use new TEST macro with
	suffix.
	* libsframe.decode/frecnt-1.c: Likewise.
	* libsframe.decode/frecnt-2.c: Likewise.
	* libsframe.encode/encode-1.c: Likewise.
	* libsframe.find/findfre-1.c: Likewise.
	* libsframe.find/findfunc-1.c: Likewise.
	* libsframe.find/plt-findfre-1.c: Likewise.
	* libsframe.find/plt-findfre-2.c: Likewise.
	* sframe-test.h: Move the TEST macro definition to this
	testsuite header.
diff --git a/libsframe/testsuite/libsframe.decode/be-flipping.c b/libsframe/testsuite/libsframe.decode/be-flipping.c
index c8b627b..5f97594 100644
--- a/libsframe/testsuite/libsframe.decode/be-flipping.c
+++ b/libsframe/testsuite/libsframe.decode/be-flipping.c
@@ -47,16 +47,6 @@
   size_t sf_size;
   uint8_t rep_block_size;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   /* Test setup.  */
   fp = fopen (DATA, "r");
   if (fp == NULL)
@@ -87,15 +77,15 @@
      the host running the test is a little-endian system.  This endian-flipped
      copy of the buffer is kept internally in dctx.  */
   dctx = sframe_decode (sf_buf, sf_size, &err);
-  TEST ("be-flipping: Decoder setup", dctx != NULL);
+  TEST (dctx != NULL, "be-flipping-1: Decoder setup");
 
   unsigned int fde_cnt = sframe_decoder_get_num_fidx (dctx);
-  TEST ("be-flipping: Decoder FDE count", fde_cnt == 1);
+  TEST (fde_cnt == 1, "be-flipping-1: Decoder FDE count");
 
   err = sframe_decoder_get_funcdesc_v2 (dctx, 0, &nfres, &fsize, &fstart,
 					&finfo, &rep_block_size);
-  TEST ("be-flipping: Decoder get FDE", err == 0);
-  TEST ("be-flipping: Decoder FRE count", nfres == 5);
+  TEST (err == 0, "be-flipping-1: Decoder get FDE");
+  TEST (nfres == 5, "be-flipping-1: Decoder FRE count");
 
   free (sf_buf);
   sf_buf = NULL;
@@ -105,6 +95,6 @@
 
 setup_fail:
   sframe_decoder_free (&dctx);
-  fail ("be-flipping: Test setup");
+  fail ("be-flipping-1: Test setup");
   return 1;
 }
diff --git a/libsframe/testsuite/libsframe.decode/frecnt-1.c b/libsframe/testsuite/libsframe.decode/frecnt-1.c
index 9cf55ef..53d70a0 100644
--- a/libsframe/testsuite/libsframe.decode/frecnt-1.c
+++ b/libsframe/testsuite/libsframe.decode/frecnt-1.c
@@ -38,16 +38,6 @@
   size_t sf_size;
   uint8_t rep_block_size;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   /* Test Setup.  */
   fp = fopen (DATA, "r");
   if (fp == NULL)
@@ -68,18 +58,18 @@
   /* Execute tests.  */
   sf_size = fread (sf_buf, 1, st.st_size, fp);
   fclose (fp);
-  TEST ("frecnt-1: Read data", sf_size != 0);
+  TEST (sf_size != 0, "frecnt-1: Read data");
 
   dctx = sframe_decode (sf_buf, sf_size, &err);
-  TEST ("frecnt-1: Decoder setup", dctx != NULL);
+  TEST (dctx != NULL, "frecnt-1: Decoder setup");
 
   unsigned int fde_cnt = sframe_decoder_get_num_fidx (dctx);
-  TEST ("frecnt-1: Decoder FDE count", fde_cnt == 1);
+  TEST (fde_cnt == 1, "frecnt-1: Decoder FDE count");
 
   err = sframe_decoder_get_funcdesc_v2 (dctx, 0, &nfres, &fsize, &fstart,
 					&finfo, &rep_block_size);
-  TEST ("frecnt-1: Decoder get FDE", err == 0);
-  TEST ("frecnt-1: Decoder FRE count", nfres == 4);
+  TEST (err == 0, "frecnt-1: Decoder get FDE");
+  TEST (nfres == 4, "frecnt-1: Decoder FRE count");
 
   free (sf_buf);
   sf_buf = NULL;
diff --git a/libsframe/testsuite/libsframe.decode/frecnt-2.c b/libsframe/testsuite/libsframe.decode/frecnt-2.c
index 9136e9a..a8ca69f 100644
--- a/libsframe/testsuite/libsframe.decode/frecnt-2.c
+++ b/libsframe/testsuite/libsframe.decode/frecnt-2.c
@@ -40,16 +40,6 @@
   size_t sf_size;
   uint8_t rep_block_size;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   fp = fopen (DATA, "r");
   if (fp == NULL)
     goto setup_fail;
@@ -69,20 +59,20 @@
   /* Execute tests.  */
   sf_size = fread (sf_buf, 1, st.st_size, fp);
   fclose (fp);
-  TEST ("frecnt-2: Read data", sf_size != 0);
+  TEST (sf_size != 0, "frecnt-2: Read data");
 
   dctx = sframe_decode (sf_buf, sf_size, &err);
-  TEST ("frecnt-2: Decode setup", dctx != NULL);
+  TEST (dctx != NULL, "frecnt-2: Decode setup");
 
   unsigned int fde_cnt = sframe_decoder_get_num_fidx (dctx);
-  TEST ("frecnt-2: Decode FDE count", fde_cnt == 2);
+  TEST (fde_cnt == 2, "frecnt-2: Decode FDE count");
 
   for (i = 0; i < fde_cnt; ++i)
     {
       err = sframe_decoder_get_funcdesc_v2 (dctx, i, &nfres, &fsize, &fstart,
 					    &finfo, &rep_block_size);
-      TEST ("frecnt-2: Decode get FDE", err == 0);
-      TEST ("frecnt-2: Decode get FRE", nfres == 4);
+      TEST (err == 0, "frecnt-2: Decode get FDE%d", i);
+      TEST (nfres == 4, "frecnt-2: Decode num FREs for FDE%d", i);
     }
 
   free (sf_buf);
diff --git a/libsframe/testsuite/libsframe.encode/encode-1.c b/libsframe/testsuite/libsframe.encode/encode-1.c
index da477e5..6df677e 100644
--- a/libsframe/testsuite/libsframe.encode/encode-1.c
+++ b/libsframe/testsuite/libsframe.encode/encode-1.c
@@ -151,16 +151,6 @@
   unsigned int fde_cnt = 0;
   int match_p = 0;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   sframe_vaddr = 0x4020c8;
   encode = sframe_encode (SFRAME_VERSION,
 			  SFRAME_F_FDE_FUNC_START_PCREL,
@@ -170,28 +160,28 @@
 			  &err);
 
   fde_cnt = sframe_encoder_get_num_fidx (encode);
-  TEST ("encode-1: Encoder FDE count", fde_cnt == 0);
+  TEST (fde_cnt == 0, "encode-1: Encoder FDE count");
 
   err = sframe_encoder_add_fre (encode, 1, &frep);
-  TEST ("encode-1: Encoder update workflow", err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR, "encode-1: Encoder update workflow");
 
   func1_start_vaddr = 0x401106;
   err = add_fde1 (encode, func1_start_vaddr, sframe_vaddr, 0, &func1_size);
-  TEST ("encode-1: Encoder adding FDE1", err == 0);
+  TEST (err == 0, "encode-1: Encoder adding FDE1");
 
   /* Function 2 is placed after 0x0 bytes from the end of Function 1.  */
   func2_start_vaddr = func1_start_vaddr + func1_size + 0x0;
   err = add_fde2 (encode, func2_start_vaddr, sframe_vaddr, 1, &func2_size);
-  TEST ("encode-1: Encoder adding FDE2", err == 0);
+  TEST (err == 0, "encode-1: Encoder adding FDE2");
 
   fde_cnt = sframe_encoder_get_num_fidx (encode);
-  TEST ("encode-1: Encoder FDE count", fde_cnt == 2);
+  TEST (fde_cnt == 2, "encode-1: Encoder FDE count");
 
   sframe_buf = sframe_encoder_write (encode, &sf_size, &err);
-  TEST ("encode-1: Encoder write", err == 0);
+  TEST (err == 0, "encode-1: Encoder write");
 
   match_p = data_match (sframe_buf, sf_size);
-  TEST ("encode-1: Encode buffer match", match_p == 1);
+  TEST (match_p == 1, "encode-1: Encode buffer match");
 
   sframe_encoder_free (&encode);
   return 0;
diff --git a/libsframe/testsuite/libsframe.find/findfre-1.c b/libsframe/testsuite/libsframe.find/findfre-1.c
index cbd7ab3..c904888 100644
--- a/libsframe/testsuite/libsframe.find/findfre-1.c
+++ b/libsframe/testsuite/libsframe.find/findfre-1.c
@@ -88,7 +88,8 @@
 }
 
 static
-void test_text_findfre (uint32_t text_vaddr, uint32_t sframe_vaddr)
+void test_text_findfre (const char suffix, uint32_t text_vaddr,
+			uint32_t sframe_vaddr)
 {
   sframe_encoder_ctx *encode;
   sframe_decoder_ctx *dctx;
@@ -103,16 +104,6 @@
   size_t sf_size;
   int err = 0;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   encode = sframe_encode (SFRAME_VERSION,
 			  SFRAME_F_FDE_FUNC_START_PCREL,
 			  SFRAME_ABI_AMD64_ENDIAN_LITTLE,
@@ -122,56 +113,57 @@
 
   func1_start_vaddr = text_vaddr;
   err = add_fde1 (encode, func1_start_vaddr, sframe_vaddr, 0, &func1_size);
-  TEST ("findfre-1: Adding FDE1", err == 0);
+  TEST (err == 0, "findfre-1%c: Adding FDE1", suffix);
 
   /* Function 2 is placed after 0x10 bytes from the end of Function 1.  */
   func2_start_vaddr = func1_start_vaddr + func1_size + 0x10;
   err = add_fde2 (encode, func2_start_vaddr, sframe_vaddr, 1, &func2_size);
-  TEST ("findfre-1: Adding FDE2", err == 0);
+  TEST (err == 0, "findfre-1%c: Adding FDE2", suffix);
 
   fde_cnt = sframe_encoder_get_num_fidx (encode);
-  TEST ("findfre-1: Test FDE count", fde_cnt == 2);
+  TEST (fde_cnt == 2, "findfre-1%c: Test FDE count", suffix);
 
   sframe_buf = sframe_encoder_write (encode, &sf_size, &err);
-  TEST ("findfre-1: Encoder write", err == 0);
+  TEST (err == 0, "findfre-1%c: Encoder write", suffix);
 
   dctx = sframe_decode (sframe_buf, sf_size, &err);
-  TEST ("findfre-1: Decoder setup", dctx != NULL);
+  TEST (dctx != NULL, "findfre-1%c: Decoder setup", suffix);
 
   /* Find the third FRE in first FDE.  */
   lookup_pc = func1_start_vaddr + 0x15 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find third FRE",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3),
+	"findfre-1%c: Find third FRE", suffix);
 
   /* Find an FRE for PC at the end of range covered by FRE.  */
   lookup_pc = func1_start_vaddr + 0x9 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find FRE for last PC covered by FRE",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2),
+	"findfre-1%c: Find FRE for last PC covered by FRE", suffix);
 
   /* Find the last FRE in first FDE.  */
   lookup_pc = func1_start_vaddr + 0x39 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find last FRE",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x8));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x8),
+	"findfre-1%c: Find last FRE", suffix);
 
   /* Find the second FRE in second FDE.  */
   lookup_pc = func2_start_vaddr + 0x11 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find second FRE",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x12));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x12),
+	"findfre-1%c: Find second FRE", suffix);
 
   /* Find the first FRE in second FDE.  */
   lookup_pc = func2_start_vaddr + 0x0 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find first FRE",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x10));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x10),
+	"findfre-1%c: Find first FRE", suffix);
 
   /* Find FRE for PC out of range.  Expect error code.  */
   lookup_pc = func1_start_vaddr + func1_size - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfre-1: Find FRE for out of range PC", err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR,
+	"findfre-1%c: Find FRE for out of range PC", suffix);
 
   sframe_encoder_free (&encode);
   sframe_decoder_free (&dctx);
@@ -183,11 +175,11 @@
   uint32_t text_vaddr = 0x401020;
   printf ("Testing with text_vaddr = %#x; sframe_vaddr = %#x\n", text_vaddr,
 	  sframe_vaddr);
-  test_text_findfre (text_vaddr, sframe_vaddr);
+  test_text_findfre ('a', text_vaddr, sframe_vaddr);
 
   sframe_vaddr = 0x401020;
   text_vaddr = 0x402220;
   printf ("Testing with text_vaddr = %#x; sframe_vaddr = %#x\n", text_vaddr,
 	  sframe_vaddr);
-  test_text_findfre (text_vaddr, sframe_vaddr);
+  test_text_findfre ('b', text_vaddr, sframe_vaddr);
 }
diff --git a/libsframe/testsuite/libsframe.find/findfunc-1.c b/libsframe/testsuite/libsframe.find/findfunc-1.c
index 1dcd870..14ff9c51 100644
--- a/libsframe/testsuite/libsframe.find/findfunc-1.c
+++ b/libsframe/testsuite/libsframe.find/findfunc-1.c
@@ -130,7 +130,8 @@
 }
 
 static
-void test_text_findfre (uint32_t text_vaddr, uint32_t sframe_vaddr)
+void test_text_findfre (const char suffix, uint32_t text_vaddr,
+			uint32_t sframe_vaddr)
 {
   sframe_encoder_ctx *encode;
   sframe_decoder_ctx *dctx;
@@ -147,16 +148,6 @@
   size_t sf_size;
   int err = 0;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   encode = sframe_encode (SFRAME_VERSION,
 			  SFRAME_F_FDE_FUNC_START_PCREL,
 			  SFRAME_ABI_AMD64_ENDIAN_LITTLE,
@@ -167,26 +158,26 @@
   /* Add FDE at index 0.  */
   func1_start_vaddr = text_vaddr;
   err = add_fde1 (encode, func1_start_vaddr, sframe_vaddr, 0, &func1_size);
-  TEST ("findfunc-1: Adding FDE1", err == 0);
+  TEST (err == 0, "findfunc-1%c: Adding FDE1", suffix);
 
   /* Add FDE at index 1.  */
   func2_start_vaddr = func1_start_vaddr + func1_size + 0x10;
   err = add_fde2 (encode, func2_start_vaddr, sframe_vaddr, 1, &func2_size);
-  TEST ("findfunc-1: Adding FDE2", err == 0);
+  TEST (err == 0, "findfunc-1%c: Adding FDE2", suffix);
 
   /* Add FDE at index 2.  */
   func3_start_vaddr = func2_start_vaddr + func2_size + 0x10;
   err = add_fde3 (encode, func3_start_vaddr, sframe_vaddr, 2, &func3_size);
-  TEST ("findfunc-1: Adding FDE3", err == 0);
+  TEST (err == 0, "findfunc-1%c: Adding FDE3", suffix);
 
   fde_cnt = sframe_encoder_get_num_fidx (encode);
-  TEST ("findfunc-1: Test FDE count", fde_cnt == 3);
+  TEST (fde_cnt == 3, "findfunc-1%c: Test FDE count", suffix);
 
   sframe_buf = sframe_encoder_write (encode, &sf_size, &err);
-  TEST ("findfunc-1: Encoder write", err == 0);
+  TEST (err == 0, "findfunc-1%c: Encoder write", suffix);
 
   dctx = sframe_decode (sframe_buf, sf_size, &err);
-  TEST ("findfunc-1: Decoder setup", dctx != NULL);
+  TEST (dctx != NULL, "findfunc-1%c: Decoder setup", suffix);
 
   /* Following negative tests check that libsframe APIs
      (sframe_get_funcdesc_with_addr, sframe_find_fre) work
@@ -195,46 +186,46 @@
   /* Search with PC less than the first FDE's start addr.  */
   lookup_pc = func1_start_vaddr - 0x15 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: test-1: Find FRE for PC not in range",
-	err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR,
+	"findfunc-1%c: test-1: Find FRE for PC not in range", suffix);
 
   /* Search with a PC between func1's last PC and func2's first PC.  */
   lookup_pc = func1_start_vaddr + func1_size + 0x1 - sframe_vaddr,
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: test-2: Find FRE for PC not in range",
-	err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR,
+	"findfunc-1%c: test-2: Find FRE for PC not in range", suffix);
 
   /* Search for a PC between func2's last PC and func3's first PC.  */
   lookup_pc = func2_start_vaddr + func2_size + 0x3 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: test-3: Find FRE for PC not in range",
-	err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR,
+	"findfunc-1%c: test-3: Find FRE for PC not in range", suffix);
 
   /* Search for a PC beyond the last func, i.e., > func3's last PC.  */
   lookup_pc = func3_start_vaddr + func3_size + 0x10 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: test-4: Find FRE for PC not in range",
-	err == SFRAME_ERR);
+  TEST (err == SFRAME_ERR,
+	"findfunc-1%c: test-4: Find FRE for PC not in range", suffix);
 
   /* And some positive tests... */
 
   /* Find an FRE for PC in FDE1.  */
   lookup_pc = func1_start_vaddr + 0x9 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: Find FRE in FDE1",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2),
+	"findfunc-1%c: Find FRE in FDE1", suffix);
 
   /* Find an FRE for PC in FDE2.  */
   lookup_pc = func2_start_vaddr + 0x11 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: Find FRE in FDE2",
-       (err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x12));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x12),
+	"findfunc-1%c: Find FRE in FDE2", suffix);
 
   /* Find an FRE for PC in FDE3.  */
   lookup_pc = func3_start_vaddr + 0x10 - sframe_vaddr;
   err = sframe_find_fre (dctx, lookup_pc, &frep);
-  TEST ("findfunc-1: Find FRE in FDE3",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x18));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x18),
+	"findfunc-1%c: Find FRE in FDE3", suffix);
 
   sframe_encoder_free (&encode);
   sframe_decoder_free (&dctx);
@@ -246,11 +237,11 @@
   uint32_t text_vaddr = 0x4038b0;
   printf ("Testing with text_vaddr = %#x; sframe_vaddr = %#x\n", text_vaddr,
 	  sframe_vaddr);
-  test_text_findfre (text_vaddr, sframe_vaddr);
+  test_text_findfre ('a', text_vaddr, sframe_vaddr);
 
   sframe_vaddr = 0x4038b0;
   text_vaddr = 0x4b5620;
   printf ("Testing with text_vaddr = %#x; sframe_vaddr = %#x\n", text_vaddr,
 	  sframe_vaddr);
-  test_text_findfre (text_vaddr, sframe_vaddr);
+  test_text_findfre ('b', text_vaddr, sframe_vaddr);
 }
diff --git a/libsframe/testsuite/libsframe.find/plt-findfre-1.c b/libsframe/testsuite/libsframe.find/plt-findfre-1.c
index b41588a..f62ca3f 100644
--- a/libsframe/testsuite/libsframe.find/plt-findfre-1.c
+++ b/libsframe/testsuite/libsframe.find/plt-findfre-1.c
@@ -54,7 +54,8 @@
 }
 
 static
-void test_plt_findfre (uint32_t plt_vaddr, uint32_t sframe_vaddr)
+void test_plt_findfre (const char suffix, uint32_t plt_vaddr,
+		       uint32_t sframe_vaddr)
 {
   sframe_encoder_ctx *ectx;
   sframe_decoder_ctx *dctx;
@@ -64,16 +65,6 @@
   int err = 0;
   uint32_t fde_cnt = 0;
 
-#define TEST(name, cond)                                                      \
-  do                                                                          \
-    {                                                                         \
-      if (cond)                                                               \
-	pass (name);                                                          \
-      else                                                                    \
-	fail (name);                                                          \
-    }                                                                         \
-    while (0)
-
   ectx = sframe_encode (SFRAME_VERSION, SFRAME_F_FDE_FUNC_START_PCREL,
 			SFRAME_ABI_AMD64_ENDIAN_LITTLE,
 			SFRAME_CFA_FIXED_FP_INVALID,
@@ -81,50 +72,50 @@
 			&err);
 
   err = add_plt_fde1 (ectx, plt_vaddr, sframe_vaddr, 0);
-  TEST ("plt-findfre-1: Adding FDE1 for plt", err == 0);
+  TEST (err == 0, "plt-findfre-1%c: Adding FDE1 for plt", suffix);
 
   fde_cnt = sframe_encoder_get_num_fidx (ectx);
-  TEST ("plt-findfre-1: Test FDE count", fde_cnt == 1);
+  TEST (fde_cnt == 1, "plt-findfre-1%c: Test FDE count", suffix);
 
   sframe_buf = sframe_encoder_write (ectx, &sf_size, &err);
-  TEST ("plt-findfre-1: Encoder write", err == 0);
+  TEST (err == 0, "plt-findfre-1%c: Encoder write", suffix);
 
   dctx = sframe_decode (sframe_buf, sf_size, &err);
-  TEST ("plt-findfre-1: Decoder setup", dctx != NULL);
+  TEST (dctx != NULL, "plt-findfre-1%c: Decoder setup", suffix);
 
   /* Find the first FRE in PLT1.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 0x0 - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find first FRE in PLT1",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x1));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x1),
+       "plt-findfre-1%c: Find first FRE in PLT1", suffix);
 
   /* Find the second FRE.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 0x6 - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find second FRE in PLT1",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2),
+	"plt-findfre-1%c: Find second FRE in PLT1", suffix);
 
   /* Find the last FRE.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 0xc - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find last FRE in PLT1",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3),
+	"plt-findfre-1%c: Find last FRE in PLT1", suffix);
 
   /* Find the first FRE in PLT4.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 16*3 + 0x0 - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find first FRE in PLT4",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x1));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x1),
+	"plt-findfre-1%c: Find first FRE in PLT4", suffix);
 
   /* Find the second FRE in PLT4.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 16*3 + 0x6 - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find second FRE in PLT4",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x2),
+	"plt-findfre-1%c: Find second FRE in PLT4", suffix);
 
   /* Find the last FRE in PLT4.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 16*3 + 0xc - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find last FRE in PLT4",
-	(err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3));
+  TEST ((err == 0 && sframe_fre_get_cfa_offset (dctx, &frep, &err) == 0x3),
+	"plt-findfre-1%c: Find last FRE in PLT4", suffix);
 
   /* Find no FRE for out of range PLT6.  */
   err = sframe_find_fre (dctx, (plt_vaddr + 16*5 + 0x0 - sframe_vaddr), &frep);
-  TEST ("plt-findfre-1: Find no FRE for out of range PLT6", err != 0);
+  TEST (err != 0, "plt-findfre-1%c: Find no FRE for out of range PLT6", suffix);
 
   sframe_encoder_free (&ectx);
   sframe_decoder_free (&dctx);
@@ -136,11 +127,11 @@
   uint32_t plt_vaddr = 0x401020;
   printf ("Testing with plt_vaddr = %#x; sframe_vaddr = %#x\n", plt_vaddr,
 	  sframe_vaddr);
-  test_plt_findfre (plt_vaddr, sframe_vaddr);
+  test_plt_findfre ('a', plt_vaddr, sframe_vaddr);
 
   sframe_vaddr = 0x401020;
   plt_vaddr = 0x402220;
   printf ("Testing with plt_vaddr = %#x; sframe_vaddr = %#x\n", plt_vaddr,
 	  sframe_vaddr);
-  test_plt_findfre (plt_vaddr, sframe_vaddr);
+  test_plt_findfre ('b', plt_vaddr, sframe_vaddr);
 }
diff --git a/libsframe/testsuite/libsframe.find/plt-findfre-2.c b/libsframe/testsuite/libsframe.find/plt-findfre-2.c
index 61f04f0..097d23b 100644
--- a/libsframe/testsuite/libsframe.find/plt-findfre-2.c
+++ b/libsframe/testsuite/libsframe.find/plt-findfre-2.c
@@ -105,16 +105,6 @@
   unsigned int fde_cnt = 0;
   int i;
 
-#define TEST(cond, ...)							\
-  do									\
-    {									\
-      if (cond)								\
-	pass (__VA_ARGS__);						\
-      else								\
-	fail (__VA_ARGS__);						\
-    }									\
-    while (0)
-
   ectx = sframe_encode (SFRAME_VERSION, SFRAME_F_FDE_FUNC_START_PCREL,
 			SFRAME_ABI_S390X_ENDIAN_BIG,
 			SFRAME_CFA_FIXED_FP_INVALID,
diff --git a/libsframe/testsuite/sframe-test.h b/libsframe/testsuite/sframe-test.h
index d5db96b..d8a8d37 100644
--- a/libsframe/testsuite/sframe-test.h
+++ b/libsframe/testsuite/sframe-test.h
@@ -40,4 +40,14 @@
 #include <dejagnu.h>
 #undef wait
 
+#define TEST(cond, subname, ...)					\
+  do									\
+    {									\
+      if (cond)								\
+	pass (subname, ##__VA_ARGS__);					\
+      else								\
+	fail (subname, ##__VA_ARGS__);					\
+    }									\
+    while (0)
+
 #endif /* _SFRAME_TEST_H */