Fixed an issue with an incorrect test summary when a testcase name includes a colon // Resolve #4508

This commit is contained in:
Ivan Kravets
2023-01-14 17:38:26 +02:00
parent 42f1197de8
commit de4ba4cbe1
2 changed files with 4 additions and 2 deletions

View File

@ -22,6 +22,7 @@ PlatformIO Core 6
* Updated `Clang-Tidy <https://docs.platformio.org/en/latest/plus/check-tools/clang-tidy.html>`__ check tool to v15.0.5 with new diagnostics and bugfixes
* Removed dependency on the "zeroconf" package and install it only when a user lists mDNS devices (issue with zeroconf's LGPL license)
* Show the real error message instead of "Can not remove temporary directory" when |PIOCONF| is broken (`issue #4480 <https://github.com/platformio/platformio-core/issues/4480>`_)
* Fixed an issue with an incorrect test summary when a testcase name includes a colon (`issue #4508 <https://github.com/platformio/platformio-core/issues/4508>`_)
6.1.5 (2022-11-01)
~~~~~~~~~~~~~~~~~~

View File

@ -29,10 +29,11 @@ class UnityTestRunner(TestRunnerBase):
EXTRA_LIB_DEPS = ["throwtheswitch/Unity@^2.5.2"]
# Example:
# Examples:
# test/test_foo.cpp:44:test_function_foo:FAIL: Expected 32 Was 33
# test/group/test_foo/test_main.cpp:5:test::dummy:FAIL: Expression Evaluated To FALSE
TESTCASE_PARSE_RE = re.compile(
r"(?P<source_file>[^:]+):(?P<source_line>\d+):(?P<name>[^:]+):"
r"(?P<source_file>[^:]+):(?P<source_line>\d+):(?P<name>[^\s]+):"
r"(?P<status>PASS|IGNORE|FAIL)(:\s*(?P<message>.+)$)?"
)