Recognize Ceedling 1.1.0 bullet point test listing report - #11
Conversation
Accept test lines in bullet point format, produced starting in Ceedling version 1.1.0.
| } else { | ||
| return result.stdout.split('\n').filter((value: string) => { | ||
| return value.startsWith(" - "); | ||
| return value.startsWith(" - ") || value.startsWith(" • "); |
There was a problem hiding this comment.
@mkarlesky VS Code extension became useless due to this careless change made 2 months ago.
ThrowTheSwitch/Ceedling@22372a3#diff-ac6a2a9e0178323fec1eced6fbdd08c509565f16a66de3a469342b92bf8a361bL90-R101
if files_list.size > 0
- files_list.sort.each { |filepath| puts " - #{filepath}" }
- puts "File count: #{files_list.size}"
- puts "Note: This list sourced only from your project file, not from any build directive macros in test files."
+ files_list.sort.each { |filepath| loginator.console(" • #{filepath}") }
+ loginator.console("File count: #{files_list.size}")
+ loginator.console("\nThis list sourced only from your project file, not from any build directive macros in test files.\n\n", LogLabels::NOTICE)
endThis line here should've been updated together when changing logging formats, or at least should've made them backward compatible.
|
@mrtumnus @jake-kim-sndk Thank you for these reports and very sorry for the trouble. A quick workaround would be, if possible, to simply disable rich logging with the environment variable The VS Code extension is very definitely brittle and out of date. A sizable update for it is in the works. I am sorry it is not working well for you. First priority is Ceedling itself. Focus has been on supporting 1.1.0 and the forthcoming 1.2.0 that adds the most frequently requested features. The VS Code extension was started elsewhere by a very generous community member and then handed off. We will be resuming work on it very soon. Please understand that the VS Code extension as is today is rather simplistic, has no self-tests of any kind, and is presently incapable of supporting much of what Ceedling does. It is in need of a great deal of work. We will get back to it very soon. Thank you for the PRs, etc. |
Accept test lines in bullet point format, produced starting in Ceedling version 1.1.0.
Fixes #10