migrant 1.1.2 → 1.2.0

data/Gemfile

@@ -11,4 +11,5 @@ group :development do
   gem "sqlite3"
   gem "simplecov"
   gem "terminal-table"
+  gem "term-ansicolor"
 end

data/Gemfile.lock

@@ -71,6 +71,7 @@ GEM
       simplecov-html (~> 0.4.0)
     simplecov-html (0.4.3)
     sqlite3 (1.3.3)
+    term-ansicolor (1.0.5)
     terminal-table (1.4.2)
     thor (0.14.6)
     thoughtbot-shoulda (2.11.1)
@@ -90,6 +91,7 @@ DEPENDENCIES
   rails (>= 3.0.0)
   simplecov
   sqlite3
+  term-ansicolor
   terminal-table
   thoughtbot-shoulda
   turn

data/VERSION

@@ -1 +1 @@
-1.1.2
+1.2.0

data/lib/dsl/data_type.rb

@@ -21,6 +21,11 @@ module DataType
     def column
       {:type => :string}.merge(@options)
     end
+            
+    def ==(compared_column)
+      # Ideally we should compare attributes, but unfortunately not all drivers report enough statistics for this
+      column[:type] == compared_column[:type]
+    end
 
     def mock
       @value || self.class.default_mock

data/lib/generators/templates/change_migration.erb

@@ -6,17 +6,37 @@ class <%= @activity.camelize.gsub(/\s/, '') %> < ActiveRecord::Migration
   <% @changed_columns.each do |field, options, old_options| %>
     change_column :<%= @table_name %>, :<%= field %>, :<%= options.delete(:type) %><%= (options.blank?)? '': ", "+options.inspect[1..-2] %>
   <% end -%>
+  <% @transferred_columns.each do |source, target| %>
+    puts "-- copy data from :<%= source %> to :<%= target %>"
+    <%= @table_name.classify %>.update_all("<%= target %> = <%= source %>")
+  <% end -%>  
+  <% @renamed_columns.each do |old_name, new_name| %>
+    rename_column :<%= @table_name %>, :<%= old_name %>, :<%= new_name %>
+  <% end -%>
+  <% @deleted_columns.each do |field, options| %>
+    remove_column :<%= @table_name %>, :<%= field %>
+  <% end -%>  
   <% @indexes.each do |index, options| %>
     add_index :<%= @table_name %>, <%= index.inspect %>
   <% end -%>
   end
   
   def self.down
-  <% @added_columns.each do |field, options| %>
-    remove_column :<%= @table_name %>, :<%= field %>
-  <% end -%>
+  <% @deleted_columns.each do |field, options| %>
+    add_column :<%= @table_name %>, :<%= field %>, :<%= options.delete(:type) %><%= (options.blank?)? '': ", "+options.inspect[1..-2] %>
+  <% end -%>  
+  <% @renamed_columns.each do |old_name, new_name| %>
+    rename_column :<%= @table_name %>, :<%= new_name %>, :<%= old_name %>
+  <% end -%>    
+  <% @transferred_columns.each do |source, target| %>
+    puts "-- copy data from :<%= target %> to :<%= source %>"    
+    <%= @table_name.classify %>.update_all("<%= source %> = <%= target %>")
+  <% end -%>  
   <% @changed_columns.each do |field, options, old_options| %>
     change_column :<%= @table_name %>, :<%= field %>, :<%= old_options.delete(:type) %><%= (old_options.blank?)? '': ", "+old_options.inspect[1..-2] %>
   <% end -%>
+  <% @added_columns.each do |field, options| %>
+    remove_column :<%= @table_name %>, :<%= field %>
+  <% end -%>
   end
 end

data/lib/migrant/migration_generator.rb

@@ -1,12 +1,12 @@
 require 'erubis'
+require 'term/ansicolor'
 
 module Migrant
   class MigrationGenerator
-    TABS = '  ' # Tabs to spaces * 2
-    NEWLINE = "\n  "
     def run
       # Ensure db/migrate path exists before starting
       FileUtils.mkdir_p(Rails.root.join('db', 'migrate'))
+      @possible_irreversible_migrations = false
  
       migrator = ActiveRecord::Migrator.new(:up, migrations_path)
 
@@ -24,25 +24,52 @@ module Migrant
         next if model.schema.nil? || !model.schema.requires_migration? # Skips inherited schemas (such as models with STI)
         model.reset_column_information # db:migrate doesn't do this
         @table_name = model.table_name
+        @changed_columns, @added_columns, @deleted_columns, @renamed_columns, @transferred_columns = [], [], [], [], []    
 
         if model.table_exists?
           # Structure ActiveRecord::Base's column information so we can compare it directly to the schema
           db_schema = Hash[*model.columns.collect {|c| [c.name.to_sym, Hash[*[:type, :limit].map { |type| [type, c.send(type)] }.flatten]  ] }.flatten]
-          @changed_columns, @added_columns = [], []
           model.schema.columns.to_a.sort { |a,b| a.to_s <=> b.to_s }.each do |field_name, data_type|
             begin
               if (options = data_type.structure_changes_from(db_schema[field_name]))
                 if db_schema[field_name]
-                  @changed_columns << [field_name, options, db_schema[field_name]]
+                  change_column(field_name, options, db_schema[field_name])
                 else
-                  @added_columns << [field_name, options]
+                  add_column(field_name, options)
                 end
               end
             rescue DataType::DangerousMigration
-              puts "Cannot generate migration automatically for #{model.table_name}, this would involve possible data loss on column: #{field_name}\nOld structure: #{db_schema[field_name].inspect}. New structure: #{data_type.column.inspect}\nPlease create and run this migration yourself (with the appropriate data integrity checks)"
+              log "Cannot generate migration automatically for #{model.table_name}, this would involve possible data loss on column: #{field_name}\nOld structure: #{db_schema[field_name].inspect}. New structure: #{data_type.column.inspect}\nPlease create and run this migration yourself (with the appropriate data integrity checks)", :error
               return false
             end
           end
+          
+          # Removed rows
+          unless model.schema.partial?
+            db_schema.reject { |field_name, options| field_name.to_s == model.primary_key || model.schema.columns.keys.include?(field_name) }.each do |removed_field_name, options|
+              case ask_user("#{model}: '#{removed_field_name}' is no longer in use.", (@added_columns.blank?)? %W{Destroy Ignore} : %W{Destroy Move Ignore})
+                when 'Destroy' then delete_column(removed_field_name, db_schema[removed_field_name])
+                when 'Move' then
+                  target = ask_user("Move '#{removed_field_name}' to:", @added_columns.collect(&:first))
+                  target_column = model.schema.columns[target]
+                  begin
+                    target_column.structure_changes_from(db_schema[removed_field_name])
+                    move_column(removed_field_name, target, db_schema[removed_field_name], target_column)
+                  rescue DataType::DangerousMigration
+                    case ask_user("Unable to safely move '#{removed_field_name}' to '#{target}'. Keep the original column for now?", %W{Yes No}, true)
+                      when 'No' then delete_column(removed_field_name, db_schema[removed_field_name])
+                    end
+                  end
+              end
+            end 
+          end
+          destroyed_columns = @deleted_columns.reject { |field, options| @transferred_columns.collect(&:first).include?(field) }
+          unless destroyed_columns.blank?
+            if ask_user("#{model}: '#{destroyed_columns.collect(&:first).join(', ')}' and associated data will be DESTROYED in all environments. Continue?", %W{Yes No}, true) == 'No'
+              log "Okay, not removing anything for now."
+              @deleted_columns = []
+            end
+          end
 
           # For adapters that can report indexes, add as necessary
           if ActiveRecord::Base.connection.respond_to?(:indexes)
@@ -52,10 +79,11 @@ module Migrant
             @new_indexes = @indexes.reject { |index, options| @changed_columns.detect { |c| c.first == index } || @added_columns.detect { |c| c.first == index } }
           end
 
-          next if @changed_columns.empty? && @added_columns.empty? && @indexes.empty? # Nothing to do for this table
+          next if @changed_columns.empty? && @added_columns.empty? && @renamed_columns.empty? && @transferred_columns.empty? && @deleted_columns.empty? && @indexes.empty? # Nothing to do for this table
 
           # Example: changed_table_added_something_and_modified_something
-          @activity = 'changed_'+model.table_name+[['added', @added_columns], ['modified', @changed_columns], ['indexed', @new_indexes]].reject { |v| v[1].empty? }.collect { |v| "_#{v[0]}_"+v[1].collect(&:first).join('_') }.join('_and')
+          @activity = 'changed_'+model.table_name+[['added', @added_columns], ['modified', @changed_columns], ['deleted', destroyed_columns], 
+          ['moved', @transferred_columns], ['renamed', @renamed_columns], ['indexed', @new_indexes]].reject { |v| v[1].empty? }.collect { |v| "_#{v[0]}_"+v[1].collect(&:first).join('_') }.join('_and')
           @activity = @activity.split('_')[0..2].join('_') if @activity.length >= 240 # Most filesystems will raise Errno::ENAMETOOLONG otherwise
           
           render('change_migration')
@@ -69,16 +97,80 @@ module Migrant
        
         filename = "#{migrations_path}/#{next_migration_number}_#{@activity}.rb"
         File.open(filename, 'w') { |migration| migration.write(@output) }
-        puts "Wrote #{filename}..."
+        log "Wrote #{filename}..."
       end
+      
+      if @possible_irreversible_migrations
+        log "*** One or more move operations were performed, which potentially could cause data loss on db:rollback. \n*** Please review your migrations before committing!", :warning
+      end
+      
       true
     end
 
     private
+    def add_column(name, options)
+      @added_columns << [name, options]
+    end
+    
+    def change_column(name, new_schema, old_schema)
+      @changed_columns << [name, new_schema, old_schema]
+    end
+    
+    def delete_column(name, current_structure)
+      @deleted_columns << [name, current_structure]
+    end
+    
+    def move_column(old_name, new_name, old_schema, new_schema)
+      if new_schema == old_schema
+        @renamed_columns << [old_name, new_name]
+        @added_columns.reject! { |a| a.first == new_name } # Don't add the column too
+      else
+        @possible_irreversible_migrations = true
+        @transferred_columns << [old_name, new_name] # Still need to add the column, just transfer the data afterwards
+        delete_column(old_name, old_schema)
+      end
+    end
+    
     def migrations_path
       Rails.root.join(ActiveRecord::Migrator.migrations_path)
     end
-
+    
+    include Term::ANSIColor
+    def ask_user(message, choices, warning=false)
+      mappings = choices.uniq.inject({}) do |mappings, choice|
+        choice_string = choice.to_s
+        choice_string.length.times do |i|
+          mappings.merge!(choice_string[i..i] => choice) and break unless mappings.keys.include?(choice_string[i..i])
+        end
+        mappings.merge!(choice_string => choice) unless mappings.values.include?(choice)
+        mappings
+      end
+      
+      begin
+        message = "> #{message} [#{mappings.collect { |shortcut, choice| choice.to_s.sub(shortcut, '('+shortcut+')') }.join(' / ')}]: "
+        if warning
+          STDOUT.print red, bold, message, reset
+        else
+          STDOUT.print bold, message, reset
+        end
+        STDOUT.flush
+        input = STDIN.gets.downcase
+      end until (choice = mappings.detect { |shortcut, choice| [shortcut.downcase,choice.to_s.downcase].include?(input.downcase.strip) })
+      choice.last
+    end
+    
+    def log(message, type=:info)
+      STDOUT.puts(
+        case type
+          when :error
+            [red, bold, message, reset]
+          when :warning
+            [yellow, message, reset]
+          else
+            message
+        end
+      )
+    end
     # See ActiveRecord::Generators::Migration
     # Only generating a migration to each second is a problem.. because we generate everything in the same second
     # So we have to add further "pretend" seconds. This WILL cause problems.

data/lib/migrant/model_extensions.rb

@@ -1,14 +1,14 @@
 module Migrant
   module ModelExtensions
     attr_accessor :schema
-    def structure(&block)
+    def structure(type=nil, &block)
       # Using instance_*evil* to get the neater DSL on the models.
       # So, my_field in the structure block actually calls Migrant::Schema.my_field
 
       if self.superclass == ActiveRecord::Base
         @schema ||= Schema.new
         @schema.add_associations(self.reflect_on_all_associations)
-        @schema.define_structure(&block)
+        @schema.define_structure(type, &block)
         
         @schema.validations.each do |field, validation_options|
           validations = (validation_options.class == Array)? validation_options : [validation_options]
@@ -28,6 +28,10 @@ module Migrant
     def no_structure
       structure {}
     end
+    
+    def reset_structure!
+      @schema = nil
+    end
 
     def mock(attributes={}, recursive=true)
       attribs = {}

data/lib/migrant/schema.rb

@@ -18,10 +18,13 @@ module Migrant
       @columns = Hash.new
       @indexes = Array.new
       @validations = Hash.new
+      @type = :default
     end
 
-    def define_structure(&block)
+    def define_structure(type, &block)
       @validations = Hash.new
+      @type = type if type
+
       # Runs method_missing on columns given in the model "structure" DSL
       @proxy.translate_fancy_dsl(&block) if block_given?
     end
@@ -43,9 +46,14 @@ module Migrant
     end
 
     def requires_migration?
-      !(@columns.blank? && @indexes.blank?)
+      true
     end
 
+    # If the user defines structure(:partial), irreversible changes are ignored (removing a column, for example)    
+    def partial?
+      @type == :partial
+    end    
+
     def column_migrations
       @columns.collect {|field, data| [field, data.column] } # All that needs to be migrated
     end

data/lib/tasks/db.rake

@@ -12,5 +12,17 @@ namespace :db do
       end
     end
   end
+  
+  desc "Provides a shortcut to rolling back and discarding the last migration"
+  task :downgrade => :environment do
+    Rake::Task['db:rollback'].invoke
+    Dir.chdir(Rails.root.join('db', 'migrate')) do
+      last_migration = Dir.glob('*.rb').sort.last and
+      File.unlink(last_migration) and
+      puts "Removed #{Dir.pwd}/#{last_migration}."      
+    end
+  
+    Rake::Task['db:test:clone'].invoke unless ENV['RAILS_ENV']
+  end
 end
 

data/migrant.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{migrant}
-  s.version = "1.1.2"
+  s.version = "1.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Pascal Houliston"]
-  s.date = %q{2011-04-12}
+  s.date = %q{2011-04-23}
   s.description = %q{Easier schema management for Rails that compliments your domain model.}
   s.email = %q{101pascal@gmail.com}
   s.extra_rdoc_files = [
@@ -52,6 +52,7 @@ Gem::Specification.new do |s|
     "test/rails_app/app/models/business.rb",
     "test/rails_app/app/models/business_category.rb",
     "test/rails_app/app/models/category.rb",
+    "test/rails_app/app/models/chameleon.rb",
     "test/rails_app/app/models/customer.rb",
     "test/rails_app/app/models/user.rb",
     "test/rails_app/app/views/layouts/application.html.erb",
@@ -83,15 +84,21 @@ Gem::Specification.new do |s|
     "test/test_migration_generator.rb",
     "test/test_validations.rb",
     "test/test_zzz_performance.rb",
+    "test/verified_output/migrations/added_incompatible_spot_and_deleted_spots.rb",
     "test/verified_output/migrations/business_id.rb",
+    "test/verified_output/migrations/chameleons_added_new_longer_spots_and_moved_new_spots.rb",
     "test/verified_output/migrations/create_business_categories.rb",
     "test/verified_output/migrations/create_businesses.rb",
     "test/verified_output/migrations/create_categories.rb",
+    "test/verified_output/migrations/create_chameleons.rb",
     "test/verified_output/migrations/create_reviews.rb",
     "test/verified_output/migrations/create_users.rb",
     "test/verified_output/migrations/created_at.rb",
+    "test/verified_output/migrations/deleted_incompatible_spot.rb",
+    "test/verified_output/migrations/deleted_spots.rb",
     "test/verified_output/migrations/estimated_value_notes.rb",
-    "test/verified_output/migrations/landline.rb"
+    "test/verified_output/migrations/landline.rb",
+    "test/verified_output/migrations/renamed_old_spots.rb"
   ]
   s.homepage = %q{http://github.com/pascalh1011/migrant}
   s.require_paths = ["lib"]
@@ -105,6 +112,7 @@ Gem::Specification.new do |s|
     "test/rails_app/app/models/business.rb",
     "test/rails_app/app/models/business_category.rb",
     "test/rails_app/app/models/category.rb",
+    "test/rails_app/app/models/chameleon.rb",
     "test/rails_app/app/models/customer.rb",
     "test/rails_app/app/models/user.rb",
     "test/rails_app/config/application.rb",
@@ -126,15 +134,21 @@ Gem::Specification.new do |s|
     "test/test_migration_generator.rb",
     "test/test_validations.rb",
     "test/test_zzz_performance.rb",
+    "test/verified_output/migrations/added_incompatible_spot_and_deleted_spots.rb",
     "test/verified_output/migrations/business_id.rb",
+    "test/verified_output/migrations/chameleons_added_new_longer_spots_and_moved_new_spots.rb",
     "test/verified_output/migrations/create_business_categories.rb",
     "test/verified_output/migrations/create_businesses.rb",
     "test/verified_output/migrations/create_categories.rb",
+    "test/verified_output/migrations/create_chameleons.rb",
     "test/verified_output/migrations/create_reviews.rb",
     "test/verified_output/migrations/create_users.rb",
     "test/verified_output/migrations/created_at.rb",
+    "test/verified_output/migrations/deleted_incompatible_spot.rb",
+    "test/verified_output/migrations/deleted_spots.rb",
     "test/verified_output/migrations/estimated_value_notes.rb",
-    "test/verified_output/migrations/landline.rb"
+    "test/verified_output/migrations/landline.rb",
+    "test/verified_output/migrations/renamed_old_spots.rb"
   ]
 
   if s.respond_to? :specification_version then
@@ -150,6 +164,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<sqlite3>, [">= 0"])
       s.add_development_dependency(%q<simplecov>, [">= 0"])
       s.add_development_dependency(%q<terminal-table>, [">= 0"])
+      s.add_development_dependency(%q<term-ansicolor>, [">= 0"])
     else
       s.add_dependency(%q<rails>, [">= 3.0.0"])
       s.add_dependency(%q<faker>, [">= 0"])
@@ -160,6 +175,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<sqlite3>, [">= 0"])
       s.add_dependency(%q<simplecov>, [">= 0"])
       s.add_dependency(%q<terminal-table>, [">= 0"])
+      s.add_dependency(%q<term-ansicolor>, [">= 0"])
     end
   else
     s.add_dependency(%q<rails>, [">= 3.0.0"])
@@ -171,6 +187,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<sqlite3>, [">= 0"])
     s.add_dependency(%q<simplecov>, [">= 0"])
     s.add_dependency(%q<terminal-table>, [">= 0"])
+    s.add_dependency(%q<term-ansicolor>, [">= 0"])
   end
 end
 

data/test/helper.rb

@@ -10,7 +10,7 @@ SimpleCov.adapters.define 'migrant' do
   add_group 'Core Extensions', '/lib/migrant'
   add_group 'Schema Data Types', '/lib/datatype'
 end
-SimpleCov.start 'migrant'
+#SimpleCov.start 'migrant'
 ENV['RAILS_ENV'] = 'test'
 
 require 'rubygems'
@@ -47,6 +47,22 @@ class Profiler
   end
 end
 
+# Mock some stubs on STDIN's eigenclass so we can fake user input
+class << STDIN
+  # Simple mock for simulating user inputs
+  def _mock_responses(*responses)
+    @_responses ||= []
+    @_responses += responses
+  end
+  
+  def gets
+    raise "STDIN.gets() called but no mocks to return. Did you set them up with _mock_responses()?" if @_responses.blank?
+    @_responses.slice!(0).tap do |response|
+      STDOUT.puts "{ANSWERED WITH: #{response}}"
+    end
+  end
+end
+
 # Reset database
 db_path = File.join(File.dirname(__FILE__), 'rails_app', 'db', 'test.sqlite3')
 File.delete(db_path) if File.exists?(db_path)

data/test/rails_app/app/models/chameleon.rb

@@ -0,0 +1,5 @@
+class Chameleon < ActiveRecord::Base
+  structure do
+    spots
+  end
+end

data/test/test_migration_generator.rb

@@ -21,7 +21,7 @@ class TestMigrationGenerator < Test::Unit::TestCase
           to_test = File.open(migration_file, 'r') { |r| r.read}.strip
           File.open(File.join(File.dirname(__FILE__), 'verified_output', 'migrations', template+'.rb'), 'r') do |file|
             while (line = file.gets)
-              assert_not_nil(to_test.match(line.strip), "Generated migration #{migration_file} missing line: #{line}")
+              assert_not_nil(to_test.gsub(/[\.\(\)]/, '').match(line.gsub(/[\.\(\)]/, '').strip), "Generated migration #{migration_file} missing line: #{line.gsub(/[\.\(\)]/, '').strip}")
             end
           end
           rake_migrate
@@ -152,6 +152,64 @@ class TestMigrationGenerator < Test::Unit::TestCase
       generate_migrations
       rake_migrate
     end
+
+    should "remove columns when requested and confirmed by the user" do
+      Chameleon.structure 
+      Chameleon.reset_structure!
+      Chameleon.no_structure
+      
+      STDIN._mock_responses('D', 'y')
+      run_against_template('deleted_incompatible_spot')
+    end
+    
+    should "not remove columns when the user does not confirm" do
+      Chameleon.reset_structure!
+      Chameleon.no_structure
+      
+      STDIN._mock_responses('D', 'n')
+      generate_migrations
+      rake_migrate
+      Chameleon.structure do
+        spots
+      end
+    end
+    
+    should "successfully rename a column missing from the schema to a new column specified by the user" do
+      Chameleon.structure do
+        old_spots
+      end
+      generate_migrations
+      rake_migrate
+      Chameleon.reset_structure!
+      Chameleon.structure do
+        new_spots
+      end
+      STDIN._mock_responses('M', 'new_spots')
+      run_against_template('renamed_old_spots')
+    end
+    
+    should "transfer data to an new incompatible column if the operation is safe" do
+      Chameleon.reset_column_information
+      Chameleon.create!(:new_spots => "22")
+      Chameleon.reset_structure!      
+      Chameleon.structure do
+        new_longer_spots "100", :as => :text
+      end
+      STDIN._mock_responses('M', 'new_longer_spots', 'M')
+      run_against_template('chameleons_added_new_longer_spots_and_moved_new_spots')
+      Chameleon.reset_column_information
+      assert_equal(Chameleon.first.new_longer_spots, "22")
+    end
+    
+    should "remove any column if a user elects to when a column can't be moved due to incompatible types" do
+      Chameleon.reset_structure!  
+      Chameleon.structure do
+        incompatible_spot 5
+      end
+      
+      STDIN._mock_responses('M', 'incompatible_spot', 'N', 'Y')
+      run_against_template('added_incompatible_spot_and_deleted_spots')
+    end
   end
 end
 

data/test/verified_output/migrations/added_incompatible_spot_and_deleted_spots.rb

@@ -0,0 +1,11 @@
+class ChangedChameleonsAddedIncompatibleSpotAndDeletedSpots < ActiveRecord::Migration
+  def self.up
+    add_column :chameleons, :incompatible_spot, :integer
+    remove_column :chameleons, :spots
+  end
+  
+  def self.down
+    add_column :chameleons, :spots, :string, :limit=>255
+    remove_column :chameleons, :incompatible_spot
+  end
+end

data/test/verified_output/migrations/chameleons_added_new_longer_spots_and_moved_new_spots.rb

@@ -0,0 +1,15 @@
+class ChangedChameleonsAddedNewLongerSpotsAndMovedNewSpots < ActiveRecord::Migration
+  def self.up
+    add_column :chameleons, :new_longer_spots, :text
+    puts "-- copy data from :new_spots to :new_longer_spots"
+    Chameleon.update_all("new_longer_spots = new_spots")
+    remove_column :chameleons, :new_spots
+  end
+  
+  def self.down
+    add_column :chameleons, :new_spots, :string, :limit=>255
+    puts "-- copy data from :new_longer_spots to :new_spots"    
+    Chameleon.update_all("new_spots = new_longer_spots")
+    remove_column :chameleons, :new_longer_spots
+  end
+end

data/test/verified_output/migrations/create_chameleons.rb

@@ -0,0 +1,11 @@
+class CreateChameleons < ActiveRecord::Migration
+  def self.up
+    create_table :chameleons do |t|
+      t.string :spots
+    end
+  end
+
+  def self.down
+    drop_table :chameleons
+  end
+end

data/test/verified_output/migrations/deleted_incompatible_spot.rb

@@ -0,0 +1,9 @@
+class ChangedChameleonsDeletedIncompatibleSpot < ActiveRecord::Migration
+  def self.up
+    remove_column :chameleons, :incompatible_spot
+  end
+  
+  def self.down
+    add_column :chameleons, :incompatible_spot, :integer, :limit=>nil
+  end
+end

data/test/verified_output/migrations/deleted_spots.rb

@@ -0,0 +1,9 @@
+class ChangedChameleonsDeletedSpots < ActiveRecord::Migration
+  def self.up
+    remove_column :chameleons, :spots
+  end
+
+  def self.down
+    add_column :chameleons, :spots, :string, :limit=>255
+  end
+end

data/test/verified_output/migrations/renamed_old_spots.rb

@@ -0,0 +1,9 @@
+class ChangedChameleonsRenamedOldSpots < ActiveRecord::Migration
+  def self.up
+    rename_column :chameleons, :old_spots, :new_spots
+  end
+  
+  def self.down
+    rename_column :chameleons, :new_spots, :old_spots
+  end
+end

metadata

@@ -1,8 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: migrant
 version: !ruby/object:Gem::Version 
+  hash: 31
   prerelease: 
-  version: 1.1.2
+  segments: 
+  - 1
+  - 2
+  - 0
+  version: 1.2.0
 platform: ruby
 authors: 
 - Pascal Houliston
@@ -10,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-12 00:00:00 +00:00
+date: 2011-04-23 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -20,9 +25,14 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 7
+        segments: 
+        - 3
+        - 0
+        - 0
         version: 3.0.0
-  type: :runtime
   prerelease: false
+  type: :runtime
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: faker
@@ -31,9 +41,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :runtime
   prerelease: false
+  type: :runtime
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
   name: thoughtbot-shoulda
@@ -42,9 +55,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id003
 - !ruby/object:Gem::Dependency 
   name: ansi
@@ -53,9 +69,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id004
 - !ruby/object:Gem::Dependency 
   name: jeweler
@@ -64,9 +83,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id005
 - !ruby/object:Gem::Dependency 
   name: turn
@@ -75,9 +97,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id006
 - !ruby/object:Gem::Dependency 
   name: sqlite3
@@ -86,9 +111,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id007
 - !ruby/object:Gem::Dependency 
   name: simplecov
@@ -97,9 +125,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id008
 - !ruby/object:Gem::Dependency 
   name: terminal-table
@@ -108,10 +139,27 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
-  type: :development
   prerelease: false
+  type: :development
   version_requirements: *id009
+- !ruby/object:Gem::Dependency 
+  name: term-ansicolor
+  requirement: &id010 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  prerelease: false
+  type: :development
+  version_requirements: *id010
 description: Easier schema management for Rails that compliments your domain model.
 email: 101pascal@gmail.com
 executables: []
@@ -157,6 +205,7 @@ files:
 - test/rails_app/app/models/business.rb
 - test/rails_app/app/models/business_category.rb
 - test/rails_app/app/models/category.rb
+- test/rails_app/app/models/chameleon.rb
 - test/rails_app/app/models/customer.rb
 - test/rails_app/app/models/user.rb
 - test/rails_app/app/views/layouts/application.html.erb
@@ -188,15 +237,21 @@ files:
 - test/test_migration_generator.rb
 - test/test_validations.rb
 - test/test_zzz_performance.rb
+- test/verified_output/migrations/added_incompatible_spot_and_deleted_spots.rb
 - test/verified_output/migrations/business_id.rb
+- test/verified_output/migrations/chameleons_added_new_longer_spots_and_moved_new_spots.rb
 - test/verified_output/migrations/create_business_categories.rb
 - test/verified_output/migrations/create_businesses.rb
 - test/verified_output/migrations/create_categories.rb
+- test/verified_output/migrations/create_chameleons.rb
 - test/verified_output/migrations/create_reviews.rb
 - test/verified_output/migrations/create_users.rb
 - test/verified_output/migrations/created_at.rb
+- test/verified_output/migrations/deleted_incompatible_spot.rb
+- test/verified_output/migrations/deleted_spots.rb
 - test/verified_output/migrations/estimated_value_notes.rb
 - test/verified_output/migrations/landline.rb
+- test/verified_output/migrations/renamed_old_spots.rb
 has_rdoc: true
 homepage: http://github.com/pascalh1011/migrant
 licenses: []
@@ -211,7 +266,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 146286865
+      hash: 3
       segments: 
       - 0
       version: "0"
@@ -220,6 +275,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
 requirements: []
 
@@ -236,6 +294,7 @@ test_files:
 - test/rails_app/app/models/business.rb
 - test/rails_app/app/models/business_category.rb
 - test/rails_app/app/models/category.rb
+- test/rails_app/app/models/chameleon.rb
 - test/rails_app/app/models/customer.rb
 - test/rails_app/app/models/user.rb
 - test/rails_app/config/application.rb
@@ -257,12 +316,18 @@ test_files:
 - test/test_migration_generator.rb
 - test/test_validations.rb
 - test/test_zzz_performance.rb
+- test/verified_output/migrations/added_incompatible_spot_and_deleted_spots.rb
 - test/verified_output/migrations/business_id.rb
+- test/verified_output/migrations/chameleons_added_new_longer_spots_and_moved_new_spots.rb
 - test/verified_output/migrations/create_business_categories.rb
 - test/verified_output/migrations/create_businesses.rb
 - test/verified_output/migrations/create_categories.rb
+- test/verified_output/migrations/create_chameleons.rb
 - test/verified_output/migrations/create_reviews.rb
 - test/verified_output/migrations/create_users.rb
 - test/verified_output/migrations/created_at.rb
+- test/verified_output/migrations/deleted_incompatible_spot.rb
+- test/verified_output/migrations/deleted_spots.rb
 - test/verified_output/migrations/estimated_value_notes.rb
 - test/verified_output/migrations/landline.rb
+- test/verified_output/migrations/renamed_old_spots.rb