Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CppUMock inputParameterForwarding #785

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

KisImre
Copy link
Contributor

@KisImre KisImre commented Oct 15, 2015

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
Copy link
Contributor Author

KisImre commented May 9, 2016

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

@basvodde
Copy link
Member

basvodde commented May 9, 2016

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

@KisImre
Copy link
Contributor Author

KisImre commented Jul 28, 2016

Ping.

@basvodde
Copy link
Member

Pong!

@coveralls
Copy link

coveralls commented Mar 8, 2018

Coverage Status

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

@KisImre
Copy link
Contributor Author

KisImre commented Mar 8, 2018

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
Copy link
Contributor Author

KisImre commented Nov 4, 2019

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.

None yet

3 participants