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

Added moduleForModel helper with test #9

Merged
merged 1 commit into from
Mar 12, 2014

Conversation

coderberry
Copy link

No description provided.

@@ -5,7 +5,7 @@ import Ember from 'ember';
export default function moduleForComponent(name, description, callbacks) {
var resolver = testResolver.get();

moduleFor('component:' + name, description, callbacks, function(container, context) {
moduleFor('component:' + name, description, callbacks, function(container, context, defaultSubject) {
Copy link
Contributor

Choose a reason for hiding this comment

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

moduleForComponent doesn't use defaultSubject, do we need this here?

Copy link
Author

Choose a reason for hiding this comment

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

No. I only added it because it was available. I can remove it if it makes more sense.

@stefanpenner
Copy link
Member

Relative is good though

@abuiles
Copy link
Member

abuiles commented Mar 11, 2014

@stefanpenner cool, we are almost good to go then 👍

@ryanflorence
Copy link
Contributor

man, I need to add a note to not include the dist/ in pull requests ...

ryanflorence added a commit that referenced this pull request Mar 12, 2014
Added moduleForModel helper with test
@ryanflorence ryanflorence merged commit f0ed18f into emberjs:master Mar 12, 2014
@coderberry coderberry deleted the module-for-model branch March 12, 2014 16:26
@ryanflorence
Copy link
Contributor

v0.1.5 is published now with these changes

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.

4 participants