Skip to content

Commit b0579ab

Browse files
authored
btest: add assert whenl manual_reference argument is mistakenly set. NFC (#17458)
1 parent 20e3bf6 commit b0579ab

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

tests/common.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1692,7 +1692,7 @@ def btest_exit(self, filename, assert_returncode=0, *args, **kwargs):
16921692
return self.btest(filename, *args, **kwargs)
16931693

16941694
def btest(self, filename, expected=None, reference=None,
1695-
reference_slack=0, manual_reference=False, post_build=None,
1695+
reference_slack=0, manual_reference=None, post_build=None,
16961696
args=None, message='.', also_proxied=False,
16971697
url_suffix='', timeout=None, also_wasm2js=False,
16981698
manually_trigger_reftest=False, extra_tries=1,
@@ -1710,6 +1710,9 @@ def btest(self, filename, expected=None, reference=None,
17101710
self.reftest(test_file(reference), manually_trigger=manually_trigger_reftest)
17111711
if not manual_reference:
17121712
args += ['--pre-js', 'reftest.js', '-sGL_TESTING']
1713+
else:
1714+
# manual_reference only makes sense for reference tests
1715+
assert manual_reference is None
17131716
outfile = 'test.html'
17141717
args += [filename, '-o', outfile]
17151718
# print('all args:', args)

tests/test_browser.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -929,7 +929,7 @@ def post():
929929
</body>''')
930930
create_file('test.html', html)
931931

932-
self.btest('sdl_key_proxy.c', '223092870', args=['--proxy-to-worker', '--pre-js', 'pre.js', '-sEXPORTED_FUNCTIONS=_main,_one', '-lSDL', '-lGL'], manual_reference=True, post_build=post)
932+
self.btest('sdl_key_proxy.c', '223092870', args=['--proxy-to-worker', '--pre-js', 'pre.js', '-sEXPORTED_FUNCTIONS=_main,_one', '-lSDL', '-lGL'], post_build=post)
933933

934934
def test_canvas_focus(self):
935935
self.btest_exit('canvas_focus.c')
@@ -979,7 +979,7 @@ def post():
979979

980980
create_file('test.html', html)
981981

982-
self.btest('keydown_preventdefault_proxy.cpp', '300', args=['--proxy-to-worker', '-sEXPORTED_FUNCTIONS=_main'], manual_reference=True, post_build=post)
982+
self.btest('keydown_preventdefault_proxy.cpp', '300', args=['--proxy-to-worker', '-sEXPORTED_FUNCTIONS=_main'], post_build=post)
983983

984984
def test_sdl_text(self):
985985
create_file('pre.js', '''

0 commit comments

Comments
 (0)