teaspoon 0.7.9 → 0.8.0

data/spec/teaspoon/formatters/clean_formatter_spec.rb

@@ -0,0 +1,37 @@
+require "spec_helper"
+
+describe Teaspoon::Formatters::CleanFormatter do
+
+  let(:passing_spec) { double(passing?: true) }
+  let(:pending_spec) { double(passing?: false, pending?: true, description: "_description_") }
+  let(:failing_spec) { double(passing?: false, pending?: false, description: "_description_", message: "_message_", link: "_link_") }
+
+  before do
+    @log = ""
+    STDOUT.stub(:print) { |s| @log << s }
+  end
+
+  describe "#result" do
+
+    let(:result) { double(elapsed: 3.1337, coverage: nil) }
+
+    before do
+      subject.run_count = 666
+    end
+
+    describe "with failures" do
+
+      before do
+        subject.failures << failing_spec
+      end
+
+      it "logs the failures but not the failure commands" do
+        subject.result(result)
+        expect(@log).to eq("\n\nFailures:\n\n  1) _description_\n\e[31m     Failure/Error: _message_\n\e[0m\nFinished in 3.1337 seconds\n\e[31m666 examples, 1 failure\e[0m\n")
+      end
+
+    end
+
+  end
+
+end

data/spec/teaspoon/formatters/documentation_formatter_spec.rb

@@ -0,0 +1,127 @@
+require "spec_helper"
+
+describe Teaspoon::Formatters::DocumentationFormatter do
+
+  let(:suite) { double(label: "_suite_", level: @level || 0)}
+  let(:passing_spec) { double(passing?: true, label: "_passing_") }
+  let(:pending_spec) { double(passing?: false, pending?: true, label: "_pending_", description: "_description_") }
+  let(:failing_spec) { double(passing?: false, pending?: false, label: "_failing_", description: "_description_", message: "_message_", link: "_link_") }
+
+  before do
+    @log = ""
+    STDOUT.stub(:print) { |s| @log << s }
+  end
+
+  describe "#suite" do
+
+    it "logs a properly indented suite message" do
+      @level = 2
+      subject.suite(suite)
+      expect(@log).to eq("    _suite_\n")
+      expect(subject.instance_variable_get(:@level)).to eq(2)
+    end
+
+  end
+
+  describe "#spec" do
+
+    before do
+      subject.instance_variable_set(:@level, 1)
+      subject.instance_variable_set(:@last_suite, {})
+      subject.instance_variable_set(:@stdout, "line1\nline2\n")
+    end
+
+    it "logs an indented green label on passing results" do
+      subject.spec(passing_spec)
+      expect(@log).to eq("\e[32m    _passing_\e[0m\n\e[36m      # line1\n      # line2\e[0m\n")
+    end
+
+    it "logs an indented yellow label on pending results" do
+      subject.instance_variable_set(:@level, 1)
+      subject.instance_variable_set(:@last_suite, {})
+      subject.spec(pending_spec)
+      expect(@log).to eq("\e[33m    _pending_ (PENDING)\e[0m\n\e[36m      # line1\n      # line2\e[0m\n")
+    end
+
+    it "logs an indented red label (with stdout) on failing results" do
+      subject.spec(failing_spec)
+      expect(@log).to eq("\e[31m    _failing_ (FAILED - 1)\e[0m\n\e[36m      # line1\n      # line2\e[0m\n")
+    end
+
+  end
+
+  describe "#error" do
+
+    let(:result) { double(message: "_message_", trace: [{"file" => "http://127.0.0.1:31337/assets/path/file.js?foo=true&body=1", "line" => 42, "function" => "notAnAnonFunc"}]) }
+
+    it "logs the error" do
+      subject.error(result)
+      expect(@log).to eq("\e[31m_message_\e[0m\n\e[36m  # path/file.js?foo=true:42 -- notAnAnonFunc\e[0m\n\n")
+    end
+
+  end
+
+  describe "#result" do
+
+    let(:result) { double(elapsed: 3.1337, coverage: nil) }
+
+    before do
+      subject.run_count = 666
+    end
+
+    describe "with no failures" do
+
+      it "logs the stats" do
+        subject.result(result)
+        expect(@log).to eq("\nFinished in 3.1337 seconds\n\e[32m666 examples, 0 failures\e[0m\n")
+      end
+
+    end
+
+    describe "with failures" do
+
+      before do
+        subject.failures << failing_spec
+      end
+
+      it "logs the failures" do
+        subject.result(result)
+        expect(@log).to eq("\nFailures:\n\n  1) _description_\n\e[31m     Failure/Error: _message_\n\e[0m\nFinished in 3.1337 seconds\n\e[31m666 examples, 1 failure\e[0m\n\nFailed examples:\n\n\e[31mteaspoon -s default --filter=\"_link_\"\e[0m\n")
+      end
+
+    end
+
+    describe "with pending" do
+
+      before do
+        subject.pendings << pending_spec
+      end
+
+      it "logs the pending specs" do
+        subject.result(result)
+        expect(@log).to eq("\nPending:\n\e[33m  _description_\e[0m\n\e[36m    # Not yet implemented\n\e[0m\nFinished in 3.1337 seconds\n\e[33m666 examples, 0 failures, 1 pending\e[0m\n")
+      end
+
+    end
+
+  end
+
+  describe "#coverage" do
+
+    it "logs the coverage" do
+      subject.coverage("_text_\n\n_text_summary_")
+      expect(@log).to eq("\n_text_\n\n_text_summary_\n")
+    end
+
+  end
+
+  describe "#threshold_failure" do
+
+    it "logs the threshold failures" do
+      subject.threshold_failure("_was_not_met_\n_was_not_met_")
+      expect(@log).to eq("\e[31m\n_was_not_met_\n_was_not_met_\n\e[0m\n")
+    end
+
+  end
+
+end

data/spec/teaspoon/formatters/dot_formatter_spec.rb

@@ -1,9 +1,11 @@
 require "spec_helper"
-require "teaspoon/formatters/dot_formatter"
-require "teaspoon/result"
 
 describe Teaspoon::Formatters::DotFormatter do
 
+  let(:passing_spec) { double(passing?: true) }
+  let(:pending_spec) { double(passing?: false, pending?: true, description: "_description_") }
+  let(:failing_spec) { double(passing?: false, pending?: false, description: "_description_", message: "_message_", link: "_link_") }
+
   before do
     @log = ""
     STDOUT.stub(:print) { |s| @log << s }
@@ -11,110 +13,104 @@ describe Teaspoon::Formatters::DotFormatter do
 
   describe "#spec" do
 
-    it "tracks totals" do
-      subject.total = 41
-      subject.should_receive(:log).with(".", 32)
-      subject.spec(Teaspoon::Result.build_from_json("status" => "passed"))
-      expect(subject.total).to be(42)
-    end
-
-    it "logs a green . on pass" do
-      subject.should_receive(:log).with(".", 32)
-      subject.spec(Teaspoon::Result.build_from_json("status" => "passed"))
+    it "logs a green . on passing results" do
+      subject.spec(passing_spec)
+      expect(@log).to eq("\e[32m.\e[0m")
     end
 
-    it "logs a yellow * on pending" do
-      subject.should_receive(:log).with("*", 33)
-      subject.spec(Teaspoon::Result.build_from_json("status" => "pending"))
+    it "logs a yellow * on pending results" do
+      subject.spec(pending_spec)
+      expect(@log).to eq("\e[33m*\e[0m")
     end
 
-    it "logs a red F on anything else" do
-      subject.should_receive(:log).with("F", 31)
-      subject.spec(Teaspoon::Result.build_from_json("status" => "foo"))
+    it "logs a red F on failing results" do
+      subject.spec(failing_spec)
+      expect(@log).to eq("\e[31mF\e[0m")
     end
 
   end
 
   describe "#error" do
 
+    let(:result) { double(message: "_message_", trace: [{"file" => "http://127.0.0.1:31337/assets/path/file.js?foo=true&body=1", "line" => 42, "function" => "notAnAnonFunc"}]) }
+
     it "logs the error" do
