Merged Tag v0.3.0 (from xcpretty/xcpretty) and branch fix/duration_of_failed_tests_workaroundinto into repository #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Support Aggregate Targets
The Sonar JUnit parser fails if the time attribute is not present on failed tests. This patch adds the time attribute for these tests with a fixed duration of "0". Adding the real test duration was considered however it cannot be done easily with the current xcpretty implementation: xcpretty parses Xcode output line by line whereas a failed test outputs at least 2 lines of log: one that shows the failed test and the cause, another that displays the duration (quite the same line as a passing test). A considered approach would be to keep a context and output a transformed log line only if the context is full (in the case of failed tests, only if the cause and the duration have been parsed).
Formatter.finish: give formatters the ability to know the end
Custom Reporters
README: Fix the path of the json compilation database
Projects may decide to use their own compiler toolchain for building, in which case the path to the clang executable will likely not contain the /usr prefix. This commit thus removes that prefix from the corresponding matcher.
…ze mode (xcpretty#182) Was producing cruft of bad output for our console logs on our build machines.
…cpretty#224) * Fixed issue where the wrong syntax formatter was being used for HTML * Removed redundant self
Minor changes to the matcher regex's
turns out we need it for gem release...
Add support for custom compiler toolchains
Add matcher for 'No profile matching 'TargetName' found:' dependency error
* Parse screenshot name to better match against suite name * fix potential crash on empty array * remove whitespace * Fix whitespace again * Fix all whitespace ever
* Parse screenshot name to better match against suite name * fix potential crash on empty array * remove whitespace * Fix whitespace again * Fix all whitespace ever * Improve screenshot matching against Test Suites with namespaces
* Improve Code sign error checking * Optimize parser by reducing the number of separate regular expressions * Add Swift errors in Check dependencies phase
* Display screenshots next to tests * Tidy report output * Fix report output hiding tests * Fix failing test * Add support for screenshots containing suite name * Fix failing test - had to shorten filename of screenshot * Fix rubocop issue
* Add a failing feature showing the issue * Add a matcher to mark test as failed when xcodebuild restarts testing.
* Add catch-all formatter * Fix linter warning * Add test for format_other
ObjectAlchemist
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should fix Idean/sonar-swift#159