fail_fast 0.5.1 → 0.5.2

data/spec/base/base_commands/failed_spec.rb

@@ -0,0 +1,19 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe 'failed?' do
+  example "is false when there are no errors" do
+    FailFast(SIMPLE_FILE_PATH ).check_now.but_fail_later {}
+    FailFast.failed?.should be_false
+  end
+
+  example "is true when there is 1 block and 1 error" do
+    FailFast(UNKNOWN_FILE_PATH).check_now.but_fail_later {} # <-- 1 error here
+    FailFast.failed?.should be_true
+  end
+
+  example "is true when the error block is followed by and error-free empty file block (BUG FIX)" do
+    FailFast(UNKNOWN_FILE_PATH).check_now.but_fail_later {} # <-- 1 error here
+    FailFast(EMPTY_FILE_PATH  ).check_now.but_fail_later {} #no errors in the last block
+    FailFast.failed?.should be_true
+  end
+end

data/spec/base/base_commands/only_if_spec.rb

@@ -0,0 +1,15 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe "only_if" do
+  it_should_not_raise_an_error('only_if false does not run the block') {
+    only_if false do
+      fail 'never executed'
+    end
+  }
+
+  it_should_raise_a_direct_error('message 456', :fail, 'only_if true runs the block') {
+    only_if true do
+      fail 'message 456'
+    end
+  }
+end

data/spec/base/base_commands/skip_if_spec.rb

@@ -0,0 +1,29 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe "skip_if" do
+  it_should_not_raise_an_error('skip_if(true) does not run the block') {
+    skip_if true do
+      fail 'never executed'
+    end
+  }
+
+  it_should_raise_a_direct_error('message 456', :fail, 'skip_if(false) runs the block') {
+    skip_if false do
+      fail 'message 456'
+    end
+  }
+end
+
+describe "skip_if" do
+  it_should_not_raise_an_error('skip_if(true) does not run the block') {
+    skip_if true do
+      fail 'never executed'
+    end
+  }
+
+  it_should_raise_a_direct_error('message 456', :fail, 'skip_if(false) runs the block') {
+    skip_if false do
+      fail 'message 456'
+    end
+  }
+end

data/spec/base/errors_storage_spec.rb

@@ -4,7 +4,7 @@ describe "errors" do
   before(:each) { capture_stdout }
   after( :each) { restore_stdout }
 
-  it "should use the filename and the prefix as key" do
+  it "uses the filename and the prefix as key" do
     begin
       FailFast('invalid_file_path', 'a_prefix').check() do
         has_value_for :unknown

data/spec/base/multiple_blocks_support_spec.rb

@@ -4,7 +4,7 @@ describe "FailFast.check_now.but_fail_later" do
   before(:all) { capture_stdout }
   after( :all) { restore_stdout }
 
-  it 'should not raise an error on the check(..) call' do
+  it 'does not raise an error on the check(..) call' do
     lambda {
       FailFast(UNKNOWN_FILE_PATH).check_now.but_fail_later {}
     }.should_not raise_error
@@ -21,13 +21,13 @@ describe "FailFast.check_now.but_fail_later" do
       FailFast(EMPTY_FILE_PATH  ).check_now.but_fail_later {} #no errors in the last block
     end
 
-    it("should detect and collect all the errors") {
+    it("detects and collect all the errors") {
       FailFast.errors_db.errors_for(FailFast::ErrorDb.key_for(UNKNOWN_FILE_PATH)).collect(&:kind).should == [:config_file_not_found]
       FailFast.errors_db.errors_for(FailFast::ErrorDb.key_for(SIMPLE_FILE_PATH )).collect(&:kind).should == [:missing_value, :missing_value]
       FailFast.errors_db.errors_for(FailFast::ErrorDb.key_for(EMPTY_FILE_PATH  )).collect(&:kind).should == []
     }
     context "after FailFast.fail_now" do
-      it "should raise an error" do
+      it "raises  an error" do
         lambda {
           FailFast.fail_now
         }.should raise_error(ExitTriggered)

data/spec/base/{not_linked_to_a_file_spec.rb → not_linked_to_a_config_file_spec.rb}

@@ -5,14 +5,14 @@ describe "check not linked to a config file" do
   after( :each) { restore_stdout }
 
 
-  it "should not raise an error when there are no checks" do
+  it "does not raise an error when there are no checks" do
     lambda {
       FailFast().check do end
     }.should_not raise_error
     FailFast.failed?.should be_false
   end
 
-  it "should raise an error when there is a failing check" do
+  it "raises  an error when there is a failing check" do
     lambda {
       FailFast().check do
         fail 'error'
@@ -21,7 +21,7 @@ describe "check not linked to a config file" do
     FailFast.failed?.should be_true
   end
 
-  it "should raise a delayed error when there is a failing check" do
+  it "raises  a delayed error when there is a failing check" do
     lambda {
       FailFast().check_now.but_fail_later do
         fail 'error'

data/spec/base/{file_is_empty_spec.rb → when_config_file_is_empty_spec.rb}

@@ -5,14 +5,14 @@ describe "ConfigCheck on an empty file" do
   after( :each) { restore_stdout }
 
 
-  it "should not raise an error when there are no checks" do
+  it "does not raise an error when there are no checks" do
     lambda {
       FailFast(EMPTY_FILE_PATH).check do end
     }.should_not raise_error
     FailFast.failed?.should be_false
   end
 
-  it "should raise an error when there is a failing check" do
+  it "raises  an error when there is a failing check" do
     lambda {
       FailFast(EMPTY_FILE_PATH).check do
         has_value_for :anykey
@@ -21,7 +21,7 @@ describe "ConfigCheck on an empty file" do
     FailFast.failed?.should be_true
   end
 
-  it "should raise a delayed error when there is a failing check" do
+  it "raises  a delayed error when there is a failing check" do
     lambda {
       FailFast(EMPTY_FILE_PATH).check_now.but_fail_later do
         has_value_for :anykey

data/spec/base/{file_is_missing_spec.rb → when_config_file_is_missing_spec.rb}

@@ -4,12 +4,12 @@ describe "ConfigCheck on an unknown file" do
   before(:all) { capture_stdout }
   after( :all) { restore_stdout }
 
-  it "should not raise an error in new()" do
+  it "does not raise an error in new()" do
     FailFast(UNKNOWN_FILE_PATH)
     fail if FailFast.failed?
   end
 
-  it "should raise an error in fail_fast()" do
+  it "raises  an error in fail_fast()" do
     lambda {
       FailFast(UNKNOWN_FILE_PATH).check do end
     }.should raise_error(ExitTriggered)

data/spec/extensions/{check_file_system_spec.rb → file_or_directory_exists_spec.rb}

@@ -6,6 +6,10 @@ a_file  = SPEC_DIR + '/fixtures/simple.yml'
 
 describe 'directory_exists' do
   context '' do
+    it_should_return_true( 'when the directory exists'          ) { directory_exists a_dir }
+    it_should_return_false('when the directory does not exists' ) { directory_exists 'XYZ'  }
+    it_should_return_false('when the directory is a dir'        ) { directory_exists a_file  }
+
     it_should_not_raise_an_error('when the directory exists'                                            ) { directory_exists a_dir }
 
     it_should_raise_a_direct_error('XYZ',     :directory_not_found, 'when the directory does not exist' ) { directory_exists 'XYZ' }
@@ -13,6 +17,10 @@ describe 'directory_exists' do
   end
 
   context '_for' do
+    it_should_return_true( 'when the file exists'          ) { directory_exists_for 'test/a_directory' }
+    it_should_return_false('when the file does not exists' ) { directory_exists_for 'test/UNKNOWN'  }
+    it_should_return_false('when the file is a dir'        ) { directory_exists_for 'test/a_file'  }
+
     it_should_not_raise_an_error('when the directory exists'                                            ) { directory_exists_for 'test/a_directory' }
 
     it_should_raise_an_error('test/a_file',   :directory_not_found, 'when the path points to a File'    ) { directory_exists_for 'test/a_file'      }
@@ -20,7 +28,7 @@ describe 'directory_exists' do
     it_should_raise_an_error('not_a_valid_key',:missing_value,      'when the key is invalid'           ) { directory_exists_for 'not_a_valid_key'  }
   end
 
-  it "should accept a custom message for the 5 cases" do
+  it "accepts a custom message for the 5 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
       directory_exists_for 'test/a_file', :message => 'a_custom_message'
       directory_exists_for 'test/email',  :message => 'a_custom_message_2'
@@ -36,6 +44,10 @@ end
 
 describe 'file_exists' do
   context '' do
+    it_should_return_true( 'when the file exists'          ) { file_exists a_file }
+    it_should_return_false('when the file does not exists' ) { file_exists 'XYZ'  }
+    it_should_return_false('when the file is a dir'        ) { file_exists a_dir  }
+
     it_should_not_raise_an_error('when the file exists'                                                 ) { file_exists a_file}
 
     it_should_raise_a_direct_error('XYZ',     :file_not_found, 'when the file does not exist'           ) { file_exists 'XYZ' }
@@ -43,6 +55,10 @@ describe 'file_exists' do
   end
 
   context '_for' do
+    it_should_return_true( 'when the file exists'          ) { file_exists_for 'test/a_file' }
+    it_should_return_false('when the file does not exists' ) { file_exists_for 'test/UNKNOWN'  }
+    it_should_return_false('when the file is a dir'        ) { file_exists_for 'test/a_directory'  }
+
     it_should_not_raise_an_error('when the file exists'                                                 ) { file_exists_for 'test/a_file' }
 
     it_should_raise_an_error('test/a_directory', :file_not_found,    'when the path points to a dir'    ) { file_exists_for 'test/a_directory'  }
@@ -50,7 +66,7 @@ describe 'file_exists' do
     it_should_raise_an_error('not_a_valid_key',  :missing_value,     'when the key is invalid'          ) { file_exists_for 'not_a_valid_key'   }
   end
 
-  it "should accept a custom message for the 5 cases" do
+  it "accepts a custom message for the 5 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
       file_exists_for 'test/a_directory', :message => 'a_custom_message'
       file_exists_for 'test/email',       :message => 'a_custom_message_2'

data/spec/extensions/{check_active_record_db_spec.rb → has_active_record_db_spec.rb}

@@ -6,14 +6,16 @@ describe 'has_ar_db' do
       conn = Object.new ; conn.should_receive(:active?).and_return(true)
       ActiveRecord::Base.should_receive(:connection).and_return(conn)
 
-      has_active_record_db :host => 'localhost', :adapter => 'mysql', :database=> 'a-db', :username => 'root'
+      result = has_active_record_db :host => 'localhost', :adapter => 'mysql', :database=> 'a-db', :username => 'root'
+      result.should == true
     }
 
     it_should_raise_a_direct_error('ze-error-message', :active_record_db_connection_error, 'when the connection fails' ) {
       exception = StandardError.new('ze-error-message')
       ActiveRecord::Base.should_receive(:establish_connection).and_raise(exception)
 
-      has_active_record_db valid_connection_options = {}
+      result = has_active_record_db valid_connection_options = {}
+      result.should == false
     }
   end
 
@@ -22,22 +24,25 @@ describe 'has_ar_db' do
       conn = Object.new ; conn.should_receive(:active?).and_return(true)
       ActiveRecord::Base.should_receive(:connection).and_return(conn)
 
-      has_active_record_db_for 'db_connection'
+      result = has_active_record_db_for 'db_connection'
+      result.should == true
     }
 
     it_should_raise_an_error('db_connection', :active_record_db_connection_error, 'when the connection fails' ) {
       exception = StandardError.new('an-error-message')
       ActiveRecord::Base.should_receive(:establish_connection).and_raise(exception)
 
-      has_active_record_db_for 'db_connection'
+      result = has_active_record_db_for 'db_connection'
+      result.should == false
     }
 
     it_should_raise_an_error('invalid_key', :missing_value, 'when the key is invalid' ) {
-      has_active_record_db_for 'invalid_key'
+      result = has_active_record_db_for 'invalid_key'
+      result.should == false
     }
   end
 
-  it "should accept a custom message for those 3 cases" do
+  it "accepts a custom message for those 3 cases" do
     exception = StandardError.new('an-error-message')
     ActiveRecord::Base.should_receive(:establish_connection).twice.and_raise(exception)
 

data/spec/extensions/{check_email_spec.rb → has_email_spec.rb}

@@ -2,19 +2,24 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe 'has_email_for()' do
   it_should_not_raise_an_error('when the value is an email') {
-    has_email_for 'test/email'
+    result = has_email_for 'test/email'
+    result.should == true
   }
   it_should_raise_an_error('test/url', :not_an_email, 'when the value is not an email') {
-    has_email_for 'test/url'
+    result = has_email_for 'test/url'
+    result.should == false
   }
   it_should_raise_an_error('invalid_key', :missing_value, 'when the key is invalid') {
-    has_email_for 'invalid_key'
+    result = has_email_for 'invalid_key'
+    result.should == false
   }
 
-  it "should accept a custom message for the 2 cases" do
+  it "accepts a custom message for the 2 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
-      has_email_for 'invalid_key', :message => 'a_custom_message'
-      has_email_for 'test/url',    :message => 'a_custom_message_2'
+      result = has_email_for 'invalid_key', :message => 'a_custom_message'
+      result.should == false
+      result = has_email_for 'test/url',    :message => 'a_custom_message_2'
+      result.should == false
     end
     messages = FailFast.errors_db.errors_for(FailFast.errors_db.keys.first).collect { |e| e.message }
     messages.should =~ %w(a_custom_message a_custom_message_2)

data/spec/extensions/{check_mongo_db_spec.rb → has_mongoDB_spec.rb}

@@ -9,11 +9,16 @@ describe 'has_mongoDB' do
       it_should_raise_a_direct_error('localhost', :mongoDB_server_not_found, 'when the mongoDB server connection failed') {
         has_mongoDB 'localhost'
       }
+      it_should_return_false('when the mongoDB server connection failed') {
+        has_mongoDB 'localhost'
+      }
 
-      it "should accept a custom message for this case" do
+      it "accepts a custom message for this case" do
         FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
-          has_mongoDB 'localhost',  :message => 'a_custom_message'
-          has_mongoDB 'localhost',  :timeout => 1, :message => 'a_custom_message_2'
+          result = has_mongoDB 'localhost',  :message => 'a_custom_message'
+          result.should == false
+          result = has_mongoDB 'localhost',  :timeout => 1, :message => 'a_custom_message_2'
+          result.should == false
         end
         messages = FailFast.errors_db.errors_for(FailFast.errors_db.keys.first).collect { |e| e.message }
         messages.should =~ %w(a_custom_message a_custom_message_2)
@@ -27,13 +32,17 @@ describe 'has_mongoDB' do
         Mongo::Connection.should_receive(:new).and_return(conn)
       end
 
+      it_should_return_true('when the mongoDB server responds'                         ) { has_mongoDB 'localhost'                   }
+      it_should_return_true('when the mongoDB server responds and the db can be found' ) { has_mongoDB 'localhost', 'sample_mongoDB' }
+      it_should_return_false('when the database cannot be found on the mongoDB'        ) { has_mongoDB 'localhost', 'not_a_known_db' }
+
       it_should_not_raise_an_error('when the mongoDB server responds'                         ) { has_mongoDB 'localhost'                   }
       it_should_not_raise_an_error('when the mongoDB server responds and the db can be found' ) { has_mongoDB 'localhost', 'sample_mongoDB' }
 
       it_should_raise_a_direct_error('not_a_known_db', :mongoDB_db_not_found, 'when the database cannot be found on the mongoDB') {
         has_mongoDB 'localhost', 'not_a_known_db'
       }
-      it "should accept a custom message for this case" do
+      it "accepts a custom message for this case" do
         FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
           has_mongoDB 'localhost', 'not_a_known_db',  :message => 'a_custom_message'
         end
@@ -47,14 +56,19 @@ describe 'has_mongoDB' do
     context 'when the mongo server cannot be reached' do
       before(:each) { Mongo::Connection.should_receive(:new).any_number_of_times.and_raise(Mongo::ConnectionFailure) }
 
+      it_should_return_false('when the mongoDB server connection failed') { has_mongoDB_for 'test/unreachable_mongoDB_server' }
+      it_should_return_false('when the key is invalid'                  ) { has_mongoDB_for 'not_a_valid_key' }
+
       it_should_raise_an_error('test/unreachable_mongoDB_server', :mongoDB_server_not_found, 'when the mongoDB server connection failed') {
         has_mongoDB_for 'test/unreachable_mongoDB_server'
       }
       it_should_raise_an_error('not_a_valid_key', :missing_value, 'when the key is invalid') { has_mongoDB_for 'not_a_valid_key' }
-      it "should accept a custom message for those 2 cases" do
+      it "accepts a custom message for those 2 cases" do
         FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
-          has_mongoDB_for 'not_a_valid_key',                  :message => 'a_custom_message'
-          has_mongoDB_for 'test/unreachable_mongoDB_server',  :message => 'a_custom_message_2'
+          result = has_mongoDB_for 'not_a_valid_key',                  :message => 'a_custom_message'
+          result.should == false
+          result = has_mongoDB_for 'test/unreachable_mongoDB_server',  :message => 'a_custom_message_2'
+          result.should == false
         end
         messages = FailFast.errors_db.errors_for(FailFast.errors_db.keys.first).collect { |e| e.message }
         messages.should =~ %w(a_custom_message a_custom_message_2)
@@ -68,6 +82,8 @@ describe 'has_mongoDB' do
         Mongo::Connection.should_receive(:new).and_return(conn)
       end
 
+      it_should_return_true('when the mongoDB server responds and the database can be found') { has_mongoDB_for 'test/mongoDB' }
+
       it_should_not_raise_an_error('when the mongoDB server responds and the database can be found') {
         has_mongoDB_for 'test/mongoDB'
       }
@@ -75,7 +91,7 @@ describe 'has_mongoDB' do
       it_should_raise_an_error('test/unknown_mongoDB_db', :mongoDB_db_not_found,'when the database cannot be found on the mongoDB') {
         has_mongoDB_for 'test/unknown_mongoDB_db'
       }
-      it "should accept a custom message for this case" do
+      it "accepts a custom message for this case" do
         FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
           has_mongoDB_for 'test/unknown_mongoDB_db',  :message => 'a_custom_message'
         end
@@ -83,7 +99,10 @@ describe 'has_mongoDB' do
         messages.should =~ %w(a_custom_message)
       end
 
-      it_should_not_raise_an_error('when :ignore_database => false desactivate the db check') {
+      it_should_return_true('when :ignore_database => false deactivates the db check') {
+        has_mongoDB_for 'test/unknown_mongoDB_db', :check_database => false
+      }
+      it_should_not_raise_an_error('when :ignore_database => false deactivates the db check') {
         has_mongoDB_for 'test/unknown_mongoDB_db', :check_database => false
       }
     end

data/spec/extensions/{check_url_spec.rb → has_url_spec.rb}

@@ -8,8 +8,11 @@ before(:all) do
     FakeWeb.register_uri(:get, "http://localhost:3200"        , :body => "I'm reachable!")
 end
 
+  it_should_return_true(       'when the value is an url') { has_url_for 'test/url' }
   it_should_not_raise_an_error('when the value is an url') { has_url_for 'test/url' }
   it_should_not_raise_an_error("when the domain is 'localhost'") { has_url_for 'test/url_localhost' }
+
+  it_should_return_false(                            'when the value is not an url') { has_url_for 'test/email' }
   it_should_raise_an_error('test/email', :not_a_url, 'when the value is not an url') { has_url_for 'test/email' }
 
   it_should_raise_an_error('not_a_valid_key', :missing_value ,'when the value is blank or absent') { has_url_for 'not_a_valid_key' }
@@ -21,7 +24,7 @@ end
     has_url_for 'test/url_not_reachable', :reachable => true
   }
 
-  it "should accept a custom message for the 3 cases" do
+  it "accepts a custom message for the 3 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
       has_url_for 'test/email',                                   :message => 'a_custom_message'
       has_url_for 'inconnu',                                      :message => 'a_custom_message_2'

data/spec/extensions/{check_value_spec.rb → has_value_spec.rb}

@@ -1,27 +1,27 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe 'value_of()' do
-  it 'should return nil when the value is blank' do
+  it 'returns nil when the value is blank' do
     FailFast(SIMPLE_FILE_PATH).check { @@val = value_of(:key_with_blank_value) }
     @@val.should be_nil
   end
 
-  it 'should return nil when the key is unknown' do
+  it 'returns nil when the key is unknown' do
     FailFast(EMPTY_FILE_PATH).check { @@val = value_of(:anykey) }
     @@val.should be_nil
   end
-  it 'should return the value when it is a leaf'do
+  it 'returns the value when it is a leaf'do
     FailFast(SIMPLE_FILE_PATH).check { @@val = value_of(:first_key) }
     @@val.should == 'premier'
   end
 
-  it 'should return the values in a hash when the value is a tree' do
+  it 'returns the values in a hash when the value is a tree' do
     FailFast(SIMPLE_FILE_PATH).check { @@val = value_of(:test) }
     @@val.should be_a(Hash)
     @@val['mongoDB']['host'].should == 'localhost'
   end
 
-  it 'should preprend the prefix to the key' do
+  it 'prepends the prefix to the key' do
     FailFast(SIMPLE_FILE_PATH, 'test').check { @@val = value_of(:mongoDB) }
     @@val['host'].should == 'localhost'
   end
@@ -31,12 +31,16 @@ describe 'has_value_for()' do
   before { capture_stdout }
   after  { restore_stdout }
 
-  it "should accept a custom message for the 4 cases" do
+  it "accepts a custom message for the 4 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
-      has_value_for 'inconnu',   /localhost/,       :message => 'a_custom_message'
-      has_value_for :first_key,  /nomatchhere/,     :message => 'a_custom_message_2'
-      has_value_for :letter_x,   :in => ('a'..'b'), :message => 'a_custom_message_3'
-      has_value_for :letter_x,   :in => [6, "a"],   :message => 'a_custom_message_4'
+      result = has_value_for 'inconnu',   /localhost/,       :message => 'a_custom_message'
+      result.should == false
+      result = has_value_for :first_key,  /nomatchhere/,     :message => 'a_custom_message_2'
+      result.should == false
+      result = has_value_for :letter_x,   :in => ('a'..'b'), :message => 'a_custom_message_3'
+      result.should == false
+      result = has_value_for :letter_x,   :in => [6, "a"],   :message => 'a_custom_message_4'
+      result.should == false
     end
 
     messages = FailFast.errors_db.errors_for(FailFast.errors_db.keys.first).collect { |e| e.message }
@@ -101,7 +105,7 @@ describe 'has_value_for()' do
   end
 
   context 'when a prefix is specified' do
-    it "should not raise an error when a prefix is used" do
+    it "does not raise an error when a prefix is used" do
       lambda {
         FailFast(SIMPLE_FILE_PATH, 'test').check do
           has_value_for 'mongoDB/host',    /localhost/

data/spec/extensions/{check_is_on_path_spec.rb → is_on_path_spec.rb}

@@ -5,25 +5,29 @@ describe 'is_on_path' do
 
   context '' do
     it_should_not_raise_an_error('when the app is on the path') {
-      is_on_path 'ls'
+      result = is_on_path 'ls'
+      result.should == true
     }
 
     it_should_raise_a_direct_error('ls987654321', :not_on_path, 'when the app is not on the path' ) {
-      is_on_path 'ls987654321'
+      result = is_on_path 'ls987654321'
+      result.should == false
     }
   end
 
   context '_for' do
     it_should_not_raise_an_error('when the app is on the path') {
-      is_on_path_for 'app_on_path'
+      result = is_on_path_for 'app_on_path'
+      result.should == true
     }
 
     it_should_raise_an_error('app_not_on_path', :not_on_path, 'when the app is not on the path' ) {
-      is_on_path_for 'app_not_on_path'
+      result = is_on_path_for 'app_not_on_path'
+      result.should == false
     }
   end
 
-  it "should accept a custom message for the 3 cases" do
+  it "accepts a custom message for the 3 cases" do
     FailFast(SIMPLE_FILE_PATH).check_now.but_fail_later do
       is_on_path 'ls987654321',         :message => 'a_custom_message'
       is_on_path_for 'INCONNU',         :message => 'a_custom_message_2'

data/spec/spec_helper.rb

@@ -3,7 +3,7 @@ require 'fail_fast'
 require 'spec'
 require 'spec/autorun'
 require 'fakeweb'
-gem 'mongo', '1.0'
+#gem 'mongo', '1.0'
 require 'mongo'
 
 SPEC_DIR = File.dirname(File.expand_path(__FILE__))
@@ -32,8 +32,43 @@ module DSLMacros
   end
   module ClassMethods
 
+    def it_should_return_true(msg, &block)
+      it "returns true #{msg}" do
+        capture_stdout
+        begin
+          FailFast(SIMPLE_FILE_PATH).check.but_fail_later do
+            raise "BUG : @@errorz should be empty \n#{errors.inspect}"  unless errors.empty?
+            result = self.instance_eval(&block)
+            #TODO : BETTER ERROR REPORTING
+            result.should == true
+          end
+        rescue => e
+          raise e
+        end
+        restore_stdout
+      end
+    end
+
+
+    def it_should_return_false(msg, &block)
+      it "returns false #{msg}" do
+        capture_stdout
+        begin
+          FailFast(SIMPLE_FILE_PATH).check.but_fail_later do
+            raise "BUG : @@errorz should be empty \n#{errors.inspect}"  unless errors.empty?
+            result = self.instance_eval(&block)
+            #TODO : BETTER ERROR REPORTING
+            result.should == false
+          end
+        rescue => e
+          raise e
+        end
+        restore_stdout
+      end
+    end
+
     def it_should_not_raise_an_error(msg, &block)
-      it "should not raise an error #{msg}" do
+      it "does not raise an error #{msg}" do
         capture_stdout
         begin
           FailFast(SIMPLE_FILE_PATH).check do
@@ -52,7 +87,7 @@ module DSLMacros
     end
 
     def it_should_raise_an_error(key, kind, msg, &block)
-      it "should raise an error #{kind}-#{key}-#{msg}" do
+      it "raises an error #{kind}-#{key}-#{msg}" do
         capture_stdout
         begin
           FailFast(SIMPLE_FILE_PATH).check do
@@ -75,7 +110,7 @@ module DSLMacros
       end
     end
     def it_should_raise_a_direct_error(value, kind, msg, &block)
-      it "should raise an error #{kind}-#{value}-#{msg}" do
+      it "raises an error #{kind}-#{value}-#{msg}" do
         capture_stdout
         begin
           FailFast(SIMPLE_FILE_PATH).check do