diff options
author | Dino Maric <dino.onex@gmail.com> | 2017-08-08 16:47:38 +0200 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2017-08-08 09:47:38 -0500 |
commit | 4b72bee47bb590efac498ad2494efa769350aabd (patch) | |
tree | fbaf8f950f14bcaff4fd43b5ccd02c957cd161a1 /activestorage | |
parent | 6089b3140dedc087d80d67fe4c60f280db9e8d6e (diff) | |
download | rails-4b72bee47bb590efac498ad2494efa769350aabd.tar.gz rails-4b72bee47bb590efac498ad2494efa769350aabd.tar.bz2 rails-4b72bee47bb590efac498ad2494efa769350aabd.zip |
ActiveStorage:Add migrations per rails engine conventions (#30111)
* Add migrations per rails engine conventions
* Fix failing tests
Diffstat (limited to 'activestorage')
-rw-r--r-- | activestorage/db/migrate/20170806125915_create_active_storage_tables.rb (renamed from activestorage/lib/active_storage/migration.rb) | 2 | ||||
-rw-r--r-- | activestorage/lib/tasks/activestorage.rake | 11 | ||||
-rw-r--r-- | activestorage/test/database/setup.rb | 3 |
3 files changed, 4 insertions, 12 deletions
diff --git a/activestorage/lib/active_storage/migration.rb b/activestorage/db/migrate/20170806125915_create_active_storage_tables.rb index 2e35e163cd..6eab7e0fa0 100644 --- a/activestorage/lib/active_storage/migration.rb +++ b/activestorage/db/migrate/20170806125915_create_active_storage_tables.rb @@ -1,4 +1,4 @@ -class ActiveStorageCreateTables < ActiveRecord::Migration[5.1] +class CreateActiveStorageTables < ActiveRecord::Migration[5.1] def change create_table :active_storage_blobs do |t| t.string :key diff --git a/activestorage/lib/tasks/activestorage.rake b/activestorage/lib/tasks/activestorage.rake index 1d386e67df..d9e240b141 100644 --- a/activestorage/lib/tasks/activestorage.rake +++ b/activestorage/lib/tasks/activestorage.rake @@ -1,13 +1,6 @@ -require "fileutils" - namespace :activestorage do desc "Copy over the migration needed to the application" - task :install do - migration_file_path = "db/migrate/#{Time.now.utc.strftime("%Y%m%d%H%M%S")}_active_storage_create_tables.rb" - FileUtils.mkdir_p Rails.root.join("db/migrate") - FileUtils.cp File.expand_path("../../active_storage/migration.rb", __FILE__), Rails.root.join(migration_file_path) - puts "Copied migration to #{migration_file_path}" - - puts "Now run rails db:migrate to create the tables for Active Storage" + task install: :environment do + Rake::Task["active_storage_engine:install:migrations"].invoke end end diff --git a/activestorage/test/database/setup.rb b/activestorage/test/database/setup.rb index b12038ee68..610b927cc3 100644 --- a/activestorage/test/database/setup.rb +++ b/activestorage/test/database/setup.rb @@ -1,6 +1,5 @@ -require "active_storage/migration" require_relative "create_users_migration" ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:") -ActiveStorageCreateTables.migrate(:up) +ActiveRecord::Migrator.migrate File.expand_path("../../../db/migrate", __FILE__) ActiveStorageCreateUsers.migrate(:up) |