thegarage-gitx 2.12.0 → 2.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9c042898972c5d2092ec9f3a579a75e32c448a95
-  data.tar.gz: a8b5e3cd62170304f12cfdb4000e2325ad77325d
+  metadata.gz: 3970bcceec2fdffef729443705a5533f089e25b5
+  data.tar.gz: c42d584f27788dfd2491149895442665ce04fb92
 SHA512:
-  metadata.gz: a0aca7c57d79390ecfc96913132303a830fa258f6e3dd5d0a5f26de44b584a5cb840324f14ee160b06127cb37c8068eeaad86c5a6ff315fa3b23945ee35db624
-  data.tar.gz: cb4eed326c8c2ddeb0554d88a8722c84a06e40aab83856f0ee61051a13c9a061ce0b565f9da71b1b8d82b6d56c22a0b6d13f561c472edb72ca3aaac7f5c3b6d8
+  metadata.gz: 09941f7c3d161f8a7e8bb9f1167484713d2d12dc40d8fc4ecfddc93a37f51d93ee20319919650205b34ebe8a96ac83c0e0e042a83f2a3f3da237091357bd04c3
+  data.tar.gz: d8960f7276396f466d4c606b1e73355cd2a2db010502f1292e787fccc04b5f06bbe17ff411119bfc68fdbd66e28ef0472fb97ea9134abf9a7128c0e50a1d5fee

data/README.md

@@ -26,7 +26,7 @@ update the local feature branch with latest remote changes plus upstream release
 
 integrate the current feature branch into an aggregate branch (ex: prototype, staging)
 
-## git review
+## git review <feature_branch_name (optional, default: current_branch)>
 
 create a pull request on github for peer review of the current branch.  This command is re-runnable
 in order to re-assign pull requests.
@@ -41,13 +41,14 @@ options:
 NOTE: the `--bump` option will also update the pull request commit status to mark the branch as 'pending peer review'.
 This setting is cleared when a reviewer approves or rejects the pull request.
 
-## git release
+## git release <feature_branch_name (optional, default: current_branch)
 
-release the current feature branch to master.  This operation will perform the following:
+release the feature branch to master.  This operation will perform the following:
 
-* pull in latest code from remote branch
-* merge in latest code from master branch
+* pull latest code from remote feature branch
+* pull latest code from master branch
 * prompt user to confirm they actually want to perform the release
+* check if pull request commit status is currently successful
 * merge current branch into master
 * (optional) cleanup merged branches from remote server
 

data/lib/thegarage/gitx/cli/release_command.rb

@@ -14,11 +14,12 @@ module Thegarage
 
         desc 'release', 'release the current branch to production'
         method_option :cleanup, :type => :boolean, :desc => 'cleanup merged branches after release'
-        def release
+        def release(branch = nil)
           return unless yes?("Release #{current_branch.name} to production? (y/n)", :green)
 
-          branch = current_branch.name
+          branch ||= current_branch.name
           assert_not_protected_branch!(branch, 'release')
+          checkout_branch(branch)
           execute_command(UpdateCommand, :update)
 
           find_or_create_pull_request(branch)

data/lib/thegarage/gitx/cli/review_command.rb

@@ -1,7 +1,6 @@
 require 'thor'
 require 'thegarage/gitx'
 require 'thegarage/gitx/cli/base_command'
-require 'thegarage/gitx/cli/update_command'
 require 'thegarage/gitx/github'
 
 module Thegarage
@@ -41,10 +40,10 @@ module Thegarage
         method_option :approve, :type => :boolean, :desc => 'approve the pull request an post comment on pull request'
         method_option :reject, :type => :boolean, :desc => 'reject the pull request an post comment on pull request'
         # @see http://developer.github.com/v3/pulls/
-        def review
+        def review(branch = nil)
           fail 'Github authorization token not found' unless authorization_token
 
-          branch = current_branch.name
+          branch ||= current_branch.name
           pull_request = find_or_create_pull_request(branch)
           bump_pull_request(pull_request) if options[:bump]
           approve_pull_request(pull_request) if options[:approve]

data/lib/thegarage/gitx/github.rb

