From 644f27cd86baa82c22ec233e9442316a000bc0d5 Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Thu, 19 Sep 2024 14:35:18 -0700 Subject: [PATCH] Remove redundant code in test_sdl2_gl_frames_swap. NFC The post_build function here exists solely in order to patch out (remove) the auto running on doRefTest. However the test is already built with `manual_reference=True` which already does this. --- test/test_browser.py | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/test/test_browser.py b/test/test_browser.py index 186bad2d23580..fbb2b3e4d59bb 100644 --- a/test/test_browser.py +++ b/test/test_browser.py @@ -3063,13 +3063,7 @@ def test_sdl2_canvas_write(self): @requires_graphics_hardware def test_sdl2_gl_frames_swap(self): - def post_build(): - self.post_manual_reftest() - reftest = read_file('reftest.js') - reftest2 = reftest.replace("Module['postRun'] = doReftest;", '') # we don't want the very first frame - assert reftest != reftest2 - create_file('reftest.js', reftest2) - self.reftest('test_sdl2_gl_frames_swap.c', 'test_sdl2_gl_frames_swap.png', args=['--proxy-to-worker', '-sGL_TESTING', '-sUSE_SDL=2'], manual_reference=True, post_build=post_build) + self.reftest('test_sdl2_gl_frames_swap.c', 'test_sdl2_gl_frames_swap.png', args=['--proxy-to-worker', '-sGL_TESTING', '-sUSE_SDL=2'], manual_reference=True, post_build=self.post_manual_reftest) @requires_graphics_hardware def test_sdl2_ttf(self):