From 0cc6c46fe9711d2377ff1ae6c55a03b3d1267874 Mon Sep 17 00:00:00 2001
From: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri, 19 Nov 2010 10:42:33 -0800
Subject: [PATCH] testing a non-invertible migration case

---
 .../test/cases/invertable_migration_test.rb   | 26 ++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/activerecord/test/cases/invertable_migration_test.rb b/activerecord/test/cases/invertable_migration_test.rb
index ab08cf6fe2..b4c1dccb22 100644
--- a/activerecord/test/cases/invertable_migration_test.rb
+++ b/activerecord/test/cases/invertable_migration_test.rb
@@ -2,16 +2,28 @@ require "cases/helper"
 
 module ActiveRecord
   class InvertableMigrationTest < ActiveRecord::TestCase
-    class InvertableMigration < ActiveRecord::Migration
+    class SilentMigration < ActiveRecord::Migration
+      def write(text = '')
+        # sssshhhhh!!
+      end
+    end
+
+    class InvertableMigration < SilentMigration
       def change
         create_table("horses") do |t|
           t.column :content, :text
           t.column :remind_at, :datetime
         end
       end
+    end
 
-      def write(text = '')
-        # sssshhhhh!!
+    class NonInvertableMigration < SilentMigration
+      def change
+        create_table("horses") do |t|
+          t.column :content, :text
+          t.column :remind_at, :datetime
+        end
+        remove_column "horses", :content
       end
     end
 
@@ -21,6 +33,14 @@ module ActiveRecord
       end
     end
 
+    def test_no_reverse
+      migration = NonInvertableMigration.new
+      migration.migrate(:up)
+      assert_raises(IrreversibleMigration) do
+        migration.migrate(:down)
+      end
+    end
+
     def test_up
       migration = InvertableMigration.new
       migration.migrate(:up)