From 14e2ba8a7da047f6394120913ff668680af3871b Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Mon, 22 Jan 2018 17:22:10 -0800 Subject: tests/fbo-storage-formats: Make subtest names predictable Currently the name of the subtest relies on checking the GL state of the command to know whether the framebuffer is completely or incomplete. That is a problem for enumerating subtests, we would need to know ahead of time whether the framebuffer is complete or not. Instead print the completeness of the test apart from the subtest name, and name the subtest name just the format name. This also makes the name the same when the test fails and when in passes. Signed-off-by: Dylan Baker Reviewed-by: Fabian Bieler --- tests/fbo/fbo-storage-formats.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'tests') diff --git a/tests/fbo/fbo-storage-formats.c b/tests/fbo/fbo-storage-formats.c index 91b80e09d..9cf8c675a 100644 --- a/tests/fbo/fbo-storage-formats.c +++ b/tests/fbo/fbo-storage-formats.c @@ -200,11 +200,10 @@ test(void) pass = GL_FALSE; } else { GLenum status = glCheckFramebufferStatus(GL_FRAMEBUFFER); - piglit_report_subtest_result(PIGLIT_PASS, - "%s (%s)", - name, - (status == GL_FRAMEBUFFER_COMPLETE ? - "complete" : "incomplete")); + printf("%s is %s\n", + name, + (status == GL_FRAMEBUFFER_COMPLETE ? "complete" : "incomplete")); + piglit_report_subtest_result(PIGLIT_PASS, "%s", name); } } -- cgit v1.2.3