lita-jls 0.0.11

data/spec/lita-jls/github_url_parser_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+require 'lita-jls/github_url_parser'
+
+describe LitaJLS::GithubUrlParser do
+  subject { LitaJLS::GithubUrlParser }
+
+  context 'when the url is a repository' do
+    let(:url) { 'https://github.com/elasticsearch/logstash' }
+    let(:parser) { subject.parse(url) }
+
+    it 'extract the different parts of the url' do
+      expect(parser.user).to eq('elasticsearch')
+      expect(parser.project).to eq('logstash')
+      expect(parser.pr).to eq(nil)
+    end
+
+    it 'returns the git url of the repository' do
+      expect(parser.git_url).to eq('https://github.com/elasticsearch/logstash')
+    end
+
+    it 'should validates the presence of #url, #project and valid github url' do
+      expect { parser.validate! }.not_to raise_error
+    end
+
+
+    it 'raise an exception if the domain is not on github' do
+      subject.parse('http://gitlab/blah/test')
+      expect { subject.validate! }.to raise_error
+    end
+    
+    it 'raise an exception if it cannot extract the user' do
+      subject.parse('http://github.com/')
+      expect { subject.validate! }.to raise_error
+    end
+
+    it 'raise an exception if it cannot extract the project' do
+      subject.parse('http://github.com/elasticsearch')
+      expect { subject.validate! }.to raise_error
+    end
+  end
+
+  context 'when the url is pull request' do
+    let(:url) { 'https://github.com/elasticsearch/logstash/pull/2' }
+    let(:parser) { subject.parse(url, { :link => :pull_request }) }
+
+    it 'extract the different parts of the url' do
+      expect(parser.user).to eq('elasticsearch')
+      expect(parser.project).to eq('logstash')
+      expect(parser.pr).to eq('2')
+    end
+
+    it 'returns the git url of the repository' do
+      expect(parser.git_url).to eq('https://github.com/elasticsearch/logstash')
+    end
+
+    it 'should validates the presence of #url, #project, #pr and valid github url' do
+      expect { parser.validate! }.not_to raise_error
+    end
+
+    it 'raise an exception if the domain is not on github' do
+      subject.parse('http://gitlab/blah/test/pull/2', { :link => :pull_request })
+      expect { subject.validate! }.to raise_error
+    end
+
+    it 'raise an exception if it cannot extract the user' do
+      subject.parse('http://github.com/', { :link => :pull_request })
+      expect { subject.validate! }.to raise_error
+    end
+
+    it 'raise an exception if it cannot extract the project' do
+      subject.parse('http://github.com/elasticsearch',  { :link => :pull_request })
+      expect { subject.validate! }.to raise_error
+    end
+
+    it 'raise an exception if it cannot extract the pull request number' do
+      subject.parse('http://github.com/elasticsearch/logstash/pull/',  { :link => :pull_request })
+      expect { subject.validate! }.to raise_error
+    end
+  end
+end

data/spec/lita/handlers/jls_spec.rb

@@ -0,0 +1,211 @@
+require "spec_helper"
+require "insist"
+require "stud/temporary"
+require "fileutils"
+
+describe Lita::Handlers::Jls, :lita_handler => true do
+  it do
+    messages = [
+      "merge https://github.com/foo/bar/pull/123 bar",
+      "merge https://github.com/user/foo/pull/123 bar 1.x master",
+      "merge https://github.com/elasticsearch/fancypants/pull/123 bar",
+      "merge https://github.com/some-test/another-thing/pull/123 bar baz fizz",
+      "merge? https://github.com/some-test/another-thing/pull/123 bar fancy-pants",
+    ]
+    messages.each do |m|
+      is_expected.to route_command(m).to(:merge)
+    end
+  end
+
+  it "routes `(tableflip)` to :tableflip" do
+    is_expected.to route_command("(tableflip)").to(:tableflip)
+  end
+
+  it "routes `publish` to :publish" do
+    is_expected.to route_command("publish https://github.com/foo/bar").with_authorization_for(:logstash).to(:publish)
+  end
+
+  it "routes `why computer so bad` to :pop_exception" do
+    is_expected.to route_command("why computer so bad?").to(:pop_exception)
+    is_expected.to route_command("why computers so bad?").to(:pop_exception)
+    is_expected.to route_command("explain").to(:pop_exception)
+  end
+
+  it "routes migrate_pr to :migrate_pr with proper arguments" do
+    is_expected.to route_command("migrate_pr https://github.com/elasticsearch/logstash/pull/1452 "\
+      "https://github.com/logstash-plugins/logstash-codec-line").to(:migrate_pr)
+  end
+
+  context "test migrate dummy pr", :network => true do
+    it "does migrate_pr correctly" do
+      VCR.use_cassette("successful_migrate_pr") do
+        send_command("migrate_pr https://github.com/elasticsearch/logstash/pull/1452 "\
+          "https://github.com/logstash-plugins/logstash-codec-line")
+      end
+    end
+  end
+
+  context "when dealing with exceptions", :network => true do
+    let(:bad_exception) { double("exception", :backtrace => 'line X', :message => '0 != 1', :exception => 'not working') }
+
+    it 'should pop the exception saved to redis' do
+      subject.push_exception(bad_exception)
+      send_command("Why computers so bad?")
+
+      expect(replies[1]).to match(/^exception: #{bad_exception.exception}/)
+      expect(replies[2]).to match(/^message: #{bad_exception.message}/)
+      expect(replies[3]).to match(/^backtrace: #{bad_exception.backtrace}/)
+    end
+
+    it 'doesnt return an exception if the list is empty' do
+      send_command("Why computers so bad?")
+      expect(replies[1]).to match(/No exception saved/)
+    end
+
+    it 'permits to add context keys to the exception' do
+      project = 'elasticsearch/jarvis'
+      subject.push_exception(bad_exception, :project => project)
+
+      send_command("Why computers so bad?")
+      expect(replies.last).to match(/^project: #{project}/)
+    end
+  end
+
+  context "#merge", :network => true do
+    before do
+      original_git = subject.method(:git)
+
+      allow(subject.config).to receive(:cla_uri).and_return(ENV['ELASTICSEARCH_CLA_URL'])
+
+      allow(subject).to receive(:git).with(any_args) do |gitdir, *args|
+        # Ignore any 'git push' attempts
+        if args[0] == "push" # git("push", ...)
+          nil
+        else
+          original_git.call(gitdir, *args)
+        end
+      end
+    end
+
+    it "should reply successfully if the merge works" do
+      VCR.use_cassette("successful_clacheck", :tag => :internal, :match_requests_on => [:path]) do
+        allow(subject).to receive(:github_issue_label) { nil }
+        allow(subject).to receive(:github_issue_comment) { nil }
+
+        expect(subject).to receive(:github_issue_label).with("jordansissel/this-is-only-a-test", 1, [])
+
+        send_command("merge https://github.com/jordansissel/this-is-only-a-test/pull/1 master")
+        insist { replies.last } == "(success) jordansissel/this-is-only-a-test#1 merged into: master"
+      end
+    end
+
+    it "should properly handle long commit messages" do
+      VCR.use_cassette("successful_clacheck_long_commit", :tag => :internal, :match_requests_on => [:path]) do
+        allow(subject).to receive(:github_issue_comment) { nil }
+        send_command("merge https://github.com/jordansissel/this-is-only-a-test/pull/4 master")
+        insist { replies.last } == "(success) jordansissel/this-is-only-a-test#4 merged into: master"
+
+        repodir = subject.instance_eval { gitdir("this-is-only-a-test") }
+        Dir.chdir(repodir) do
+          log = `git log --format="%B" -n 1 HEAD`.chomp
+          insist { log } == "One two three four five six seven eight.
+
+Nine ten eleven.
+
+Twelve?
+* Thirteen
+* Fourteen
+
+Fixes #4
+"
+        end
+      end
+    end
+  end
+end
+
+if ENV["DEBUG"]
+  logger = Cabin::Channel.get
+  logger.level = :debug if ENV["DEBUG"]
+end
+
+module Fixture
+  class Util
+    include LitaJLS::Util
+    public(:gitdir, :clone_at, :github_issue_label)
+
+    def workdir(*args)
+      return @workdir if @workdir
+      @workdir = Stud::Temporary.directory("lita-jls-testing")
+    end
+  end
+end
+
+#module Stud::Temporary
+  #alias_method :directory_orig, :directory
+  #def directory(*args)
+    #r = directory_orig(*args)
+    #puts r => caller[0]
+    #r
+  #end
+#end
+
+describe LitaJLS::Util do
+  subject do
+    Fixture::Util.new
+  end
+
+  after do
+    FileUtils.rm_rf(subject.workdir)
+  end
+
+  context "#gitdir" do
+    let(:dir) { subject.gitdir("whatever") }
+    after { Dir.rmdir(dir) }
+    it "should be a string" do
+      insist { dir }.is_a?(String)
+    end
+    it "starts with the workdir" do
+      insist { dir } =~ Regexp.new("^" + Regexp.escape(subject.workdir))
+    end
+  end
+
+  context "#clone_at" do
+    let(:url) { 'https://github.com/jordansissel/this-is-only-a-test' }
+    let(:repo) { Stud::Temporary.directory('lita-jls-testing') }
+
+    before do
+      insist { repo } =~ /lita-jls-testing/ # just in case.
+    end
+
+    after do
+      FileUtils.rm_rf(repo)
+    end
+
+    it "should clone" do
+      subject.clone_at(url, repo)
+      expect(File.directory?(repo)).to eq(true)
+      insist { File }.directory?(File.join(repo, ".git"))
+    end
+  end
+
+  context "#github_issue_label" do
+    let(:issue) do
+      expect(subject.github_client).to receive(:add_labels_to_an_issue).with("#{user}/#{project}", pr.to_i, labels)
+      github_issue_label("jordansissel/this-is-only-a-test", 1, [ "one", "two", "three" ])
+    end
+
+    it 'should not raise and exception if the label is an empty list' do
+      expect { subject.github_issue_label("jordansissel/this-is-only-a-test", 1, []) }.not_to raise_error
+    end
+  end
+
+  context "#workdir" do
+    it "should return the same value on multiple invocations" do
+      expected = subject.workdir("foo")
+      5.times do
+        insist { subject.workdir("foo") } == expected
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,31 @@
+require "simplecov"
+require "coveralls"
+require "vcr"
+require "lita/rspec"
+require "lita-jls"
+require "uri"
+
+Lita.version_3_compatibility_mode = false
+
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter
+]
+
+SimpleCov.start { add_filter "/spec/" }
+
+VCR.configure do |config|
+  config.cassette_library_dir = File.join(File.dirname(__FILE__), 'fixtures', 'vcr_cassettes')
+  config.hook_into :webmock
+ # config.debug_logger = true
+  
+  config.before_record do |interaction|
+    interaction.response.headers.delete('Set-Cookie')
+    interaction.request.headers.delete('Authorization')
+  end
+
+  config.before_record(:internal) do |interaction|
+    uri = URI.parse(interaction.request.uri)
+    interaction.request.uri.sub!(/:\/\/.*#{Regexp.escape(uri.host)}/, "://test.local" )
+  end
+end

metadata

@@ -0,0 +1,338 @@
+--- !ruby/object:Gem::Specification
+name: lita-jls
+version: !ruby/object:Gem::Version
+  version: 0.0.11
+platform: ruby
+authors:
+- Jordan Sissel
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-01-23 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: lita
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.3'
+- !ruby/object:Gem::Dependency
+  name: cabin
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: faraday
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: octokit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: netrc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mbox
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: insist
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: gems
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.8.3
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.8.3
+- !ruby/object:Gem::Dependency
+  name: semverly
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: stud
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: vcr
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Some stuff for the lita.io bot
+email:
+- jls@semicomplete.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- Gemfile
+- LICENSE
+- README.md
+- Rakefile
+- lib/lita-jls.rb
+- lib/lita-jls/bot_builder.rb
+- lib/lita-jls/github_url_parser.rb
+- lib/lita-jls/repository.rb
+- lib/lita-jls/util.rb
+- lib/lita/handlers/jls.rb
+- lita-jls.gemspec
+- locales/en.yml
+- spec/fixtures/bad_project/Gemfile
+- spec/fixtures/logstash-codec-edn/logstash-codec-edn.gemspec
+- spec/fixtures/logstash.gemspec
+- spec/fixtures/project_with_version_file/lib/testmore/version.rb
+- spec/fixtures/project_with_version_file/project_with_version_file.gemspec
+- spec/fixtures/vcr_cassettes/fetch_version_of_logstash-output-s3.yml
+- spec/fixtures/vcr_cassettes/gem_doesnt_exist.yml
+- spec/fixtures/vcr_cassettes/successful_clacheck.yml
+- spec/fixtures/vcr_cassettes/successful_clacheck_long_commit.yml
+- spec/fixtures/vcr_cassettes/successful_migrate_pr.yml
+- spec/lita-jls/bot_builder_spec.rb
+- spec/lita-jls/github_url_parser_spec.rb
+- spec/lita/handlers/jls_spec.rb
+- spec/spec_helper.rb
+homepage: http://example.com/
+licenses:
+- MIT
+metadata:
+  lita_plugin_type: handler
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.4.3
+signing_key: 
+specification_version: 4
+summary: Some stuff for the lita.io bot
+test_files:
+- spec/fixtures/bad_project/Gemfile
+- spec/fixtures/logstash-codec-edn/logstash-codec-edn.gemspec
+- spec/fixtures/logstash.gemspec
+- spec/fixtures/project_with_version_file/lib/testmore/version.rb
+- spec/fixtures/project_with_version_file/project_with_version_file.gemspec
+- spec/fixtures/vcr_cassettes/fetch_version_of_logstash-output-s3.yml
+- spec/fixtures/vcr_cassettes/gem_doesnt_exist.yml
+- spec/fixtures/vcr_cassettes/successful_clacheck.yml
+- spec/fixtures/vcr_cassettes/successful_clacheck_long_commit.yml
+- spec/fixtures/vcr_cassettes/successful_migrate_pr.yml
+- spec/lita-jls/bot_builder_spec.rb
+- spec/lita-jls/github_url_parser_spec.rb
+- spec/lita/handlers/jls_spec.rb
+- spec/spec_helper.rb