Skip to content

Commit

Permalink
Merge pull request #69 from CodeOfficer/punctual-punctuation
Browse files Browse the repository at this point in the history
punctual punctuation
  • Loading branch information
rwjblue committed Jul 1, 2015
2 parents eedfaf4 + 59634c5 commit fbee787
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion lib/ember-test-helpers/test-module.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default Klass.extend({
initNeeds: function() {
this.needs = [this.subjectName];
if (this.callbacks.needs) {
this.needs = this.needs.concat(this.callbacks.needs)
this.needs = this.needs.concat(this.callbacks.needs);
delete this.callbacks.needs;
}
},
Expand Down
4 changes: 2 additions & 2 deletions tests/test-module-for-component-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ var BoringColor = Ember.Component.extend({
var stateIndicatesInDOM = (this._state === 'inDOM');
var actuallyInDOM = Ember.$.contains(document, this.$()[0]);

ok((actuallyInDOM === true) && (actuallyInDOM === stateIndicatesInDOM), 'component should still be in the DOM')
ok((actuallyInDOM === true) && (actuallyInDOM === stateIndicatesInDOM), 'component should still be in the DOM');
}
});

Expand Down Expand Up @@ -199,7 +199,7 @@ moduleForComponent('boring-color', 'component:boring-color -- still in DOM in wi
});

test("className", function(){
expect(1)
expect(1);
// the assertion is in the willDestroyElement() hook of the component
});

Expand Down
4 changes: 2 additions & 2 deletions tests/test-module-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ moduleFor('component:x-foo', 'component:x-foo -- without needs or `integration:

test("knows nothing about our non-subject component", function() {
var otherComponent = this.container.lookup('component:not-the-subject');
equal(null, otherComponent, "We shouldn't know about a non-subject component")
equal(null, otherComponent, "We shouldn't know about a non-subject component");
});

moduleFor('component:x-foo', 'component:x-foo -- when needing another component', {
Expand All @@ -146,7 +146,7 @@ test("needs gets us the component we need", function() {

moduleFor('component:x-foo', 'component:x-foo -- `integration: true`', {
beforeSetup: function() {
setupRegistry()
setupRegistry();
ok(!this.callbacks.integration, "integration property should be removed from callbacks");
ok(this.isIntegration, "isIntegration should be set when we set `integration: true` in callbacks");
},
Expand Down

0 comments on commit fbee787

Please sign in to comment.