Skip to content

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

@angelskieglazki

Copy link
Copy Markdown

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

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.

@angelskieglazki

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.

@basvodde

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.

@jacob-keller

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.

@basvodde

Copy link
Copy Markdown
Member

@jacob-keller Expecting PR soon :P

@jacob-keller

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

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