From 4a1a2123a351f9214d24b058f1a79102ea44ce2e Mon Sep 17 00:00:00 2001
From: "Davis W. Frank" <dwfrank@infe.ws>
Date: Sat, 25 Feb 2012 11:58:54 -0800
Subject: [PATCH] Fixes https://github.com/pivotal/jasmine-gem/issues/65 and
 https://www.pivotaltracker.com/story/show/24902023

---
 spec/html/HTMLReporterSpec.js | 31 +++++++++++++++++++++++--------
 src/html/HtmlReporter.js      |  2 +-
 2 files changed, 24 insertions(+), 9 deletions(-)

diff --git a/spec/html/HTMLReporterSpec.js b/spec/html/HTMLReporterSpec.js
index 84ee3a96..3545527b 100644
--- a/spec/html/HTMLReporterSpec.js
+++ b/spec/html/HTMLReporterSpec.js
@@ -44,8 +44,23 @@ describe("HtmlReporter", function() {
     expect(htmlReporter.specFilter(fakeSpec("not run this"))).toBeFalsy();
   });
 
-  describe('Matcher reporting', function () {
-    var getResultMessageDiv = function (body) {
+  describe("running without any specs", function() {
+    var runner;
+    beforeEach(function() {
+      runner = env.currentRunner();
+      env.addReporter(htmlReporter);
+    });
+
+    it("should not error", function() {
+      var exec = function() {
+        runner.execute();
+      };
+      expect(exec).not.toThrow();
+    });
+  });
+
+  describe('Matcher reporting', function() {
+    var getResultMessageDiv = function(body) {
       var divs = body.getElementsByTagName("div");
       for (var i = 0; i < divs.length; i++) {
         if (divs[i].className.match(/resultMessage/)) {
@@ -55,7 +70,7 @@ describe("HtmlReporter", function() {
     };
 
     var runner, spec, fakeTimer;
-    beforeEach(function () {
+    beforeEach(function() {
       fakeTimer = new jasmine.FakeTimer();
       env.setTimeout = fakeTimer.setTimeout;
       env.clearTimeout = fakeTimer.clearTimeout;
@@ -70,9 +85,9 @@ describe("HtmlReporter", function() {
       env.addReporter(htmlReporter);
     });
 
-    describe('toContain', function () {
-      it('should show actual and expected', function () {
-        spec.runs(function () {
+    describe('toContain', function() {
+      it('should show actual and expected', function() {
+        spec.runs(function() {
           this.expect('foo').toContain('bar');
         });
         runner.execute();
@@ -85,7 +100,7 @@ describe("HtmlReporter", function() {
     });
   });
 
-  describe("failure messages (integration)", function () {
+  describe("failure messages (integration)", function() {
     var spec, results, expectationResult;
 
     it("should add the failure message to the DOM (non-toEquals matchers)", function() {
@@ -168,7 +183,7 @@ describe("HtmlReporter", function() {
   });
 
   describe('#reportSpecStarting', function() {
-    beforeEach(function () {
+    beforeEach(function() {
       env.describe("suite 1", function() {
         env.it("spec 1", function() {
         });
diff --git a/src/html/HtmlReporter.js b/src/html/HtmlReporter.js
index c70a9a1e..b69fc9d1 100644
--- a/src/html/HtmlReporter.js
+++ b/src/html/HtmlReporter.js
@@ -24,7 +24,7 @@ jasmine.HtmlReporter = function(_doc) {
   };
 
   self.reportRunnerResults = function(runner) {
-    reporterView.complete();
+    reporterView && reporterView.complete();
   };
 
   self.reportSuiteResults = function(suite) {