teabag 0.7.0 → 0.7.1

data/spec/spec_helper.rb

@@ -5,14 +5,14 @@ require File.expand_path("../dummy/config/environment", __FILE__)
 require "rspec/rails"
 require "rspec/autorun"
 require "capybara/rails"
-require 'capybara/poltergeist'
+#require 'capybara/poltergeist'
 require "aruba/api"
 
 require "ostruct"
 
 Dir[File.expand_path("../support/**/*.rb", __FILE__)].each { |f| require f }
 
-Capybara.javascript_driver = :poltergeist
+#Capybara.javascript_driver = :poltergeist
 
 RSpec.configure do |config|
   config.infer_base_class_for_anonymous_controllers = false

data/spec/teabag/configuration_spec.rb

@@ -46,6 +46,7 @@ describe Teabag::Configuration do
   after do
     Teabag::Configuration.mount_at = "/teabag"
     Teabag::Configuration.suites.delete("test_suite")
+    Teabag::Configuration.server = nil
   end
 
   it "has the default configuration" do
@@ -60,11 +61,14 @@ describe Teabag::Configuration do
     expect(subject.suites).to be_a(Hash)
     expect(subject.coverage).to eq(false)
     expect(subject.coverage_reports).to eq(["text-summary"])
+    expect(subject.server).to be_nil
   end
 
   it "allows setting various configuration options" do
     Teabag.configuration.mount_at = "/teabags_are_awesome"
     expect(subject.mount_at).to eq("/teabags_are_awesome")
+    Teabag.configuration.server = :webrick
+    expect(subject.server).to eq(:webrick)
   end
 
   it "allows defining suites" do

data/spec/teabag/formatters/base_formatter_spec.rb

@@ -34,7 +34,7 @@ describe Teabag::Formatters::BaseFormatter do
     end
 
     it "doesn't log when suppressing logs" do
-      Teabag.configuration.should_receive(:suppress_log).and_return(true)
+      subject.should_receive(:suppress_logs?).and_return(true)
       Teabag::Coverage.should_not_receive(:new)
       subject.send(:log_coverage, "_data_")
     end

data/spec/teabag/formatters/tap_formatter_spec.rb

@@ -0,0 +1,124 @@
+require "spec_helper"
+require "teabag/formatters/tap_formatter"
+require "teabag/result"
+
+describe Teabag::Formatters::TapFormatter do
+
+  before do
+    @log = ""
+    STDOUT.stub(:print) { |s| @log << s }
+  end
+
+  describe "#runner" do
+
+    let(:json) { {"start" => "_start_", "total" => 20} }
+
+    it "logs the information" do
+      result = Teabag::Result.build_from_json(json)
+      subject.should_receive(:log).with("1..20")
+      subject.runner(result)
+    end
+
+  end
+
+  describe "#spec" do
+
+    describe "passing spec" do
+
+      let(:json) { {"status" => "passed"} }
+
+      it "calls passing_spec" do
+        result = Teabag::Result.build_from_json(json)
+        subject.should_receive(:passing_spec)
+        subject.spec(result)
+      end
+
+    end
+
+    describe "pending spec" do
+
+      let(:json) { {"status" => "pending"} }
+
+      it "calls pending_spec" do
+        result = Teabag::Result.build_from_json(json)
+        subject.should_receive(:pending_spec)
+        subject.spec(result)
+      end
+
+    end
+
+    describe "failing spec" do
+
+      let(:json) { {"status" => "fail"} }
+
+      it "calls failing_spec" do
+        result = Teabag::Result.build_from_json(json)
+        subject.should_receive(:failing_spec)
+        subject.spec(result)
+      end
+
+    end
+
+  end
+
+  describe "#error" do
+
+    it "keeps count" do
+      expect(subject.errors.size).to be(0)
+      subject.error("foo")
+      expect(subject.errors.size).to be(1)
+    end
+
+  end
+
+  describe "#passing_spec" do
+
+    let(:json) { {"suite" => "_suite_", "label" => "_label_"} }
+
+    it "logs the information" do
+      result = Teabag::Result.build_from_json(json)
+      subject.should_receive(:log).with("ok 42 - _suite_ _label_")
+      subject.instance_variable_set(:@total, 42)
+      subject.instance_variable_set(:@result, result)
+      subject.send(:passing_spec)
+    end
+
+  end
+
+  describe "#pending_spec" do
+
+    let(:json) { {"suite" => "_suite_", "label" => "_label_"} }
+
+    it "logs the information" do
+      result = Teabag::Result.build_from_json(json)
+      subject.should_receive(:log).with("ok 42 - [pending] _suite_ _label_")
+      subject.instance_variable_set(:@total, 42)
+      subject.instance_variable_set(:@result, result)
+      subject.send(:pending_spec)
+    end
+
+  end
+
+  describe "#failing_spec" do
+
+    let(:json) { {"suite" => "_suite_", "label" => "_label_", "message" => "_message_"} }
+
+    it "logs the information" do
+      result = Teabag::Result.build_from_json(json)
+      subject.should_receive(:log).with("not ok 42 - _suite_ _label_\n  # FAIL _message_")
+      subject.instance_variable_set(:@total, 42)
+      subject.instance_variable_set(:@result, result)
+      subject.send(:failing_spec)
+    end
+
+  end
+
+  describe "#log" do
+
+    it "calls STDOUT.print" do
+      STDOUT.should_receive(:print).with("foo\n")
+      subject.send(:log, "foo")
+    end
+  end
+
+end

data/spec/teabag/instrumentation_spec.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 require "spec_helper"
 require "rack/test"
 
@@ -64,13 +66,14 @@ describe Teabag::Instrumentation do
 
   describe ".add_to" do
 
-    let(:asset) { mock(source: "function add(a, b) { return a + b }", pathname: 'path/to/instrument.js') }
+    let(:asset) { mock(source: source, pathname: 'path/to/instrument.js') }
+    let(:source) { "function add(a, b) { return a + b } // ☃ " }
 
     before do
       Teabag::Instrumentation.stub(:add?).and_return(true)
 
       File.stub(:write)
-      subject.any_instance.stub(:instrument).and_return("_foo_")
+      subject.any_instance.stub(:instrument).and_return(source + " // instrumented")
 
       path = nil
       Dir.mktmpdir { |p| path = p }
@@ -79,7 +82,7 @@ describe Teabag::Instrumentation do
     end
 
     it "writes the file to a tmp path" do
-      File.should_receive(:write).with(@output, "function add(a, b) { return a + b }")
+      File.should_receive(:write).with(@output, "function add(a, b) { return a + b } // ☃ ")
       subject.add_to(response, env)
     end
 
@@ -90,7 +93,7 @@ describe Teabag::Instrumentation do
 
     it "replaces the response array with the appropriate information" do
       response = [666, {"Content-Type" => "application/javascript"}, asset]
-      expected = [666, {"Content-Type" => "application/javascript", "Content-Length" => "5"}, asset]
+      expected = [666, {"Content-Type" => "application/javascript", "Content-Length" => "59"}, asset]
 
       subject.add_to(response, env)
       expect(response).to eq(expected)