status_cat 0.0.8 → 0.0.9

data/spec/dummy/tmp/pids/server.pid

@@ -1 +1 @@
-7028
+58006

data/spec/helpers/status_cat/status_helper_spec.rb

@@ -33,6 +33,20 @@ describe StatusCat::StatusHelper do
 
   end
 
+  describe '#status_cell' do
+
+    it 'returns the status as a table cell' do
+      expected = "<td>foo</td>"
+      expect( helper.status_cell( 'foo' ) ).to eql( expected )
+    end
+
+    it 'converts array statuses into html lists' do
+      expected = "<td><ul><li>foo</li><li>bar</li></ul></td>"
+      expect( helper.status_cell( [ 'foo', 'bar' ] ) ).to eql( expected )
+    end
+
+  end
+
   describe '#status_style' do
 
     it 'returns a green background when the status is nil' do
@@ -69,6 +83,8 @@ describe StatusCat::StatusHelper do
 
     it 'generates a text status report' do
       all = StatusCat::Status.all
+      all.each { |s| s.stub( :status ).and_return( nil ) }
+
       helper.status_report( all ).should eql_file( 'spec/data/report.txt' )
     end
 
@@ -77,7 +93,10 @@ describe StatusCat::StatusHelper do
   describe '#status_report_format' do
 
     it 'generates a format string and length based on the max length of the given checkers' do
-      status_report_format = helper.status_report_format( StatusCat::Status.all ).to_s
+      all = StatusCat::Status.all
+      all.each { |s| s.stub( :status ).and_return( nil ) }
+
+      status_report_format = helper.status_report_format( all ).to_s
       status_report_format.should eql_file( 'spec/data/status_report_format.txt' )
     end
 

data/spec/lib/status_cat/checkers/action_mailer_spec.rb

@@ -59,7 +59,7 @@ describe StatusCat::Checkers::ActionMailer do
 
       it 'returns an error message if it can not send a message' do
         smtp = Object.new
-        smtp.stub!( :send_message ).and_raise( exception )
+        smtp.stub( :send_message ).and_raise( exception )
         Net::SMTP.should_receive( :start ).and_yield( smtp )
         checker = StatusCat::Checkers::ActionMailer.new
         checker.status.should be( exception )

data/spec/lib/status_cat/checkers/active_record_spec.rb

@@ -6,14 +6,10 @@ describe StatusCat::Checkers::ActiveRecord do
 
   it_should_behave_like 'a status checker'
 
-  it 'provides configuration' do
-    yaml =  YAML::load( ERB.new( IO.read( File.join( Rails.root, 'config', 'database.yml' ) ) ).result )
-    expected = yaml[ Rails.env ].symbolize_keys!
-    checker.config.should eql( expected )
-  end
-
   it 'constructs a value from the configuration' do
-    expected = "#{checker.config[ :adapter ]}:#{checker.config[ :username ]}@#{checker.config[ :database ]}"
+    config = { :adapter => 'postgres', :username => 'dba', :database => 'test' }
+    expect( ::ActiveRecord::Base ).to receive( :connection_config ).and_return( config )
+    expected = "#{config[ :adapter ]}:#{config[ :username ]}@#{config[ :database ]}"
     checker.value.should eql( expected )
   end
 
@@ -22,7 +18,7 @@ describe StatusCat::Checkers::ActiveRecord do
     context 'pass' do
 
       it 'passes if it can execute a query against the database' do
-        ActiveRecord::Base.connection.stub!( :execute )
+        ActiveRecord::Base.connection.stub( :execute )
         checker = StatusCat::Checkers::ActiveRecord.new
         checker.status.should be_nil
       end

data/spec/lib/status_cat/checkers/base_spec.rb

@@ -24,8 +24,8 @@ describe StatusCat::Checkers::Base do
   describe '#to_s' do
 
     before( :each ) do
-      checker.stub!( :value ).and_return( 'secret' )
-      checker.stub!( :status ).and_return( 'fail' )
+      checker.stub( :value ).and_return( 'secret' )
+      checker.stub( :status ).and_return( 'fail' )
     end
 
     it 'generates a string' do

data/spec/lib/status_cat/checkers/delayed_job_spec.rb

@@ -1,4 +1,5 @@
 require 'spec_helper'
+require 'delayed_job_active_record'
 
 describe StatusCat::Checkers::DelayedJob do
 
@@ -10,6 +11,30 @@ describe StatusCat::Checkers::DelayedJob do
 
   it_should_behave_like 'a status checker'
 
-  it 'should be specced better'
+  it 'tolerates Delayed::Job being undefined' do
+    dj = Object.send(:remove_const, :Delayed)
+    expect( checker.status ).to_not be_nil
+    Object.const_set( :Delayed, dj )
+  end
+
+  it 'fails if there is an exception' do
+    ActiveRecord::Base.connection.should_receive( :execute ).and_raise( :error )
+    expect( checker.status ).to_not be_nil
+  end
+
+  it 'fails if there are expired jobs' do
+    ActiveRecord::Base.connection.should_receive( :execute ).and_return( [1], [1] )
+    expect( checker.status ).to_not be_nil
+  end
+
+  it 'passes if there are no expired jobs' do
+    ActiveRecord::Base.connection.should_receive( :execute ).and_return( [1], [0] )
+    expect( checker.status ).to be_nil
+  end
+
+  it 'uses the job count as the value' do
+    ActiveRecord::Base.connection.should_receive( :execute ).and_return( [1], [0] )
+    expect( checker.value ).to eql( 1 )
+  end
 
 end

