table_warnings 0.0.7 → 1.0.0

data/test/test_warn_if_nonexistent_owner_except.rb

@@ -0,0 +1,67 @@
+require 'helper'
+
+class Gente < ActiveRecord::Base
+  col :llc_name
+end
+Gente.auto_upgrade!
+
+class PetRojo < ActiveRecord::Base
+  col :handler_id, :type => :integer
+  col :trainer_id
+  belongs_to :handler, :class_name => 'Gente'
+  belongs_to :trainer, :class_name => 'Gente', :primary_key => :llc_name
+  warn_if_nonexistent_owner_except :handler
+end
+PetRojo.auto_upgrade!
+
+class PetAzul < ActiveRecord::Base
+  col :handler_id, :type => :integer
+  col :trainer_id
+  belongs_to :handler, :class_name => 'Gente'
+  belongs_to :trainer, :class_name => 'Gente', :primary_key => :llc_name
+  warn_if_nonexistent_owner_except :trainer
+end
+PetAzul.auto_upgrade!
+
+class PetVerde < ActiveRecord::Base
+  col :handler_id, :type => :integer
+  col :trainer_id
+  belongs_to :handler, :class_name => 'Gente'
+  belongs_to :trainer, :class_name => 'Gente', :primary_key => :llc_name
+  warn_if_nonexistent_owner_except :trainer, :allow_null => true
+end
+PetVerde.auto_upgrade!
+
+describe TableWarnings do
+  describe :warn_if_nonexistent_owner_except do
+    before do
+      Gente.force_create! :llc_name => 'My Small Business, LLC'
+    end
+    it "takes a single belongs-to association name" do
+      assert_causes_warning PetRojo, /nonexistent.*trainer/i do
+        PetRojo.force_create!
+      end
+    end
+    it "checks the value of foreign keys not just their presence" do
+      assert_causes_warning PetRojo, /nonexistent.*trainer/i do
+        PetRojo.force_create! :trainer_id => 999999
+      end
+    end
+    it "doesn't raise false warnings" do
+      assert_does_not_cause_warning PetRojo do
+        PetRojo.force_create! :trainer_id => Gente.first.llc_name
+      end
+    end
+    it "regards nulls as nonexistent even if the association primary key column contains nulls" do
+      assert_causes_warning PetAzul, /nonexistent.*handler/i do
+        PetAzul.force_create!
+      end
+    end
+    it "allows nulls if explicitly requested" do
+      assert_does_not_cause_warning PetVerde do
+        PetVerde.force_create!
+      end
+    end
+
+  end
+end

data/test/test_warn_if_nulls_except.rb

@@ -0,0 +1,120 @@
+require 'helper'
+
+class PetA < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls_except :birthday
+end
+PetA.auto_upgrade!
+
+class PetB < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls_except :birthday, :gender
+end
+PetB.auto_upgrade!
+
+class PetC < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls_except /irthda/
+end
+PetC.auto_upgrade!
+
+class PetD < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls_except /irthda/, /ende/
+end
+PetD.auto_upgrade!
+
+class PetE < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :certified, :type => :boolean
+  warn_if_nulls_except :birthday, :conditions => { :certified => true }
+end
+PetE.auto_upgrade!
+
+class PetF < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :sire
+  col :certified, :type => :boolean
+  warn_if_nulls_except :birthday, :gender, :conditions => { :certified => true }
+end
+PetF.auto_upgrade!
+
+class PetG < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :certified, :type => :boolean
+  warn_if_nulls_except /irthda/, :conditions => { :certified => true }
+end
+PetG.auto_upgrade!
+
+class PetH < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :sire
+  col :certified, :type => :boolean
+  warn_if_nulls_except /irthda/, /ende/, :conditions => { :certified => true }
+end
+PetH.auto_upgrade!
+
+describe TableWarnings do
+  describe :warn_if_nulls_except do
+    it "takes a single column" do
+      assert_causes_warning PetA, /null.*gender/i do
+        PetA.force_create!
+      end
+    end
+    it "takes multiple columns" do
+      assert_does_not_cause_warning PetB do
+        PetB.force_create!
+      end
+    end
+    it "takes a single regexp" do
+      assert_causes_warning PetC, /null.*gender/i do
+        PetC.force_create!
+      end
+    end
+    it "takes multiple regexps" do
+      assert_does_not_cause_warning PetD do
+        PetD.force_create!
+      end
+    end
+    it "takes a single column and a condition" do
+      assert_does_not_cause_warning PetE do
+        PetE.force_create!
+      end
+      assert_causes_warning PetE, /null.*gender/i do
+        PetE.force_create! :certified => true
+      end
+    end
+    it "takes multiple columns and a condition" do
+      assert_does_not_cause_warning PetF do
+        PetF.force_create!
+      end
+      assert_causes_warning PetF, /null.*sire/i do
+        PetF.force_create! :certified => true
+      end
+    end
+    it "takes a single regexp and a condition" do
+      assert_does_not_cause_warning PetG do
+        PetG.force_create!
+      end
+      assert_causes_warning PetG, /null.*gender/i do
+        PetG.force_create! :certified => true
+      end
+    end
+    it "takes multiple regexps and a condition" do
+      assert_does_not_cause_warning PetH do
+        PetH.force_create!
+      end
+      assert_causes_warning PetH, /null.*sire/i do
+        PetH.force_create! :certified => true
+      end
+    end
+  end
+end

data/test/test_warn_if_nulls_in.rb

@@ -0,0 +1,114 @@
+require 'helper'
+
+class Pet1 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  warn_if_nulls :birthday
+end
+Pet1.auto_upgrade!
+
+class Pet2 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls :birthday, :gender
+end
+Pet2.auto_upgrade!
+
+class Pet3 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  warn_if_nulls /irthda/
+end
+Pet3.auto_upgrade!
+
+class Pet4 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  warn_if_nulls /irthda/, /ende/
+end
+Pet4.auto_upgrade!
+
+class Pet5 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :certified, :type => :boolean
+  warn_if_nulls :birthday, :conditions => { :certified => true }
+end
+Pet5.auto_upgrade!
+
+class Pet6 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :certified, :type => :boolean
+  warn_if_nulls :birthday, :gender, :conditions => { :certified => true }
+end
+Pet6.auto_upgrade!
+
+class Pet7 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :certified, :type => :boolean
+  warn_if_nulls /irthda/, :conditions => { :certified => true }
+end
+Pet7.auto_upgrade!
+
+class Pet8 < ActiveRecord::Base
+  col :birthday, :type => :datetime
+  col :gender
+  col :certified, :type => :boolean
+  warn_if_nulls /irthda/, /ende/, :conditions => { :certified => true }
+end
+Pet8.auto_upgrade!
+
+describe TableWarnings do
+  describe :warn_if_nulls do
+    it "takes a single column" do
+      assert_causes_warning Pet1, /null.*birthday/i do
+        Pet1.force_create!
+      end
+    end
+    it "takes multiple columns" do
+      assert_causes_warning Pet2, [/null.*birthday/i, /null.*gender/i] do
+        Pet2.force_create!
+      end
+    end
+    it "takes a single regexp" do
+      assert_causes_warning Pet3, /null.*birthday/i do
+        Pet3.force_create!
+      end
+    end
+    it "takes multiple regexps" do
+      assert_causes_warning Pet4, [/null.*birthday/i, /null.*gender/i] do
+        Pet4.force_create!
+      end
+    end
+    it "takes a single column and a condition" do
+      assert_does_not_cause_warning Pet5 do
+        Pet5.force_create!
+      end
+      assert_causes_warning Pet5, /null.*birthday/i do
+        Pet5.force_create! :certified => true
+      end
+    end
+    it "takes multiple columns and a condition" do
+      assert_does_not_cause_warning Pet6 do
+        Pet6.force_create!
+      end
+      assert_causes_warning Pet6, [/null.*birthday/i, /null.*gender/i] do
+        Pet6.force_create! :certified => true
+      end
+    end
+    it "takes a single regexp and a condition" do
+      assert_does_not_cause_warning Pet7 do
+        Pet7.force_create!
+      end
+      assert_causes_warning Pet7, /null.*birthday/i do
+        Pet7.force_create! :certified => true
+      end
+    end
+    it "takes multiple regexps and a condition" do
+      assert_does_not_cause_warning Pet8 do
+        Pet8.force_create!
+      end
+      assert_causes_warning Pet8, [/null.*birthday/i, /null.*gender/i] do
+        Pet8.force_create! :certified => true
+      end
+    end
+  end
+end

data/test/test_warn_unless_range.rb

@@ -0,0 +1,18 @@
+require 'helper'
+
+class PetUno < ActiveRecord::Base
+  col :age, :type => :integer
+  # sad!
+  warn_unless_range :age, :min => 0, :max => 20
+end
+PetUno.auto_upgrade!
+
+describe TableWarnings do
+  describe :warn_unless_range do
+    it "takes a single column and range" do
+      assert_causes_warning PetUno, /range.*age/i do
+        PetUno.force_create! :age => 100
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: table_warnings
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 1.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-04 00:00:00.000000000Z
+date: 2012-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &2154337840 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2154337840
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &2154337420 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +37,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2154337420
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: fastercsv
-  requirement: &2154337000 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +53,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2154337000
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
-  requirement: &2154336580 !ruby/object:Gem::Requirement
+  name: active_record_inline_schema
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +69,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2154336580
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: mini_record-compat
-  requirement: &2154336160 !ruby/object:Gem::Requirement
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +85,31 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2154336160
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: sqlite3
-  requirement: &2154335740 !ruby/object:Gem::Requirement
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-reporters
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,7 +117,28 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2154335740
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Validate an entire [ActiveRecord] table, checking for things like blank
   rows or total number of rows
 email:
@@ -86,19 +148,31 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- CHANGELOG
 - Gemfile
 - README.rdoc
 - Rakefile
 - lib/table_warnings.rb
-- lib/table_warnings/config.rb
+- lib/table_warnings/arbitrary.rb
+- lib/table_warnings/blank.rb
+- lib/table_warnings/column.rb
+- lib/table_warnings/debug.rb
+- lib/table_warnings/exclusive.rb
+- lib/table_warnings/nonexistent_owner.rb
+- lib/table_warnings/null.rb
+- lib/table_warnings/range.rb
+- lib/table_warnings/registry.rb
+- lib/table_warnings/scout.rb
+- lib/table_warnings/size.rb
 - lib/table_warnings/version.rb
-- lib/table_warnings/warning.rb
-- lib/table_warnings/warning/arbitrary.rb
-- lib/table_warnings/warning/blank.rb
-- lib/table_warnings/warning/size.rb
 - table_warnings.gemspec
 - test/helper.rb
-- test/test_table_warnings.rb
+- test/test_combinations.rb
+- test/test_warn_if_nonexistent_owner.rb
+- test/test_warn_if_nonexistent_owner_except.rb
+- test/test_warn_if_nulls_except.rb
+- test/test_warn_if_nulls_in.rb
+- test/test_warn_unless_range.rb
 homepage: https://github.com/seamusabshere/table_warnings
 licenses: []
 post_install_message: 
@@ -119,11 +193,17 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: table_warnings
-rubygems_version: 1.8.6
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: It's called validations to remind people of per-record validations, but it
   works on the table, and is meant to be used after a table is completely populated.
 test_files:
 - test/helper.rb
-- test/test_table_warnings.rb
+- test/test_combinations.rb
+- test/test_warn_if_nonexistent_owner.rb
+- test/test_warn_if_nonexistent_owner_except.rb
+- test/test_warn_if_nulls_except.rb
+- test/test_warn_if_nulls_in.rb
+- test/test_warn_unless_range.rb
+has_rdoc: