From 0629f4fd9c4c30d7abe362bf75ac91271885bdc8 Mon Sep 17 00:00:00 2001 From: Florian Staudacher Date: Fri, 30 Mar 2012 16:40:08 +0200 Subject: [PATCH] added tests for resetting height on close --- spec/javascripts/app/views/publisher_view_spec.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/spec/javascripts/app/views/publisher_view_spec.js b/spec/javascripts/app/views/publisher_view_spec.js index 885001ec1..2fbc5908d 100644 --- a/spec/javascripts/app/views/publisher_view_spec.js +++ b/spec/javascripts/app/views/publisher_view_spec.js @@ -16,13 +16,19 @@ describe("app.views.Publisher", function() { }); describe("#close", function() { - it("removes the 'active' class from the publisher element", function(){ - $(this.view.el).removeClass("closed"); + beforeEach(function() { + this.view.open($.Event()); + }); - expect($(this.view.el)).not.toHaveClass("closed"); + it("removes the 'active' class from the publisher element", function(){ this.view.close($.Event()); expect($(this.view.el)).toHaveClass("closed"); }) + + it("resets the element's height", function() { + this.view.close($.Event()); + expect($(this.view.el).find("#status_message_fake_text").attr("style")).not.toContain("height"); + }); }); describe("#clear", function() {