FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

print list of all failed test at the end by angelskieglazki · Pull Request #1245 · cpputest/cpputest · GitHub

print list of all failed test at the end - #1245

Open
angelskieglazki wants to merge 13 commits into
cpputest:masterfrom
angelskieglazki:master
Open

print list of all failed test at the end#1245
angelskieglazki wants to merge 13 commits into
cpputest:masterfrom
angelskieglazki:master

Conversation

Copy link
Copy Markdown

TestResult.h:
TestResult.cpp:  print list of all failed test at the end.

basvodde commented Oct 5, 2019

Copy link
Copy Markdown
Member

I don't think we want this by default.

Also, I cannot merge it as it is as it is using the SdtC++ list in the code, which in CppUTest code shouldn't be used.

Copy link
Copy Markdown
Author

I don't think we want this by default.

I understand. Can you advise how best to do how to make this function optional?

Also, I cannot merge it as it is as it is using the SdtC++ list in the code, which in CppUTest code shouldn't be used.

I can write list and called them SimpleList.h/cpp.

Copy link
Copy Markdown
Member

I'm hesitant to merge this as I'm not sure it is worth the extra code. Have used CppUTest on lots of large products and never needed a summary feature....

Would love to hear whether others disagree/agree.

Copy link
Copy Markdown
Contributor

I'd personally rather have CppUTest support something like TAP or another format that could be parsed by other common tools.

Copy link
Copy Markdown
Member

@jacob-keller Expecting PR soon :P

Copy link
Copy Markdown
Contributor

@jacob-keller Expecting PR soon :P

Hah, we'll see when I can get some time at $DAYJOB for this

This file contains hidden or 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL