lita-github 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bd7cf1b35725c7ee19e36efdd1c4717ead4e7dfd
-  data.tar.gz: dc58dac08313f2ec192911ca292681b146068297
+  metadata.gz: 0bca7c5436b3f7d3858b5374d633dbf2ce49a78d
+  data.tar.gz: 0f13f80249167fb9cfb48b835c02c377af0994e9
 SHA512:
-  metadata.gz: 1f5e87d33ffe37ca8ab4f1b584e32766b649d7a440e7252702d8b27584ea0dcc31e997a446515e7f09ab57daf62290fb03591bfc22578c6d2e146d122d3c4cfd
-  data.tar.gz: 5f7337752aa592d3c56941cd3ffe4e9e891262875e9e774863e74401b295ac13b7ac800bf9b19fac8d1e03f7ccb5530b59646b0831ebf2e5865220b9397ef8a4
+  metadata.gz: d69a87f51a1b4a4d3ce7ad68bc471e29207da530cb192e9d45773a7608d32f1c97f35543030af4b940bc5b9665b5caa609b847232da733ad79ef0a9ef9f99c06
+  data.tar.gz: df92f580a280732f33141f74fe5eb8897ac599eb6f876770fb505cea5c777cc4c66af535171db12ac01147e57a09345880f2472048f8cc5fde56046539642677

data/README.md

@@ -59,6 +59,8 @@ Here is the current functionality:
 * `!gh repo delete PagerDuty/lita-github`
   * Deletes the repo you specify, requires confirmation before doing so
   * **Note:** This method is disabled by default, you need to enable it by setting `config.handlers.github.repo_delete_enabled = true` in your configuration file
+* `!gh repo rename PagerDuty/lita-github better-lita-github`
+  * Renames the repo you specify -- requires confirmation
 * `!gh repo teams PagerDuty/lita-github`
   * list all of the teams currently attached to a repo
 * `!gh repo team add <TEAM_ID|TEAM_SLUG> PagerDuty/lita-github`

data/lib/lita-github/r.rb

@@ -26,6 +26,9 @@ module LitaGithub
     #  key1:value key2:"value2" key3:'a better value'
     OPT_REGEX ||= /((?:\s+?[a-zA-Z0-9_]+?):(?:(?:".+?")|(?:'.+?')|(?:[a-zA-Z0-9_]+)))/
 
+    # regex matcher for a repo name (not including the org).
+    REPO_NAME_REGEX = '(?<repo_name>[a-zA-Z0-9_\-]+)'
+
     # regex matcher for: Org/repo
     REPO_REGEX = '(?:(?<org>[a-zA-Z0-9_\-]+)(?:\s+?)?\/)?(?:\s+?)?(?<repo>[a-zA-Z0-9_\-]+)'
   end

data/lib/lita-github/version.rb

@@ -21,7 +21,7 @@
 # @author Tim Heckman <tim@pagerduty.com>
 module LitaGithub
   # lita-github version
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 
   # lita-github version split amongst different revisions
   MAJOR_VERSION, MINOR_VERSION, REVISION = VERSION.split('.').map(&:to_i)

data/lib/lita/handlers/github.rb

@@ -81,6 +81,7 @@ module Lita
 
         # Lita::Handlers::GithubRepo
         config.repo_create_enabled              = true
+        config.repo_rename_enabled              = true
         config.repo_delete_enabled              = false
         config.repo_team_add_enabled            = false
         config.repo_team_rm_enabled             = false

data/lib/lita/handlers/github_repo.rb

@@ -68,6 +68,17 @@ module Lita
         }
       )
 
