rspec 0.7.5.1 → 0.8.0

data/spec/spec/runner/heckler_spec.rb

@@ -0,0 +1,14 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+unless ['i386-mswin32', 'java'].index(PLATFORM)
+  require 'spec/runner/heckle_runner'
+
+  context "Heckler" do
+    specify "should run context_runner on tests_pass?" do
+      context_runner = mock("context_runner")
+      context_runner.should_receive(:run)
+      heckler = Spec::Runner::Heckler.new('Array', 'push', context_runner)
+
+      heckler.tests_pass?
+    end
+  end
+end

data/spec/spec/runner/kernel_ext_spec.rb

@@ -0,0 +1,16 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+module Spec
+module Runner
+context "KernelExt" do
+  specify "should add context method to kernel" do
+      lambda do
+        context "" do
+          
+        end
+      end.should_not_raise
+    
+  end
+end
+end
+end

data/spec/spec/runner/noisy_backtrace_tweaker_spec.rb

@@ -0,0 +1,45 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+module Spec
+  module Runner
+    context "NoisyBacktraceTweaker" do
+      setup do
+        @error = RuntimeError.new
+        @tweaker = NoisyBacktraceTweaker.new
+      end
+        
+      specify "should leave anything in lib spec dir" do
+        ["expectations", "mocks", "runner", "stubs"].each do |child|
+          @error.set_backtrace(["/lib/spec/#{child}/anything.rb"])
+          @tweaker.tweak_backtrace(@error, "spec name")
+          @error.backtrace.should_not_be_empty
+        end
+      end
+
+      specify "should leave anything in spec dir" do
+        @error.set_backtrace(["/lib/spec/expectations/anything.rb"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_not_be_empty
+      end
+
+      specify "should leave bin spec" do
+        @error.set_backtrace(["bin/spec:"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_not_be_empty
+      end
+
+      specify "should not barf on nil backtrace" do
+        lambda do
+          @tweaker.tweak_backtrace(@error, "spec name")
+        end.should_not_raise
+      end
+      
+      specify "should clean up double slashes" do
+        @error.set_backtrace(["/a//b/c//d.rb"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_include "/a/b/c/d.rb"
+      end
+      
+    end
+  end
+end

data/spec/spec/runner/object_ext_spec.rb

@@ -0,0 +1,11 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+module Spec
+  module Runner
+    context "ObjectExt" do
+      specify "should add copy_instance_variables_from to object" do
+        Object.new.should_respond_to :copy_instance_variables_from
+      end
+    end
+  end
+end

data/spec/spec/runner/option_parser_spec.rb

@@ -0,0 +1,269 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+context "OptionParser" do
+  setup do
+    @out = StringIO.new
+    @err = StringIO.new
+    @parser = Spec::Runner::OptionParser.new
+  end
+
+  def parse(args)
+    @parser.parse(args, @err, @out, true)
+  end
+
+  specify "should accept dry run option" do
+    options = parse(["--dry-run"])
+    options.dry_run.should_be(true)
+  end
+
+  specify "should eval and use custom formatter when none of the builtins" do
+    options = parse(["--format", "Custom::Formatter"])
+    options.formatter_type.should equal(Custom::Formatter)
+  end
+
+  specify "should not be verbose by default" do
+    options = parse([])
+    options.verbose.should_be(nil)
+  end
+
+  specify "should not use colour by default" do
+    options = parse([])
+    options.colour.should_be(nil)
+  end
+
+  specify "should print help to stdout" do
+    options = parse(["--help"])
+    @out.rewind
+    @out.read.should_match(/Usage: spec \[options\] \(FILE\|DIRECTORY\|GLOB\)\+/n)
+  end
+
+  specify "should print instructions about how to fix bad formatter" do
+    options = parse(["--format", "Custom::BadFormatter"])
+    @err.string.should_match(/Couldn't find formatter class Custom::BadFormatter/n)
+  end
+
+  specify "should print usage to err if no dir specified" do
+    options = parse([])
+    @err.string.should_match(/Usage: spec/)
+  end
+
+  specify "should print version to stdout" do
+    options = parse(["--version"])
+    @out.rewind
+    @out.read.should_match(/RSpec-\d+\.\d+\.\d+ \(r\d+\) - BDD for Ruby\nhttp:\/\/rspec.rubyforge.org\/\n/n)
+  end
+  
+  specify "should accept -o option" do
+    options = parse(["-o", "#{File.expand_path(File.dirname(__FILE__))}/output_file.txt"])
+    options.out.should_be_an_instance_of File
+    options.out.path.should == "#{File.expand_path(File.dirname(__FILE__))}/output_file.txt"
+    File.delete(options.out.path) rescue nil
+  end
+
+  specify "should require file when require specified" do
+    lambda do
+      parse(["--require", "whatever"])
+    end.should_raise(LoadError)
+  end
+
+  specify "should select dry run for rdoc formatter" do
+    options = parse(["--format", "rdoc"])
+    options.dry_run.should_be(true)
+  end
+
+  specify "should support c option" do
+    options = parse(["-c"])
+    options.colour.should_be(true)
+  end
+
+  specify "should support queens colour option" do
+    options = parse(["--colour"])
+    options.colour.should_be(true)
+  end
+
+  specify "should support us color option" do
+    options = parse(["--color"])
+    options.colour.should_be(true)
+  end
+
+  specify "should support single spec with s option" do
+    options = parse(["-s", "something or other"])
+    options.spec_name.should_eql("something or other")
+  end
+
+  specify "should support single spec with spec option" do
+    options = parse(["--spec", "something or other"])
+    options.spec_name.should_eql("something or other")
+  end
+
+  specify "should use html formatter when format is h" do
+    options = parse(["--format", "h"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::HtmlFormatter)
+  end
+
+  specify "should use html formatter when format is html" do
+    options = parse(["--format", "html"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::HtmlFormatter)
+  end
+
+  specify "should use noisy backtrace tweaker with b option" do
+    options = parse(["-b"])
+    options.backtrace_tweaker.should_be_instance_of(Spec::Runner::NoisyBacktraceTweaker)
+  end
+
+  specify "should use noisy backtrace tweaker with backtrace option" do
+    options = parse(["--backtrace"])
+    options.backtrace_tweaker.should_be_instance_of(Spec::Runner::NoisyBacktraceTweaker)
+  end
+
+  specify "should use quiet backtrace tweaker by default" do
+    options = parse([])
+    options.backtrace_tweaker.should_be_instance_of(Spec::Runner::QuietBacktraceTweaker)
+  end
+
+  specify "should use progress bar formatter by default" do
+    options = parse([])
+    options.formatter_type.should equal(Spec::Runner::Formatter::ProgressBarFormatter)
+  end
+
+  specify "should use rdoc formatter when format is r" do
+    options = parse(["--format", "r"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::RdocFormatter)
+  end
+
+  specify "should use rdoc formatter when format is rdoc" do
+    options = parse(["--format", "rdoc"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::RdocFormatter)
+  end
+
+  specify "should use specdoc formatter when format is s" do
+    options = parse(["--format", "s"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::SpecdocFormatter)
+  end
+
+  specify "should use specdoc formatter when format is specdoc" do
+    options = parse(["--format", "specdoc"])
+    options.formatter_type.should equal(Spec::Runner::Formatter::SpecdocFormatter)
+  end
+
+  specify "should support diff option when format is not specified" do
+    options = parse(["--diff"])
+    options.diff_format.should_be :unified
+  end
+
+  specify "should use unified diff format option when format is unified" do
+    options = parse(["--diff", "unified"])
+    options.diff_format.should_be :unified
+    options.differ_class.should_be Spec::Expectations::Differs::Default
+  end
+
+  specify "should use context diff format option when format is context" do
+    options = parse(["--diff", "context"])
+    options.diff_format.should eql(:context)
+    options.differ_class.should_eql Spec::Expectations::Differs::Default
+  end
+
+  specify "should use custom diff format option when format is a custom format" do
+    options = parse(["--diff", "Custom::Formatter"])
+    options.diff_format.should_be :custom
+    options.differ_class.should_eql Custom::Formatter
+  end
+
+  specify "should print instructions about how to fix bad differ" do
+    options = parse(["--diff", "Custom::BadFormatter"])
+    @err.string.should_match(/Couldn't find differ class Custom::BadFormatter/n)
+  end
+
+  specify "should support --line to identify spec" do
+    spec_parser = mock("spec_parser")
+    @parser.instance_variable_set('@spec_parser', spec_parser)
+
+    file_factory = mock("File")
+    file_factory.should_receive(:file?).and_return(true)
+    file_factory.should_receive(:open).and_return("fake_io")
+    @parser.instance_variable_set('@file_factory', file_factory)
+
+    spec_parser.should_receive(:spec_name_for).with("fake_io", 169).and_return("some spec")
+
+    options = parse(["some file", "--line", "169"])
+    options.spec_name.should_eql("some spec")
+    File.__verify
+  end
+
+  specify "should fail with error message if file is dir along with --line" do
+    spec_parser = mock("spec_parser")
+    @parser.instance_variable_set('@spec_parser', spec_parser)
+
+    file_factory = mock("File")
+    file_factory.should_receive(:file?).and_return(false)
+    file_factory.should_receive(:directory?).and_return(true)
+    @parser.instance_variable_set('@file_factory', file_factory)
+
+    options = parse(["some file", "--line", "169"])
+    @err.string.should_match(/You must specify one file, not a directory when using the --line option/n)
+  end
+
+  specify "should fail with error message if file is dir along with --line" do
+    spec_parser = mock("spec_parser")
+    @parser.instance_variable_set('@spec_parser', spec_parser)
+
+    file_factory = mock("File")
+    file_factory.should_receive(:file?).and_return(false)
+    file_factory.should_receive(:directory?).and_return(false)
+    @parser.instance_variable_set('@file_factory', file_factory)
+
+    options = parse(["some file", "--line", "169"])
+    @err.string.should_match(/some file does not exist/n)
+  end
+
+  specify "should fail with error message if more than one files are specified along with --line" do
+    spec_parser = mock("spec_parser")
+    @parser.instance_variable_set('@spec_parser', spec_parser)
+
+    options = parse(["some file", "some other file", "--line", "169"])
+    @err.string.should_match(/Only one file can be specified when using the --line option/n)
+  end
+
+  specify "should fail with error message if --spec and --line are used simultaneously" do
+    spec_parser = mock("spec_parser")
+    @parser.instance_variable_set('@spec_parser', spec_parser)
+
+    options = parse(["some file", "--spec", "some spec", "--line", "169"])
+    @err.string.should_match(/You cannot use both --line and --spec/n)
+  end
+
+  if(PLATFORM != "i386-mswin32")
+    specify "should heckle when --heckle is specified (and platform is not windows)" do
+      options = parse(["--heckle", "Spec"])
+      options.heckle_runner.should_be_instance_of(Spec::Runner::HeckleRunner)
+    end
+  else
+    specify "should barf when --heckle is specified (and platform is windows)" do
+      lambda do
+        options = parse(["--heckle", "Spec"])
+      end.should_raise(StandardError, "Heckle not supported on Windows")
+    end
+  end
+   
+  specify "should read options from file when --options is specified" do
+    Spec::Runner::CommandLine.should_receive(:run).with(["--diff", "--colour"], @err, @out, true, true)
+    options = parse(["--options", File.dirname(__FILE__) + "/spec.opts"])
+  end
+
+  specify "should append options from file when --options is specified" do
+    Spec::Runner::CommandLine.should_receive(:run).with(["some/spec.rb", "--diff", "--colour"], @err, @out, true, true)
+    options = parse(["some/spec.rb", "--options", File.dirname(__FILE__) + "/spec.opts"])
+  end
+   
+  specify "should save config to file when --generate-options is specified" do
+    FileUtils.rm 'spec.opts' rescue nil
+    options = parse(["--colour", "--generate-options", "spec.opts", "--diff"])
+    File.open('spec.opts').read.should == "--colour\n--diff\n"
+    FileUtils.rm 'spec.opts' rescue nil
+  end
+
+  specify "should call DrbCommandLine when --drb is specified" do
+    Spec::Runner::DrbCommandLine.should_receive(:run).with(["some/spec.rb", "--diff", "--colour"], @err, @out, true, true)
+    options = parse(["some/spec.rb", "--diff", "--drb", "--colour"])
+  end
+end

data/spec/spec/runner/quiet_backtrace_tweaker_spec.rb

@@ -0,0 +1,47 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+module Spec
+  module Runner
+    context "QuietBacktraceTweaker" do
+      setup do
+        @error = RuntimeError.new
+        @tweaker = QuietBacktraceTweaker.new
+      end
+
+      specify "should not barf on nil backtrace" do
+        lambda do
+          @tweaker.tweak_backtrace(@error, "spec name")
+        end.should_not_raise
+      end
+
+      specify "should remove anything from textmate ruby bundle" do
+        @error.set_backtrace(["/Applications/TextMate.app/Contents/SharedSupport/Bundles/Ruby.tmbundle/Support/tmruby.rb:147"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_be_empty
+      end
+
+      specify "should remove anything in lib spec dir" do
+        ["expectations", "mocks", "runner", "callback"].each do |child|
+          element="/lib/spec/#{child}/anything.rb"
+          @error.set_backtrace([element])
+          @tweaker.tweak_backtrace(@error, "spec name")
+          unless (@error.backtrace.empty?)
+            raise("Should have tweaked away '#{element}'")
+          end
+        end
+      end
+
+      specify "should remove bin spec" do
+        @error.set_backtrace(["bin/spec:"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_be_empty
+      end
+      
+      specify "should clean up double slashes" do
+        @error.set_backtrace(["/a//b/c//d.rb"])
+        @tweaker.tweak_backtrace(@error, "spec name")
+        @error.backtrace.should_include "/a/b/c/d.rb"
+      end
+    end
+  end
+end

data/spec/spec/runner/reporter_spec.rb

@@ -0,0 +1,126 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+module Spec
+module Runner
+context "Reporter" do
+    setup do
+        @io = StringIO.new
+        @backtrace_tweaker = Spec::Mocks::Mock.new("backtrace tweaker", {
+          :auto_verify => false
+        })
+        @formatter = Spec::Mocks::Mock.new("formatter", {
+          :auto_verify => false
+        })
+        @reporter = Reporter.new(@formatter, @backtrace_tweaker)
+      
+    end
+    specify "should account for context in stats" do
+        @formatter.should_receive(:add_context).with("context", true)
+        @reporter.add_context("context")
+      
+    end
+    specify "should account for spec and error in stats for pass" do
+        @formatter.should_receive(:add_context)
+        @formatter.should_receive(:spec_started).with("spec")
+        @formatter.should_receive(:spec_failed).with("spec", 1, failure)
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_failure).with(1, :anything)
+        @formatter.should_receive(:dump_summary).with(:anything, 1, 1)
+        @backtrace_tweaker.should_receive(:tweak_backtrace)
+        @reporter.add_context("context")
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec", RuntimeError.new)
+        @reporter.dump
+      
+    end
+    specify "should account for spec in stats for pass" do
+        @formatter.should_receive(:spec_started)
+        @formatter.should_receive(:spec_passed)
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_summary).with(:anything, 1, 0)
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec")
+        @reporter.dump
+      
+    end
+    specify "should delegate to backtrace tweaker" do
+        @formatter.should_receive(:add_context)
+        @formatter.should_receive(:spec_failed)
+        @backtrace_tweaker.should_receive(:tweak_backtrace)
+        @reporter.add_context("context")
+        @reporter.spec_finished("spec", RuntimeError.new)
+        @backtrace_tweaker.__verify
+      
+    end
+    specify "should handle multiple contexts with same name" do
+        @formatter.should_receive(:add_context).exactly(3).times
+        @formatter.should_receive(:spec_started).exactly(3).times
+        @formatter.should_receive(:spec_passed).exactly(3).times
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_summary).with(:anything, 3, 0)
+        @reporter.add_context("context")
+        @reporter.spec_started("spec 1")
+        @reporter.spec_finished("spec 1")
+        @reporter.add_context("context")
+        @reporter.spec_started("spec 2")
+        @reporter.spec_finished("spec 2")
+        @reporter.add_context("context")
+        @reporter.spec_started("spec 3")
+        @reporter.spec_finished("spec 3")
+        @reporter.dump
+      
+    end
+    specify "should handle multiple specs same name" do
+      
+        error=RuntimeError.new
+        @formatter.should_receive(:add_context).exactly(2).times
+        @formatter.should_receive(:spec_started).with("spec").exactly(4).times
+        @formatter.should_receive(:spec_passed).with("spec").exactly(2).times
+        @formatter.should_receive(:spec_failed).with("spec", 1, failure)
+        @formatter.should_receive(:spec_failed).with("spec", 2, failure)
+        @formatter.should_receive(:dump_failure).exactly(2).times
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_summary).with(:anything, 4, 2)
+        @backtrace_tweaker.should_receive(:tweak_backtrace)
+        @reporter.add_context("context")
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec")
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec", error)
+        @reporter.add_context("context")
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec")
+        @reporter.spec_started("spec")
+        @reporter.spec_finished("spec", error)
+        @reporter.dump
+      
+    end
+    specify "should push context to formatter" do
+        @formatter.should_receive(:add_context).never
+        @reporter.add_context("context")
+      
+    end
+    specify "should push stats to reporter even with no data" do
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_summary).with(:anything, 0, 0)
+        @reporter.dump
+      
+    end
+    specify "should push time to reporter" do
+        @formatter.should_receive(:start).with(5)
+        @formatter.should_receive(:start_dump)
+        @formatter.should_receive(:dump_summary) do |time, a, b|
+          time.to_s.should_match(/[0-9].[0-9|e|-]+/)
+        end
+        @reporter.start(5)
+        @reporter.end
+        @reporter.dump
+      
+    end
+    def failure
+      Mocks::DuckTypeArgConstraint.new(:header, :exception)
+    end
+  
+end
+end
+end