[10/15] tests/fbo-storage-formats: print subtest result for skip too

Submitted by Dylan Baker on Jan. 23, 2018, 1:22 a.m.

Details

Message ID 9a37a26683a0da14016f99c43c5c0778a14f8f6c.1516670408.git-series.dylan@pnwbakers.com
State New
Headers show
Series "Series without cover letter" ( rev: 13 ) in Piglit

Not browsing as part of any series.

Commit Message

Dylan Baker Jan. 23, 2018, 1:22 a.m.
Because we always want to have the same number of subtests printed, or
they'll show up as "NOTRUN" instead of "SKIP" in the summary.

Signed-off-by: Dylan Baker <dylan.c.baker@intel.com>
---
 tests/fbo/fbo-storage-formats.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/tests/fbo/fbo-storage-formats.c b/tests/fbo/fbo-storage-formats.c
index 9cf8c67..7f13684 100644
--- a/tests/fbo/fbo-storage-formats.c
+++ b/tests/fbo/fbo-storage-formats.c
@@ -190,8 +190,10 @@  test(void)
 	for (i = 0; i < ARRAY_SIZE(formats); i++) {
 		const char *name = piglit_get_gl_enum_name(formats[i].format);
 
-		if (!have_extension[formats[i].extension])
+		if (!have_extension[formats[i].extension]) {
+			piglit_report_subtest_result(PIGLIT_SKIP, "%s", name);
 			continue;
+		}
 
 		glRenderbufferStorageEXT(GL_RENDERBUFFER_EXT, formats[i].format,
 					 piglit_width, piglit_height);