Skip to content

Commit 9fb31f7

Browse files
committed
[test] Pass filename explicitly to compile_btest helper. NFC
This allows us to use `compiler_for` to determine whether to run `EMCC` or `EMXX` appropriately.
1 parent 946c850 commit 9fb31f7

File tree

3 files changed

+63
-64
lines changed

3 files changed

+63
-64
lines changed

test/common.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2106,7 +2106,7 @@ def reftest(self, expected, manually_trigger=False):
21062106
setupRefTest();
21072107
''' % (reporting, basename, int(manually_trigger)))
21082108

2109-
def compile_btest(self, args, reporting=Reporting.FULL):
2109+
def compile_btest(self, filename, args, reporting=Reporting.FULL):
21102110
# Inject support code for reporting results. This adds an include a header so testcases can
21112111
# use REPORT_RESULT, and also adds a cpp file to be compiled alongside the testcase, which
21122112
# contains the implementation of REPORT_RESULT (we can't just include that implementation in
@@ -2120,10 +2120,13 @@ def compile_btest(self, args, reporting=Reporting.FULL):
21202120
# also compile in report_result.c and forice-include report_result.h
21212121
args += ['-I' + TEST_ROOT,
21222122
'-include', test_file('report_result.h'),
2123+
'-Wno-deprecated',
21232124
test_file('report_result.c')]
21242125
if EMTEST_BROWSER == 'node':
21252126
args.append('-DEMTEST_NODE')
2126-
self.run_process([EMCC] + self.get_emcc_args() + args)
2127+
if not os.path.exists(filename):
2128+
filename = test_file(filename)
2129+
self.run_process([compiler_for(filename), filename] + self.get_emcc_args() + args)
21272130

21282131
def btest_exit(self, filename, assert_returncode=0, *args, **kwargs):
21292132
"""Special case of btest that reports its result solely via exiting
@@ -2166,10 +2169,10 @@ def btest(self, filename, expected=None, reference=None,
21662169
# manual_reference only makes sense for reference tests
21672170
assert manual_reference is None
21682171
outfile = output_basename + '.html'
2169-
args += [filename, '-o', outfile]
2172+
args += ['-o', outfile]
21702173
# print('all args:', args)
21712174
utils.delete_file(outfile)
2172-
self.compile_btest(args, reporting=reporting)
2175+
self.compile_btest(filename, args, reporting=reporting)
21732176
self.assertExists(outfile)
21742177
if post_build:
21752178
post_build()

0 commit comments

Comments
 (0)