Ticket #10855: drop_table_if_exists_with_test.diff

activerecord/test/cases/migration_test.rb

@@ -181 +181 @@
-      Person.connection.drop_table :testings rescue nil
-    end
-
+    # PostgreSQL returns an error when dropping a table that doesn't exist,
+    # unless you use <tt>IF EXISTS</tt>. <tt>:force => true</tt> uses this syntax.
+    if current_adapter?(:PostgreSQLAdapter)
+      def test_drop_table_force
+        assert_nothing_raised do
+          Person.connection.drop_table :does_not_exist, :force => true
+        end
+      end
+    end
+
-    # SQL Server, Sybase, and SQLite3 will not allow you to add a NOT NULL
-    # column to a table without a default value.
-    unless current_adapter?(:SQLServerAdapter, :SybaseAdapter, :SQLiteAdapter)

activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb

@@ -587 +587 @@
-        execute "ALTER TABLE #{name} RENAME TO #{new_name}"
-      end
-
+      # Drops a table from the database.
+      def drop_table(table_name, options = {})
+        args = options[:force] ? " IF EXISTS" : ""
+        execute "DROP TABLE#{args} #{quote_table_name(table_name)}"
+      end
+
-      # Adds a new column to the named table.
-      # See TableDefinition#column for details of the options you can use.
-      def add_column(table_name, column_name, type, options = {})