affairs_of_state 0.1.2 → 0.1.3

data/.travis.yml

@@ -2,4 +2,5 @@ language: ruby
 rvm:
   - "1.9.3"
   - "2.0.0"
+  - "2.1.0"
 script: bundle exec rspec

data/lib/affairs_of_state/version.rb

@@ -1,3 +1,3 @@
 module AffairsOfState
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/spec/affairs_of_state_spec.rb

@@ -8,53 +8,51 @@ describe AffairsOfState do
     end
 
     it "should set the constant" do
-      Pie::STATUSES.should == ["active", "inactive", "cancelled"]
+      expect(Pie::STATUSES).to eq(["active", "inactive", "cancelled"])
     end
 
     it "should validate the column is set" do
-      p = Pie.new :status => nil
-      p.should_not be_valid
+      expect(Pie.new(status: nil)).to_not be_valid
     end
 
     it "should validate that we're not setting it to something stupid" do
-      p = Pie.new :status => "delicious_pie"
-      p.should_not be_valid
+      expect(Pie.new(status: "delicious_pie")).to_not be_valid
     end
 
     describe "boolean methods" do
       it "should find the set status" do
-        p = Pie.new :status => "active"
-        p.active?.should be_true
+        p = Pie.new status: "active"
+        expect(p.active?).to be_truthy
       end
 
       it "should not find if a different status is set" do
-        p = Pie.new :status => "inactive"
-        p.cancelled?.should be_false
+        p = Pie.new status: "inactive"
+        expect(p.cancelled?).to be_falsy
       end
     end
 
     describe "update methods" do
       it "should set the value" do
-        p = Pie.create! :status => "active"
-        p.inactive!.should be_true
-        p.status.should == "inactive"
+        p = Pie.create! status: "active"
+        expect(p.inactive!).to be_truthy
+        expect(p.status).to eq("inactive")
       end
     end
 
     it "should provide a method to pass to dropdowns" do
-      Pie.statuses_for_select.should == [["Active", "active"], ["Inactive", "inactive"], ["Cancelled", "cancelled"]]
+      expect(Pie.statuses_for_select).to eq([["Active", "active"], ["Inactive", "inactive"], ["Cancelled", "cancelled"]])
     end
 
     describe "scopes" do
       it "should have a finder to match the status name" do
-        Pie.create! :status => "active"
-        Pie.create! :status => "inactive"
-        Pie.create! :status => "active"
-        Pie.create! :status => "cancelled"
-
-        Pie.active.size.should == 2
-        Pie.inactive.size.should == 1
-        Pie.cancelled.size.should == 1
+        Pie.create! status: "active"
+        Pie.create! status: "inactive"
+        Pie.create! status: "active"
+        Pie.create! status: "cancelled"
+
+        expect(Pie.active.size).to eq(2)
+        expect(Pie.inactive.size).to eq(1)
+        expect(Pie.cancelled.size).to eq(1)
       end
     end
 
@@ -67,12 +65,11 @@ describe AffairsOfState do
     class Pie2 < ActiveRecord::Base
       self.table_name = "pies"
 
-      affairs_of_state :active, :inactive, :column => :super_status
+      affairs_of_state :active, :inactive, column: :super_status
     end
 
     it "should validate the column is set" do
-      p = Pie2.new :status => nil, :super_status => "active"
-      p.should be_valid
+      expect(Pie2.new(status: nil, super_status: "active")).to be_valid
     end
   end
 
@@ -80,12 +77,11 @@ describe AffairsOfState do
     class Pie3 < ActiveRecord::Base
       self.table_name = "pies"
 
-      affairs_of_state :active, :inactive, :allow_blank => true
+      affairs_of_state :active, :inactive, allow_blank: true
     end
 
     it "should validate the column is set" do
-      p = Pie3.new :status => nil
-      p.should be_valid
+      expect(Pie3.new(status: nil)).to be_valid
     end
   end
 
@@ -97,7 +93,7 @@ describe AffairsOfState do
     end
 
     it "should work too if that's what floats your boat" do
-      Pie4::STATUSES.should == ["on", "off"]
+      expect(Pie4::STATUSES).to eq(["on", "off"])
     end
   end
 
@@ -105,11 +101,11 @@ describe AffairsOfState do
     class Pie5 < ActiveRecord::Base
       self.table_name = "pies"
 
-      affairs_of_state :active, :inactive, :scopes => false
+      affairs_of_state :active, :inactive, scopes: false
     end
 
     it "should work too if that's what floats your boat" do
-      Pie5.should_not respond_to(:active)
+      expect(Pie5).to_not respond_to(:active)
     end
   end
 
@@ -117,7 +113,7 @@ describe AffairsOfState do
     class Pie6 < ActiveRecord::Base
       self.table_name = "pies"
 
-      affairs_of_state :active, :inactive, :if => lambda{|p| p.is_going_to_validate }
+      affairs_of_state :active, :inactive, if: lambda{|p| p.is_going_to_validate }
 
       attr_accessor :is_going_to_validate
     end
@@ -126,14 +122,14 @@ describe AffairsOfState do
       p = Pie6.new
       p.is_going_to_validate = true
       p.status = "pie"
-      p.should_not be_valid
+      expect(p).to_not be_valid
     end
 
     it "should not enforce the validation if the :if param evaluates to false" do
       p = Pie6.new
       p.is_going_to_validate = false
       p.status = "pie"
-      p.should be_valid
+      expect(p).to be_valid
     end
   end
 
@@ -141,7 +137,7 @@ describe AffairsOfState do
     class Pie7 < ActiveRecord::Base
       self.table_name = "pies"
 
-      affairs_of_state :active, :inactive, :if => :validation_method?
+      affairs_of_state :active, :inactive, if: :validation_method?
 
       attr_accessor :is_going_to_validate
 
@@ -154,14 +150,14 @@ describe AffairsOfState do
       p = Pie7.new
       p.is_going_to_validate = true
       p.status = "pie"
-      p.should_not be_valid
+      expect(p).to_not be_valid
     end
 
     it "should not enforce the validation if the :if param evaluates to false" do
       p = Pie7.new
       p.is_going_to_validate = false
       p.status = "pie"
-      p.should be_valid
+      expect(p).to be_valid
     end
   end
 

data/spec/spec_helper.rb

@@ -3,12 +3,12 @@ require 'affairs_of_state'
 require 'pry'
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
 end
 
 
 ## Create an AR model to test with
+I18n.enforce_available_locales = false
 
 ActiveRecord::Base.establish_connection(
   :adapter => "sqlite3",

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: affairs_of_state
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2014-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &81631330 !ruby/object:Gem::Requirement
+  requirement: &78218490 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>'
@@ -21,10 +21,10 @@ dependencies:
         version: '3.0'
   type: :runtime
   prerelease: false
-  version_requirements: *81631330
+  version_requirements: *78218490
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &81631020 !ruby/object:Gem::Requirement
+  requirement: &78218080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *81631020
+  version_requirements: *78218080
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &81630660 !ruby/object:Gem::Requirement
+  requirement: &78217850 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *81630660
+  version_requirements: *78217850
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: &81630350 !ruby/object:Gem::Requirement
+  requirement: &78217630 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *81630350
+  version_requirements: *78217630
 description: Add a simple state to a gem, without all the hassle of a complex state
   machine.
 email: