paratrooper 1.3.2 → 1.4.0

data/.bundle/config

@@ -0,0 +1,2 @@
+---
+BUNDLE_BIN: bin

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    paratrooper (1.3.2)
+    paratrooper (1.4.0)
       heroku-api (~> 0.3)
       netrc (~> 0.7)
       rendezvous (~> 0.0.1)
@@ -11,9 +11,9 @@ GEM
   specs:
     coderay (1.0.8)
     diff-lcs (1.1.3)
-    excon (0.20.1)
-    heroku-api (0.3.9)
-      excon (~> 0.20.1)
+    excon (0.21.0)
+    heroku-api (0.3.10)
+      excon (~> 0.21.0)
     method_source (0.8.1)
     netrc (0.7.7)
     pry (0.9.10)

data/lib/paratrooper/deploy.rb

@@ -1,6 +1,7 @@
 require 'paratrooper/heroku_wrapper'
 require 'paratrooper/system_caller'
 require 'paratrooper/notifiers/screen_notifier'
+require 'paratrooper/pending_migration_check'
 
 module Paratrooper
 
@@ -8,7 +9,7 @@ module Paratrooper
   #
   class Deploy
     attr_reader :app_name, :notifiers, :system_caller, :heroku, :tag_name,
-      :match_tag, :protocol, :deployment_host
+      :match_tag, :protocol, :deployment_host, :migration_check, :debug
 
     # Public: Initializes a Deploy
     #
@@ -28,15 +29,19 @@ module Paratrooper
     #                               application (optional, default: 'http').
     #            :deployment_host - String host name to be used in git URL
     #                               (optional, default: 'heroku.com').
+    #            :migration_check - Object responsible for checking pending
+    #                               migrations (optional).
     def initialize(app_name, options = {})
       @app_name        = app_name
       @notifiers       = options[:notifiers] || [Notifiers::ScreenNotifier.new]
       @heroku          = options[:heroku] || HerokuWrapper.new(app_name, options)
       @tag_name        = options[:tag]
       @match_tag       = options[:match_tag_to] || 'master'
-      @system_caller   = options[:system_caller] || SystemCaller.new
+      @system_caller   = options[:system_caller] || SystemCaller.new(debug)
       @protocol        = options[:protocol] || 'http'
       @deployment_host = options[:deployment_host] || 'heroku.com'
+      @debug           = options[:debug] || false
+      self.migration_check = options[:migration_check]
     end
 
     def setup
@@ -88,6 +93,7 @@ module Paratrooper
     # Public: Runs rails database migrations on your application.
     #
     def run_migrations
+      return unless pending_migrations?
       notify(:run_migrations)
       heroku.run_migrations
     end
@@ -155,6 +161,14 @@ module Paratrooper
       git_remote(deployment_host, app_name)
     end
 
+    def pending_migrations?
+      migration_check.migrations_waiting?
+    end
+
+    def migration_check=(obj)
+      @migration_check = obj || PendingMigrationCheck.new(match_tag, heroku, system_caller)
+    end
+
     # Internal: Calls commands meant to go to system
     #
     # call - String version of system command

data/lib/paratrooper/heroku_wrapper.rb

@@ -35,6 +35,12 @@ module Paratrooper
       rendezvous.start(:url => data['rendezvous_url'])
     end
 
+    def last_deploy_commit
+      data = heroku_api.get_releases(app_name).body
+      return nil if data.empty?
+      data.last['commit']
+    end
+
     private
     def app_domain_name
       if custom_domain_response

data/lib/paratrooper/pending_migration_check.rb

@@ -0,0 +1,23 @@
+require 'paratrooper/system_caller'
+
+module Paratrooper
+  class PendingMigrationCheck
+    attr_accessor :diff, :heroku, :match_tag_name, :system_caller
+
+    def initialize(match_tag_name, heroku_wrapper, system_caller)
+      self.heroku         = heroku_wrapper
+      self.match_tag_name = match_tag_name
+      self.system_caller  = system_caller
+    end
+
+    def migrations_waiting?
+      call = %Q[git diff --shortstat #{last_deployed_commit} #{match_tag_name} -- db/migrate]
+      self.diff = system_caller.execute(call)
+      !diff.strip.empty?
+    end
+
+    def last_deployed_commit
+      heroku.last_deploy_commit
+    end
+  end
+end

data/lib/paratrooper/system_caller.rb

@@ -1,7 +1,19 @@
 module Paratrooper
   class SystemCaller
+    attr_accessor :debug
+
+    def initialize(debug = false)
+      self.debug = debug
+    end
+
     def execute(call)
-      system(call)
+      debug_message_for(call)
+      `#{call}`
+    end
+
+    private
+    def debug_message_for(call)
+      p "DEBUG: #{call}" if debug
     end
   end
 end

data/lib/paratrooper/version.rb

@@ -1,3 +1,3 @@
 module Paratrooper
-  VERSION = "1.3.2"
+  VERSION = "1.4.0"
 end

data/spec/paratrooper/deploy_spec.rb

@@ -10,7 +10,8 @@ describe Paratrooper::Deploy do
     {
       heroku: heroku,
       notifiers: [],
-      system_caller: system_caller
+      system_caller: system_caller,
+      migration_check: migration_check
     }
   end
   let(:options) { Hash.new }
@@ -24,6 +25,7 @@ describe Paratrooper::Deploy do
     )
   end
   let(:system_caller) { double(:system_caller) }
+  let(:migration_check) { double(:migration_check) }
   let(:domain_response) do
     double(:domain_response, body: [{'domain' => 'application_url'}])
   end
@@ -175,14 +177,31 @@ describe Paratrooper::Deploy do
       system_caller.stub(:execute)
     end
 
-    it 'sends notification' do
-      deployer.should_receive(:notify).with(:run_migrations).once
-      deployer.run_migrations
+    context "when new migrations are waiting to be run" do
+      before do
+        migration_check.stub(:migrations_waiting?).and_return(true)
+      end
+
+      it 'sends notification' do
+        deployer.should_receive(:notify).with(:run_migrations).once
+        deployer.run_migrations
+      end
+
+      it 'pushes repo to heroku' do
+        heroku.should_receive(:run_migrations)
+        deployer.run_migrations
+      end
     end
 
-    it 'pushes repo to heroku' do
-      heroku.should_receive(:run_migrations)
-      deployer.run_migrations
+    context "when no migrations are available to be run" do
+      before do
+        migration_check.stub(:migrations_waiting?).and_return(false)
+      end
+
+      specify "heroku is not notified to run migrations" do
+        heroku.should_not_receive(:run_migrations)
+        deployer.run_migrations
+      end
     end
   end
 

data/spec/paratrooper/heroku_wrapper_spec.rb

@@ -107,4 +107,29 @@ describe Paratrooper::HerokuWrapper do
       end
     end
   end
+
+  describe "#last_deploy_commit" do
+    context "when deploy data is returned" do
+      let(:response) do
+        double(:response, body: [{ 'commit' => 'SHA' }])
+      end
+      it "returns string of last deployed commit" do
+        heroku_api.should_receive(:get_releases).with(app_name)
+          .and_return(response)
+        expect(wrapper.last_deploy_commit).to eq('SHA')
+      end
+    end
+
+    context "when no deploys have happened yet" do
+      let(:response) do
+        double(:response, body: [])
+      end
+
+      it "returns nil" do
+        heroku_api.should_receive(:get_releases).with(app_name)
+          .and_return(response)
+        expect(wrapper.last_deploy_commit).to eq(nil)
+      end
+    end
+  end
 end

data/spec/paratrooper/pending_migration_check_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+require 'paratrooper/pending_migration_check'
+
+describe Paratrooper::PendingMigrationCheck do
+  let(:migration_check) do
+    described_class.new(match_tag_name, heroku_wrapper, system_caller)
+  end
+  let(:system_caller) { double(:system_caller) }
+  let(:heroku_wrapper) do
+    double(:heroku_wrapper, last_deploy_commit: last_deployed_commit)
+  end
+  let(:last_deployed_commit) { nil }
+
+  describe "#migrations_waiting?" do
+    let(:match_tag_name) { "MATCH" }
+    let(:last_deployed_commit) { "LAST_DEPLOYED_COMMIT" }
+
+    it "calls out to heroku for latest deploy's commit" do
+      system_caller.stub(:execute).and_return("")
+      heroku_wrapper.should_receive(:last_deploy_commit)
+      migration_check.migrations_waiting?
+    end
+
+    context "and migrations are in diff" do
+      it "returns true" do
+        expected_call = %Q[git diff --shortstat LAST_DEPLOYED_COMMIT MATCH -- db/migrate]
+        system_caller.should_receive(:execute).with(expected_call)
+          .and_return("DIFF")
+        expect(migration_check.migrations_waiting?).to be_true
+      end
+    end
+
+    context "and migrations are not in diff" do
+      let(:match_tag_name) { 'master' }
+      let(:last_deployed_commit) { "LAST_DEPLOYED_COMMIT" }
+
+      it "returns false" do
+        expected_call = %Q[git diff --shortstat LAST_DEPLOYED_COMMIT master -- db/migrate]
+        system_caller.should_receive(:execute).with(expected_call)
+          .and_return("")
+        expect(migration_check.migrations_waiting?).to be_false
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: paratrooper
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-15 00:00:00.000000000 Z
+date: 2013-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -116,6 +116,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .bundle/config
 - .gitignore
 - .ruby-version
 - Gemfile
@@ -129,6 +130,7 @@ files:
 - lib/paratrooper/local_api_key_extractor.rb
 - lib/paratrooper/notifier.rb
 - lib/paratrooper/notifiers/screen_notifier.rb
+- lib/paratrooper/pending_migration_check.rb
 - lib/paratrooper/system_caller.rb
 - lib/paratrooper/version.rb
 - paratrooper.gemspec
@@ -138,6 +140,7 @@ files:
 - spec/paratrooper/local_api_key_extractor_spec.rb
 - spec/paratrooper/notifier_spec.rb
 - spec/paratrooper/notifiers/screen_notifier_spec.rb
+- spec/paratrooper/pending_migration_check_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/mattpolito/paratrooper
 licenses: []
@@ -170,4 +173,5 @@ test_files:
 - spec/paratrooper/local_api_key_extractor_spec.rb
 - spec/paratrooper/notifier_spec.rb
 - spec/paratrooper/notifiers/screen_notifier_spec.rb
+- spec/paratrooper/pending_migration_check_spec.rb
 - spec/spec_helper.rb