teabag 0.3.5 → 0.3.6

data/spec/javascripts/teabag/base/reporters/html/base_view_spec.coffee

@@ -0,0 +1,88 @@
+describe "Teabag.Reporters.BaseView", ->
+
+  beforeEach ->
+    @base = new Teabag.Reporters.BaseView()
+
+  describe "constructor", ->
+
+    it "calls build", ->
+      spy = spyOn(Teabag.Reporters.BaseView.prototype, "build")
+      new Teabag.Reporters.BaseView()
+      expect(spy).toHaveBeenCalled()
+
+
+  describe "#build", ->
+
+    it "builds an element", ->
+      @base.build("foo")
+      expect(@base.el.className).toBe("foo")
+
+
+  describe "#appendTo", ->
+
+    it "calls appendChild on the passed element", ->
+      el = {appendChild: ->}
+      spy = spyOn(el, "appendChild")
+      @base.appendTo(el)
+      expect(spy).toHaveBeenCalledWith(@base.el)
+
+
+  describe "#append", ->
+
+    it "calls appendChild on our element", ->
+      @base.el = {appendChild: ->}
+      spy = spyOn(@base.el, "appendChild")
+      otherEl = {}
+      @base.append(otherEl)
+      expect(spy).toHaveBeenCalledWith(otherEl)
+
+
+  describe "#createEl", ->
+
+    it "creates an element with a className", ->
+      el = @base.createEl("em", "foo")
+      expect(el.tagName).toBe("EM")
+      expect(el.className).toBe("foo")
+
+
+  describe "#findEl", ->
+
+    it "finds an element and momoizes it", ->
+      @base.findEl("controls")
+      expect(@base.elements["controls"]).toBeDefined()
+
+
+  describe "#setText", ->
+
+    it "finds an el and sets it's innerText", ->
+      el = {innerHTML: "bar"}
+      spy = spyOn(@base, "findEl").andReturn(el)
+      @base.setText("foo-id", "foo")
+      expect(spy).toHaveBeenCalledWith("foo-id")
+      expect(el.innerHTML).toBe("foo")
+
+
+  describe "#setHtml", ->
+
+    it "finds an el and sets it's innerHTML", ->
+      el = {innerHTML: "bar"}
+      spy = spyOn(@base, "findEl").andReturn(el)
+      @base.setHtml("foo-id", "foo")
+      expect(spy).toHaveBeenCalledWith("foo-id")
+      expect(el.innerHTML).toBe("foo")
+
+
+  describe "#setClass", ->
+
+    it "finds an el and sets a class on it", ->
+      el = {className: "bar"}
+      spy = spyOn(@base, "findEl").andReturn(el)
+      @base.setClass("foo-id", "foo")
+      expect(spy).toHaveBeenCalledWith("foo-id")
+      expect(el.className).toBe("foo")
+
+
+  describe "#htmlSafe", ->
+
+    it "makes a string html safe", ->
+      expect(@base.htmlSafe("<div></div>")).toEqual("&lt;div&gt;&lt;/div&gt;")

data/spec/javascripts/teabag/base/reporters/html_spec.coffee

@@ -34,309 +34,309 @@ describe "Teabag.Reporters.HTML", ->
       expect(@readConfigSpy).toHaveBeenCalled()
 
 
-    describe "#build", ->
+  describe "#build", ->
 
-      beforeEach ->
-        @el = {}
-        @buildSpy.andCallThrough()
-        @buildLayoutSpy = spyOn(@reporter, "buildLayout")
-        @findElSpy = spyOn(@reporter, "findEl").andReturn(@el)
-        @setTextSpy = spyOn(@reporter, "setText")
-        @showConfigurationSpy = spyOn(@reporter, "showConfiguration")
-        @buildProgressSpy = spyOn(@reporter, "buildProgress")
-        spyOn(@reporter, "envInfo").andReturn("library 1.0.0")
-        @reporter.build()
+    beforeEach ->
+      @el = {}
+      @buildSpy.andCallThrough()
+      @buildLayoutSpy = spyOn(@reporter, "buildLayout")
+      @findElSpy = spyOn(@reporter, "findEl").andReturn(@el)
+      @setTextSpy = spyOn(@reporter, "setText")
+      @showConfigurationSpy = spyOn(@reporter, "showConfiguration")
+      @buildProgressSpy = spyOn(@reporter, "buildProgress")
+      spyOn(@reporter, "envInfo").andReturn("library 1.0.0")
+      @reporter.build()
 
-      it "calls buildLayout", ->
-        expect(@buildLayoutSpy).toHaveBeenCalled()
+    it "calls buildLayout", ->
+      expect(@buildLayoutSpy).toHaveBeenCalled()
 
-      it "finds the element and assigns it", ->
-        expect(@findElSpy).toHaveBeenCalledWith("report-all")
-        expect(@reporter.el).toBe(@el)
+    it "finds the element and assigns it", ->
+      expect(@findElSpy).toHaveBeenCalledWith("report-all")
+      expect(@reporter.el).toBe(@el)
 
-      it "sets the env-info and version", ->
-        expect(@setTextSpy).toHaveBeenCalledWith("env-info", "library 1.0.0")
-        expect(@setTextSpy).toHaveBeenCalledWith("version", Teabag.version)
+    it "sets the env-info and version", ->
+      expect(@setTextSpy).toHaveBeenCalledWith("env-info", "library 1.0.0")
+      expect(@setTextSpy).toHaveBeenCalledWith("version", Teabag.version)
 
-      it "makes the toggles clickable", ->
-        expect(@findElSpy).toHaveBeenCalledWith("toggles")
-        expect(@el.onclick).toBe(@reporter.toggleConfig)
+    it "makes the toggles clickable", ->
+      expect(@findElSpy).toHaveBeenCalledWith("toggles")
+      expect(@el.onclick).toBe(@reporter.toggleConfig)
 
-      it "calls showConfiguration", ->
-        expect(@showConfigurationSpy).toHaveBeenCalled()
+    it "calls showConfiguration", ->
+      expect(@showConfigurationSpy).toHaveBeenCalled()
 
-      it "calls buildProgress", ->
-        expect(@buildProgressSpy).toHaveBeenCalled()
+    it "calls buildProgress", ->
+      expect(@buildProgressSpy).toHaveBeenCalled()
 
 
-    describe "#buildLayout", ->
+  describe "#buildLayout", ->
 
-      beforeEach ->
-        @el = {}
-        @createElSpy = spyOn(@reporter, "createEl").andReturn(@el)
-        @appendChildSpy = spyOn(document.body, "appendChild")
-        @reporter.buildLayout()
+    beforeEach ->
+      @el = {}
+      @createElSpy = spyOn(@reporter, "createEl").andReturn(@el)
+      @appendChildSpy = spyOn(document.body, "appendChild")
+      @reporter.buildLayout()
 
-      it "creates an element and appends it to the body", ->
-        expect(@createElSpy).toHaveBeenCalledWith("div")
-        expect(@appendChildSpy).toHaveBeenCalledWith(@el)
-        expect(@el.innerHTML).toContain("Teabag")
+    it "creates an element and appends it to the body", ->
+      expect(@createElSpy).toHaveBeenCalledWith("div")
+      expect(@appendChildSpy).toHaveBeenCalledWith(@el)
+      expect(@el.innerHTML).toContain("Teabag")
 
 
-    describe "#buildProgress", ->
+  describe "#buildProgress", ->
 
