forked from espressif/esp-idf
Merge branch 'fix/idf_ci_script' into 'master'
fix(ci): Move error handling to correct scope See merge request espressif/esp-idf!25777
This commit is contained in:
@@ -64,12 +64,12 @@ def get_pytest_cases(
|
|||||||
cmd.extend(['-k', filter_expr])
|
cmd.extend(['-k', filter_expr])
|
||||||
res = pytest.main(cmd, plugins=[collector])
|
res = pytest.main(cmd, plugins=[collector])
|
||||||
|
|
||||||
if res.value != ExitCode.OK:
|
if res.value != ExitCode.OK:
|
||||||
if res.value == ExitCode.NO_TESTS_COLLECTED:
|
if res.value == ExitCode.NO_TESTS_COLLECTED:
|
||||||
print(f'WARNING: no pytest app found for target {target} under paths {", ".join(paths)}')
|
print(f'WARNING: no pytest app found for target {target} under paths {", ".join(paths)}')
|
||||||
else:
|
else:
|
||||||
print(buf.getvalue())
|
print(buf.getvalue())
|
||||||
raise RuntimeError(f'pytest collection failed at {", ".join(paths)} with command \"{" ".join(cmd)}\"')
|
raise RuntimeError(f'pytest collection failed at {", ".join(paths)} with command \"{" ".join(cmd)}\"')
|
||||||
|
|
||||||
cases.extend(collector.cases)
|
cases.extend(collector.cases)
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user