fides 0.0.10 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a2f7ad27b8fcf4775b3d93dbf0b81e92189d35cb
-  data.tar.gz: 80f98cbff22f202f192d171085026d1ffe334c71
+  metadata.gz: 92258eb65e77c3d3fd6881caa7fa5037e9a502d0
+  data.tar.gz: 688bada85c59802fa3d7bb442389f6c3ff47009a
 SHA512:
-  metadata.gz: b68a7524614d75f1632852a822fa042167d08a3b0cb8984e4f231fd2fd313a97b304992d8a827a16b63e70e0fb21159d1679af769aa130345fa3dfe5360dd522
-  data.tar.gz: 1e7488197348b392d03897830d57b4097ca9349d61fcd676ad1610282ef576fff53d096cdd06bc727d9499ba1f6b1992f2f6e138d5667afdb9db6c7159cf65a3
+  metadata.gz: 24558f5c56565abfe349078dab792678d820569b83430287f56408e159b544fd95ad423eaf71e5baa644dcd3b9885373049de7c4b31a2dc54358c460ff7f7cc8
+  data.tar.gz: 8d0d5c243156a0bd7a3301260097351e0c6b77c215a15fba32dc45ff3009fd43b3a7c84097d114e2f04877be6d1d3c5f09d50fce6a2bdb34afd34cf09b5a852f

data/README.md

@@ -1,15 +1,18 @@
 # Fides
 
-This gem adds helper methods for use in Rails migrations that creates SQL triggers to enforce the 
-integrity of polymorphic associations at the database level.
+Adds methods for use in Rails migrations to enforce polymorphic associations at the database level.
 
-When an attempt is made to create or update a record in a polymorphic table the SQL trigger checks for
-the existance of the referred-to model of the specified type in the other table. If it doesn't exist in 
-it throws a descriptive SQL exception.
+### Longer Description
 
-Similarly, when an attempt is made to delete a record from a table that could be pointed to from the
-polymorphic table, a SQL trigger checks that there are no references to it in the polymorphic table. If
-there are it throws a SQL exception suggesting changing the delete order.
+The methods that this gem exposes are for use in Rails migrations and create SQL Triggers to enforce
+the data integrity of Polymorphic Associations.
+
+The triggers are invoked by the database before an insert or update on the polymorphic table, and before 
+a delete on tables referred to by the polymorphic table.
+
+If an insert/update is attempted on the polymorphic table with a record that refers to a non-existent 
+record in another table then a SQL error is raised. If a delete is attempted from a table that is 
+referred to by a record in the polymorphic table then a SQL error is raised.
 
 ## Installation
 
@@ -63,12 +66,12 @@ If you're using Rails < version 3.1, then use Fides in your migration like this:
     
     end
 
-Fides assumes the use of Rails conventions, so if there's something that needs overriding just open a
-bug or shoot me a pull request.
-
 ## Caveats
 
-Fides currently only functions with PostgreSQL. Please feel free to contribute other adapters.
+Fides assumes the use of Rails conventions, so if you find a case for something that needs overriding, 
+please feel free to submit a bug or send a pull request.
+
+Fides currently only functions with PostgreSQL. Please feel free to contribute other adapters as desired.
 
 ## Contributing
 

data/Rakefile

@@ -5,7 +5,7 @@ require 'rake/testtask'
  
 Rake::TestTask.new do |t|
   t.libs << 'lib/fides'
-  t.test_files = FileList['test/lib/fides/*_test.rb']
+  t.test_files = FileList['test/lib/*_test.rb', 'test/lib/fides/*_test.rb']
   t.verbose = true
 end
  

data/fides.gemspec

@@ -21,6 +21,7 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency "activesupport"
   spec.add_dependency "activerecord"
+  spec.add_dependency "rails"
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"

data/lib/fides/database_adapter.rb

@@ -0,0 +1,15 @@
+module Fides
+  module SqlWriter
+
+    def self.included(base)
+      base.extend(ClassMethods)
+    end
+
+    module ClassMethods
+      def strip_non_essential_spaces(string_to_strip)
+        return string_to_strip.gsub(/\s{2,}|\\n/, " ").strip
+      end
+    end
+
+  end
+end

data/lib/fides/database_adapter_error.rb

@@ -0,0 +1,7 @@
+module Fides
+  class DatabaseAdapterError < StandardError
+    def initialize(adapter)
+      super("Fides doesn't work with the #{adapter} database adapter.")
+    end
+  end
+end

data/lib/fides/postgresql.rb

@@ -0,0 +1,95 @@
+module Fides
+
+  class Postgresql
+    include SqlWriter
+
+    def self.get_create_function_sql(interface_name, models, polymorphic_model)
+
+      sql = "DROP FUNCTION IF EXISTS check_#{interface_name}_create_integrity() CASCADE;"
+
+      sql << %{
+        CREATE FUNCTION check_#{interface_name}_create_integrity() RETURNS TRIGGER AS '
+          BEGIN
+            IF NEW.#{interface_name}_type = ''#{models[0]}'' AND EXISTS (
+                SELECT id FROM #{models[0].constantize.table_name} WHERE id = NEW.#{interface_name}_id) THEN
+              RETURN NEW;
+      }
+
+      models[1..-1].each do |model|
+        sql << %{
+          ELSEIF NEW.#{interface_name}_type = ''#{model}'' AND EXISTS (
+              SELECT id FROM #{model.constantize.table_name} WHERE id = NEW.#{interface_name}_id) THEN
+            RETURN NEW;
+        }
+      end
+      
+      sql << %{
+            ELSE
+              RAISE EXCEPTION ''No % model with id %.'', NEW.#{interface_name}_type, NEW.#{interface_name}_id;
+              RETURN NULL;
+            END IF;
+          END'
+          LANGUAGE plpgsql;
+
+          CREATE TRIGGER check_#{interface_name}_create_integrity_trigger 
+            BEFORE INSERT OR UPDATE ON #{polymorphic_model.constantize.table_name} 
+            FOR EACH ROW EXECUTE PROCEDURE check_#{interface_name}_create_integrity();
+      }
+
+      return strip_non_essential_spaces(sql)
+    end
+
+    def self.get_delete_function_sql(interface_name, models, polymorphic_model)
+      polymorphic_model_table_name = polymorphic_model.constantize.table_name
+      
+      sql = ""
+      sql << %{
+        CREATE FUNCTION check_#{interface_name}_delete_integrity() RETURNS TRIGGER AS '
+          BEGIN
+            IF TG_TABLE_NAME = ''#{models[0].constantize.table_name}'' AND EXISTS (
+              SELECT id FROM #{polymorphic_model_table_name} 
+              WHERE #{interface_name}_type = ''#{models[0]}'' AND #{interface_name}_id = OLD.id) THEN
+              RAISE EXCEPTION ''There are records in #{polymorphic_model_table_name} that refer to %. You must delete those records first.'', OLD;
+      }
+
+      models[1..-1].each do |model|
+        sql << %{
+          ELSEIF TG_TABLE_NAME = ''#{model.constantize.table_name}'' AND EXISTS (
+            SELECT id FROM #{polymorphic_model_table_name} 
+            WHERE #{interface_name}_type = ''#{model}'' AND #{interface_name}_id = OLD.id) THEN
+              RAISE EXCEPTION ''There are records in #{polymorphic_model_table_name} that refer to %. You must delete those records first.'', OLD;
+        }
+      end
+
+      sql << %{
+            ELSE
+              RETURN NULL;
+            END IF;
+          END'
+        LANGUAGE plpgsql;
+      }
+
+      models.each do |model|
+        table_name = model.constantize.table_name
+        
+        sql << %{
+          CREATE TRIGGER check_#{table_name}_delete_integrity_trigger 
+            BEFORE DELETE ON #{table_name} 
+            FOR EACH ROW EXECUTE PROCEDURE check_#{interface_name}_delete_integrity();
+        }
+      end
+
+      return strip_non_essential_spaces(sql)
+    end
+
+    def self.get_drop_function_sql(interface_name)
+      sql = %{
+        DROP FUNCTION IF EXISTS check_#{interface_name}_create_integrity() CASCADE;
+        DROP FUNCTION IF EXISTS check_#{interface_name}_delete_integrity() CASCADE;
+      }
+      return strip_non_essential_spaces(sql)
+    end
+
+  end #class
+
+end #module

data/lib/fides/sql_writer.rb

@@ -0,0 +1,15 @@
+module Fides
+  module SqlWriter
+
+    def self.included(base)
+      base.extend(ClassMethods)
+    end
+
+    module ClassMethods
+      def strip_non_essential_spaces(string_to_strip)
+        return string_to_strip.gsub(/\s{2,}|\\n/, " ").strip
+      end
+    end
+
+  end
+end

data/lib/fides/sqlite3.rb

@@ -0,0 +1,18 @@
+require 'fides/sql_writer'
+
+module Fides
+
+  class Sqlite3
+    include SqlWriter
+
+    def self.get_create_function_sql(interface_name, models, polymorphic_model)
+      raise "Sqlite3 not yet implemented"
+    end
+
+    def self.get_delete_function_sql(interface_name, models, polymorphic_model)
+      raise "Sqlite3 not yet implemented"
+    end
+
+  end
+
+end

data/lib/fides/version.rb

@@ -1,3 +1,3 @@
 module Fides
-  VERSION = "0.0.10"
+  VERSION = "0.0.11"
 end

data/lib/fides.rb

@@ -1,120 +1,60 @@
-require "fides/version"
-require 'active_support/concern'
+require 'rails'
 require 'active_record'
+require 'active_support/concern'
 require 'active_support/inflector'
+require 'fides/version'
+require 'fides/sql_writer'
+require 'fides/postgresql'
+require 'fides/sqlite3'
+require 'fides/database_adapter_error'
 
 module Fides
 
+  SUPPORTED_ADAPTERS = ["postgresql"]
+
   extend ActiveSupport::Concern
 
   def add_polymorphic_triggers(opts)
     raise ArgumentError, "missing :associated_models from options hash" if !opts.has_key?(:associated_models) 
     raise ArgumentError, "missing :polymorphic_model from options hash" if !opts.has_key?(:polymorphic_model)
+
     associated_models = opts[:associated_models]
     polymorphic_model = opts[:polymorphic_model]
-    interface_name = opts.has_key?(:interface_name) ? opts[:interface_name] : interface_name(polymorphic_model)
+    interface = opts.has_key?(:interface_name) ? opts[:interface_name] : interface_name(polymorphic_model)
 
-    sql = get_create_function_sql(interface_name, associated_models, polymorphic_model)
-    sql << get_delete_function_sql(interface_name, associated_models, polymorphic_model)
+    fides_sql_generator = get_sql_generator_class
+
+    sql = fides_sql_generator.get_create_function_sql(interface, associated_models, polymorphic_model)
+    sql << fides_sql_generator.get_delete_function_sql(interface, associated_models, polymorphic_model)
 
     execute sql
   end
 
   def remove_polymorphic_triggers(opts)
     raise ArgumentError, "missing :polymorphic_model from options hash" if !opts.has_key?(:polymorphic_model)
+    
     polymorphic_model = opts[:polymorphic_model]
-    interface_name = opts.has_key?(:interface_name) ? opts[:interface_name] : interface_name(polymorphic_model)
+    interface = opts.has_key?(:interface_name) ? opts[:interface_name] : interface_name(polymorphic_model)
 
-    execute %{
-      DROP FUNCTION IF EXISTS check_#{interface_name}_create_integrity() CASCADE;
-      DROP FUNCTION IF EXISTS check_#{interface_name}_delete_integrity() CASCADE;
-    }
+    sql = get_sql_generator_class.get_drop_function_sql(interface)
+
+    execute sql
   end
 
   private
 
+  def get_sql_generator_class
+    db_adapter = ActiveRecord::Base.configurations[Rails.env]['adapter']
+    raise DatabaseAdapterError.new(db_adapter) unless SUPPORTED_ADAPTERS.include?(db_adapter)
+    return "Fides::#{db_adapter.capitalize}".constantize
+  end
+
   # TODO: Is it safe to just grab the first polymorphic association?
   def interface_name(model_name)
     model_name.constantize.reflect_on_all_associations.select { |r| r if r.options[:polymorphic] }.first.name
   end
 
-  def get_create_function_sql(interface_name, models, polymorphic_model)
-    sql = "DROP FUNCTION IF EXISTS check_#{interface_name}_create_integrity() CASCADE;"
-    
-    sql << %{
-      CREATE FUNCTION check_#{interface_name}_create_integrity() RETURNS TRIGGER AS '
-        BEGIN
-          IF NEW.#{interface_name}_type = ''#{models[0]}'' AND EXISTS (
-              SELECT id FROM #{models[0].constantize.table_name} WHERE id = NEW.#{interface_name}_id) THEN
-            RETURN NEW;
-    }
-
-    models[1..-1].each do |model|
-      sql << %{
-        ELSEIF NEW.#{interface_name}_type = ''#{model}'' AND EXISTS (
-            SELECT id FROM #{model.constantize.table_name} WHERE id = NEW.#{interface_name}_id) THEN
-          RETURN NEW;
-      }
-    end
-    
-    sql << %{
-          ELSE
-            RAISE EXCEPTION ''No % model with id %.'', NEW.#{interface_name}_type, NEW.#{interface_name}_id;
-            RETURN NULL;
-          END IF;
-        END'
-        LANGUAGE plpgsql;
-
-        CREATE TRIGGER check_#{interface_name}_create_integrity_trigger 
-          BEFORE INSERT OR UPDATE ON #{polymorphic_model.constantize.table_name} 
-          FOR EACH ROW EXECUTE PROCEDURE check_#{interface_name}_create_integrity();
-    }
-
-    return sql
-  end
-
-  def get_delete_function_sql(interface_name, models, polymorphic_model)
-    polymorphic_model_table_name = polymorphic_model.constantize.table_name
-    
-    sql = ""
-    sql << %{
-      CREATE FUNCTION check_#{interface_name}_delete_integrity() RETURNS TRIGGER AS '
-        BEGIN
-          IF TG_TABLE_NAME = ''#{models[0].constantize.table_name}'' AND EXISTS (
-            SELECT id FROM #{polymorphic_model_table_name} 
-            WHERE #{interface_name}_type = ''#{models[0]}'' AND #{interface_name}_id = OLD.id) THEN
-            RAISE EXCEPTION ''There are records in #{polymorphic_model_table_name} that refer to %. You must delete those records first.'', OLD;
-    }
-
-    models[1..-1].each do |model|
-      sql << %{
-        ELSEIF TG_TABLE_NAME = ''#{model.constantize.table_name}'' AND EXISTS (
-          SELECT id FROM #{polymorphic_model_table_name} 
-          WHERE #{interface_name}_type = ''#{model}'' AND #{interface_name}_id = OLD.id) THEN
-            RAISE EXCEPTION ''There are records in #{polymorphic_model_table_name} that refer to %. You must delete those records first.'', OLD;
-      }
-    end
-
-    sql << %{
-          ELSE
-            RETURN NULL;
-          END IF;
-        END'
-      LANGUAGE plpgsql;
-    }
-
-    models.each do |model|
-      table_name = model.constantize.table_name
-      
-      sql << %{
-        CREATE TRIGGER check_#{table_name}_delete_integrity_trigger 
-          BEFORE DELETE ON #{table_name} 
-          FOR EACH ROW EXECUTE PROCEDURE check_#{interface_name}_delete_integrity();
-      }
-    end
-
-    return sql
-  end
+ 
 end
 
 class ActiveRecord::Migration

data/test/lib/fides/database_adapter_test.rb

@@ -0,0 +1,4 @@
+require_relative '../../test_helper'
+
+describe Fides::SqlWriter do
+end

data/test/lib/fides/postgresql_test.rb

@@ -0,0 +1,78 @@
+require_relative '../../test_helper'
+ 
+describe Fides::Postgresql do
+
+  let(:subject) { Fides::Postgresql }
+
+  it 'responds to #get_create_function_sql' do
+    assert_respond_to subject, :get_create_function_sql
+  end
+
+  it 'responds to #get_delete_function_sql' do
+    assert_respond_to subject, :get_delete_function_sql
+  end
+
+  it 'returns an expected SQL string' do
+    sql = subject.get_create_function_sql("imageable", ["Product", "Employee"], "Picture")
+    assert_equal subject.strip_non_essential_spaces(CREATE_SQL), sql
+  end
+
+  it 'returns an expected SQL string' do
+    sql = subject.get_delete_function_sql("imageable", ["Product", "Employee"], "Picture")
+    assert_equal subject.strip_non_essential_spaces(DELETE_SQL), sql 
+  end
+
+  CREATE_SQL = %{
+
+    DROP FUNCTION IF EXISTS check_imageable_create_integrity() CASCADE;
+
+    CREATE FUNCTION check_imageable_create_integrity() RETURNS TRIGGER AS '
+      BEGIN
+        IF NEW.imageable_type = ''Product'' AND EXISTS (
+          SELECT id FROM products WHERE id = NEW.imageable_id) THEN
+          RETURN NEW;
+        ELSEIF NEW.imageable_type = ''Employee'' AND EXISTS (
+          SELECT id FROM employees WHERE id = NEW.imageable_id) THEN
+          RETURN NEW;
+        ELSE
+          RAISE EXCEPTION ''No % model with id %.'', NEW.imageable_type, NEW.imageable_id;
+          RETURN NULL;
+        END IF;
+      END'
+    LANGUAGE plpgsql;
+
+    CREATE TRIGGER check_imageable_create_integrity_trigger
+      BEFORE INSERT OR UPDATE ON pictures
+      FOR EACH ROW EXECUTE PROCEDURE check_imageable_create_integrity();
+
+  }
+
+  DELETE_SQL = %{
+
+    CREATE FUNCTION check_imageable_delete_integrity() RETURNS TRIGGER AS '
+      BEGIN
+        IF TG_TABLE_NAME = ''products'' AND EXISTS (
+          SELECT id FROM pictures
+          WHERE imageable_type = ''Product'' AND imageable_id = OLD.id) THEN
+          RAISE EXCEPTION ''There are records in pictures that refer to %. You must delete those records first.'', OLD;
+        ELSEIF TG_TABLE_NAME = ''employees'' AND EXISTS (
+          SELECT id FROM pictures
+          WHERE imageable_type = ''Employee'' AND imageable_id = OLD.id) THEN
+          RAISE EXCEPTION ''There are records in pictures that refer to %. You must delete those records first.'', OLD;
+        ELSE
+          RETURN NULL;
+        END IF;
+      END'
+    LANGUAGE plpgsql;
+
+    CREATE TRIGGER check_products_delete_integrity_trigger
+      BEFORE DELETE ON products
+      FOR EACH ROW EXECUTE PROCEDURE check_imageable_delete_integrity();
+
+    CREATE TRIGGER check_employees_delete_integrity_trigger
+      BEFORE DELETE ON employees
+      FOR EACH ROW EXECUTE PROCEDURE check_imageable_delete_integrity();
+
+  }
+
+end

data/test/lib/fides_test.rb

@@ -0,0 +1,82 @@
+require_relative '../test_helper'
+ 
+describe Fides do
+
+  before do
+    class Picture < ActiveRecord::Base
+      def self.reflect_on_all_associations
+        [MyTestAssociaiton.new]
+      end
+    end
+    class Product < ActiveRecord::Base; end
+    class Employee < ActiveRecord::Base; end
+    class MyTestMigration < ActiveRecord::Migration
+      def execute(blah)
+      end
+    end
+    class MyTestAssociaiton
+      def options
+        {:polymorphic => true}
+      end
+      def name
+        @imageable
+      end
+    end
+    @my_test_migration = MyTestMigration.new
+  end
+
+  it "responds to add_polymorphic_triggers" do
+    assert_respond_to @my_test_migration, :add_polymorphic_triggers
+  end
+
+  it "responds to remove_polymorphic_triggers" do
+    assert_respond_to @my_test_migration, :remove_polymorphic_triggers
+  end
+
+  it "includes the ability to use of the constantize method" do
+    assert_equal "Picture".constantize, Picture
+  end
+
+  it "raises and exception if :associated_models isn't a parameter of #add_polymorphic_triggers" do
+    exception = assert_raises(ArgumentError) { 
+      @my_test_migration.add_polymorphic_triggers(:polymorphic_model => "Picture") 
+    }
+    assert_match /associated_models/, exception.message
+  end
+
+  it "raises and exception if :polymorphic_model isn't a parameter of #add_polymorphic_triggers" do
+    exception = assert_raises(ArgumentError) { 
+      @my_test_migration.add_polymorphic_triggers(:associated_models => ["Product", "Employee"]) 
+    }
+    assert_match /polymorphic_model/, exception.message
+  end
+
+  it "raise an error if the database adapter isn't supported" do
+    Rails.stub :env, "development" do
+      ActiveRecord::Base.stub :configurations, { "development" => { "adapter" => "fakedb" } } do
+        exception = assert_raises(Fides::DatabaseAdapterError) { 
+          @my_test_migration.add_polymorphic_triggers(
+            :polymorphic_model => "Picture", 
+            :associated_models => ["Product", "Employee"]
+          )
+        }
+        assert_match /fakedb/, exception.message
+      end
+    end
+  end
+
+  it "runs silently" do
+    assert_silent do
+      Rails.stub :env, "development" do
+        ActiveRecord::Base.stub :configurations, { "development" => { "adapter" => "postgresql" } } do
+          @my_test_migration.add_polymorphic_triggers(
+            :polymorphic_model => "Picture", 
+            :associated_models => ["Product", "Employee"],
+            :interface_name => "imageable"
+          )
+        end
+      end
+    end
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fides
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors:
 - Martin Kraft
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-28 00:00:00.000000000 Z
+date: 2013-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -38,6 +38,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -108,9 +122,16 @@ files:
 - Rakefile
 - fides.gemspec
 - lib/fides.rb
+- lib/fides/database_adapter.rb
+- lib/fides/database_adapter_error.rb
+- lib/fides/postgresql.rb
+- lib/fides/sql_writer.rb
+- lib/fides/sqlite3.rb
 - lib/fides/version.rb
-- test/lib/fides/fides_test.rb
+- test/lib/fides/database_adapter_test.rb
+- test/lib/fides/postgresql_test.rb
 - test/lib/fides/version_test.rb
+- test/lib/fides_test.rb
 - test/test_helper.rb
 homepage: https://github.com/mkraft/fides
 licenses:
@@ -138,6 +159,8 @@ specification_version: 4
 summary: Creates SQL triggers from Rails migrations to enforce the integrity of  polymorphic
   associations at the database level.
 test_files:
-- test/lib/fides/fides_test.rb
+- test/lib/fides/database_adapter_test.rb
+- test/lib/fides/postgresql_test.rb
 - test/lib/fides/version_test.rb
+- test/lib/fides_test.rb
 - test/test_helper.rb

data/test/lib/fides/fides_test.rb

@@ -1,42 +0,0 @@
-require_relative '../../test_helper'
- 
-describe Fides do
-
-  before do
-    class Picture < ActiveRecord::Base; end
-    class MyTestMigration < ActiveRecord::Migration; end
-    class MyTestAssociaiton; end
-    @my_test_association = MyTestAssociaiton.new
-    @my_test_migration = MyTestMigration.new
-  end
-
-  it "responds to add_polymorphic_triggers" do
-    assert_respond_to @my_test_migration, :add_polymorphic_triggers
-  end
-
-  it "responds to remove_polymorphic_triggers" do
-    assert_respond_to @my_test_migration, :remove_polymorphic_triggers
-  end
-
-  it "includes the ability to use of the constantize method" do
-
-    assert_equal "Picture".constantize, Picture
-  end
-
-
-  it "raises and exception if :associated_models isn't a parameter of #add_polymorphic_triggers" do
-    exception = assert_raises(ArgumentError) { 
-      MyTestMigration.add_polymorphic_triggers(:polymorphic_model => "Picture") 
-    }
-    assert_match /associated_models/, exception.message
-  end
-
-  it "raises and exception if :polymorphic_model isn't a parameter of #add_polymorphic_triggers" do
-    exception = assert_raises(ArgumentError) { 
-      MyTestMigration.add_polymorphic_triggers(:associated_models => ["Product", "Employee"]) 
-    }
-    assert_match /polymorphic_model/, exception.message
-  end
-
- 
-end