-      beforeEach ->
-        @progress = {appendTo: ->}
-        @findElSpy = spyOn(@reporter, "findEl").andReturn("_element_")
-        @createSpy = spyOn(Teabag.Reporters.HTML.ProgressView, "create").andReturn(@progress)
-        @appendToSpy = spyOn(@progress, "appendTo")
-        @reporter.buildProgress()
+    beforeEach ->
+      @progress = {appendTo: ->}
+      @findElSpy = spyOn(@reporter, "findEl").andReturn("_element_")
+      @createSpy = spyOn(Teabag.Reporters.HTML.ProgressView, "create").andReturn(@progress)
+      @appendToSpy = spyOn(@progress, "appendTo")
+      @reporter.buildProgress()
 
-      it "calls create on ProgressView", ->
-        expect(@createSpy).toHaveBeenCalledWith(true)
+    it "calls create on ProgressView", ->
+      expect(@createSpy).toHaveBeenCalledWith(true)
 
-      it "appends the progress element to the dom", ->
-        expect(@appendToSpy).toHaveBeenCalledWith("_element_")
+    it "appends the progress element to the dom", ->
+      expect(@appendToSpy).toHaveBeenCalledWith("_element_")
 
 
-    describe "#reportRunnerStarting", ->
+  describe "#reportRunnerStarting", ->
 
-      beforeEach ->
-        @setTextSpy = spyOn(@reporter, "setText")
-        @reporter.reportRunnerStarting(total: 42)
+    beforeEach ->
+      @setTextSpy = spyOn(@reporter, "setText")
+      @reporter.reportRunnerStarting(total: 42)
 
-      it "gets the totals", ->
-        expect(@reporter.total.exist).toBe(42)
+    it "gets the totals", ->
+      expect(@reporter.total.exist).toBe(42)
 
-      it "sets the duration text", ->
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-duration", "...")
+    it "sets the duration text", ->
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-duration", "...")
 
 
-    describe "#reportSpecStarting", ->
+  describe "#reportSpecStarting", ->
 
-      it "creates a SpecView", ->
-        @reporter.config["build-full-report"] = true
-        spy = spyOn(Teabag.Reporters.HTML, "SpecView")
-        @reporter.reportSpecStarting(@jasmineSpec)
-        expect(spy).toHaveBeenCalled()
+    it "creates a SpecView", ->
+      @reporter.config["build-full-report"] = true
+      spy = spyOn(Teabag.Reporters.HTML, "SpecView")
+      @reporter.reportSpecStarting(@jasmineSpec)
+      expect(spy).toHaveBeenCalled()
 
-      it "doesn't create the SpecView if we're not building the full report", ->
-        @reporter.config["build-full-report"] = false
-        spy = spyOn(Teabag.Reporters.HTML, "SpecView")
-        @reporter.reportSpecStarting(@jasmineSpec)
-        expect(spy).wasNotCalled()
+    it "doesn't create the SpecView if we're not building the full report", ->
+      @reporter.config["build-full-report"] = false
+      spy = spyOn(Teabag.Reporters.HTML, "SpecView")
+      @reporter.reportSpecStarting(@jasmineSpec)
+      expect(spy).wasNotCalled()
 
-      it "tracks the start time of the spec", ->
-        @reporter.specStart = undefined
-        @reporter.reportSpecStarting(@jasmineSpec)
-        expect(@reporter.specStart).toBeDefined()
+    it "tracks the start time of the spec", ->
+      @reporter.specStart = undefined
+      @reporter.reportSpecStarting(@jasmineSpec)
+      expect(@reporter.specStart).toBeDefined()
 
 
-    describe "#reportSpecResults", ->
+  describe "#reportSpecResults", ->
 
-      beforeEach ->
-        @updateProgressSpy = spyOn(@reporter, "updateProgress")
-        @updateStatusSpy = spyOn(@reporter, "updateStatus")
+    beforeEach ->
+      @updateProgressSpy = spyOn(@reporter, "updateProgress")
+      @updateStatusSpy = spyOn(@reporter, "updateStatus")
 
-      it "increases the total run count", ->
-        @reporter.total.run = 41
-        @reporter.reportSpecResults(@jasmineSpec)
-        expect(@reporter.total.run).toBe(42)
+    it "increases the total run count", ->
+      @reporter.total.run = 41
+      @reporter.reportSpecResults(@jasmineSpec)
+      expect(@reporter.total.run).toBe(42)
 
-      it "calls updateProgress", ->
-        @reporter.reportSpecResults(@jasmineSpec)
-        expect(@updateProgressSpy).toHaveBeenCalled()
+    it "calls updateProgress", ->
+      @reporter.reportSpecResults(@jasmineSpec)
+      expect(@updateProgressSpy).toHaveBeenCalled()
 
-      it "calls updateStatus", ->
-        @reporter.reportSpecResults(@jasmineSpec)
-        expect(@updateStatusSpy).toHaveBeenCalledWith(@jasmineSpec)
+    it "calls updateStatus", ->
+      @reporter.reportSpecResults(@jasmineSpec)
+      expect(@updateStatusSpy).toHaveBeenCalledWith(@jasmineSpec)
 
 
-    describe "#reportRunnerResults", ->
+  describe "#reportRunnerResults", ->
 
-      beforeEach ->
-        @setTextSpy = spyOn(@reporter, "setText")
-        @setStatusSpy = spyOn(@reporter, "setStatus")
-        @updateProgressSpy = spyOn(@reporter, "updateProgress")
-        @elapsedTimeSpy = spyOn(@reporter, "elapsedTime").andReturn("1.000s")
-        @reporter.total = {run: 666, exist: 42, failures: 5, passes: 10, skipped: 15}
+    beforeEach ->
+      @setTextSpy = spyOn(@reporter, "setText")
+      @setStatusSpy = spyOn(@reporter, "setStatus")
+      @updateProgressSpy = spyOn(@reporter, "updateProgress")
+      @elapsedTimeSpy = spyOn(@reporter, "elapsedTime").andReturn("1.000s")
+      @reporter.total = {run: 666, exist: 42, failures: 5, passes: 10, skipped: 15}
 
-      it "does nothing if there were no tests run", ->
-        @reporter.total.run = 0
-        @reporter.reportRunnerResults()
-        expect(@setTextSpy).wasNotCalled()
+    it "does nothing if there were no tests run", ->
+      @reporter.total.run = 0
+      @reporter.reportRunnerResults()
+      expect(@setTextSpy).wasNotCalled()
 
-      it "sets the duration text", ->
-        @reporter.total = {run: 666, exist: 42}
-        @reporter.reportRunnerResults()
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-duration", "1.000s")
+    it "sets the duration text", ->
+      @reporter.total = {run: 666, exist: 42}
+      @reporter.reportRunnerResults()
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-duration", "1.000s")
 
-      it "sets the status to passed if there are no failures", ->
-        @reporter.total.failures = 0
-        @reporter.reportRunnerResults()
-        expect(@setStatusSpy).toHaveBeenCalledWith("passed")
+    it "sets the status to passed if there are no failures", ->
+      @reporter.total.failures = 0
+      @reporter.reportRunnerResults()
+      expect(@setStatusSpy).toHaveBeenCalledWith("passed")
 
-      it "displays the total passes", ->
-        @reporter.reportRunnerResults()
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-passes", 10)
+    it "displays the total passes", ->
+      @reporter.reportRunnerResults()
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-passes", 10)
 
-      it "displays the total failures", ->
-        @reporter.reportRunnerResults()
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-failures", 5)
+    it "displays the total failures", ->
+      @reporter.reportRunnerResults()
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-failures", 5)
 
-      it "displays the total skipped", ->
-        @reporter.reportRunnerResults()
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-skipped", 15)
+    it "displays the total skipped", ->
+      @reporter.reportRunnerResults()
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-skipped", 15)
 
-      it "calls updateProgress", ->
-        @reporter.reportRunnerResults()
-        expect(@updateProgressSpy).toHaveBeenCalled()
+    it "calls updateProgress", ->
+      @reporter.reportRunnerResults()
+      expect(@updateProgressSpy).toHaveBeenCalled()
 
 
-    describe "#updateStat", ->
+  describe "#updateStat", ->
 
-      beforeEach ->
-        @setTextSpy = spyOn(@reporter, "setText")
+    beforeEach ->
+      @setTextSpy = spyOn(@reporter, "setText")
 
-      it "does nothing if we're not displaying progress", ->
-        @reporter.config["display-progress"] = false
-        @reporter.updateStat("name", 42)
-        expect(@setTextSpy).wasNotCalled()
+    it "does nothing if we're not displaying progress", ->
+      @reporter.config["display-progress"] = false
+      @reporter.updateStat("name", 42)
+      expect(@setTextSpy).wasNotCalled()
 
 
-      it "sets the text of the stat we want to set", ->
-        @reporter.updateStat("name", 42)
-        expect(@setTextSpy).toHaveBeenCalledWith("stats-name", 42)
+    it "sets the text of the stat we want to set", ->
+      @reporter.updateStat("name", 42)
+      expect(@setTextSpy).toHaveBeenCalledWith("stats-name", 42)
 
 
-    describe "#updateStatus", ->
+  describe "#updateStatus", ->
 
-      beforeEach ->
-        @updateStatSpy = spyOn(@reporter, "updateStat")
-        @setStatusSpy = spyOn(@reporter, "setStatus")
-        @findElSpy = spyOn(@reporter, "findEl").andReturn(appendChild: ->)
+    beforeEach ->
+      @updateStatSpy = spyOn(@reporter, "updateStat")
+      @setStatusSpy = spyOn(@reporter, "setStatus")
+      @findElSpy = spyOn(@reporter, "findEl").andReturn(appendChild: ->)
 
-      describe "skipped", ->
+    describe "skipped", ->
 
-        it "updates the statistic", ->
-          @jasmineSpecResults.skipped = true
-          @reporter.updateStatus(@jasmineSpec)
-          expect(@updateStatSpy).toHaveBeenCalledWith("skipped", 1)
+      it "updates the statistic", ->
+        @jasmineSpecResults.skipped = true
+        @reporter.updateStatus(@jasmineSpec)
+        expect(@updateStatSpy).toHaveBeenCalledWith("skipped", 1)
 
-      describe "pass", ->
+    describe "pass", ->
 
-        it "updates the statistic", ->
-          @reporter.updateStatus(@jasmineSpec)
-          expect(@updateStatSpy).toHaveBeenCalledWith("passes", 1)
+      it "updates the statistic", ->
+        @reporter.updateStatus(@jasmineSpec)
+        expect(@updateStatSpy).toHaveBeenCalledWith("passes", 1)
 
-        it "calls updateState on the view", ->
-          @reporter.reportView = updateState: ->
-          spy = spyOn(@reporter.reportView, "updateState")
-          @reporter.updateStatus(@jasmineSpec)
-          expect(spy.argsForCall[0][0]).toBe("passed")
+      it "calls updateState on the view", ->
+        @reporter.reportView = updateState: ->
+        spy = spyOn(@reporter.reportView, "updateState")
+        @reporter.updateStatus(@jasmineSpec)
+        expect(spy.argsForCall[0][0]).toBe("passed")
 
-      describe "failure", ->
+    describe "failure", ->
 
-        beforeEach ->
-          @jasmineSpecResults.passed = -> false
+      beforeEach ->
+        @jasmineSpecResults.passed = -> false
 
-        it "updates the statistic", ->
-          @reporter.updateStatus(@jasmineSpec)
-          expect(@updateStatSpy).toHaveBeenCalledWith("failures", 1)
+      it "updates the statistic", ->
+        @reporter.updateStatus(@jasmineSpec)
+        expect(@updateStatSpy).toHaveBeenCalledWith("failures", 1)
 