-      trace = {"file" => "http://127.0.0.1:31337/assets/path/file.js?foo=true&body=1", "line" => 42, "function" => "notAnAnonFunc"}
-      subject.error("message" => "_message_", "trace" => [trace])
-      expect(@log).to eq("\e[31m_message_\n\e[0m\e[36m  # path/file.js?foo=true:42 -- notAnAnonFunc\n\e[0m\n")
+      subject.error(result)
+      expect(@log).to eq("\e[31m_message_\e[0m\n\e[36m  # path/file.js?foo=true:42 -- notAnAnonFunc\e[0m\n\n")
+    end
+
+  end
+
+  describe "#console" do
+
+    it "logs the message" do
+      subject.console("_message_\n")
+      expect(@log).to eq("_message_\n")
     end
 
   end
 
   describe "#result" do
 
+    let(:result) { double(elapsed: 3.1337, coverage: nil) }
+
     before do
-      subject.total = 666
+      subject.run_count = 666
     end
 
     describe "with no failures" do
 
-      it "logs the details" do
-        subject.result("elapsed" => 0.31337)
-        expect(@log).to eq("\n\nFinished in 0.31337 seconds\n\e[32m666 examples, 0 failures\n\e[0m")
+      it "logs the stats" do
+        subject.result(result)
+        expect(@log).to eq("\n\nFinished in 3.1337 seconds\n\e[32m666 examples, 0 failures\e[0m\n")
       end
 
     end
 
     describe "with failures" do
 
-      it "logs the details and raises an exception" do
-        subject.failures << Teaspoon::Result.build_from_json("label" => "some spec", "suite" => "full description", "message" => "some message", "link" => "full description")
-        expect{
-          subject.result("elapsed" => 0.31337)
-        }.to raise_error(Teaspoon::Failure)
-        expect(@log).to eq("\n\nFailures:\n\n  1) full description some spec\n\e[31m     Failure/Error: some message\n\n\e[0mFinished in 0.31337 seconds\n\e[31m666 examples, 1 failure\n\e[0m\nFailed examples:\n\n\e[31mteaspoon -s default --filter=\"full description\"\n\e[0m\n")
-        expect(subject.failures.length).to be(1)
+      before do
+        subject.failures << failing_spec
       end
 
-      describe "when fail_fast is false" do
-
-        it "doesn't raise the exception" do
-          Teaspoon.configuration.should_receive(:fail_fast).and_return(false)
-          subject.failures << Teaspoon::Result.build_from_json("message" => "some message")
-          subject.result("elapsed" => 0.31337)
-          expect(subject.failures.length).to be(1)
-        end
-
+      it "logs the failures" do
+        subject.result(result)
+        expect(@log).to eq("\n\nFailures:\n\n  1) _description_\n\e[31m     Failure/Error: _message_\n\e[0m\nFinished in 3.1337 seconds\n\e[31m666 examples, 1 failure\e[0m\n\nFailed examples:\n\n\e[31mteaspoon -s default --filter=\"_link_\"\e[0m\n")
       end
 
     end
 
     describe "with pending" do
 
-      it "logs the details" do
-        subject.pendings << Teaspoon::Result.build_from_json("label" => "some spec", "suite" => "full description")
-        subject.result("elapsed" => 0.31337)
-        expect(@log).to eq("\n\nPending:\n\e[33m  full description some spec\n\e[0m\e[36m    # Not yet implemented\n\n\e[0mFinished in 0.31337 seconds\n\e[33m666 examples, 0 failures, 1 pending\n\e[0m")
+      before do
+        subject.pendings << pending_spec
+      end
+
+      it "logs the pending specs" do
+        subject.result(result)
+        expect(@log).to eq("\n\nPending:\n\e[33m  _description_\e[0m\n\e[36m    # Not yet implemented\n\e[0m\nFinished in 3.1337 seconds\n\e[33m666 examples, 0 failures, 1 pending\e[0m\n")
       end
 
     end
 
   end
 
-  describe "#exception" do
+  describe "#coverage" do
 
-    it "raises" do
-      expect{ subject.exception }.to raise_error(Teaspoon::RunnerException)
+    it "logs the coverage" do
+      subject.coverage("_text_\n\n_text_summary_")
+      expect(@log).to eq("\n_text_\n\n_text_summary_\n")
     end
 
   end
 
-  describe "#log" do
+  describe "#threshold_failure" do
 
-    it "calls STDOUT.write" do
-      STDOUT.should_receive(:print).with("foo")
-      subject.send(:log, "foo")
+    it "logs the threshold failures" do
+      subject.threshold_failure("_was_not_met_\n_was_not_met_")
+      expect(@log).to eq("\e[31m\n_was_not_met_\n_was_not_met_\n\e[0m\n")
     end
 
-    it "colorizes" do
-      STDOUT.should_receive(:print).with("\e[31mfoo\e[0m")
-      subject.send(:log, "foo", 31)
-    end
   end
 
 end

data/spec/teaspoon/formatters/json_formatter_spec.rb

@@ -0,0 +1,77 @@
+require "spec_helper"
+
+describe Teaspoon::Formatters::JsonFormatter do
+
+  let(:hash) { {original_json: "_original_json_"} }
+  let(:result) { double(hash) }
+
+  describe "#runner" do
+
+    let(:result) { double(hash.merge(total: 42)) }
+
+    it "logs the original json" do
+      subject.should_receive(:log_result).with(result)
+      subject.runner(result)
+    end
+
+  end
+
+  describe "#suite" do
+
+    it "logs the original json" do
+      subject.should_receive(:log_result).with(result)
+      subject.suite(result)
+    end
+
+  end
+
+  describe "#spec" do
+
+    let(:result) { double(hash.merge(passing?: true)) }
+
+    it "logs the original json" do
+      subject.should_receive(:log_result).with(result)
+      subject.spec(result)
+    end
+
+  end
+
+  describe "#error" do
+
+    it "logs the original json" do
+      subject.should_receive(:log_result).with(result)
+      subject.error(result)
+    end
+
+  end
+
+  describe "#exception" do
+
+    it "logs the original json" do
+      subject.should_receive(:log_result).with(result)
+      subject.exception(result)
+    end
+
+  end
+
+  describe "#console" do
+
+    it "logs the message as json" do
+      subject.should_receive(:log_line).with(%Q{{"type":"console","log":"_message_"}})
+      subject.console("_message_")
+    end
+
+  end
+
+  describe "#result" do
+
+    let(:result) { double(hash.merge(coverage: nil)) }
+
+    it "logs the original json" do
+      subject.should_receive(:log_str).with("_original_json_")
+      subject.result(result)
+    end
+
+  end
+
+end

data/spec/teaspoon/formatters/junit_formatter_spec.rb

@@ -1,9 +1,11 @@
 require "spec_helper"
-require "teaspoon/formatters/junit_formatter"
-require "teaspoon/result"
 
 describe Teaspoon::Formatters::JunitFormatter do
 
+  let(:passing_spec) { double(passing?: true, suite: "_suite_name_", label: "_passing_label_") }
+  let(:pending_spec) { double(passing?: false, pending?: true, suite: "_suite_name_", label: "_pending_label_") }
+  let(:failing_spec) { double(passing?: false, pending?: false, suite: "_suite_name_", label: "_failing&label_", message: "_failure_message_") }
+
   before do
     @log = ""
     STDOUT.stub(:print) { |s| @log << s }
@@ -11,65 +13,100 @@ describe Teaspoon::Formatters::JunitFormatter do
 
   describe "#runner" do
 
-    let(:json) { {"start" => "_start_", "total" => 20} }
+    let(:result) { double(start: "_start_", total: 42) }
+
+    before do
+      subject.instance_variable_set(:@suite_name, "not_default")
+    end
 
-    it "logs the information" do
-      result = Teaspoon::Result.build_from_json(json)
-      subject.should_receive(:log).with("<?xml version=\"1.0\" encoding=\"UTF-8\"?>")
-      subject.should_receive(:log).with("<testsuites name=\"jasmine\"><testsuite name=\"\" tests=\"20\">")
+    it "starts the suite" do
       subject.runner(result)
+      expect(@log).to eq(%Q{<?xml version="1.0" encoding="UTF-8"?>\n<testsuites name="Teaspoon">\n<testsuite name="not_default" tests="42" time="_start_">\n})
     end
 
   end
 
-  describe "#spec" do
+  describe "#suite" do
+
+    let(:result) { double(label: "_label_") }
 
-    describe "passing spec" do
+    it "calls #log_end_suite" do
+      subject.should_receive(:log_end_suite)
+      subject.suite(result)
+    end
 
-      let(:json) { {"status" => "passed", "suite" => "My Suite", "label" => "My Label"} }
+    it "logs the start of the testsuite" do
+      subject.suite(result)
+      expect(@log).to eq(%Q{<testsuite name="_label_">\n})
+    end
 
-      it "calls passing_spec" do
-        result = Teaspoon::Result.build_from_json(json)
-        subject.spec(result)
+  end
 
-        @log.should include "<testcase classname=\"My Suite\" name=\"My Label\"></testcase>"
-      end
+  describe "#spec" do
 
+    it "logs a passing testcase on passing results" do
+      subject.spec(passing_spec)
+      expect(@log).to eq(%Q{<testcase classname="_suite_name_" name="_passing_label_">\n</testcase>\n})
     end
 
-    describe "pending spec" do
+    it "logs a skipped testcase on pending results" do
+      subject.spec(pending_spec)
+      expect(@log).to eq(%Q{<testcase classname="_suite_name_" name="_pending_label_">\n  <skipped/>\n</testcase>\n})
+    end
 
-      let(:json) { {"status" => "pending", "suite" => "My Suite", "label" => "My Label"} }
+    it "logs a failing testcase with the message on failing results" do
+      subject.spec(failing_spec)
+      expect(@log).to include(%Q{<testcase classname="_suite_name_" name="_failing&amp;label_">\n})
+      expect(@log).to include(%Q{  <failure type="AssertionFailed">\n<![CDATA[\n_failure_message_\n]]>\n</failure>\n})
+      expect(@log).to include(%Q{</testcase>\n})
+    end
 
-      it "calls pending_spec" do
-        result = Teaspoon::Result.build_from_json(json)
-        subject.spec(result)
-        @log.should include "<testcase classname=\"My Suite\" name=\"My Label\"><skipped /></testcase>"
-      end
+    it "includes any stdout" do
+      subject.instance_variable_set(:@stdout, "_stdout_")
+      subject.spec(passing_spec)
+      expect(@log).to eq(%Q{<testcase classname="_suite_name_" name="_passing_label_">\n<system-out>\n<![CDATA[\n_stdout_\n]]>\n</system-out>\n</testcase>\n})
+    end
 
+  end
+
+  describe "#result" do
+
+    let(:result) { double(coverage: nil) }
+
+    it "closes the last suite" do
+      subject.should_receive(:log_end_suite)
+      subject.result(result)
     end
 
-    describe "failing spec" do
+  end
 
-      let(:json) { {"status" => "fail", "suite" => "My Suite", "label" => "My Label", "message" => "Error, oh no"} }
+  describe "#coverage" do
 
-      it "calls failing_spec" do
-        result = Teaspoon::Result.build_from_json(json)
-        subject.spec(result)
-        @log.should include "<testcase classname=\"My Suite\" name=\"My Label\">
-<failure type=\"AssertionFailed\">Error, oh no</failure>
-</testcase>"
-      end
+    it "logs the coverage" do
+      subject.coverage("_text_\n\n_text_summary_")
+      expect(@log).to eq(%Q{<testsuite name="Coverage summary" tests="0">\n<properties>\n<![CDATA[\n_text_\n_text_summary_\n]]>\n<properties>\n</testsuite>\n})
+    end
+
+  end
 
+  describe "#threshold_failure" do
+
+    it "logs the threshold failures" do
+      subject.threshold_failure("_was_not_met_\n_was_not_met_")
+      expect(@log).to include(%Q{<testsuite name="Coverage thresholds" tests="1">\n})
+      expect(@log).to include(%Q{<testcase classname="Coverage thresholds" name="were not met">\n})
+      expect(@log).to include(%Q{  <failure type="AssertionFailed">\n<![CDATA[\n_was_not_met_\n_was_not_met_\n]]>\n</failure>\n})
+      expect(@log).to include(%Q{</testcase>\n})
+      expect(@log).to include(%Q{</testsuite>\n})
     end
 
   end
-  describe "#result" do
 
-    it "ends the suite" do
-      subject.result('result')
+  describe "#complete" do
 
-      @log.should include "</testsuite></testsuites>"
+    it "logs the closing suite tags" do
+      subject.complete(2)
+      expect(@log).to include(%Q{</testsuite>\n</testsuites>})
     end
 
   end