data/spec/lib/status_cat/checkers/s3_spec.rb

@@ -1,15 +1,39 @@
 require 'spec_helper'
+require 'aws-sdk'
 
 describe StatusCat::Checkers::S3 do
 
   let( :checker ) { StatusCat::Checkers::S3.new.freeze }
 
   before( :each ) do
-
+    AWS::S3.stub( :new ).and_return( @s3 = double( AWS::S3 ) )
   end
 
   it_should_behave_like 'a status checker'
 
-  it 'should be specced better'
+  it 'tolerates AWS being undefined' do
+    aws = Object.send(:remove_const, :AWS)
+    expect( checker.status ).to_not be_nil
+    Object.const_set( :AWS, aws )
+  end
+
+  it 'uses the aws access key as the value' do
+    expect( checker.value ).to eql( AWS.config.access_key_id )
+  end
+
+  it 'fails if there is an exception talking to S3' do
+    expect( @s3 ).to receive( :buckets ).and_raise( 'error' )
+    expect( checker.status ).to_not be_nil
+  end
+
+  it 'fails if there are no S3 buckets' do
+      expect( @s3 ).to receive( :buckets ).and_return( [] )
+      expect( checker.status ).to_not be_nil
+    end
+
+  it 'passes if it finds S3 buckets' do
+    expect( @s3 ).to receive( :buckets ).and_return( [ 1 ] )
+    expect( checker.status ).to be_nil
+  end
 
 end

data/spec/lib/status_cat/checkers/stripe_spec.rb

@@ -1,15 +1,44 @@
 require 'spec_helper'
+require 'stripe'
 
 describe StatusCat::Checkers::Stripe do
 
   let( :checker ) { StatusCat::Checkers::Stripe.new.freeze }
+  let( :email ) { 'foo@bar.com' }
 
   before( :each ) do
-
+    Stripe::Account.stub( :retrieve ).and_return( @stripe = double( Stripe::Account ) )
   end
 
   it_should_behave_like 'a status checker'
 
-  it 'should be specced better'
+  it 'tolerates Stripe being undefined' do
+    stripe = Object.send(:remove_const, :Stripe)
+    expect( checker.status ).to_not be_nil
+    Object.const_set( :Stripe, stripe )
+  end
+
+  it 'fails if there is an exception talking to Stripe' do
+    expect( @stripe ).to receive( :email ).and_raise( 'error' )
+    expect( checker.status ).to_not be_nil
+  end
+
+  it 'fails if charging is not enabled' do
+    expect( @stripe ).to receive( :charge_enabled ).and_return( false )
+    expect( @stripe ).to receive( :email ).and_return( email )
+    expect( checker.status ).to_not be_nil
+  end
+
+  it 'passes if charging is enabled' do
+    expect( @stripe ).to receive( :charge_enabled ).and_return( true )
+    expect( @stripe ).to receive( :email ).and_return( email )
+    expect( checker.status ).to be_nil
+  end
+
+  it 'uses the account email as the value' do
+    expect( @stripe ).to receive( :charge_enabled ).and_return( true )
+    expect( @stripe ).to receive( :email ).and_return( email )
+    expect( checker.value ).to eql( email )
+  end
 
 end

data/spec/lib/status_cat/config_spec.rb

@@ -17,7 +17,7 @@ describe StatusCat::Config do
     end
 
     it 'has an #authorize accessor' do
-      config.authorize.should_not be_nil
+      config.authorize = StatusCat::Config::NIL_PROC
       config.authorize = config.authorize
       config.authorize.should eql( config.authorize )
     end

data/spec/lib/status_cat/status_spec.rb

@@ -101,7 +101,7 @@ describe StatusCat::Status do
       @pass = StatusCat::Checkers::Base.new
 
       @fail = StatusCat::Checkers::Base.new
-      @fail.stub!( :status ).and_return( :fail )
+      @fail.stub( :status ).and_return( :fail )
     end
 
     it 'returns only failed checkers from ::all' do

data/spec/spec_helper.rb

@@ -2,6 +2,12 @@
 ENV['RAILS_ENV'] ||= 'test'
 
 require 'simplecov'
+require 'coveralls'
+
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter
+]
 
 SimpleCov.start 'rails' do
   add_filter '/vendor/'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: status_cat
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors:
 - Rich Humphrey
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-19 00:00:00.000000000 Z
+date: 2014-04-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -110,6 +110,106 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.7.3
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.8.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.8.2
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.7.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.7.0
+- !ruby/object:Gem::Dependency
+  name: spec_cat
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.0
+- !ruby/object:Gem::Dependency
+  name: aws-sdk
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.38'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.38.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.38'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.38.0
+- !ruby/object:Gem::Dependency
+  name: delayed_job_active_record
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 4.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 4.0.0
 description: |2
       This engine makes monitoring the status of your Rails environment easier.
 
@@ -269,7 +369,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: A Rails engine for checking system health