lita-github 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dc3526e11e2ee29500e53d039037b006f74137b9
-  data.tar.gz: f66a2729e9335198069b8b88fa45e46d3cfb57f7
+  metadata.gz: 83ef88d9ef27a589ab808c94f5acbee5c45890f3
+  data.tar.gz: 72f83141ec52828b41e684fce7a60371ef0f1270
 SHA512:
-  metadata.gz: 0404f2c6e7846c9ae078809738d894c0a18a4a5ddb134c8e1626abd84c3a2c2cd1459ae91599e458cee909e3ec4d385d57226dd32280bcbf764351267f29092a
-  data.tar.gz: a2bc91f92334ae48fc92c630903eb5edbeecc06b63a7c2534722bc94f3c7b6b0b3c453d3e2e9e3c94bc5ad0d46e830d17cfdae6f84c51ec5efa0b19dceda25f7
+  metadata.gz: 299cbb26470c5d2d99d3eaaf34bf61b5d5de8e88b6a0b701a24e09d02eb492b4e7db75e3642a729a4a6326f75635c4fe0c933a725ed1dfa1ad3c94c0c5ffca65
+  data.tar.gz: 3ca05325963bcc50abb0f39bb91f54acee4147699b513a3d06d272d4f8ed473b6d4d60292ab6b45877267387239a3c3ccb68b9833b9bba7710eaca892269f394

data/.gitignore

@@ -29,6 +29,7 @@ build/
 # Gemfile.lock
 # .ruby-version
 # .ruby-gemset
+Gemfile.lock
 
 # unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
 .rvmrc

data/.travis.yml

@@ -1,6 +1,7 @@
 branches:
   only:
     - master
+env: CODECLIMATE_REPO_TOKEN=3ffc4d9148b686fd07872259719fc6458a5521213e73e1f7b9db8546292f622a
 services:
   - redis-server
 language: ruby

data/README.md

@@ -1,5 +1,11 @@
 lita-github
 ===========
+[![Build Status](https://img.shields.io/travis/PagerDuty/lita-github/master.svg)](https://travis-ci.org/PagerDuty/lita-github)
+[![MIT License](https://img.shields.io/badge/license-Apache%202.0-brightgreen.svg)](https://tldrlegal.com/license/apache-license-2.0-(apache-2.0))
+[![RubyGems :: lita-github Gem Version](http://img.shields.io/gem/v/lita-github.svg)](https://rubygems.org/gems/lita-github)
+[![Code Climate Quality](https://img.shields.io/codeclimate/github/PagerDuty/lita-github.svg)](https://codeclimate.com/github/PagerDuty/lita-github)
+[![Code Climate Coverage](http://img.shields.io/codeclimate/coverage/github/PagerDuty/lita-github.svg)](https://codeclimate.com/github/PagerDuty/lita-github)
+
 Copyright 2014 PagerDuty, Inc.
 
 Lita handler for GitHub-related operations. This include administrative management of organizations and repository management/operations.

data/lib/lita-github.rb

@@ -22,3 +22,4 @@ Lita.load_locales Dir[File.expand_path(
 
 require 'lita-github/version'
 require 'lita/handlers/github'
+require 'lita/handlers/github_repo'

data/lib/lita-github/filters.rb

@@ -0,0 +1,24 @@
+# -*- coding: UTF-8 -*-
+#
+# Copyright 2014 PagerDuty, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+module LitaGithub
+  # Github handler common-use method filters
+  module Filters
+    def func_disabled?(method)
+      config.send("#{method}_enabled".to_sym) ? false : true
+    end
+  end
+end

data/lib/lita-github/org.rb

@@ -0,0 +1,24 @@
+# -*- coding: UTF-8 -*-
+#
+# Copyright 2014 PagerDuty, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+module LitaGithub
+  # Github handler common-use Organization methods
+  module Org
+    def organization(name)
+      name.nil? || name.empty? ? config.default_org : name
+    end
+  end
+end

data/lib/lita-github/r.rb

@@ -22,9 +22,9 @@ module LitaGithub
 
     # option regex, letting you scan the command string of things like:
     # key1:value key2:value, etc.
-    OPT_REGEX ||= /((?:\s+?[a-zA-Z0-9_]+?):(?:[a-zA-Z0-9_]+))(?:\s+?)?/
+    OPT_REGEX ||= /((?:\s+?[a-zA-Z0-9_]+?):(?:[a-zA-Z0-9_]+))/
 
     # regex matcher for: Org/repo
-    REPO_REGEX = '(?:(?<org>\w+)(?:\s+?)?\/)?(?:\s+?)?(?<repo>\w+)'
+    REPO_REGEX = '(?:(?<org>[a-zA-Z0-9_\-]+)(?:\s+?)?\/)?(?:\s+?)?(?<repo>[a-zA-Z0-9_\-]+)'
   end
 end

data/lib/lita-github/repo.rb

@@ -0,0 +1,28 @@
+# -*- coding: UTF-8 -*-
+#
+# Copyright 2014 PagerDuty, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+module LitaGithub
+  # Github handler common-use Repository methods
+  module Repo
+    def rpo(org, repo)
+      "#{org}/#{repo}"
+    end
+
+    def repo?(r)
+      octo.repository?(r)
+    end
+  end
+end

data/lib/lita-github/version.rb

@@ -16,6 +16,6 @@
 
 # Administer your Hub of Gits with Lita!
 module LitaGithub
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
   MAJ, MIN, REV = VERSION.split('.').map(&:to_i)
 end

data/lib/lita/handlers/github.rb

@@ -14,7 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'octokit'
 require 'lita-github/version'
 require 'lita-github/r'
 require 'lita-github/config'
@@ -49,6 +48,8 @@ module Lita
       )
 
       def self.default_config(config)
+        # when setting default configuration values please remember one thing:
+        # secure and safe by default
         config.repo_delete_enabled = false
         config.repo_private_default = true
       end

data/lib/lita/handlers/github_repo.rb

@@ -0,0 +1,191 @@
+# -*- coding: UTF-8 -*-
+#
+# Copyright 2014 PagerDuty, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'lita-github/r'
+require 'lita-github/config'
+require 'lita-github/octo'
+require 'lita-github/org'
+require 'lita-github/repo'
+require 'lita-github/filters'
+
+module Lita
+  # Lita Handler
+  module Handlers
+    # GitHub Lita Handler
+    class GithubRepo < Handler
+      include LitaGithub::Config  # Github handler Lita configuration methods
+      include LitaGithub::Octo    # Github handler common-use Octokit methods
+      include LitaGithub::Org     # Github handler common-use Organization methods
+      include LitaGithub::Repo    # Github handler common-use Repository methods
+      include LitaGithub::Filters # Github handler common-use method filters
+
+      on :loaded, :setup_octo # from LitaGithub::Octo
+
+      route(
+        /#{LitaGithub::R::A_REG}repo\s+?(?:create|new)\s+?#{LitaGithub::R::REPO_REGEX}.*$/,
+        :repo_create,
+        command: true,
+        help: {
+          'gh repo create PagerDuty/lita-github private:true team:heckman' =>
+            'Create the PagerDuty/lita-github repo, make it private, use team with "heckman" slug',
+          'gh repo new PagerDuty/lita-github' =>
+            'create new repo using the default privacy/team settings'
+        }
+      )
+
+      route(
+        /#{LitaGithub::R::A_REG}repo\s+?delete\s+?#{LitaGithub::R::REPO_REGEX}/,
+        :repo_delete, command: true, confirmation: true,
+        help: {
+          'gh repo delete PagerDuty/lita-github' => 'Delete the PagerDuty/lita-github repo'
+        }
+      )
+
+      route(
+        /#{LitaGithub::R::A_REG}repo\s+?info\s+?#{LitaGithub::R::REPO_REGEX}/,
+        :repo_info,
+        command: true,
+        help: {
+          'gh repo info PagerDuty/lita-github' => 'Display some information about the repo'
+        }
+      )
+
+      def repo_create(response)
+        org, repo = repo_match(response)
+
+        if repo?(rpo(org, repo))
+          return response.reply(t('repo_create.exists', org: org, repo: repo))
+        end
+
+        opts = extrapolate_create_opts(command_opts(response.args.join(' ')), org)
+
+        response.reply(create_repo(org, repo, opts))
+      end
+
+      def repo_delete(response)
+        return response.reply(t('method_disabled')) if func_disabled?(__method__)
+
+        org, repo = repo_match(response)
+
+        return response.reply(t('repo_delete.not_found', org: org, repo: repo)) unless repo?(rpo(org, repo))
+
+        response.reply(delete_repo(org, repo))
+      end
+
+      def repo_info(response)
+        org, repo = repo_match(response)
+        opts = {}
+        r_obj = octo.repository(rpo(org, repo))
+        p_obj = octo.pull_requests(rpo(org, repo))
+
+        opts[:repo]             = r_obj[:full_name]
+        opts[:description]      = r_obj[:description]
+        opts[:private]          = r_obj[:private]
+        opts[:url]              = r_obj[:html_url]
+        opts[:raw_issues_count] = r_obj[:open_issues_count]
+        opts[:pr_count]         = p_obj.length
+        opts[:issues_count]     = opts[:raw_issues_count] - opts[:pr_count]
+
+        response.reply(t('repo_info.reply', opts))
+      end
+
+      private
+
+      def command_opts(cmd)
+        o = {}
+        cmd.scan(LitaGithub::R::OPT_REGEX).flatten.compact.each do |opt|
+          k, v = opt.strip.split(':')
+          k = k.to_sym
+          o[k] = v unless o.key?(k)
+        end
+        o
+      end
+
+      def repo_match(response)
+        md = response.match_data
+        [organization(md['org']), md['repo']]
+      end
+
+      def extrapolate_create_opts(opts, org)
+        opts[:organization] = org
+
+        if opts.key?(:team)
+          opts[:team_id] = team_by_slug(opts[:team], org) || team_by_slug(config.default_team_slug, org)
+        else
+          opts[:team_id] = team_by_slug(config.default_team_slug, org)
+        end unless opts.key?(:team_id)
+        opts
+      end
+
+      def team_by_slug(slug, org)
+        octo.organization_teams(org).each do |team|
+          return team[:id] if team[:slug] == slug
+        end
+        nil
+      end
+
+      def should_repo_be_private?(value)
+        if value.nil? || value.empty?
+          config.repo_private_default
+        else
+          privacy_decider(value)
+        end
+      end
+
+      def privacy_decider(value)
+        case value.downcase
+        when 'true'
+          true
+        when 'false'
+          false
+        else # when some invalud value...
+          config.repo_private_default
+        end
+      end
+
+      def create_repo(org, repo, opts)
+        reply = nil
+        begin
+          octo.create_repository(repo, opts)
+        ensure
+          if repo?(rpo(org, repo))
+            repo_url = "https://github.com/#{rpo(org, repo)}"
+            reply = t('repo_create.pass', org: org, repo: repo, repo_url: repo_url)
+          else
+            reply = t('repo_create.fail', org: org, repo: repo)
+          end
+        end
+        reply
+      end
+
+      def delete_repo(org, repo)
+        reply = nil
+        begin
+          octo.delete_repository(rpo(org, repo))
+        ensure
+          if repo?(rpo(org, repo))
+            reply = t('repo_delete.fail', org: org, repo: repo)
+          else
+            reply = t('repo_delete.pass', org: org, repo: repo)
+          end
+        end
+        reply
+      end
+    end
+
+    Lita.register_handler(GithubRepo)
+  end
+end

data/lita-github.gemspec

@@ -40,8 +40,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rubocop', '~> 0.25.0'
   s.add_development_dependency 'rspec', '~> 3.0'
   s.add_development_dependency 'fuubar', '~> 2.0'
-  s.add_development_dependency 'coveralls', '~> 0.7'
-  s.add_development_dependency 'simplecov', '~> 0.9'
+  s.add_development_dependency 'codeclimate-test-reporter', '~> 0.4', '>= 0.4.0'
 
   s.add_runtime_dependency 'lita', '~> 3.3'
   s.add_runtime_dependency 'octokit', '~> 3.3'

data/locales/en.yml

@@ -15,6 +15,7 @@ en:
         repo_delete:
           pass: "Deleted %{org}/%{repo}"
           fail: "Unable to delete %{org}/%{repo}"
+          not_found: "That repo (%{org}/%{repo}) does not exist"
         repo_info:
           reply: "%{repo} (private:%{private}) :: %{url}\nDesc: %{description}\nIssues: %{issues_count} PRs: %{pr_count}"
       github_pr:

data/spec/spec_helper.rb

@@ -16,12 +16,8 @@
 
 require 'rspec'
 require 'lita/rspec'
-# require 'coveralls'
-# require 'simplecov'
+require 'codeclimate-test-reporter'
 
-# SimpleCov.formatter = Coveralls::SimpleCov::Formatter
-# SimpleCov.start do
-#   add_filter 'spec/'
-# end
+CodeClimate::TestReporter.start
 
 require 'lita-github'

data/spec/unit/lita-github/filters_spec.rb

@@ -0,0 +1,31 @@
+# -*- coding: UTF-8 -*-
+
+require 'spec_helper'
+
+describe LitaGithub::Filters do
+  include LitaGithub::Filters
+
+  describe '.func_disabled?' do
+    before do
+      @cfg = double('Lita::Configuration', test_enabled: true)
+      allow(self).to receive(:config).and_return(@cfg)
+    end
+
+    context 'when enabled' do
+      it 'should return false' do
+        expect(func_disabled?(:test)).to be_falsey
+      end
+    end
+
+    context 'when disabled' do
+      before do
+        @cfg = double('Lita::Configuration', test_enabled: false)
+        allow(self).to receive(:config).and_return(@cfg)
+      end
+
+      it 'should return true' do
+        expect(func_disabled?(:test)).to be_truthy
+      end
+    end
+  end
+end

data/spec/unit/lita-github/org_spec.rb

@@ -0,0 +1,32 @@
+# -*- coding: UTF-8 -*-
+
+require 'spec_helper'
+
+describe LitaGithub::Org do
+  include LitaGithub::Org
+
+  describe '.organization' do
+    before do
+      @cfg = double('Lita::Config', default_org: 'GrapeDuty')
+      allow(self).to receive(:config).and_return(@cfg)
+    end
+
+    context 'when name provided is nil' do
+      it 'should use the default' do
+        expect(organization(nil)).to eql 'GrapeDuty'
+      end
+    end
+
+    context 'when name provided is empty string ("")' do
+      it 'should use the default' do
+        expect(organization('')).to eql 'GrapeDuty'
+      end
+    end
+
+    context 'when name provided is not nil or not empty string' do
+      it 'should use the name provided' do
+        expect(organization('testing')).to eql 'testing'
+      end
+    end
+  end
+end

data/spec/unit/lita-github/r_spec.rb

@@ -46,25 +46,41 @@ describe LitaGithub::R do
 
     context 'it should match' do
       it 'test:pass' do
-        expect(subject.match(' test:pass')).to_not be_nil
+        s = ' test:pass'
+        m = s.scan(subject).flatten.compact
+        expect(m).to eql [' test:pass']
       end
 
       it 'test7_pass:pAss_test' do
-        expect(subject.match(' test7_pass:pAss_test')).to_not be_nil
+        s = ' test7_pass:pAss_test'
+        m = s.scan(subject).flatten.compact
+        expect(m).to eql [' test7_pass:pAss_test']
+      end
+
+      it 'test:pass bacon:always test:coverage' do
+        s = ' test:pass bacon:always test:coverage'
+        m = s.scan(subject).flatten.compact
+        expect(m).to eql [' test:pass', ' bacon:always', ' test:coverage']
       end
     end
 
     context 'it should not match' do
       it 'test-stuff:fail' do
-        expect(subject.match(' test-stuff:fail')).to be_nil
+        s = ' test-stuff:fail'
+        m = s.scan(subject).flatten.compact
+        expect(m).to be_empty
       end
 
       it 'test: fail' do
-        expect(subject.match(' test: fail')).to be_nil
+        s = ' test: fail'
+        m = s.scan(subject).flatten.compact
+        expect(m).to be_empty
       end
 
-      it 'test:fail' do
-        expect(subject.match('test:fail')).to be_nil
+      it 'test:fail (no leading space)' do
+        s = 'test:fail'
+        m = s.scan(subject).flatten.compact
+        expect(m).to be_empty
       end
     end
   end

data/spec/unit/lita-github/repo_spec.rb

@@ -0,0 +1,37 @@
+# -*- coding: UTF-8 -*-
+
+require 'spec_helper'
+
+describe LitaGithub::Repo do
+  include LitaGithub::Repo
+
+  describe '.rpo' do
+    it 'should return the provided arguments in a Repo-like string' do
+      org = 'GrapeDuty'
+      repo = 'lita-test'
+      expect(rpo(org, repo)).to eql "#{org}/#{repo}"
+    end
+  end
+
+  describe '.repo?' do
+    before do
+      @octo = double('Octokit::Client', repository?: true)
+      allow(self).to receive(:octo).and_return(@octo)
+    end
+
+    context 'when repo exists' do
+      subject { repo?('GrapeDuty/lita-test') }
+      it { should be_truthy }
+    end
+
+    context 'when repo does not exist' do
+      before do
+        @octo = double('Octokit::Client', repository?: false)
+        allow(self).to receive(:octo).and_return(@octo)
+      end
+
+      subject { repo?('GrapeDuty/lita-test') }
+      it { should be_falsey }
+    end
+  end
+end

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

@@ -0,0 +1,326 @@
+# -*- coding: UTF-8 -*-
+#
+# Copyright 2014 GrapeDuty, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'spec_helper'
+
+describe Lita::Handlers::GithubRepo, lita_handler: true do
+  # repo_create command routing
+  it { routes_command('gh repo create GrapeDuty/lita-test').to(:repo_create) }
+  it { routes_command('gh repo new GrapeDuty/lita-test').to(:repo_create) }
+  it { routes_command('gh repo new lita-test').to(:repo_create) }
+  it { routes_command('gh repo new GrapeDuty/lita-test private:true team:heckman').to(:repo_create) }
+  it { routes_command('gh repo new GrapeDuty/lita-test private:true randomunparseabletext ').to(:repo_create) }
+
+  # repo_delete command routing
+  it { routes_command('gh repo delete GrapeDuty/lita-test').to(:repo_delete) }
+  it { routes_command('gh repo delete lita-test').to(:repo_delete) }
+
+  # repo_info command routing
+  it { routes_command('gh repo info GrapeDuty/lita-test').to(:repo_info) }
+  it { routes_command('gh repo info lita-test').to(:repo_info) }
+
+  let(:github_repo) { Lita::Handlers::GithubRepo.new('robot') }
+  let(:github_org) { 'GrapeDuty' }
+  let(:disabled_reply) { 'Sorry, this function has been disabled in the config' }
+
+  ####
+  # Helper Methods
+  ####
+  describe '.privacy_decider' do
+    before do
+      c_obj = double('Lita::Configuration', repo_private_default: :dummyvalue)
+      allow(github_repo).to receive(:config).and_return(c_obj)
+    end
+
+    it 'should return true when value is "true"' do
+      expect(github_repo.send(:privacy_decider, 'true')).to be_truthy
+    end
+
+    it 'should return true when value is "True"' do
+      expect(github_repo.send(:privacy_decider, 'True')).to be_truthy
+    end
+
+    it 'should return false when value is "false"' do
+      expect(github_repo.send(:privacy_decider, 'false')).to be_falsey
+    end
+
+    it 'should return the default when the value is something unknown' do
+      expect(github_repo.send(:privacy_decider, 'something')).to eql :dummyvalue
+    end
+  end
+
+  describe '.should_repo_be_private?' do
+    before do
+      @c_obj = double('Lita::Configuration', repo_private_default: true)
+      allow(github_repo).to receive(:config).and_return(@c_obj)
+    end
+
+    it 'should return the default if value nil' do
+      expect(@c_obj).to receive(:repo_private_default).and_return(:dummyvalue)
+      expect(github_repo.send(:should_repo_be_private?, nil)).to eql :dummyvalue
+    end
+
+    it 'should return the default if value empty string ("")' do
+      expect(@c_obj).to receive(:repo_private_default).and_return(:dummyvalue)
+      expect(github_repo.send(:should_repo_be_private?, '')).to eql :dummyvalue
+    end
+
+    it 'should call privacy_decider() with the value if not nil or empty' do
+      expect(github_repo).to receive(:privacy_decider).with('ohai').and_return(:dummyvalue)
+      expect(github_repo.send(:should_repo_be_private?, 'ohai')).to eql :dummyvalue
+    end
+  end
+
+  describe '.team_by_slug' do
+    before do
+      @teams = [
+        { id: 1, slug: 'hi' },
+        { id: 42, slug: 'heckman' },
+        { id: 84, slug: 'orwell' }
+      ]
+      @octo_obj = double('Octokit::Client', organization_teams: @teams)
+      allow(github_repo).to receive(:octo).and_return(@octo_obj)
+    end
+
+    it 'should return the team id of the team matching the slug' do
+      expect(@octo_obj).to receive(:organization_teams).with(github_org).and_return(@teams)
+      expect(github_repo.send(:team_by_slug, 'heckman', github_org)).to eql 42
+      expect(github_repo.send(:team_by_slug, 'orwell', github_org)).to eql 84
+      expect(github_repo.send(:team_by_slug, 'unknown', github_org)).to be_nil
+    end
+
+    it 'should return nil if unknown' do
+      expect(github_repo.send(:team_by_slug, 'unknown', 'x')).to be_nil
+    end
+  end
+
+  describe '.extrapolate_create_opts' do
+    before do
+      @eco_opts = {}
+      @c_obj = double('Lita::Configuration', default_team_slug: 'h3ckman')
+      allow(github_repo).to receive(:config).and_return(@c_obj)
+      allow(github_repo).to receive(:team_by_slug).and_return(42)
+    end
+
+    it 'should set the :organization key and :team_id key' do
+      h = { organization: github_org, team_id: 42 }
+      expect(github_repo.send(:extrapolate_create_opts, @eco_opts, github_org)).to eql h
+    end
+
+    context 'when options contains :team and no :team_id' do
+      context 'when given a valid slug' do
+        before { @eco_opts = { team: 'heckman' } }
+
+        it 'should set the :team_id key' do
+          h = { organization: github_org, team_id: 84 }.merge!(@eco_opts)
+          expect(github_repo).to receive(:team_by_slug).with('heckman', github_org).and_return(84)
+          expect(github_repo.send(:extrapolate_create_opts, @eco_opts, github_org)).to eql h
+        end
+      end
+
+      context 'when given an invalid slug' do
+        it 'should set the team to the default' do
+          h = { organization: github_org, team_id: 42 }.merge!(@eco_opts)
+          expect(github_repo).to receive(:team_by_slug).with('h3ckman', github_org).and_return(42)
+          expect(github_repo.send(:extrapolate_create_opts, @eco_opts, github_org)).to eql h
+        end
+      end
+    end
+
+    context 'when there is a :team_id key' do
+      before { @eco_opts = { team_id: 44 } }
+
+      it 'should just leave it alone...' do
+        h = { organization: github_org }.merge!(@eco_opts)
+        expect(github_repo).not_to receive(:team_by_slug)
+        expect(github_repo.send(:extrapolate_create_opts, @eco_opts, github_org)).to eql h
+      end
+    end
+  end
+
+  describe '.repo_match' do
+    let(:resp_obj) do
+      md_mock = { 'org' => github_org, 'repo' => 'lita-test' }
+      double('Lita::Response', match_data: md_mock)
+    end
+
+    it 'should return the Org/Repo match' do
+      expect(github_repo.send(:repo_match, resp_obj)).to eql [github_org, 'lita-test']
+    end
+  end
+
+  describe '.command_opts' do
+    it 'should find the valid options' do
+      o = ' private:true team:heckman bacon:always bacon:sometimes'
+      co = github_repo.send(:command_opts, o)
+      expect(co).to be_an_instance_of Hash
+      expect(co[:private]).to eql 'true'
+      expect(co[:team]).to eql 'heckman'
+      expect(co[:bacon]).to eql 'always' # of course it's always
+    end
+  end
+
+  describe '.create_repo' do
+    before do
+      allow(github_repo).to receive(:octo).and_return(double('Octokit::Client', create_repository: nil))
+    end
+
+    context 'when repo created' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(true)
+      end
+
+      it 'should confirm succesfful creation' do
+        opts = { private: true, team_id: 42, organization: github_org }
+        expect(github_repo.send(:create_repo, github_org, 'lita-test', opts))
+          .to eql 'Created GrapeDuty/lita-test: https://github.com/GrapeDuty/lita-test'
+      end
+    end
+
+    context 'when repo not created' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(false)
+      end
+
+      it 'should confirm failure' do
+        opts = { private: true, team_id: 42, organization: github_org }
+        expect(github_repo.send(:create_repo, github_org, 'lita-test', opts))
+          .to eql 'Unable to create GrapeDuty/lita-test'
+      end
+    end
+  end
+
+  describe '.delete_repo' do
+    before do
+      allow(github_repo).to receive(:octo).and_return(double('Octokit::Client', delete_repository: nil))
+    end
+
+    context 'when repo deleted' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(false)
+      end
+
+      it 'should confirm successful delete' do
+        expect(github_repo.send(:delete_repo, github_org, 'lita-test'))
+          .to eql 'Deleted GrapeDuty/lita-test'
+      end
+    end
+
+    context 'when repo not deleted' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(true)
+      end
+
+      it 'should reply with failure message' do
+        expect(github_repo.send(:delete_repo, github_org, 'lita-test'))
+          .to eql 'Unable to delete GrapeDuty/lita-test'
+      end
+    end
+  end
+
+  ####
+  # Handlers
+  ####
+  describe '.repo_info' do
+    before do
+      repo = {
+        full_name: "#{github_org}/lita-test",
+        description: 'unit testing',
+        private: true,
+        html_url: "https://stubbed.github.com/#{github_org}/lita-test",
+        open_issues_count: 10
+      }
+      pr = [nil, nil, nil, nil, nil]
+      @octo_obj = double('Octokit::Client', repository: repo, pull_requests: pr)
+      allow(github_repo).to receive(:octo).and_return(@octo_obj)
+    end
+
+    it 'should return some repo info' do
+      send_command('gh repo info GrapeDuty/lita-test')
+      r = "GrapeDuty/lita-test (private:true) :: https://stubbed.github.com/#{github_org}/lita-test\n" \
+          "Desc: unit testing\n" \
+          'Issues: 5 PRs: 5'
+      expect(replies.last).to eql r
+    end
+  end
+
+  describe '.repo_delete' do
+    before do
+      allow(github_repo).to receive(:func_disabled?).and_return(false)
+      allow(github_repo).to receive(:delete_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 delete_repo()' do
+      send_command("gh repo delete #{github_org}/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 delete #{github_org}/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 delete #{github_org}/lita-test")
+        expect(replies.last).to eql 'That repo (GrapeDuty/lita-test) does not exist'
+      end
+    end
+  end
+
+  describe '.repo_create' do
+    before do
+      @opts = { private: true, team_id: 42, organization: github_org }
+      allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(false)
+      allow(github_repo).to receive(:extrapolate_create_opts).and_return(@opts)
+      allow(github_repo).to receive(:create_repo).and_return('hello from PAX prime!')
+    end
+
+    context 'when repo already exists' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(true)
+      end
+
+      it 'should tell you it already exists' do
+        send_command("gh repo create #{github_org}/lita-test")
+        expect(replies.last).to eql 'Unable to create GrapeDuty/lita-test as it already exists'
+      end
+    end
+
+    context 'when repo does not exist' do
+      before do
+        allow(github_repo).to receive(:repo?).with("#{github_org}/lita-test").and_return(false)
+      end
+
+      it 'should reply with the return of create_repo()' do
+        expect(github_repo).to receive(:extrapolate_create_opts).and_return(@opts)
+        send_command("gh repo create #{github_org}/lita-test")
+        expect(replies.last).to eql 'hello from PAX prime!'
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-github
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Tim Heckman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-27 00:00:00.000000000 Z
+date: 2014-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -81,33 +81,25 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: coveralls
+  name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.7'
-- !ruby/object:Gem::Dependency
-  name: simplecov
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+        version: '0.4'
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: 0.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.4.0
 - !ruby/object:Gem::Dependency
   name: lita
   requirement: !ruby/object:Gem::Requirement
@@ -162,23 +154,30 @@ files:
 - ".rubocop.yml"
 - ".travis.yml"
 - Gemfile
-- Gemfile.lock
 - LICENSE
 - README.md
 - Rakefile
 - lib/lita-github.rb
 - lib/lita-github/config.rb
+- lib/lita-github/filters.rb
 - lib/lita-github/octo.rb
+- lib/lita-github/org.rb
 - lib/lita-github/r.rb
+- lib/lita-github/repo.rb
 - lib/lita-github/version.rb
 - lib/lita/handlers/github.rb
+- lib/lita/handlers/github_repo.rb
 - lita-github.gemspec
 - locales/en.yml
 - spec/spec_helper.rb
 - spec/unit/lita-github/config_spec.rb
+- spec/unit/lita-github/filters_spec.rb
 - spec/unit/lita-github/octo_spec.rb
+- spec/unit/lita-github/org_spec.rb
 - spec/unit/lita-github/r_spec.rb
+- spec/unit/lita-github/repo_spec.rb
 - spec/unit/lita-github/version_spec.rb
+- spec/unit/lita/handlers/github_repo_spec.rb
 - spec/unit/lita/handlers/github_spec.rb
 homepage: https://github.com/PagerDuty/lita-github
 licenses:
@@ -208,7 +207,11 @@ summary: GitHub repo/org management in Lita; handler not ready for production
 test_files:
 - spec/spec_helper.rb
 - spec/unit/lita-github/config_spec.rb
+- spec/unit/lita-github/filters_spec.rb
 - spec/unit/lita-github/octo_spec.rb
+- spec/unit/lita-github/org_spec.rb
 - spec/unit/lita-github/r_spec.rb
+- spec/unit/lita-github/repo_spec.rb
 - spec/unit/lita-github/version_spec.rb
+- spec/unit/lita/handlers/github_repo_spec.rb
 - spec/unit/lita/handlers/github_spec.rb

data/Gemfile.lock

@@ -1,113 +0,0 @@
-PATH
-  remote: .
-  specs:
-    lita-github (0.0.1)
-      lita (~> 3.3)
-      lita-confirmation
-      octokit (~> 3.3)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    addressable (2.3.6)
-    ast (2.0.0)
-    coveralls (0.7.0)
-      multi_json (~> 1.3)
-      rest-client
-      simplecov (>= 0.7)
-      term-ansicolor
-      thor
-    diff-lcs (1.2.5)
-    docile (1.1.5)
-    faraday (0.9.0)
-      multipart-post (>= 1.2, < 3)
-    fuubar (2.0.0)
-      rspec (~> 3.0)
-      ruby-progressbar (~> 1.4)
-    http_router (0.11.1)
-      rack (>= 1.0.0)
-      url_mount (~> 0.2.1)
-    i18n (0.6.11)
-    ice_nine (0.11.0)
-    lita (3.3.1)
-      bundler (>= 1.3)
-      faraday (>= 0.8.7)
-      http_router (>= 0.11.1)
-      i18n (>= 0.6.9)
-      ice_nine (>= 0.11.0)
-      multi_json (>= 1.7.7)
-      puma (>= 2.7.1)
-      rack (>= 1.5.2)
-      rb-readline (>= 0.5.1)
-      redis-namespace (>= 1.3.0)
-      thor (>= 0.18.1)
-    lita-confirmation (0.0.1)
-      lita (>= 3.3)
-    mime-types (2.3)
-    multi_json (1.10.1)
-    multipart-post (2.0.0)
-    netrc (0.7.7)
-    octokit (3.3.1)
-      sawyer (~> 0.5.3)
-    parser (2.2.0.pre.4)
-      ast (>= 1.1, < 3.0)
-      slop (~> 3.4, >= 3.4.5)
-    powerpack (0.0.9)
-    puma (2.9.0)
-      rack (>= 1.1, < 2.0)
-    rack (1.5.2)
-    rainbow (2.0.0)
-    rake (10.3.2)
-    rb-readline (0.5.1)
-    redis (3.1.0)
-    redis-namespace (1.5.1)
-      redis (~> 3.0, >= 3.0.4)
-    rest-client (1.7.2)
-      mime-types (>= 1.16, < 3.0)
-      netrc (~> 0.7)
-    rspec (3.0.0)
-      rspec-core (~> 3.0.0)
-      rspec-expectations (~> 3.0.0)
-      rspec-mocks (~> 3.0.0)
-    rspec-core (3.0.4)
-      rspec-support (~> 3.0.0)
-    rspec-expectations (3.0.4)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.0.0)
-    rspec-mocks (3.0.4)
-      rspec-support (~> 3.0.0)
-    rspec-support (3.0.4)
-    rubocop (0.25.0)
-      parser (>= 2.2.0.pre.4, < 3.0)
-      powerpack (~> 0.0.6)
-      rainbow (>= 1.99.1, < 3.0)
-      ruby-progressbar (~> 1.4)
-    ruby-progressbar (1.5.1)
-    sawyer (0.5.5)
-      addressable (~> 2.3.5)
-      faraday (~> 0.8, < 0.10)
-    simplecov (0.9.0)
-      docile (~> 1.1.0)
-      multi_json
-      simplecov-html (~> 0.8.0)
-    simplecov-html (0.8.0)
-    slop (3.6.0)
-    term-ansicolor (1.3.0)
-      tins (~> 1.0)
-    thor (0.19.1)
-    tins (1.3.0)
-    url_mount (0.2.1)
-      rack
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  bundler (~> 1.5)
-  coveralls (~> 0.7)
-  fuubar (~> 2.0)
-  lita-github!
-  rake (~> 10.2)
-  rspec (~> 3.0)
-  rubocop (~> 0.25.0)
-  simplecov (~> 0.9)