rspec-core 2.5.0 → 2.6.0

data/spec/autotest/rspec_spec.rb

@@ -6,135 +6,13 @@ describe Autotest::Rspec2 do
   let(:ruby_cmd) { "ruby" }
 
   before do
-    RSpec.stub(:warn_deprecation)
     File.stub(:exist?) { false }
-    @orig_argv = ARGV.dup; ARGV.clear
   end
 
-  after  { ARGV.clear; ARGV.concat(@orig_argv) }
-
-  context "with ARGV" do
-    it "passes ARGV to command" do
-      ARGV.concat(%w[-c -f d])
-      rspec_autotest.cl_args.should eq(%w[-c -f d --tty])
-    end
-
-    context "with --skip-bundler" do
-      before do
-        ARGV.concat(%w[--skip-bundler])
-      end
-      it "extracts --skip-bundler" do
-        rspec_autotest.cl_args.should eq(%w[--tty])
-      end
-
-      it "sets skip_bundler? true" do
-        rspec_autotest.skip_bundler?.should be_true
-      end
-    end
-  end
-
-  describe "handling bundler" do
-    context "gemfile, no prefix" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { true }
-      end
-
-      it "warns of deprecation of implicit inclusion of 'bundle exec'" do
-        RSpec.should_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({})
-      end
-
-      it "includes bundle exec" do
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should match(/bundle exec/)
-      end
-    end
-
-    context "gemfile, no prefix, --skip-bundler" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { true }
-        ARGV.concat(%w[--skip-bundler])
-      end
-
-      it "does not warn" do
-        RSpec.should_not_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({})
-      end
-
-      it "does not include bundle exec" do
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should_not match(/bundle exec/)
-      end
-    end
-
-    context "no gemfile, prefix" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { false }
-      end
-
-      it "does not warn" do
-        RSpec.should_not_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({'a' => 'b'})
-      end
-
-      it "includes bundle exec" do
-        rspec_autotest.prefix = "bundle exec "
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should match(/bundle exec/)
-      end
-    end
-
-    context "gemfile, prefix" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { true }
-        rspec_autotest.prefix = "bundle exec "
-      end
-
-      it "does not warn" do
-        RSpec.should_not_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({'a' => 'b'})
-      end
-
-      it "includes bundle exec" do
-        rspec_autotest.prefix = "bundle exec "
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should match(/bundle exec/)
-      end
-    end
-
-    context "gemfile, prefix, --skip-bundler" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { true }
-        rspec_autotest.prefix = "bundle exec "
-        rspec_autotest.stub(:skip_bundler?) { true }
-      end
-
-      it "does not warn" do
-        RSpec.should_not_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({'a' => 'b'})
-      end
-
-      it "does not include bundle exec" do
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should_not match(/bundle exec/)
-      end
-    end
-
-    context "no gemfile, no prefix" do
-      before do
-        File.stub(:exist?).with("./Gemfile") { false }
-      end
-
-      it "does not warn" do
-        RSpec.should_not_receive(:warn_deprecation)
-        rspec_autotest.make_test_cmd({'a' => 'b'})
-      end
-
-      it "does not include bundle exec" do
-        rspec_autotest.
-          make_test_cmd({'a' => 'b'}).should_not match(/bundle exec/)
-      end
-    end
+  it "uses autotest's prefix" do
+    rspec_autotest.prefix = "this is the prefix "
+    rspec_autotest.
+      make_test_cmd({'a' => 'b'}).should match(/this is the prefix/)
   end
 
   describe "commands" do

data/spec/rspec/core/command_line_spec.rb

@@ -4,6 +4,42 @@ require 'tmpdir'
 
 module RSpec::Core
   describe CommandLine do
+
+    describe "#run" do
+      include_context "spec files"
+
+      let(:out)    { StringIO.new }
+      let(:err)    { StringIO.new }
+
+      def config_options(argv=[])
+        options = RSpec::Core::ConfigurationOptions.new(argv)
+        options.parse_options
+        options
+      end
+
+      def command_line(args)
+        RSpec::Core::CommandLine.new(config_options(args))
+      end
+
+      def config_options(argv=[])
+        options = RSpec::Core::ConfigurationOptions.new(argv)
+        options.parse_options
+        options
+      end
+
+      it "returns 0 if spec passes" do
+        err, out = StringIO.new, StringIO.new
+        result = command_line([passing_spec_filename]).run(err, out)
+        result.should be(0)
+      end
+
+      it "returns 1 if spec passes" do
+        err, out = StringIO.new, StringIO.new
+        result = command_line([failing_spec_filename]).run(err, out)
+        result.should be(1)
+      end
+    end
+
     context "given an Array of options" do
       it "assigns ConfigurationOptions built from Array to @options" do
         config_options = ConfigurationOptions.new(%w[--color])
@@ -50,8 +86,6 @@ module RSpec::Core
         config.stub(:run_hook)
 
         config.should_receive(:load_spec_files)
-        config.should_receive(:configure_mock_framework)
-        config.should_receive(:configure_expectation_framework)
 
         world.should_receive(:announce_inclusion_filter)
         world.should_receive(:announce_exclusion_filter)

data/spec/rspec/core/configuration_spec.rb

@@ -5,7 +5,7 @@ module RSpec::Core
 
   describe Configuration do
 
-    let(:config) { subject }
+    let(:config) { Configuration.new }
 
     describe "#load_spec_files" do
 
@@ -30,6 +30,17 @@ module RSpec::Core
       end
     end
 
+    describe "#treat_symbols_as_metadata_keys_with_true_values?" do
+      it 'defaults to false' do
+        config.treat_symbols_as_metadata_keys_with_true_values?.should be_false
+      end
+
+      it 'can be set to true' do
+        config.treat_symbols_as_metadata_keys_with_true_values = true
+        config.treat_symbols_as_metadata_keys_with_true_values?.should be_true
+      end
+    end
+
     describe "#mock_framework" do
       it "defaults to :rspec" do
         config.should_receive(:require).with('rspec/core/mocking/with_rspec')
@@ -100,16 +111,46 @@ module RSpec::Core
       end
     end
 
-    context "setting the files to run" do
+    describe "#expecting_with_rspec?" do
+      before(:each) do
+        # prevent minitest assertions from being required and included,
+        # as that causes problems in some of our specs.
+        config.stub(:require)
+      end
+
+      it "returns false by default" do
+        config.should_not be_expecting_with_rspec
+      end
+
+      it "returns true when `expect_with :rspec` has been configured" do
+        config.expect_with :rspec
+        config.should be_expecting_with_rspec
+      end
+
+      it "returns true when `expect_with :rspec, :stdlib` has been configured" do
+        config.expect_with :rspec, :stdlib
+        config.should be_expecting_with_rspec
+      end
+
+      it "returns true when `expect_with :stdlib, :rspec` has been configured" do
+        config.expect_with :stdlib, :rspec
+        config.should be_expecting_with_rspec
+      end
 
+      it "returns false when `expect_with :stdlib` has been configured" do
+        config.expect_with :stdlib
+        config.should_not be_expecting_with_rspec
+      end
+    end
+
+    describe "#files_to_run" do
       it "loads files not following pattern if named explicitly" do
         file = "./spec/rspec/core/resources/a_bar.rb"
         config.files_or_directories_to_run = file
         config.files_to_run.should == [file]
       end
 
-      describe "with default --pattern" do
-
+      context "with default pattern" do
         it "loads files named _spec.rb" do
           dir = "./spec/rspec/core/resources"
           config.files_or_directories_to_run = dir
@@ -121,80 +162,70 @@ module RSpec::Core
           config.files_or_directories_to_run = file
           config.files_to_run.should == [file]
         end
-
       end
+    end
 
-      describe "with explicit pattern (single)" do
+    %w[pattern= filename_pattern=].each do |setter|
+      describe "##{setter}" do
+        context "with single pattern" do
+          before { config.send(setter, "**/*_foo.rb") }
+          it "loads files following pattern" do
+            file = File.expand_path(File.dirname(__FILE__) + "/resources/a_foo.rb")
+            config.files_or_directories_to_run = file
+            config.files_to_run.should include(file)
+          end
 
-        before do
-          config.filename_pattern = "**/*_foo.rb"
-        end
+          it "loads files in directories following pattern" do
+            dir = File.expand_path(File.dirname(__FILE__) + "/resources")
+            config.files_or_directories_to_run = dir
+            config.files_to_run.should include("#{dir}/a_foo.rb")
+          end
 
