On 18 October 2016 at 18:33, Chad Versace <chadvers...@chromium.org> wrote: > Run with `bin/gl_basic_test --platform=surfaceless_egl`. > > All the tests fail! The rgba pixels returned by glReadPixels are > black (0x0, 0x0, 0x0, 0xff), but the test expects magenta (0xff, 0x0, > 0xff, 0xff). I don't know why. > > Cc: Gurchetan Singh <gurchetansi...@chromium.org> > Cc: Haixia Shi <h...@chromium.org> > --- > tests/functional/gl_basic_test.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/tests/functional/gl_basic_test.c > b/tests/functional/gl_basic_test.c > index 4a90dfd..c3c7774 100644 > --- a/tests/functional/gl_basic_test.c > +++ b/tests/functional/gl_basic_test.c > @@ -703,6 +703,17 @@ CREATE_TESTSUITE(WAFFLE_PLATFORM_GLX, glx) > > #endif // WAFFLE_HAS_GLX > > +#ifdef WAFFLE_HAS_SURFACELESS_EGL > + > +#define unit_test_make(name) \ > + cmocka_unit_test_setup_teardown(name, setup_surfaceless_egl, > gl_basic_fini) > + > +CREATE_TESTSUITE(WAFFLE_PLATFORM_SURFACELESS_EGL, surfaceless_egl) > + > +#undef unit_test_make > + Which reminds me that we might want to add support for testsuite(s) where each test, uses the same setup/teardown [to cmocka] and drop the above define/undef foo is related (rather nasty) macros.
Either way, with the small fixes in 4 and 5 the series is Reviewed-by: Emil Velikov <emil.l.veli...@gmail.com> Separating surfaceless/null and "platform FOO does not display anything" wss a great move, imho. Nicely done ! Emil _______________________________________________ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle