status_cat 0.1.0 → 0.1.1

data/spec/lib/status_cat/engine_spec.rb

@@ -1,9 +1,6 @@
-require 'spec_helper'
-
 describe StatusCat::Engine do
 
   it 'isolates the StatusCat namespace' do
-    StatusCat::Engine.isolated.should be_true
+    expect( StatusCat::Engine.isolated ).to be( true )
   end
-
 end

data/spec/lib/status_cat/status_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe StatusCat::Status do
 
   #############################################################################
@@ -10,14 +8,13 @@ describe StatusCat::Status do
 
     it 'returns an array of enabled checkers' do
       enabled = StatusCat::Config.instance.enabled
-      StatusCat::Config.instance.should_receive( :enabled ).and_return( enabled )
+      expect( StatusCat::Config.instance ).to receive( :enabled ).and_return( enabled )
 
       all = StatusCat::Status.all
-      all.should be_an_instance_of( Array )
-      all.length.should eql( StatusCat::Checkers::Base.descendants.length )
-      all.each { |checker| checker.should be_a_kind_of( StatusCat::Checkers::Base ) }
+      expect( all ).to be_an_instance_of( Array )
+      expect( all.length ).to eql( StatusCat::Checkers::Base.descendants.length )
+      all.each { |checker| expect( checker ).to be_a_kind_of( StatusCat::Checkers::Base ) }
     end
-
   end
 
   #############################################################################
@@ -27,9 +24,9 @@ describe StatusCat::Status do
   describe '::check' do
 
     def checker_array_should_have_names( actual, expected )
-      actual.should be_a_kind_of( Array )
-      actual.length.should eql( expected.length )
-      actual.each_index { |i| actual[ i ].name.should be( expected[ i ] ) }
+      expect( actual ).to be_a_kind_of( Array )
+      expect( actual.length ).to eql( expected.length )
+      actual.each_index { |i| expect( actual[ i ].name ).to be( expected[ i ] ) }
     end
 
     it 'defaults to all checkers' do
@@ -45,7 +42,7 @@ describe StatusCat::Status do
     end
 
     it 'returns a single checker when given a single symbolic name' do
-      StatusCat::Status.check( :active_record ).should be_an_instance_of( StatusCat::Checkers::ActiveRecord )
+      expect( StatusCat::Status.check( :active_record ) ).to be_an_instance_of( StatusCat::Checkers::ActiveRecord )
     end
 
     it 'returns an array of checkers when given an array of symbolic name' do
@@ -53,7 +50,6 @@ describe StatusCat::Status do
       checkers = StatusCat::Status.check( names )
       checker_array_should_have_names( checkers, names )
     end
-
   end
 
   #############################################################################
@@ -65,17 +61,16 @@ describe StatusCat::Status do
     it 'delivers email if ::failed is not empty' do
       failed = [ StatusCat::Checkers::Base.new ]
       mail =  StatusCat::StatusMailer.failure( failed )
-      StatusCat::Status.should_receive( :failed ).and_return( failed )
-      StatusCat::StatusMailer.should_receive( :failure ).with( failed ).and_return( mail )
+      expect( StatusCat::Status ).to receive( :failed ).and_return( failed )
+      expect( StatusCat::StatusMailer ).to receive( :failure ).with( failed ).and_return( mail )
       StatusCat::Status.cron
     end
 
     it 'does not email when ::failed is empty' do
-      StatusCat::Status.should_receive( :failed ).and_return( [] )
-      StatusCat::StatusMailer.should_not_receive( :failure )
+      expect( StatusCat::Status ).to receive( :failed ).and_return( [] )
+      expect( StatusCat::StatusMailer ).to_not receive( :failure )
       StatusCat::Status.cron
     end
-
   end
 
   #############################################################################
@@ -85,10 +80,9 @@ describe StatusCat::Status do
   describe '::factory' do
 
     it 'constructs a checker given its symbolic name' do
-      StatusCat::Status.factory( :action_mailer ).should be_an_instance_of( StatusCat::Checkers::ActionMailer )
-      StatusCat::Status.factory( :active_record ).should be_an_instance_of( StatusCat::Checkers::ActiveRecord )
+      expect( StatusCat::Status.factory( :action_mailer ) ).to be_an_instance_of( StatusCat::Checkers::ActionMailer )
+      expect( StatusCat::Status.factory( :active_record ) ).to be_an_instance_of( StatusCat::Checkers::ActiveRecord )
     end
-
   end
 
   #############################################################################
@@ -101,20 +95,18 @@ describe StatusCat::Status do
       @pass = StatusCat::Checkers::Base.new
 
       @fail = StatusCat::Checkers::Base.new
-      @fail.stub( :status ).and_return( :fail )
+      allow( @fail ).to receive( :status ).and_return( :fail )
     end
 
     it 'returns only failed checkers from ::all' do
-      StatusCat::Status.should_receive( :all ).and_return( [ @pass, @fail ] )
-      StatusCat::Status.failed.should eql( [ @fail ] )
+      expect(  StatusCat::Status ).to receive( :all ).and_return( [ @pass, @fail ] )
+      expect( StatusCat::Status.failed ).to eql( [ @fail ] )
     end
 
     it 'returns an empty list if all checkers pass' do
-      StatusCat::Status.should_receive( :all ).and_return( [ @pass ] )
-      StatusCat::Status.failed.should eql( [] )
+      expect( StatusCat::Status ).to receive( :all ).and_return( [ @pass ] )
+      expect( StatusCat::Status.failed ).to eql( [] )
     end
-
   end
-
 end
 

data/spec/lib/status_cat/version_spec.rb

@@ -1,11 +1,8 @@
-require 'spec_helper'
-
 describe 'version' do
 
   it 'has a version constant' do
-    StatusCat::VERSION.should_not be_nil
-    StatusCat::VERSION.should be_an_instance_of( String )
-    StatusCat::VERSION.should =~ /\d+\.\d+\.\d+/
+    expect( StatusCat::VERSION ).to_not be_nil
+    expect( StatusCat::VERSION ).to be_an_instance_of( String )
+    expect( StatusCat::VERSION ).to match( /\d+\.\d+\.\d+/ )
   end
-
 end

data/spec/lib/status_cat_spec.rb

@@ -1,21 +1,16 @@
-require 'spec_helper'
-
 describe StatusCat do
 
   describe '::config' do
 
     it 'returns the configuration' do
-      StatusCat.config.should be( StatusCat::Config.instance )
+      expect( StatusCat.config ).to be( StatusCat::Config.instance )
     end
-
   end
 
   describe '::configure' do
 
     it 'yields the configuration' do
-      StatusCat.configure { |config| config.should be( StatusCat::Config.instance ) }
+      StatusCat.configure { |config| expect( config ).to be( StatusCat::Config.instance ) }
     end
-
   end
-
 end

data/spec/lib/tasks/status_cat.rake_spec.rb

@@ -1,4 +1,3 @@
-require 'spec_helper'
 require 'rake'
 
 describe 'status_cat rake tasks' do
@@ -13,20 +12,16 @@ describe 'status_cat rake tasks' do
   describe 'rake status_cat:check' do
 
     it 'puts the status report' do
-      Kernel.should_receive( :puts ).with( kind_of( String ) )
+      expect( Kernel ).to receive( :puts ).with( kind_of( String ) )
       @rake[ 'status_cat:check' ].invoke
     end
-
   end
 
   describe 'rake status_cat:cron' do
 
     it 'calls StatusCat::Status.cron' do
-      StatusCat::Status.should_receive( :cron )
+      expect( StatusCat::Status ).to receive( :cron )
       @rake[ 'status_cat:cron' ].invoke
     end
-
   end
-
-
 end

data/spec/mailers/status_cat/status_mailer_spec.rb

@@ -1,10 +1,8 @@
-require 'spec_helper'
-
 describe StatusCat::StatusMailer do
 
   it 'adds StatusCat::StatusHelper as a template helper' do
     modules = StatusCat::StatusMailer._helpers.included_modules
-    modules.should include( StatusCat::StatusHelper )
+    expect( modules ).to include( StatusCat::StatusHelper )
   end
 
   describe '#failure' do
@@ -13,26 +11,24 @@ describe StatusCat::StatusMailer do
     let( :config ) { StatusCat.config }
 
     it 'uses the configured from address' do
-      mail.from.should_not be_nil
-      mail.from.should eql( [ config.from ] )
+      expect( mail.from ).to_not be_nil
+      expect( mail.from ).to eql( [ config.from ] )
     end
 
     it 'uses the configured to address' do
-      mail.to.should_not be_nil
-      mail.to.should eql( [ config.to ] )
+      expect( mail.to ).to_not be_nil
+      expect( mail.to ).to eql( [ config.to ] )
     end
 
     it 'uses the configured subject' do
-      mail.subject.should_not be_nil
-      mail.subject.should eql( config.subject )
+      expect( mail.subject ).to_not be_nil
+      expect( mail.subject ).to eql( config.subject )
     end
 
     it 'generates a multipart email with both text and html' do
-      mail.parts.first.content_type.should eql( 'text/plain; charset=UTF-8' )
-      mail.parts.second.content_type.should eql( 'text/html; charset=UTF-8' )
+      expect( mail.parts.first.content_type ).to eql( 'text/plain; charset=UTF-8' )
+      expect( mail.parts.second.content_type ).to eql( 'text/html; charset=UTF-8' )
     end
-
   end
-
 end
 

data/spec/spec_helper.rb

@@ -16,10 +16,14 @@ end
 
 require File.expand_path('../dummy/config/environment', __FILE__)
 require 'rspec/rails'
-require 'rspec/autorun'
 
 require 'spec_cat'
 
+require 'fitgem'
+require 'httparty'
+require 'ruby-sendhub'
+require 'twilio-ruby'
+
 ENGINE_ROOT = File.join(File.dirname(__FILE__), '..')
 
 # Requires supporting ruby files with custom matchers and macros, etc,
@@ -47,10 +51,26 @@ RSpec.configure do |config|
   # automatically. This will be the default behavior in future versions of
   # rspec-rails.
   config.infer_base_class_for_anonymous_controllers = false
+  config.infer_spec_type_from_file_location!
 
   # Run specs in random order to surface order dependencies. If you find an
   # order dependency and want to debug it, you can fix the order by providing
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = 'random'
+
+  config.before( :each ) do
+    allow( Fitgem::Client ).to receive( :new ).and_return( @fitgem = double( Fitgem::Client ) )
+    allow( @fitgem ).to receive( :user_info ).and_return( {} )
+
+    allow( HTTParty ).to receive( :get ).and_return( double( "response", :code => 200 ) )
+
+    allow( SendHub ).to receive( :new ).and_return( @send_hub = double( SendHub ) )
+    allow( @send_hub ).to receive( :get_contacts ).and_return( {} )
+
+    allow( Twilio::REST::Client ).to receive( :new ).and_return( @twilio = double( Twilio::REST::Client ) )
+    allow( @twilio ).to receive( :account ).and_return( @twilio_account = double( Twilio::REST::Account ) )
+    allow( @twilio_account ).to receive( :messages ).and_return( @twilio_messages = double( Twilio::REST::Messages ) )
+    allow( @twilio_messages ).to receive( :total ).and_return( 0 )
+  end
 end

data/spec/support/shared/checker.rb

@@ -1,25 +1,23 @@
 shared_examples_for 'a status checker' do
 
   it 'inherits from StatusCat::Checker' do
-    checker.should be_a_kind_of( StatusCat::Checkers::Base )
+    expect( checker ).to be_a_kind_of( StatusCat::Checkers::Base )
   end
 
   describe 'attributes' do
 
     it 'has a name reader' do
       expected = checker.class.to_s.split( '::' ).last.underscore.to_sym
-      checker.name.should eql( expected )
+      expect( checker.name ).to eql( expected )
     end
 
     it 'has a value reader' do
-      checker.value.should eql( checker.value )
+      expect( checker.value ).to eql( checker.value )
     end
 
     it 'has a status reader' do
-      checker.status.should eql( checker.status )
+      expect( checker.status ).to eql( checker.status )
     end
-
   end
-
 end
 

data/spec/views/status_cat/status/index.html.erb_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe 'status_cat/status/index.html.erb' do
 
   before( :each ) do
@@ -9,16 +7,16 @@ describe 'status_cat/status/index.html.erb' do
 
   it 'includes an h1 tag' do
     render
-    rendered.should have_selector( :h1, :content => t( :h1, :scope => :status_cat )  )
+    expect( rendered ).to match( /<h1>#{t( :h1, :scope => :status_cat )}<\/h1>/ )
   end
 
   it 'uses the status_title helper'  do
-    view.should_receive( :status_title )
+    expect( view ).to receive( :status_title )
     render
   end
 
   it 'uses the status_table helper'  do
-    view.should_receive( :status_table ).with( @checkers )
+    expect( view ).to receive( :status_table ).with( @checkers )
     render
   end
 

data/spec/views/status_cat/status_mailer/failure.html.erb_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe 'status_cat/status_mailer/failure.html.erb' do
 
   before( :each ) do
@@ -8,7 +6,7 @@ describe 'status_cat/status_mailer/failure.html.erb' do
   end
 
   it 'uses the status_table helper'  do
-    view.should_receive( :status_table ).with( @checkers )
+    expect( view ).to receive( :status_table ).with( @checkers )
     render
   end
 

data/spec/views/status_cat/status_mailer/failure.text.erb_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe 'status_cat/status_mailer/failure.text.erb' do
 
   before( :each ) do
@@ -8,8 +6,7 @@ describe 'status_cat/status_mailer/failure.text.erb' do
   end
 
   it 'uses the status_table helper'  do
-    view.should_receive( :status_report ).with( @checkers )
+    expect( view ).to receive( :status_report ).with( @checkers )
     render
   end
-
 end