diff options
author | Matthew Draper <matthew@trebex.net> | 2017-12-03 04:19:27 +1030 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-03 04:19:27 +1030 |
commit | 6a7787218b8d75913b58d7bed198495e8d29e34e (patch) | |
tree | ec324f35f9dd2d839016579855c2dac0a6dde90a /activerecord/test/cases/migration | |
parent | 4c4d092431f45471f90badb81eaa2011843cc39b (diff) | |
parent | dd6338a0699f2301d4b2fc8653688b4c4183cee5 (diff) | |
download | rails-6a7787218b8d75913b58d7bed198495e8d29e34e.tar.gz rails-6a7787218b8d75913b58d7bed198495e8d29e34e.tar.bz2 rails-6a7787218b8d75913b58d7bed198495e8d29e34e.zip |
Merge pull request #31230 from dinahshi/postgresql_extract_sql
Extract sql fragment generators from PostgreSQL adapter
Diffstat (limited to 'activerecord/test/cases/migration')
-rw-r--r-- | activerecord/test/cases/migration/compatibility_test.rb | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/activerecord/test/cases/migration/compatibility_test.rb b/activerecord/test/cases/migration/compatibility_test.rb index 2fef2f796e..0fd55e6ae4 100644 --- a/activerecord/test/cases/migration/compatibility_test.rb +++ b/activerecord/test/cases/migration/compatibility_test.rb @@ -126,6 +126,23 @@ module ActiveRecord end assert_match(/LegacyMigration < ActiveRecord::Migration\[4\.2\]/, e.message) end + + if current_adapter?(:PostgreSQLAdapter) + class Testing < ActiveRecord::Base + end + + def test_legacy_change_column_with_null_executes_update + migration = Class.new(ActiveRecord::Migration[5.1]) { + def migrate(x) + change_column :testings, :foo, :string, null: false, default: "foobar" + end + }.new + + t = Testing.create! + ActiveRecord::Migrator.new(:up, [migration]).migrate + assert_equal ["foobar"], Testing.all.map(&:foo) + end + end end end end |