erikdoe / ocmock

Mock objects for Objective-C
http://ocmock.org
Apache License 2.0
2.16k stars 606 forks source link

Error when ignoreNonObjectArgs is unnecessary. #414

Open dmaclach opened 4 years ago

dmaclach commented 4 years ago

This encourages proper use of the APIS and keeps the testing code cleaner and easier to read.

I would like to probably actually implement this as a warning that could be turned into an error (see #413)

erikdoe commented 4 years ago

Agree on warning by default. Making this an error will cause a lot of unexpected problems for existing test suites.

dmaclach commented 4 years ago

I will happily refactor this on top of #424 if that gets in.