Skip to content

Commit

Permalink
Fixed an issue with an incorrect test summary when a testcase name in…
Browse files Browse the repository at this point in the history
…cludes a colon // Resolve #4508
  • Loading branch information
ivankravets committed Jan 14, 2023
1 parent 42f1197 commit de4ba4c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions HISTORY.rst
Original file line number Diff line number Diff line change
Expand Up @@ -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)
~~~~~~~~~~~~~~~~~~
Expand Down
5 changes: 3 additions & 2 deletions platformio/test/runners/unity.py
Original file line number Diff line number Diff line change
Expand Up @@ -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>.+)$)?"
)

Expand Down

0 comments on commit de4ba4c

Please sign in to comment.