configuration_spec.rb in lucid-0.3.0

- old
+ new

@@ -1,16 +1,16 @@
-require_relative '../spec_helper'
+require 'spec_helper'
 
 module Lucid
   describe Configuration do
     describe '.default' do
       subject { Configuration.default }
 
       it 'has an autoload_code_paths containing default Lucid folders' do
-        subject.autoload_code_paths.should include('common')
-        subject.autoload_code_paths.should include('steps')
-        subject.autoload_code_paths.should include('pages')
+        subject.autoload_code_paths.should include 'common'
+        subject.autoload_code_paths.should include 'steps'
+        subject.autoload_code_paths.should include 'pages'
       end
     end
 
     describe 'supports custom user options' do
       let(:user_options) { { :autoload_code_paths => ['library/common'] } }
@@ -22,10 +22,12 @@
     end
   end
   
   module CLI
     describe Configuration do
+
+      attr_reader :out, :error
       
       module ExposeOptions
         attr_reader :options
       end
       
@@ -37,16 +39,17 @@
         File.stub(:directory?).and_return(true)
         File.stub(:file?).and_return(true)
         Dir.stub(:[]).and_return(files)
       end
       
-      def with_this_configuration_file(info)
+      def with_this_configuration(info)
+        Dir.stub(:glob).with('{,.config/,config/}lucid{.yml,.yaml}').and_return(['lucid.yml'])
         File.stub(:exist?).and_return(true)
         profile_file = info.is_a?(Hash) ? info.to_yaml : info
         IO.stub(:read).with('lucid.yml').and_return(profile_file)
       end
-      
+
       it 'should require driver.rb files first' do
         with_these_files('/common/support/browser.rb', '/common/support/driver.rb')
         config.parse(%w{--require /common})
         
         config.library_context.should == %w(
@@ -75,21 +78,43 @@
       end
 
       it 'should default to a specs directory when no information is provided' do
         File.stub(:directory?).and_return(true)
         Dir.stub(:[]).with('specs/**/*.spec').and_return(['lucid.spec'])
+        Dir.stub(:[]).with('specs/**/*.feature').and_return(['lucid.spec'])
+        Dir.stub(:[]).with('specs/**/*.story').and_return(['lucid.spec'])
         config.parse(%w{})
         config.spec_files.should == ['lucid.spec']
       end
       
       it 'should search for all specs in the specified directory' do
         File.stub(:directory?).and_return(true)
-        Dir.stub(:[]).with('specs/**/*.spec').and_return(["lucid.spec"])
+        Dir.stub(:[]).with('specs/**/*.spec').and_return(['lucid.spec'])
+        Dir.stub(:[]).with('specs/**/*.feature').and_return(['lucid.spec'])
+        Dir.stub(:[]).with('specs/**/*.story').and_return(['lucid.spec'])
         config.parse(%w{specs/})
         config.spec_files.should == ['lucid.spec']
       end
-      
+
+      it 'should return the correct spec file type for feature file' do
+        File.stub(:directory?).and_return(true)
+        Dir.stub(:[]).with('specs/**/*.spec').and_return(['lucid.feature'])
+        Dir.stub(:[]).with('specs/**/*.feature').and_return(['lucid.feature'])
+        Dir.stub(:[]).with('specs/**/*.story').and_return(['lucid.feature'])
+        config.parse(%w{specs/})
+        config.spec_files.should == ['lucid.feature']
+      end
+
+      it 'should return the correct spec file type for story file' do
+        File.stub(:directory?).and_return(true)
+        Dir.stub(:[]).with('specs/**/*.spec').and_return(['lucid.story'])
+        Dir.stub(:[]).with('specs/**/*.feature').and_return(['lucid.story'])
+        Dir.stub(:[]).with('specs/**/*.story').and_return(['lucid.story'])
+        config.parse(%w{specs/})
+        config.spec_files.should == ['lucid.story']
+      end
+
       it 'should preserve the order of the spec files' do
         config.parse(%w{test_b.spec test_c.spec test_a.spec})
         config.spec_files.should == %w[test_b.spec test_c.spec test_a.spec]
       end
       
@@ -140,10 +165,16 @@
       
       it 'should be able to use a --verbose option' do
         config.parse(%w{--verbose})
         config.options[:verbose].should be_true
       end
+
+      it 'uses the default profile when no profile is defined' do
+        with_this_configuration({'default' => '--require test_file'})
+        config.parse(%w{--format progress})
+        config.options[:require].should include('test_file')
+      end
       
       describe 'generating output' do
       
         it 'should be able to use an --out option' do
           config.parse(%w{--out report.txt})
@@ -164,9 +195,133 @@
 
       it 'should accept --no-color option' do
         Lucid::Term::ANSIColor.should_receive(:coloring=).with(false)
         config = Configuration.new(StringIO.new)
         config.parse(['--no-color'])
+      end
+
+      it 'should accept multiple --format options and put the STDOUT one first so progress is seen' do
+        config.parse(%w{--format standard --out output.txt --format progress})
+        config.formats.should == [['progress', out], ['standard', 'output.txt']]
+      end
+
+      it 'should not accept multiple --format options when both use implicit STDOUT' do
+        lambda do
+          config.parse(%w{--format pretty --format progress})
+        end.should raise_error('All but one formatter must use --out, only one can print to each stream (or STDOUT)')
+      end
+
+      it 'should accept same --format options with implicit STDOUT, and keep only one' do
+        config.parse(%w{--format standard --format standard})
+        config.formats.should == [['standard', out]]
+      end
+
+      it 'should not accept multiple --out streams pointing to the same place' do
+        lambda do
+          config.parse(%w{--format standard --out file1 --format progress --out file1})
+        end.should raise_error('All but one formatter must use --out, only one can print to each stream (or STDOUT)')
+      end
+
+      it 'should associate --out to previous --format' do
+        config.parse(%w{--format progress --out file1 --format profile --out file2})
+        config.formats.should == [['progress', 'file1'], ['profile', 'file2']]
+      end
+
+      it 'should accept same --format options with same --out streams and keep only one' do
+        config.parse(%w{--format html --out file --format standard --format html --out file})
+        config.formats.should == [['standard', out], ['html', 'file']]
+      end
+
+      it 'should accept same --format options with different --out streams' do
+        config.parse(%w{--format html --out file1 --format html --out file2})
+        config.formats.should == [['html', 'file1'], ['html', 'file2']]
+      end
+
+      it 'should accept multiple --name options' do
+        config.parse(['--name', 'User logs in', '--name', 'User signs up'])
+
+        config.options[:name_regexps].should include(/User logs in/)
+        config.options[:name_regexps].should include(/User signs up/)
+      end
+
+      it 'should accept multiple -n options' do
+        config.parse(['-n', 'User logs in', '-n', 'User signs up'])
+
+        config.options[:name_regexps].should include(/User logs in/)
+        config.options[:name_regexps].should include(/User signs up/)
+      end
+
+      it 'should allow specifying environment variables in profiles' do
+        with_this_configuration({'selenium' => 'DRIVER=selenium'})
+        config.parse(['--profile', 'selenium'])
+        ENV['DRIVER'].should == 'selenium'
+        config.spec_files.should_not include('DRIVER=selenium')
+      end
+
+      describe 'Dry run execution' do
+        it 'returns true when --dry-run was specified on in the arguments' do
+          config.parse(['--dry-run'])
+          config.dry_run?.should be_true
+        end
+
+        it 'returns true when --dry-run was specified in a profile' do
+          with_this_configuration({'default' => '--dry-run'})
+          config.parse([])
+          config.dry_run?.should be_true
+        end
+
+        it 'returns false by default' do
+          config.parse([])
+          config.dry_run?.should be_false
+        end
+      end
+      
+      describe 'Specifying matcher type' do
+        it 'returns the matcher type when it was set' do
+          config.parse(['--matcher-type', 'classic'])
+          config.matcher_type.should eql :classic
+        end
+
+        it 'returns the matcher type when it was set with shorthand option' do
+          config.parse(['-I', 'classic'])
+          config.matcher_type.should eql :classic
+        end
+
+        it 'returns the default matcher type if it was not set' do
+          config.parse([])
+          config.matcher_type.should eql :regexp
+        end
+      end
+
+      describe 'handling tags' do
+        it 'returns an empty expression when no tags are specified' do
+          config.parse([])
+          config.tag_expression.should be_empty
+        end
+
+        it 'returns an expression when tags are specified' do
+          config.parse(['--tags','@smoke'])
+          config.tag_expression.should_not be_empty
+        end
+      end
+
+      describe 'Getting a backtrace of errors' do
+        before do
+          Lucid.use_full_backtrace = false
+        end
+
+        it 'should show full backtrace when --backtrace is called for' do
+          config = App.new(['--backtrace'])
+          begin
+            'x'.should == 'y'
+          rescue => e
+            e.backtrace[0].should_not == "#{__FILE__}:#{__LINE__ - 2}"
+          end
+        end
+        
+        after do
+          Lucid.use_full_backtrace = false
+        end
       end
       
     end
   end
 end
\ No newline at end of file