guard-migrate 0.0.1 → 0.0.2

data/README.rdoc

@@ -1,6 +1,37 @@
-= guard-migrate
+= Guard::Migrate
 
-Description goes here.
+Migrate guard allows you to keep you migrations up to date while developing without needing to rerun them every time. I developed it because when I am first working out a project, the schema can change several times and this saves me several downs/ups/resets.
+
+- This should be compatible with 2.x rails, but has only currently been tested on 3.x rails.
+
+= Install
+
+Please be sure to have {Guard}[https://github.com/guard/guard] installed before continue.
+
+Install the gem:
+
+    gem install guard-migrate
+
+Add it to your Gemfile (inside test group):
+
+    gem 'guard-migrate'
+
+Add guard definition to your Guardfile by running this command:
+
+    guard init migrate
+
+== Options
+
+Available options:
+
+* :run_on_start - this will run the migration task when you start, reload or run all. Defaults to false. If reset is set to true with this, then it will run a reset on the start, reload, run all instead of just a regular migrate
+* :test_clone - this will run the with the additional `db:test:clone` to update the test database. Defaults to true.
+* :reset - this will run `rake db:migrate:reset` every time migrate is run. Defaults to false.
+* :rails_env - passing this will add "RAILS_ENV=" together with the environment.
+
+== Todos
+
+* move the customized code to a separate class/module.
 
 == Contributing to guard-migrate
  
@@ -12,6 +43,9 @@ Description goes here.
 * Make sure to add tests for it. This is important so I don't break it in a future version unintentionally.
 * Please try not to mess with the Rakefile, version, or history. If you want to have your own version, or is otherwise necessary, that is fine, but please isolate to its own commit so I can cherry-pick around it.
 
+== Authors
+
+{Geoff Lanotte}[https://github.com/glanotte]
 == Copyright
 
 Copyright (c) 2011 Geoff Lanotte. See LICENSE.txt for

data/lib/guard/migrate.rb

@@ -6,12 +6,16 @@ module Guard
     def initialize(watchers=[], options={})
       super
       
-      @reset = true unless options[:reset] == false
+      @reset = true if options[:reset] == true
       @test_clone = true unless options[:test_clone] == false
       @run_on_start = true if options[:run_on_start] == true
       @rails_env = options[:rails_env]
     end
 
+    def bundler?
+      @bundler ||= File.exist?("#{Dir.pwd}/Gemfile")
+    end
+
     def run_on_start?
       !!@run_on_start
     end
@@ -60,17 +64,25 @@ module Guard
 
     # Called on file(s) modifications
     def run_on_change(paths)
-      self.migrate
+      self.migrate(paths.map{|path| path.scan(%r{^db/migrate/(\d+).+\.rb}).flatten.first})
     end
 
-    def migrate
-      system self.rake_string
+    def migrate(paths = [])
+      return if !self.reset? && paths.empty?
+      system self.rake_string if self.reset?
+      paths.each do |path|
+        UI.info "Running #{self.rake_string(path)}"
+        system self.rake_string(path)
+      end
     end
 
-    def rake_string
-      @rake_string = 'rake'
+    def rake_string(path = nil)
+      @rake_string = ''
+      @rake_string += 'bundle exec ' if self.bundler?
+      @rake_string += 'rake'
       @rake_string += ' db:migrate'
       @rake_string += ':reset' if self.reset?
+      @rake_string += ":redo VERSION=#{path}" if !self.reset? && path && !path.empty?
       @rake_string += ' db:test:clone' if self.test_clone?
       @rake_string += " RAILS_ENV=#{self.rails_env}" if self.rails_env
       @rake_string

data/lib/guard/migrate/templates/Guardfile

@@ -1,4 +1,4 @@
 guard 'migrate' do
-  watch('db/migrate/*.rb')
+  watch(%r{^db/migrate/(\d+).+\.rb})
 end
 

data/lib/guard/migrate/version.rb

@@ -1,5 +1,5 @@
 module Guard
   module MigrateVersion
-    VERSION = "0.0.1"
+    VERSION = "0.0.2"
   end
 end

data/spec/guard/migrate_spec.rb

@@ -2,11 +2,27 @@ require 'spec_helper'
 
 describe Guard::Migrate do
   let(:options){ {}}
-  subject{ Guard::Migrate.new([], options) }
+  let(:paths){{}}
+
+  subject{ Guard::Migrate.new(paths, options) }
   
   describe "options" do
+    context "bundler" do
+      context "with a gemfile found" do
+        before{File.stub!(:exist?).and_return(true) }
+        its(:bundler?){should be_true}
+        its(:rake_string){should match(/^bundle exec rake/)}
+
+      end
+      context "with no gemfile found" do
+        before{File.stub!(:exist?).and_return(false)}
+        its(:bundler?){should_not be_true}
+        its(:rake_string){should match(/^rake/)}
+      end
+
+    end
     context "test clone" do
-      context "with no optios passed" do
+      context "with no options passed" do
         its(:test_clone?){should be_true}
         its(:rake_string){should match(/db:test:clone/)}
       end
@@ -21,15 +37,20 @@ describe Guard::Migrate do
 
     context "reset" do
       context "with no options passed" do
-        its(:reset?){should be_true}
-        its(:rake_string){should match(/rake db:migrate:reset/)}
+        its(:reset?){should_not be_true}
+
+        context "with paths" do
+          let(:paths){ ['1234'] }
+          it "rake string should attempt redo of changed migration" do
+            subject.rake_string(paths.first).should match(/rake db:migrate:redo VERSION\=1234/)
+          end
+        end
       end
 
-      context "when passed false" do
-        let(:options){ {:reset => false} }
-        its(:reset?){should_not be_true}
-        its(:rake_string){should match(/rake db:migrate/)}
-        its(:rake_string){should_not match(/rake db:migrate:reset/)}
+      context "when passed true" do
+        let(:options){ {:reset => true} }
+        its(:reset?){should be_true}
+        its(:rake_string){should match(/rake db:migrate:reset/)}
       end
     end
 
@@ -71,6 +92,22 @@ describe Guard::Migrate do
           subject.should_receive(:migrate)
           subject.run_all
         end
+
+        context "with reset set to true" do
+          let(:options){ {:run_on_start => true, :reset => true} }
+          it "should run a migrate reset on start" do
+            subject.rake_string.should match(/db:migrate:reset/)
+          end
+        end
+
+        context "with reset set to false" do
+          let(:options){ {:run_on_start => true, :reset => false} }
+          it "should run a regular migrate on start" do
+            subject.rake_string.should match(/db:migrate/)
+            subject.rake_string.should_not match(/db:migrate:reset/)
+            subject.rake_string.should_not match(/db:migrate:redo/)
+          end
+        end
       end
     end
 
@@ -88,10 +125,12 @@ describe Guard::Migrate do
     end
   end
 
-  context "migrate" do
+  context "run on change should fixup the path to only the version" do
+    ##I don't like this test much - consider refactoring
+    let(:paths){ ['db/migrate/1234_i_like_cheese.rb'] }
     it "should run the rake command" do
-      subject.should_receive(:system).with(subject.rake_string)
-      subject.run_on_change []
+      subject.should_receive(:system).with(subject.rake_string('1234'))
+      subject.run_on_change paths
     end
   end
 end

metadata

@@ -2,7 +2,7 @@
 name: guard-migrate
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors: 
 - Geoff Lanotte
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-15 00:00:00 Z
+date: 2011-05-22 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: guard