support: Add FAIL test failure helper

Add a FAIL test failure helper analogous to FAIL_RET, that does not
cause the current function to return, providing a standardized way to
report a test failure with a message supplied while permitting the
caller to continue executing, for further reporting, cleaning up, etc.

Update existing test cases that provide a conflicting definition of FAIL
by removing the local FAIL definition and then as follows:

- tst-fortify-syslog: provide a meaningful message in addition to the
  file name already added by <support/check.h>; 'support_record_failure'
  is already called by 'support_print_failure_impl' invoked by the new
  FAIL test failure helper.

- tst-ctype: no update to FAIL calls required, with the name of the file
  and the line number within of the failure site additionally included
  by the new FAIL test failure helper, and error counting plus count
  reporting upon test program termination also already provided by
  'support_record_failure' and 'support_report_failure' respectively,
  called by 'support_print_failure_impl' and 'adjust_exit_status' also
  respectively.  However in a number of places 'printf' is called and
  the error count adjusted by hand, so update these places to make use
  of FAIL instead.  And last but not least adjust the final summary just
  to report completion, with any error count following as reported by
  the test driver.

- test-tgmath2: no update to FAIL calls required, with the name of the
  file of the failure site additionally included by the new FAIL test
  failure helper.  Also there is no need to track the return status by
  hand as any call to FAIL will eventually cause the test case to return
  an unsuccesful exit status regardless of the return status from the
  test function, via a call to 'adjust_exit_status' made by the test
  driver.

Reviewed-by: DJ Delorie <dj@redhat.com>
This commit is contained in:
Maciej W. Rozycki 2024-07-26 13:21:34 +01:00
parent fe47595504
commit 1b97a9f23b
4 changed files with 18 additions and 48 deletions

View File

@ -22,7 +22,6 @@
#include <syslog.h> #include <syslog.h>
#include <string.h> #include <string.h>
#include <unistd.h> #include <unistd.h>
#include <stdio.h>
#include <support/check.h> #include <support/check.h>
#include <support/support.h> #include <support/support.h>
@ -46,18 +45,13 @@ handler (int sig)
_exit (127); _exit (127);
} }
#define FAIL() \
do { \
printf ("Failure on line %d\n", __LINE__); \
support_record_failure (); \
} while (0)
#define CHK_FAIL_START \ #define CHK_FAIL_START \
chk_fail_ok = 1; \ chk_fail_ok = 1; \
if (! setjmp (chk_fail_buf)) \ if (! setjmp (chk_fail_buf)) \
{ {
#define CHK_FAIL_END \ #define CHK_FAIL_END \
chk_fail_ok = 0; \ chk_fail_ok = 0; \
FAIL (); \ FAIL ("not supposed to reach here"); \
} }
static void static void

View File

