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

Two new options for meck #77

Merged
merged 2 commits into from
Oct 7, 2012
Merged

Conversation

norton
Copy link
Contributor

@norton norton commented Oct 6, 2012

Two commits to help when using meck for performance-like testing.

  • spawn_opt option for meck's gen server
  • no_history option to disable storing of meck's call history

@@ -142,6 +142,12 @@ new(Mod) when is_list(Mod) -> lists:foreach(fun new/1, Mod), ok.
%% This option allows you to disable that
%% feature if it causes problems.
%% </dd>
%% <dt>`{spawn_opt,list()}'</dt><dd>Specify spawn options. Typically used to
%% specify non-default, garbage collection
%% options.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any way to make it more clear that it is Erlang process spawn options, and not Meck specific spawn options?

@eproxus
Copy link
Owner

eproxus commented Oct 6, 2012

Nice stuff! I'll merge after fixes.

@norton
Copy link
Contributor Author

norton commented Oct 6, 2012

Well .... how about?

"Specify spawn options." => "Specify Erlang process spawn options."

@eproxus
Copy link
Owner

eproxus commented Oct 6, 2012

Sounds good. :-)

Typically used to specify non-default, garbage collection options.
@norton
Copy link
Contributor Author

norton commented Oct 7, 2012

I pushed those 2 updates.

@eproxus
Copy link
Owner

eproxus commented Oct 7, 2012

Thanks!

eproxus added a commit that referenced this pull request Oct 7, 2012
@eproxus eproxus merged commit 2d8a995 into eproxus:develop Oct 7, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants