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

Add basic descriptions for invocation actions. #436

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dmaclach
Copy link
Contributor

@dmaclach dmaclach commented Jun 6, 2020

Helps in debugging and error reporting.

@erikdoe erikdoe added this to the OCMock 3.7 milestone Jun 22, 2020
@erikdoe erikdoe removed this from the OCMock 3.7 milestone Jul 2, 2020
@erikdoe
Copy link
Owner

erikdoe commented Jul 2, 2020

I've taken this out of 3.7 because I think I need more thought on what the descriptions shoould look like. The mock objects just show their class, the quantifiers return a string that can be used in user (developer) messaging. Here we'd include the memory location...

@dmaclach dmaclach force-pushed the invocationActionDescriptions branch from 7c9c885 to 2fef3c6 Compare July 14, 2020 20:50
@dmaclach dmaclach force-pushed the invocationActionDescriptions branch from 2fef3c6 to b095a20 Compare August 1, 2020 00:11
@dmaclach dmaclach force-pushed the invocationActionDescriptions branch from b095a20 to 5e01c36 Compare August 13, 2020 15:49
Helps in debugging and error reporting.
@dmaclach dmaclach force-pushed the invocationActionDescriptions branch from 5e01c36 to d05e065 Compare January 2, 2021 18:31
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.

2 participants