@ -21,6 +21,8 @@
#include <stdio.h> #include <stdio.h>
#include <string.h> #include <string.h>
#include <support/check.h>
static const char lower[] = "abcdefghijklmnopqrstuvwxyz"; static const char lower[] = "abcdefghijklmnopqrstuvwxyz";
static const char upper[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; static const char upper[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
@ -53,19 +55,11 @@ static struct classes
#define nclasses (sizeof (classes) / sizeof (classes[0])) #define nclasses (sizeof (classes) / sizeof (classes[0]))
#define FAIL(str, args...) \
{ \
printf (" " str "\n", ##args); \
++errors; \
}
static int static int
do_test (void) do_test (void)
{ {
const char *cp; const char *cp;
const char *cp2; const char *cp2;
int errors = 0;
char *inpline = NULL; char *inpline = NULL;
size_t inplinelen = 0; size_t inplinelen = 0;
char *resline = NULL; char *resline = NULL;
@ -394,11 +388,8 @@ punct = %04x alnum = %04x\n",
{ {
if (((__ctype_b[(unsigned int) *inp] & classes[n].mask) != 0) if (((__ctype_b[(unsigned int) *inp] & classes[n].mask) != 0)
!= (*resp != '0')) != (*resp != '0'))
{ FAIL (" is%s('%c' = '\\x%02x') %s true\n", inpline,
printf (" is%s('%c' = '\\x%02x') %s true\n", inpline,
*inp, *inp, *resp == '1' ? "not" : "is"); *inp, *inp, *resp == '1' ? "not" : "is");
++errors;
}
++inp; ++inp;
++resp; ++resp;
} }
@ -408,11 +399,8 @@ punct = %04x alnum = %04x\n",
while (*inp != '\0') while (*inp != '\0')
{ {
if (tolower (*inp) != *resp) if (tolower (*inp) != *resp)
{ FAIL (" tolower('%c' = '\\x%02x') != '%c'\n",
printf (" tolower('%c' = '\\x%02x') != '%c'\n",
*inp, *inp, *resp); *inp, *inp, *resp);
++errors;
}
++inp; ++inp;
++resp; ++resp;
} }
@ -422,11 +410,8 @@ punct = %04x alnum = %04x\n",
while (*inp != '\0') while (*inp != '\0')
{ {
if (toupper (*inp) != *resp) if (toupper (*inp) != *resp)
{ FAIL (" toupper('%c' = '\\x%02x') != '%c'\n",
printf (" toupper('%c' = '\\x%02x') != '%c'\n",
*inp, *inp, *resp); *inp, *inp, *resp);
++errors;
}
++inp; ++inp;
++resp; ++resp;
} }
@ -436,14 +421,7 @@ punct = %04x alnum = %04x\n",
} }
if (errors != 0) printf ("Completed testing for `%s' locale\n\n\n", setlocale (LC_ALL, NULL));
{
printf (" %d error%s for `%s' locale\n\n\n", errors,
errors == 1 ? "" : "s", setlocale (LC_ALL, NULL));
return 1;
}
printf (" No errors for `%s' locale\n\n\n", setlocale (LC_ALL, NULL));
return 0; return 0;
} }

View File

@ -24,6 +24,8 @@
#include <string.h> #include <string.h>
#include <tgmath.h> #include <tgmath.h>
#include <support/check.h>
//#define DEBUG //#define DEBUG
typedef complex float cfloat; typedef complex float cfloat;
@ -87,13 +89,6 @@ enum
int count; int count;
int counts[Tlast][C_last]; int counts[Tlast][C_last];
#define FAIL(str) \
do \
{ \
printf ("%s failure on line %d\n", (str), __LINE__); \
result = 1; \
} \
while (0)
#define TEST_TYPE_ONLY(expr, rettype) \ #define TEST_TYPE_ONLY(expr, rettype) \
do \ do \
{ \ { \
@ -133,8 +128,6 @@ int counts[Tlast][C_last];
int int
test_cos (const int Vint4, const long long int Vllong4) test_cos (const int Vint4, const long long int Vllong4)
{ {
int result = 0;
TEST (cos (vfloat1), float, cos); TEST (cos (vfloat1), float, cos);
TEST (cos (vdouble1), double, cos); TEST (cos (vdouble1), double, cos);
TEST (cos (vldouble1), ldouble, cos); TEST (cos (vldouble1), ldouble, cos);
@ -152,7 +145,7 @@ test_cos (const int Vint4, const long long int Vllong4)
TEST (cos (Vcdouble1), cdouble, cos); TEST (cos (Vcdouble1), cdouble, cos);
TEST (cos (Vcldouble1), cldouble, cos); TEST (cos (Vcldouble1), cldouble, cos);
return result; return 0;
} }
int int

View File

@ -24,6 +24,11 @@
__BEGIN_DECLS __BEGIN_DECLS
/* Record a test failure, print the failure message to standard output
and pass the result of 1 through. */
#define FAIL(...) \
support_print_failure_impl (__FILE__, __LINE__, __VA_ARGS__)
/* Record a test failure, print the failure message to standard output /* Record a test failure, print the failure message to standard output
and return 1. */ and return 1. */
#define FAIL_RET(...) \ #define FAIL_RET(...) \