update from Error to FAILED
This commit is contained in:
committed by
Tamara Dahlgren
parent
f5cfcadfc5
commit
50640d4924
2
lib/spack/external/ctest_log_parser.py
vendored
2
lib/spack/external/ctest_log_parser.py
vendored
@@ -118,6 +118,8 @@ def match(self, text):
|
|||||||
"([^:]+): (Error:|error|undefined reference|multiply defined)",
|
"([^:]+): (Error:|error|undefined reference|multiply defined)",
|
||||||
"([^ :]+) ?: (error|fatal error|catastrophic error)",
|
"([^ :]+) ?: (error|fatal error|catastrophic error)",
|
||||||
"([^:]+)\\(([^\\)]+)\\) ?: (error|fatal error|catastrophic error)"),
|
"([^:]+)\\(([^\\)]+)\\) ?: (error|fatal error|catastrophic error)"),
|
||||||
|
"^FAILED",
|
||||||
|
"^==> \[\d\d\d\d-\d\d-\d\d-\d\d:\d\d:\d\d\.\d+, \d+\]\s*FAILED",
|
||||||
"^[Bb]us [Ee]rror",
|
"^[Bb]us [Ee]rror",
|
||||||
"^[Ss]egmentation [Vv]iolation",
|
"^[Ss]egmentation [Vv]iolation",
|
||||||
"^[Ss]egmentation [Ff]ault",
|
"^[Ss]egmentation [Ff]ault",
|
||||||
|
|||||||
@@ -1685,12 +1685,12 @@ def run_test(self, exe, options=[], expected=[], status=None,
|
|||||||
try:
|
try:
|
||||||
self._run_test_helper(
|
self._run_test_helper(
|
||||||
exe, options, expected, status, installed, purpose)
|
exe, options, expected, status, installed, purpose)
|
||||||
tty.msg("PASSED")
|
print("PASSED")
|
||||||
except BaseException as e:
|
except BaseException as e:
|
||||||
# print a summary of the error to the log file
|
# print a summary of the error to the log file
|
||||||
# so that cdash and junit reporters know about it
|
# so that cdash and junit reporters know about it
|
||||||
exc_type, _, tb = sys.exc_info()
|
exc_type, _, tb = sys.exc_info()
|
||||||
print('Error: %s' % e)
|
print('FAILED: %s' % e)
|
||||||
import traceback
|
import traceback
|
||||||
# remove the current call frame to get back to
|
# remove the current call frame to get back to
|
||||||
stack = traceback.extract_stack()[:-1]
|
stack = traceback.extract_stack()[:-1]
|
||||||
|
|||||||
@@ -69,7 +69,7 @@ def test_test_output(install_mockery, mock_archive, mock_fetch,
|
|||||||
assert "BEFORE TEST" in output
|
assert "BEFORE TEST" in output
|
||||||
assert "true: expect command status in [" in output
|
assert "true: expect command status in [" in output
|
||||||
assert "AFTER TEST" in output
|
assert "AFTER TEST" in output
|
||||||
assert "rror" not in output # no error
|
assert "FAILED" not in output
|
||||||
|
|
||||||
|
|
||||||
def test_test_output_on_error(mock_packages, mock_archive, mock_fetch,
|
def test_test_output_on_error(mock_packages, mock_archive, mock_fetch,
|
||||||
@@ -80,7 +80,7 @@ def test_test_output_on_error(mock_packages, mock_archive, mock_fetch,
|
|||||||
out = spack_test('run', 'test-error', fail_on_error=False)
|
out = spack_test('run', 'test-error', fail_on_error=False)
|
||||||
|
|
||||||
assert "TestFailure" in out
|
assert "TestFailure" in out
|
||||||
assert "Error: Command exited with status 1" in out
|
assert "FAILED: Command exited with status 1" in out
|
||||||
|
|
||||||
|
|
||||||
def test_test_output_on_failure(mock_packages, mock_archive, mock_fetch,
|
def test_test_output_on_failure(mock_packages, mock_archive, mock_fetch,
|
||||||
@@ -90,7 +90,6 @@ def test_test_output_on_failure(mock_packages, mock_archive, mock_fetch,
|
|||||||
out = spack_test('run', 'test-fail', fail_on_error=False)
|
out = spack_test('run', 'test-fail', fail_on_error=False)
|
||||||
|
|
||||||
assert "Expected 'not in the output' in output of `true`" in out
|
assert "Expected 'not in the output' in output of `true`" in out
|
||||||
assert "Error:" in out
|
|
||||||
assert "TestFailure" in out
|
assert "TestFailure" in out
|
||||||
|
|
||||||
|
|
||||||
@@ -109,8 +108,8 @@ def test_show_log_on_error(mock_packages, mock_archive, mock_fetch,
|
|||||||
'mock_packages', 'mock_archive', 'mock_fetch', 'install_mockery'
|
'mock_packages', 'mock_archive', 'mock_fetch', 'install_mockery'
|
||||||
)
|
)
|
||||||
@pytest.mark.parametrize('pkg_name,msgs', [
|
@pytest.mark.parametrize('pkg_name,msgs', [
|
||||||
('test-error', ['Error: Command exited', 'TestFailure']),
|
('test-error', ['FAILED: Command exited', 'TestFailure']),
|
||||||
('test-fail', ['Error: Expected', 'TestFailure'])
|
('test-fail', ['FAILED: Expected', 'TestFailure'])
|
||||||
])
|
])
|
||||||
def test_junit_output_with_failures(tmpdir, mock_test_stage, pkg_name, msgs):
|
def test_junit_output_with_failures(tmpdir, mock_test_stage, pkg_name, msgs):
|
||||||
install(pkg_name)
|
install(pkg_name)
|
||||||
@@ -151,7 +150,7 @@ def test_cdash_output_test_error(
|
|||||||
report_file = report_dir.join('test-error_Test.xml')
|
report_file = report_dir.join('test-error_Test.xml')
|
||||||
assert report_file in report_dir.listdir()
|
assert report_file in report_dir.listdir()
|
||||||
content = report_file.open().read()
|
content = report_file.open().read()
|
||||||
assert 'Error: Command exited with status 1' in content
|
assert 'FAILED: Command exited with status 1' in content
|
||||||
|
|
||||||
|
|
||||||
def test_cdash_upload_clean_test(
|
def test_cdash_upload_clean_test(
|
||||||
|
|||||||
Reference in New Issue
Block a user