-        it "calls updateState on the view", ->
-          @reporter.reportView = updateState: ->
-          spy = spyOn(@reporter.reportView, "updateState")
-          @reporter.updateStatus(@jasmineSpec)
-          expect(spy.argsForCall[0][0]).toBe("failed")
+      it "calls updateState on the view", ->
+        @reporter.reportView = updateState: ->
+        spy = spyOn(@reporter.reportView, "updateState")
+        @reporter.updateStatus(@jasmineSpec)
+        expect(spy.argsForCall[0][0]).toBe("failed")
 
-        it "creates a FailureView and appends it to the dom", ->
-          spy = spyOn(Teabag.Reporters.HTML, "FailureView").andReturn(appendTo: ->)
-          @reporter.updateStatus(@jasmineSpec)
-          expect(spy).toHaveBeenCalled()
+      it "creates a FailureView and appends it to the dom", ->
+        spy = spyOn(Teabag.Reporters.HTML, "FailureView").andReturn(appendTo: ->)
+        @reporter.updateStatus(@jasmineSpec)
+        expect(spy).toHaveBeenCalled()
 
-        it "doesn't create a FailureView if we're building the full report", ->
-          @reporter.config["build-full-report"] = true
-          spy = spyOn(Teabag.Reporters.HTML, "FailureView").andReturn(appendTo: ->)
-          @reporter.updateStatus(@jasmineSpec)
-          expect(spy).wasNotCalled()
+      it "doesn't create a FailureView if we're building the full report", ->
+        @reporter.config["build-full-report"] = true
+        spy = spyOn(Teabag.Reporters.HTML, "FailureView").andReturn(appendTo: ->)
+        @reporter.updateStatus(@jasmineSpec)
+        expect(spy).wasNotCalled()
 
-        it "sets the status", ->
-          @reporter.updateStatus(@jasmineSpec)
-          expect(@setStatusSpy).toHaveBeenCalledWith("failed")
+      it "sets the status", ->
+        @reporter.updateStatus(@jasmineSpec)
+        expect(@setStatusSpy).toHaveBeenCalledWith("failed")
 
 
-    describe "#updateProgress", ->
+  describe "#updateProgress", ->
 
-      beforeEach ->
-        @progress = {update: ->}
-        @updateSpy = spyOn(@progress, "update")
-        @reporter.progress = @progress
-        @reporter.total = {exist: 666, run: 42}
-        @reporter.updateProgress()
+    beforeEach ->
+      @progress = {update: ->}
+      @updateSpy = spyOn(@progress, "update")
+      @reporter.progress = @progress
+      @reporter.total = {exist: 666, run: 42}
+      @reporter.updateProgress()
 
-      it "calls update on the progress view", ->
-        expect(@updateSpy).toHaveBeenCalledWith(666, 42)
+    it "calls update on the progress view", ->
+      expect(@updateSpy).toHaveBeenCalledWith(666, 42)
 
 
-    describe "#showConfiguration", ->
+  describe "#showConfiguration", ->
 
-      beforeEach ->
-        @setClassSpy = spyOn(@reporter, "setClass")
+    beforeEach ->
+      @setClassSpy = spyOn(@reporter, "setClass")
 
-      it "sets the class to active on the toggle buttons for each configuration", ->
-        @reporter.showConfiguration()
-        expect(@setClassSpy).toHaveBeenCalledWith("use-catch", "active")
-        expect(@setClassSpy).toHaveBeenCalledWith("build-full-report", "")
-        expect(@setClassSpy).toHaveBeenCalledWith("display-progress", "active")
+    it "sets the class to active on the toggle buttons for each configuration", ->
+      @reporter.showConfiguration()
+      expect(@setClassSpy).toHaveBeenCalledWith("use-catch", "active")
+      expect(@setClassSpy).toHaveBeenCalledWith("build-full-report", "")
+      expect(@setClassSpy).toHaveBeenCalledWith("display-progress", "active")
 
 
-    describe "#setStatus", ->
+  describe "#setStatus", ->
 
-      it "sets the body class to the status passed in", ->
-        current = document.body.className
-        @reporter.setStatus("foo")
-        expect(document.body.className).toBe("teabag-foo")
-        document.body.className = current
+    it "sets the body class to the status passed in", ->
+      current = document.body.className
+      @reporter.setStatus("foo")
+      expect(document.body.className).toBe("teabag-foo")
+      document.body.className = current
 
 
-    describe "#setFilter", ->
+  describe "#setFilter", ->
 
-      beforeEach ->
-        @setClassSpy = spyOn(@reporter, "setClass")
-        @setHtmlSpy = spyOn(@reporter, "setHtml")
-        @reporter.setFilter("_filter_")
+    beforeEach ->
+      @setClassSpy = spyOn(@reporter, "setClass")
+      @setHtmlSpy = spyOn(@reporter, "setHtml")
+      @reporter.setFilter("_filter_")
 
-      it "sets a class and the html for the filter display", ->
-        expect(@setClassSpy).toHaveBeenCalledWith("filtered", "teabag-filtered")
-        expect(@setHtmlSpy).toHaveBeenCalledWith("filtered", "_filter_", true)
+    it "sets a class and the html for the filter display", ->
+      expect(@setClassSpy).toHaveBeenCalledWith("filtered", "teabag-filtered")
+      expect(@setHtmlSpy).toHaveBeenCalledWith("filtered", "_filter_", true)
 
 
-    describe "#readConfig", ->
+  describe "#readConfig", ->
 
-      beforeEach ->
-        @readConfigSpy.andCallThrough()
-        @config = {}
-        @cookieSpy = spyOn(@reporter, "cookie").andReturn(@config)
-        @reporter.readConfig()
+    beforeEach ->
+      @readConfigSpy.andCallThrough()
+      @config = {}
+      @cookieSpy = spyOn(@reporter, "cookie").andReturn(@config)
+      @reporter.readConfig()
 
-      it "reads the configuration from the cookie", ->
-        expect(@cookieSpy).toHaveBeenCalledWith("teabag")
-        expect(@reporter.config).toEqual(@config)
+    it "reads the configuration from the cookie", ->
+      expect(@cookieSpy).toHaveBeenCalledWith("teabag")
+      expect(@reporter.config).toEqual(@config)
 
 
-    describe "#toggleConfig", ->
+  describe "#toggleConfig", ->
 
-      beforeEach ->
-        @refreshSpy = spyOn(@reporter, "refresh")
-        @cookieSpy = spyOn(@reporter, "cookie")
-        @reporter.toggleConfig(target: {tagName: "button", getAttribute: -> "teabag-use-catch"})
+    beforeEach ->
+      @refreshSpy = spyOn(@reporter, "refresh")
+      @cookieSpy = spyOn(@reporter, "cookie")
+      @reporter.toggleConfig(target: {tagName: "button", getAttribute: -> "teabag-use-catch"})
 
-      it "toggles the configuration", ->
-        expect(@reporter.config["use-catch"]).toBe(false)
+    it "toggles the configuration", ->
+      expect(@reporter.config["use-catch"]).toBe(false)
 
-      it "sets the cookie", ->
-        expect(@cookieSpy).toHaveBeenCalledWith("teabag", @reporter.config)
+    it "sets the cookie", ->
+      expect(@cookieSpy).toHaveBeenCalledWith("teabag", @reporter.config)
 
-      it "refreshes the page", ->
-        expect(@refreshSpy).toHaveBeenCalled()
+    it "refreshes the page", ->
+      expect(@refreshSpy).toHaveBeenCalled()