diff options
author | M Joonas Pihlaja <jpihlaja@cc.helsinki.fi> | 2009-09-03 17:54:49 +0300 |
---|---|---|
committer | M Joonas Pihlaja <jpihlaja@cc.helsinki.fi> | 2009-09-03 19:27:23 +0300 |
commit | 12d0613210547b8a50dd7b21a12eb1485ee496b9 (patch) | |
tree | 87d722379b1d8b411a89421869067210f819a4d3 | |
parent | a76c36f2edd16d8e2b8f49e6409134aceab38a42 (diff) |
[test] Allow tests to XFAIL by putting the cairo_t into an error state.
The test runner was extra strict about never letting a test put
the cairo_t into an error state, and never would it check for
the expectedness status of the failure. This patch moves the
check for a test being an XFAIL above the check on the cairo_t's
final status.
-rw-r--r-- | test/cairo-test.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/test/cairo-test.c b/test/cairo-test.c index 7e93410f..55e28dc7 100644 --- a/test/cairo-test.c +++ b/test/cairo-test.c @@ -1005,13 +1005,6 @@ REPEAT: goto UNWIND_CAIRO; } - if (cairo_status (cr) != CAIRO_STATUS_SUCCESS) { - cairo_test_log (ctx, "Error: Function under test left cairo status in an error state: %s\n", - cairo_status_to_string (cairo_status (cr))); - ret = CAIRO_TEST_FAILURE; - goto UNWIND_CAIRO; - } - #if HAVE_MEMFAULT if (MEMFAULT_COUNT_FAULTS () - last_fault_count > 0 && MEMFAULT_HAS_FAULTS ()) @@ -1380,6 +1373,13 @@ REPEAT: cairo_surface_destroy (diff_image); } + if (cairo_status (cr) != CAIRO_STATUS_SUCCESS) { + cairo_test_log (ctx, "Error: Function under test left cairo status in an error state: %s\n", + cairo_status_to_string (cairo_status (cr))); + ret = CAIRO_TEST_FAILURE; + goto UNWIND_CAIRO; + } + UNWIND_CAIRO: if (test_filename != NULL) { free (test_filename); |