From bea3524e3f5a56e4dc87ae240a2c58301e73a128 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jos=C3=A9=20Valim?= Date: Sat, 24 Dec 2011 21:57:42 +0100 Subject: Last attempt to fix 1.8.7 tests. --- railties/lib/rails/generators/generated_attribute.rb | 2 +- railties/test/generators/migration_generator_test.rb | 13 +++++++++---- railties/test/generators/model_generator_test.rb | 8 +++++--- 3 files changed, 15 insertions(+), 8 deletions(-) diff --git a/railties/lib/rails/generators/generated_attribute.rb b/railties/lib/rails/generators/generated_attribute.rb index 964bd6280c..cfd58527ef 100644 --- a/railties/lib/rails/generators/generated_attribute.rb +++ b/railties/lib/rails/generators/generated_attribute.rb @@ -96,7 +96,7 @@ module Rails end def inject_options - @attr_options.blank? ? '' : ", #{@attr_options.to_s.gsub(/[{}]/, '')}" + "".tap { |s| @attr_options.each { |k,v| s << ", :#{k} => #{v.inspect}" } } end def inject_index_options diff --git a/railties/test/generators/migration_generator_test.rb b/railties/test/generators/migration_generator_test.rb index d81fc59fbf..8f352d1ea6 100644 --- a/railties/test/generators/migration_generator_test.rb +++ b/railties/test/generators/migration_generator_test.rb @@ -109,10 +109,15 @@ class MigrationGeneratorTest < Rails::Generators::TestCase assert_migration "db/migrate/#{migration}.rb" do |content| assert_method :change, content do |up| - assert_match(/add_column :books, :title, :string, :limit=>40/, up) - assert_match(/add_column :books, :content, :string, :limit=>255/, up) - assert_match(/add_column :books, :price, :decimal, :precision=>5, :scale=>2/, up) - assert_match(/add_column :books, :discount, :decimal, :precision=>3, :scale=>2/, up) + assert_match(/add_column :books, :title, :string, :limit => 40/, up) + assert_match(/add_column :books, :content, :string, :limit => 255/, up) + assert_match(/add_column :books, :price, :decimal, :precision => 5, :scale => 2/, up) + assert_match(/add_column :books, :price, :decimal,/, up) + assert_match(/, :precision => 5/, up) + assert_match(/, :scale => 2/, up) + assert_match(/add_column :books, :discount, :decimal,/, up) + assert_match(/, :precision => 3/, up) + assert_match(/, :scale => 2/, up) end assert_match(/add_index :books, :title/, content) assert_match(/add_index :books, :price/, content) diff --git a/railties/test/generators/model_generator_test.rb b/railties/test/generators/model_generator_test.rb index ffb3d2ceba..68c1015cbc 100644 --- a/railties/test/generators/model_generator_test.rb +++ b/railties/test/generators/model_generator_test.rb @@ -171,9 +171,11 @@ class ModelGeneratorTest < Rails::Generators::TestCase assert_migration "db/migrate/create_products.rb" do |content| assert_method :change, content do |up| assert_match(/create_table :products/, up) - assert_match(/t.string :title, :limit=>40/, up) - assert_match(/t.string :content, :limit=>255/, up) - assert_match(/t.decimal :price, :precision=>5, :scale=>2/, up) + assert_match(/t.string :title, :limit => 40/, up) + assert_match(/t.string :content, :limit => 255/, up) + assert_match(/t.decimal :price,/, up) + assert_match(/, :precision => 5/, up) + assert_match(/, :scale => 2/, up) end assert_match(/add_index :products, :title/, content) assert_match(/add_index :products, :price/, content) -- cgit v1.2.3