+      route(
+        /#{LitaGithub::R::A_REG}repo\s+?rename\s+?#{LitaGithub::R::REPO_REGEX}\s+?#{LitaGithub::R::REPO_NAME_REGEX}/,
+        :repo_rename,
+        command: true,
+        confirmation: true,
+        help: {
+          'gh repo rename PagerDuty/lita-github better-lita-github' =>
+            'Rename the PagerDuty/lita-github repo to PagerDuty/better-lita-github'
+        }
+      )
+
       route(
         /#{LitaGithub::R::A_REG}repo\s+?(teams|team\s+?list)\s+?#{LitaGithub::R::REPO_REGEX}/,
         :repo_teams_list,
@@ -117,6 +128,16 @@ module Lita
         response.reply(create_repo(org, repo, opts))
       end
 
+      def repo_rename(response)
+        return response.reply(t('method_disabled')) if func_disabled?(__method__)
+        org, repo = repo_match(response.match_data)
+        new_repo = response.match_data['repo_name']
+
+        return response.reply(t('not_found', org: org, repo: repo)) unless repo?(rpo(org, repo))
+
+        response.reply(rename_repo(org, repo, new_repo))
+      end
+
       def repo_delete(response)
         return response.reply(t('method_disabled')) if func_disabled?(__method__)
 
@@ -308,6 +329,21 @@ module Lita
         reply
       end
 
+      def rename_repo(org, repo, new_repo)
+        full_name = rpo(org, repo)
+        reply = nil
+        begin
+          octo.edit_repository(full_name, name: new_repo)
+        ensure
+          if repo?(rpo(org, new_repo))
+            reply = t('repo_rename.pass', org: org, old_repo: repo, new_repo: new_repo)
+          else
+            reply = t('repo_rename.fail', org: org, repo: repo)
+          end
+        end
+        reply
+      end
+
       def delete_repo(org, repo)
         full_name = rpo(org, repo)
         reply = nil

data/locales/en.yml

@@ -26,6 +26,9 @@ en:
           pass: "Created %{org}/%{repo}: %{repo_url}"
           fail: "Unable to create %{org}/%{repo}"
           exists: "Unable to create %{org}/%{repo} as it already exists"
+        repo_rename:
+          pass: "Renamed %{org}/%{old_repo} to %{org}/%{new_repo}"
+          fail: "Unable to rename %{org}/%{repo}"
         repo_delete:
           pass: "Deleted %{org}/%{repo}"
           fail: "Unable to delete %{org}/%{repo}"

data/spec/unit/lita/handlers/github_repo_spec.rb

@@ -24,6 +24,10 @@ describe Lita::Handlers::GithubRepo, lita_handler: true do
   it { is_expected.to route_command('gh repo new GrapeDuty/lita-test private:true team:heckman').to(:repo_create) }
   it { is_expected.to route_command('gh repo new GrapeDuty/lita-test private:true randomunparseabletext ').to(:repo_create) }
 
+  # repo_rename command routing
+  it { is_expected.to route_command('gh repo rename GrapeDuty/lita-test lita-test-2').to(:repo_rename) }
+  it { is_expected.to route_command('gh repo rename lita-test lita-test-2').to(:repo_rename) }
+
   # repo_delete command routing
   it { is_expected.to route_command('gh repo delete GrapeDuty/lita-test').to(:repo_delete) }
   it { is_expected.to route_command('gh repo delete lita-test').to(:repo_delete) }
@@ -303,6 +307,34 @@ describe Lita::Handlers::GithubRepo, lita_handler: true do
     end
   end
 
+  describe '.rename_repo' do
+    before do
+      allow(github_repo).to receive(:octo).and_return(double('Octokit::Client', edit_repository: nil))
+    end
+
+    context 'when repo renamed' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/better-lita-test").and_return(true)
+      end
+
+      it 'should confirm successful rename' do
+        expect(github_repo.send(:rename_repo, github_org, 'lita-test', 'better-lita-test'))
+          .to eql 'Renamed GrapeDuty/lita-test to GrapeDuty/better-lita-test'
+      end
+    end
+
+    context 'when repo not renamed' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/better-lita-test").and_return(false)
+      end
+
+      it 'should reply with failure message' do
+        expect(github_repo.send(:rename_repo, github_org, 'lita-test', 'better-lita-test'))
+          .to eql 'Unable to rename GrapeDuty/lita-test'
+      end
+    end
+  end
+
   describe '.remove_team_from_repo' do
     before do
       @octo_obj = double('Octokit::Client', remove_team_repository: true)
@@ -742,6 +774,41 @@ describe Lita::Handlers::GithubRepo, lita_handler: true do
     end
   end
 
+  describe '.repo_rename' do
+    before do
+      allow(github_repo).to receive(:func_disabled?).and_return(false)
+      allow(github_repo).to receive(:rename_repo).and_return('hello there')
+      allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(true)
+    end
+
+    it 'reply with the return from rename_repo()' do
+      send_command("gh repo rename #{github_org}/lita-test better-lita-test")
+      expect(replies.last).to eql 'hello there'
+    end
+
+    context 'when command disabled' do
+      before do
+        allow(github_repo).to receive(:func_disabled?).and_return(true)
+      end
+
+      it 'should no-op and say such if the command is disabled' do
+        send_command("gh repo rename #{github_org}/lita-test better-lita-test")
+        expect(replies.last).to eql disabled_reply
+      end
+    end
+
+    context 'when repo not found' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(false)
+      end
+
+      it 'should no-op informing you that the repo is not there' do
+        send_command("gh repo rename #{github_org}/lita-test better-lita-test")
+        expect(replies.last).to eql 'That repo (GrapeDuty/lita-test) was not found'
+      end
+    end
+  end
+
   describe '.repo_teams_list' do
     before do
       @teams = [

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-github
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Tim Heckman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-22 00:00:00.000000000 Z
+date: 2015-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler