aboutsummaryrefslogtreecommitdiff
path: root/mesonbuild/mtest.py
diff options
context:
space:
mode:
authorEli Schwartz <eschwartz@archlinux.org>2022-03-11 04:13:58 -0500
committerJussi Pakkanen <jpakkane@gmail.com>2022-03-14 01:48:16 +0200
commit4680c09aac3dc97436357ba6c8a7ac64132a1830 (patch)
tree17142f68ca4f800be99a914d1ff059f7dd556d5f /mesonbuild/mtest.py
parente54e9f58e5dce61573a96bdbef2bc3528566589e (diff)
downloadmeson-4680c09aac3dc97436357ba6c8a7ac64132a1830.zip
meson-4680c09aac3dc97436357ba6c8a7ac64132a1830.tar.gz
meson-4680c09aac3dc97436357ba6c8a7ac64132a1830.tar.bz2
mtest: fix logic bug that broke tests where the cmd is a Windows found program
In this case, the test fname might have an implicit extension and cannot be found by `os.path.isfile()`. We cannot use `shutil.which()` to handle platform differences, because not all test fnames are executable -- for example Java jars. The test representation does have an "is built" attribute which in theory should work here, because all built targets definitely have their full filename known to Meson, but it turns out to be misnamed. Rename it correctly and add an actual "is built" attribute to check. Tests which aren't built by Meson can be assumed to exist without consulting their existence on the filesystem. Fixes #10027
Diffstat (limited to 'mesonbuild/mtest.py')
-rw-r--r--mesonbuild/mtest.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/mesonbuild/mtest.py b/mesonbuild/mtest.py
index 02b0cba..b474cd4 100644
--- a/mesonbuild/mtest.py
+++ b/mesonbuild/mtest.py
@@ -1349,18 +1349,18 @@ class SingleTestRunner:
def _get_test_cmd(self) -> T.Optional[T.List[str]]:
testentry = self.test.fname[0]
- if self.options.no_rebuild and not os.path.isfile(testentry):
+ if self.options.no_rebuild and self.test.cmd_is_built and not os.path.isfile(testentry):
raise TestException(f'The test program {testentry!r} does not exist. Cannot run tests before building them.')
if testentry.endswith('.jar'):
return ['java', '-jar'] + self.test.fname
elif not self.test.is_cross_built and run_with_mono(testentry):
return ['mono'] + self.test.fname
- elif self.test.cmd_is_built and self.test.is_cross_built and self.test.needs_exe_wrapper:
+ elif self.test.cmd_is_exe and self.test.is_cross_built and self.test.needs_exe_wrapper:
if self.test.exe_wrapper is None:
# Can not run test on cross compiled executable
# because there is no execute wrapper.
return None
- elif self.test.cmd_is_built:
+ elif self.test.cmd_is_exe:
# If the command is not built (ie, its a python script),
# then we don't check for the exe-wrapper
if not self.test.exe_wrapper.found():