@@ -1,6 +1,7 @@
 require 'octokit'
 require 'fileutils'
 require 'yaml'
+require 'thegarage/gitx/cli/update_command'
 
 module Thegarage
   module Gitx
@@ -20,6 +21,7 @@ module Thegarage
       def find_or_create_pull_request(branch)
         pull_request = find_pull_request(branch)
         pull_request ||= begin
+          checkout_branch(branch)
           execute_command(Thegarage::Gitx::Cli::UpdateCommand, :update)
           pull_request = create_pull_request(branch)
           say 'Created pull request: '

data/lib/thegarage/gitx/version.rb

@@ -1,5 +1,5 @@
 module Thegarage
   module Gitx
-    VERSION = '2.12.0'
+    VERSION = '2.13.0'
   end
 end

data/spec/thegarage/gitx/cli/integrate_command_spec.rb

@@ -98,7 +98,7 @@ describe Thegarage::Gitx::Cli::IntegrateCommand do
         expect(cli).to receive(:run_cmd).with("git merge feature-branch").ordered
         expect(cli).to receive(:run_cmd).with("git push origin HEAD").ordered
         expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
-
+        expect(cli).to receive(:run_cmd).with('git checkout feature-branch').ordered
         expect(cli).to receive(:run_cmd).with("git log master...feature-branch --reverse --no-merges --pretty=format:'* %s%n%b'").and_return("2013-01-01 did some stuff").ordered
 
         stub_request(:post, 'https://api.github.com/repos/thegarage/thegarage-gitx/pulls').to_return(:status => 201, :body => new_pull_request.to_json, :headers => {'Content-Type' => 'application/json'})

data/spec/thegarage/gitx/cli/release_command_spec.rb

@@ -61,6 +61,7 @@ describe Thegarage::Gitx::Cli::ReleaseCommand do
         expect(cli).to receive(:yes?).and_return(true)
         allow(cli).to receive(:authorization_token).and_return(authorization_token)
 
+        expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
         expect(cli).to receive(:run_cmd).with("git checkout master").ordered
         expect(cli).to receive(:run_cmd).with("git pull origin master").ordered
         expect(cli).to receive(:run_cmd).with("git merge --no-ff feature-branch").ordered
@@ -74,6 +75,29 @@ describe Thegarage::Gitx::Cli::ReleaseCommand do
         should meet_expectations
       end
     end
+    context 'when target_branch is not nil and user confirms release and pull request exists with success status' do
+      before do
+        expect(cli).to receive(:execute_command).with(Thegarage::Gitx::Cli::UpdateCommand, :update)
+        expect(cli).to receive(:execute_command).with(Thegarage::Gitx::Cli::IntegrateCommand, :integrate, 'staging')
+        expect(cli).to_not receive(:execute_command).with(Thegarage::Gitx::Cli::CleanupCommand, :cleanup)
+
+        expect(cli).to receive(:yes?).and_return(true)
+        allow(cli).to receive(:authorization_token).and_return(authorization_token)
+
+        expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
+        expect(cli).to receive(:run_cmd).with("git checkout master").ordered
+        expect(cli).to receive(:run_cmd).with("git pull origin master").ordered
+        expect(cli).to receive(:run_cmd).with("git merge --no-ff feature-branch").ordered
+        expect(cli).to receive(:run_cmd).with("git push origin HEAD").ordered
+
+        VCR.use_cassette('pull_request_does_exist_with_success_status') do
+          cli.release 'feature-branch'
+        end
+      end
+      it 'runs expected commands' do
+        should meet_expectations
+      end
+    end
     context 'when user confirms release and pull request does not exist' do
       let(:new_pull_request) do
         {
@@ -96,6 +120,8 @@ describe Thegarage::Gitx::Cli::ReleaseCommand do
         expect(cli).to receive(:yes?).with('Release feature-branch to production? (y/n)', :green).and_return(true)
         expect(cli).to receive(:yes?).with('Branch status is currently: pending.  Proceed with release? (y/n)', :red).and_return(true)
 
+        expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
+        expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
         expect(cli).to receive(:run_cmd).with("git log master...feature-branch --reverse --no-merges --pretty=format:'* %s%n%b'").and_return("2013-01-01 did some stuff").ordered
         expect(cli).to receive(:run_cmd).with("git checkout master").ordered
         expect(cli).to receive(:run_cmd).with("git pull origin master").ordered
@@ -128,6 +154,7 @@ describe Thegarage::Gitx::Cli::ReleaseCommand do
         expect(cli).to receive(:yes?).and_return(true)
         allow(cli).to receive(:authorization_token).and_return(authorization_token)
 
+        expect(cli).to receive(:run_cmd).with("git checkout feature-branch").ordered
         expect(cli).to receive(:run_cmd).with("git checkout master").ordered
         expect(cli).to receive(:run_cmd).with("git pull origin master").ordered
         expect(cli).to receive(:run_cmd).with("git merge --no-ff feature-branch").ordered
@@ -140,9 +167,6 @@ describe Thegarage::Gitx::Cli::ReleaseCommand do
       it 'runs expected commands' do
         should meet_expectations
       end
-      it 'prunes merged branches (git cleanup)' do
-        should meet_expectations
-      end
     end
   end
 end

data/spec/thegarage/gitx/cli/review_command_spec.rb

@@ -43,6 +43,7 @@ describe Thegarage::Gitx::Cli::ReviewCommand do
         expect(Thegarage::Gitx::Cli::UpdateCommand).to receive(:new).and_return(fake_update_command)
 
         allow(cli).to receive(:authorization_token).and_return(authorization_token)
+        expect(cli).to receive(:run_cmd).with('git checkout feature-branch').ordered
         expect(cli).to receive(:run_cmd).with("git log master...feature-branch --reverse --no-merges --pretty=format:'* %s%n%b'").and_return("* old commit\n\n* new commit").ordered
         expect(cli).to receive(:ask_editor).with("### Changelog\n* old commit\n\n* new commit\n#{Thegarage::Gitx::Github::PULL_REQUEST_FOOTER}", anything).and_return('description')
 
@@ -59,6 +60,41 @@ describe Thegarage::Gitx::Cli::ReviewCommand do
         should meet_expectations
       end
     end
+    context 'when target branch is not nil and pull request does not exist' do
+      let(:authorization_token) { '123123' }
+      let(:changelog) { '* made some fixes' }
+      let(:fake_update_command) { double('fake update command', update: nil) }
+      let(:new_pull_request) do
+        {
+          html_url: "https://path/to/html/pull/request",
+          issue_url: "https://api/path/to/issue/url",
+          number: 10,
+          head: {
+            ref: "branch_name"
+          }
+        }
+      end
+      before do
+        expect(Thegarage::Gitx::Cli::UpdateCommand).to receive(:new).and_return(fake_update_command)
+
+        allow(cli).to receive(:authorization_token).and_return(authorization_token)
+        expect(cli).to receive(:run_cmd).with('git checkout feature-branch').ordered
+        expect(cli).to receive(:run_cmd).with("git log master...feature-branch --reverse --no-merges --pretty=format:'* %s%n%b'").and_return("* old commit\n\n* new commit").ordered
+        expect(cli).to receive(:ask_editor).with("### Changelog\n* old commit\n\n* new commit\n#{Thegarage::Gitx::Github::PULL_REQUEST_FOOTER}", anything).and_return('description')
+
+        stub_request(:post, 'https://api.github.com/repos/thegarage/thegarage-gitx/pulls').to_return(:status => 201, :body => new_pull_request.to_json, :headers => {'Content-Type' => 'application/json'})
+
+        VCR.use_cassette('pull_request_does_not_exist') do
+          cli.review 'feature-branch'
+        end
+      end
+      it 'creates github pull request' do
+        should meet_expectations
+      end
+      it 'runs expected commands' do
+        should meet_expectations
+      end
+    end
     context 'when authorization_token is missing' do
       let(:authorization_token) { nil }
       it do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: thegarage-gitx
 version: !ruby/object:Gem::Version
-  version: 2.12.0
+  version: 2.13.0
 platform: ruby
 authors:
 - Ryan Sonnek
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-13 00:00:00.000000000 Z
+date: 2015-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rugged