about summary refs log tree commit diff
path: root/argp
diff options
context:
space:
mode:
authorGabriel F. T. Gomes <gabriel@inconstante.eti.br>2018-11-06 17:44:13 -0200
committerGabriel F. T. Gomes <gabriel@inconstante.eti.br>2018-11-08 22:42:01 -0200
commit43257c335ad6b9e05fb882975e1776ff763164ee (patch)
tree0d73bd4922bd0536f72178310c14505449910fb9 /argp
parente5de3b5b72afffdebb7f4642015060a5c437d546 (diff)
downloadglibc-43257c335ad6b9e05fb882975e1776ff763164ee.tar.gz
glibc-43257c335ad6b9e05fb882975e1776ff763164ee.tar.xz
glibc-43257c335ad6b9e05fb882975e1776ff763164ee.zip
Use TEST_COMPARE_STRING in recently added test
The commit

commit 1df872fd74f730bcae3df201a229195445d2e18a
Author: Florian Weimer <fweimer@redhat.com>
Date:   Wed Nov 7 12:42:44 2018 +0100

    support: Implement TEST_COMPARE_STRING

added the new macro TEST_COMPARE_STRING, which compares the output of
functions under test against expected strings, and, when there's a
mismatch, automatically reports an error and prints the differences.
This patch adapts recently added test cases to use this new macro.

Tested for powerpc64le (as is, and locally patched to intentionally fail
and produce error output).

	* argp/tst-ldbl-argp.c (do_one_test): Use TEST_COMPARE_STRING,
	instead of manually comparing and reporting mismatching strings.
	* misc/tst-ldbl-error.c (do_one_test): Likewise.
	* misc/tst-ldbl-warn.c (do_one_test): Likewise.
Diffstat (limited to 'argp')
-rw-r--r--argp/tst-ldbl-argp.c9
1 files changed, 1 insertions, 8 deletions
diff --git a/argp/tst-ldbl-argp.c b/argp/tst-ldbl-argp.c
index 4465f812af..231abeb18c 100644
--- a/argp/tst-ldbl-argp.c
+++ b/argp/tst-ldbl-argp.c
@@ -73,14 +73,7 @@ do_one_test (const char *expected)
   struct support_capture_subprocess result;
   result = support_capture_subprocess ((void *) &do_test_call, NULL);
 
-  if (strcmp (result.err.buffer, expected) != 0)
-    {
-      support_record_failure ();
-      printf ("error:\n"
-	      "    expected: '%s'\n"
-	      "      actual: '%s'\n",
-	      expected, result.err.buffer);
-    }
+  TEST_COMPARE_STRING (result.err.buffer, expected);
 
   return 0;
 }