-        it "loads files following pattern" do
-          file = File.expand_path(File.dirname(__FILE__) + "/resources/a_foo.rb")
-          config.files_or_directories_to_run = file
-          config.files_to_run.should include(file)
+          it "does not load files in directories not following pattern" do
+            dir = File.expand_path(File.dirname(__FILE__) + "/resources")
+            config.files_or_directories_to_run = dir
+            config.files_to_run.should_not include("#{dir}/a_bar.rb")
+          end
         end
 
-        it "loads files in directories following pattern" do
-          dir = File.expand_path(File.dirname(__FILE__) + "/resources")
-          config.files_or_directories_to_run = dir
-          config.files_to_run.should include("#{dir}/a_foo.rb")
-        end
+        context "with multiple patterns" do
+          it "supports comma separated values" do
+            config.send(setter, "**/*_foo.rb,**/*_bar.rb")
+            dir = File.expand_path(File.dirname(__FILE__) + "/resources")
+            config.files_or_directories_to_run = dir
+            config.files_to_run.should include("#{dir}/a_foo.rb")
+            config.files_to_run.should include("#{dir}/a_bar.rb")
+          end
 
-        it "does not load files in directories not following pattern" do
-          dir = File.expand_path(File.dirname(__FILE__) + "/resources")
-          config.files_or_directories_to_run = dir
-          config.files_to_run.should_not include("#{dir}/a_bar.rb")
+          it "supports comma separated values with spaces" do
+            config.send(setter, "**/*_foo.rb, **/*_bar.rb")
+            dir = File.expand_path(File.dirname(__FILE__) + "/resources")
+            config.files_or_directories_to_run = dir
+            config.files_to_run.should include("#{dir}/a_foo.rb")
+            config.files_to_run.should include("#{dir}/a_bar.rb")
+          end
         end
-
       end
+    end
 
-      context "with explicit pattern (comma,separated,values)" do
-
-        before do
-          config.filename_pattern = "**/*_foo.rb,**/*_bar.rb"
-        end
-
-        it "supports comma separated values" do
-          dir = File.expand_path(File.dirname(__FILE__) + "/resources")
-          config.files_or_directories_to_run = dir
-          config.files_to_run.should include("#{dir}/a_foo.rb")
-          config.files_to_run.should include("#{dir}/a_bar.rb")
-        end
-
-        it "supports comma separated values with spaces" do
-          dir = File.expand_path(File.dirname(__FILE__) + "/resources")
-          config.files_or_directories_to_run = dir
-          config.files_to_run.should include("#{dir}/a_foo.rb")
-          config.files_to_run.should include("#{dir}/a_bar.rb")
-        end
-
+    describe "path with line number" do
+      it "assigns the line number as the filter" do
+        config.files_or_directories_to_run = "path/to/a_spec.rb:37"
+        config.filter.should == {:line_number => 37}
       end
+    end
 
-      context "with line number" do
-
-        it "assigns the line number as the filter" do
-          config.files_or_directories_to_run = "path/to/a_spec.rb:37"
-          config.filter.should == {:line_number => 37}
-        end
-
+    context "with full_description" do
+      it "overrides :focused" do
+        config.filter_run :focused => true
+        config.full_description = "foo"
+        config.filter.should_not have_key(:focused)
       end
 
-      context "with full_description" do
-        it "overrides :focused" do
-          config.filter_run :focused => true
-          config.full_description = "foo"
-          config.filter.should_not have_key(:focused)
-        end
-
-        it "assigns the example name as the filter on description" do
-          config.full_description = "foo"
-          config.filter.should == {:full_description => /foo/}
-        end
-
+      it "assigns the example name as the filter on description" do
+        config.full_description = "foo"
+        config.filter.should == {:full_description => /foo/}
       end
-
     end
 
     describe "#include" do
@@ -205,6 +236,13 @@ module RSpec::Core
         end
       end
 
+      it_behaves_like "metadata hash builder" do
+        def metadata_hash(*args)
+          config.include(InstanceLevelMethods, *args)
+          config.include_or_extend_modules.last.last
+        end
+      end
+
       context "with no filter" do
         it "includes the given module into each example group" do
           RSpec.configure do |c|
@@ -238,6 +276,13 @@ module RSpec::Core
         end
       end
 
+      it_behaves_like "metadata hash builder" do
+        def metadata_hash(*args)
+          config.extend(ThatThingISentYou, *args)
+          config.include_or_extend_modules.last.last
+        end
+      end
+
       it "extends the given module into each matching example group" do
         RSpec.configure do |c|
           c.extend(ThatThingISentYou, :magic_key => :extend)
@@ -249,7 +294,7 @@ module RSpec::Core
 
     end
 
-    describe "run_all_when_everything_filtered?" do
+    describe "#run_all_when_everything_filtered?" do
 
       it "defaults to false" do
         config.run_all_when_everything_filtered?.should be_false
@@ -350,14 +395,14 @@ module RSpec::Core
       end
     end
 
-    describe 'formatter=' do
+    describe '#formatter=' do
       it "delegates to add_formatter (better API for user-facing configuration)" do
         config.should_receive(:add_formatter).with('these','options')
         config.add_formatter('these','options')
       end
     end
 
-    describe "add_formatter" do
+    describe "#add_formatter" do
 
       it "adds to the list of formatters" do
         config.add_formatter :documentation
@@ -421,6 +466,13 @@ module RSpec::Core
     end
 
     describe "#filter_run" do
+      it_behaves_like "metadata hash builder" do
+        def metadata_hash(*args)
+          config.filter_run *args
+          config.filter
+        end
+      end
+
       it "sets the filter" do
         config.filter_run :focus => true
         config.filter[:focus].should == true
@@ -454,6 +506,13 @@ module RSpec::Core
     end
 
     describe "#filter_run_excluding" do
+      it_behaves_like "metadata hash builder" do
+        def metadata_hash(*args)
+          config.filter_run_excluding *args
+          config.exclusion_filter
+        end
+      end
+
       it "sets the filter" do
         config.filter_run_excluding :slow => true
         config.exclusion_filter[:slow].should == true
@@ -502,7 +561,7 @@ module RSpec::Core
       end
     end
 
-    describe "line_number=" do
+    describe "#line_number=" do
       before { config.stub(:warn) }
 
       it "sets the line number" do
@@ -524,9 +583,18 @@ module RSpec::Core
     end
 
     describe "#full_backtrace=" do
-      it "clears the backtrace clean patterns" do
-        config.full_backtrace = true
-        config.backtrace_clean_patterns.should == []
+      context "given true" do
+        it "clears the backtrace clean patterns" do
+          config.full_backtrace = true
+          config.backtrace_clean_patterns.should == []
+        end
+      end
+
+      context "given false" do
+        it "restores backtrace clean patterns" do
+          config.full_backtrace = false
+          config.backtrace_clean_patterns.should == RSpec::Core::Configuration::DEFAULT_BACKTRACE_PATTERNS
+        end
       end
 
       it "doesn't impact other instances of config" do
@@ -712,5 +780,17 @@ module RSpec::Core
         group.included_modules.should include(mod2)
       end
     end
+
+    describe "#alias_example_to" do
+      it_behaves_like "metadata hash builder" do
+        def metadata_hash(*args)
+          config.alias_example_to :my_example, *args
+          group = ExampleGroup.describe("group")
+          example = group.my_example("description")
+          example.metadata
+        end
+      end
+    end
+
   end
 end

data/spec/rspec/core/deprecations_spec.rb

@@ -11,6 +11,16 @@ describe "deprecations" do
       RSpec.stub(:warn_deprecation)
       Spec.should == RSpec
     end
+
+    it "doesn't include backward compatibility in const_missing backtrace" do
+      RSpec.stub(:warn_deprecation)
+      exception = nil
+      begin
+        ConstantThatDoesNotExist
+      rescue Exception => exception
+      end
+      exception.backtrace.find { |l| l =~ /lib\/rspec\/core\/backward_compatibility/ }.should be_nil
+    end
   end
 
   describe RSpec::Core::ExampleGroup do
@@ -41,5 +51,16 @@ describe "deprecations" do
       RSpec.should_receive(:deprecate)
       Spec::Rake::SpecTask
     end
+
+    it "doesn't include backward compatibility in const_missing backtrace" do
+      RSpec.stub(:warn_deprecation)
+      exception = nil
+      begin
+        Spec::Rake::ConstantThatDoesNotExist
+      rescue Exception => exception
+      end
+      exception.backtrace.find { |l| l =~ /lib\/rspec\/core\/backward_compatibility/ }.should be_nil
+    end
   end
+
 end