Skip to content

CppUMock inputParameterForwarding#785

Open
KisImre wants to merge 10 commits into
cpputest:masterfrom
KisImre:master
Open

CppUMock inputParameterForwarding#785
KisImre wants to merge 10 commits into
cpputest:masterfrom
KisImre:master

Conversation

@KisImre

@KisImre KisImre commented Oct 15, 2015

Copy link
Copy Markdown
Contributor

I finished the feature which I mentioned in the issue #781.
It can copy data from the code under test into the test case.

@KisImre

KisImre commented May 9, 2016

Copy link
Copy Markdown
Contributor Author

It's been a long time since I've posted this request. Let me know if there's any problem with it.

@basvodde

basvodde commented May 9, 2016

Copy link
Copy Markdown
Member

Mostly didn't had time to look it at carefully. Sorry.

@KisImre

KisImre commented Jul 28, 2016

Copy link
Copy Markdown
Contributor Author

Ping.

@basvodde

Copy link
Copy Markdown
Member

Pong!

@coveralls

coveralls commented Mar 8, 2018

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.001%) to 99.876% when pulling da17156 on KisImre:master into 952bf2d on cpputest:master.

@KisImre

KisImre commented Mar 8, 2018

Copy link
Copy Markdown
Contributor Author

Hi, it's been a while since I created this pull request. I've just finished merging all the changes that happened in this repo since then, so you may be able to merge this PR. Please tell me if you have any requests or questions in connection with this PR.
I don't know if the purpose of this feature is clear or not, but it comes very handy when you want to inspect the contents of a struct created by the code under test. Basically this feature provides the functionality of the withOutputParameterReturning but in the opposite direction (code under test -> test case).
It would be great if this pull request could make it into CppUTest.

@KisImre

KisImre commented Nov 4, 2019

Copy link
Copy Markdown
Contributor Author

Could you please check if this pull request can be merged?

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