nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix comment stream jasmine - bind function after spy has been defined

Browse source
This commit is contained in:
Florian Staudacher 2012-03-14 18:43:06 +01:00
parent b3868fc6dd
commit cd965e32fa
2 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
public/javascripts/app/views/comment_stream_view.js
View file

@ -14,7 +14,11 @@ app.views.CommentStream = app.views.Base.extend({
this.model.comments.bind('add', this.appendComment, this); this.model.comments.bind('add', this.appendComment, this);
this.commentTemplate = options.commentTemplate; this.commentTemplate = options.commentTemplate;
this.model.bind("commentsExpanded", this.render, this) this.setupBindings();
},
setupBindings: function() {
this.model.bind("commentsExpanded", this.render, this);
}, },
postRenderTemplate : function() { postRenderTemplate : function() {

7
spec/javascripts/app/views/comment_stream_view_spec.js
View file

@ -6,9 +6,10 @@ describe("app.views.CommentStream", function(){
describe("binds", function() { describe("binds", function() {
it("re-renders on a commentsExpanded trigger", function(){ it("re-renders on a commentsExpanded trigger", function(){
spyOn(this.view, "render") spyOn(this.view, "render");
this.view.model.trigger("commentsExpanded") this.view.setupBindings();
expect(this.view.render).toHaveBeenCalled() this.view.model.trigger("commentsExpanded");
expect(this.view.render).toHaveBeenCalled();
}) })
}) })