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

Implicit new #80

Closed
eproxus opened this issue Oct 23, 2012 · 3 comments
Closed

Implicit new #80

eproxus opened this issue Oct 23, 2012 · 3 comments

Comments

@eproxus
Copy link
Owner

eproxus commented Oct 23, 2012

This means get rid of the {not_mocked, Module} error and just run new if there is no Meck process already.

@horkhe
Copy link
Contributor

horkhe commented Oct 23, 2012

Certainly doable, but don't we trying to be too smart with that? As a user I would prefer my unit test to fail in this case. Has a need for that came from your personal experience or somebody asked for that?

@eproxus
Copy link
Owner Author

eproxus commented Oct 23, 2012

It's just something that I thought about, that you would save tons of meck:new(mod) lines for test cases that don't need mocks with options.

@edgurgel
Copy link
Contributor

@eproxus, that would mean:

meck:expect(module, ...).

I would not need to write meck:new if I used expect on a module?

Cool

eproxus added a commit that referenced this issue Apr 24, 2013
@eproxus eproxus closed this as completed Mar 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants