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

CppUMock inputParameterForwarding by KisImre · Pull Request #785 · cpputest/cpputest · GitHub

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 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 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 commented May 9, 2016

Copy link
Copy Markdown
Member

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

KisImre commented Jul 28, 2016

Copy link
Copy Markdown
Contributor Author

Ping.

Copy link
Copy Markdown
Member

Pong!

coveralls commented Mar 8, 2018
edited
Loading

Copy link
Copy Markdown

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

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 commented Nov 4, 2019

Copy link
Copy Markdown
Contributor Author

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

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