ping 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b72814d428ea3a2db8f4c7aafd0df25d4be72402bb4f1345da8f24635d07a17
-  data.tar.gz: 200b1aed0640d89e91730c7d29b7356e42ab19074c06f9d8c3b855814a4915c9
+  metadata.gz: ac62acfe2a85d874917d0af6fbbb36f0eda901912c097f84d7900174ceccdd1d
+  data.tar.gz: 82c14c8e4b00b70c99dcd1dd76c4bbebffeed6732102f3d1185dd32d3332a391
 SHA512:
-  metadata.gz: b5a1a8053177c0ee4395ddd325c7acad96a7500f3190069dc50c54a95d00ee8fe339a94b2482ecf2d1cf523f5517156d7059454e81bb8a066d0add270c0e074b
-  data.tar.gz: d6d88258ce8b7e886eafad4888db0cf1906585fbf56375f87495127f3a13a0bd8dce9b21a5255f82e248ed65256481900241c0a46024d0d34eb04a4365314458
+  metadata.gz: fa87a0acd7bd37dc8d80a5ce8fa5d4efbe3e271f906746b8127ba4d21613446f10074e56d56a6e59d3575127dca2c1ba5648b93f70c73406587016521b3204dd
+  data.tar.gz: e2b424198f0672f8785c3c8a9dd8da0b8a3a0407f72fdf7a1123dd3e398495ff1a6052d9cadc8331f8c9e805af8914190e9489d0c6ea1618ff2891f8686ea0f4

data/.circleci/config.yml

@@ -15,6 +15,10 @@ jobs:
     steps:
       - checkout
 
+      - run:
+          name: install bundler
+          command: gem install bundler:2.1.4
+
       - run:
           name: check dependencies
           command: |

data/.rubocop.yml

@@ -1,11 +1,44 @@
-inherit_from: .rubocop_todo.yml
+AllCops:
+  TargetRubyVersion: 2.4
+  Exclude:
+    - 'bin/**/*'
+    - 'vendor/**/*'
+    - 'config.ru'
 
-Style/RegexpLiteral:
-  AllowInnerSlashes: true
+Layout/LineLength:
+  Max: 100
 
-AllCops:
+Metrics/AbcSize:
+  Max: 20
+
+Metrics/BlockLength:
+  Max: 30
+  Exclude:
+    - 'test/**/*.rb'
+
+Metrics/ClassLength:
+  Exclude:
+    - 'test/**/*.rb'
+
+Metrics/MethodLength:
+  Max: 15
   Exclude:
-    - 'test/ping/*'
-    - 'ping.gemspec'
-    - 'lib/ping/version.rb'
-    - 'vendor/**/*'
+    - 'test/**/*.rb'
+
+Style/ClassAndModuleChildren:
+  Exclude:
+    - 'test/**/*.rb'
+
+Style/Documentation:
+  Enabled: false
+  Exclude:
+    - 'test/**/*'
+
+Style/HashEachMethods:
+  Enabled: true
+
+Style/HashTransformKeys:
+  Enabled: true
+
+Style/HashTransformValues:
+  Enabled: true

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in ping.gemspec

data/Rakefile

@@ -1,20 +1,18 @@
-#!/usr/bin/env rake
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rake/testtask'
+require 'rubocop/rake_task'
 
+desc 'Run tests'
+task default: :test
 Rake::TestTask.new do |t|
   t.libs << 'lib'
   t.pattern = 'test/**/*_test.rb'
   t.verbose = true
 end
 
-desc 'Run tests'
-task default: :test
-
 # append to the default rake task
-require 'rubocop/rake_task'
 RuboCop::RakeTask.new
 
-task :default do
-  Rake::Task['rubocop'].invoke
-end
+Rake::Task['test'].enhance ['rubocop']

data/lib/ping.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require 'ping/version'
 require 'ping/parser'

data/lib/ping/config.rb

@@ -1,7 +1,7 @@
+# frozen_string_literal: true
+
 module Ping
   class << self
-    attr_accessor :config
-
     def configure
       yield config
     end
@@ -11,7 +11,7 @@ module Ping
     end
 
     # Here we are implementing a reset method because
-    # for global values like this, it’s best to clean up before/after
+    # for global values like this, it's best to clean up before/after
     # each spec to ensure the system is back to a default state.
     def reset
       @config = Config.new
@@ -21,10 +21,10 @@ module Ping
   class Config
     attr_accessor :qualifiers
 
-    DEFAULT_QUALIFIERS = [
-      'close', 'closes', 'closed', 'fix', 'fixes', 'fixed', 'need', 'needs', 'needed',
-      'require', 'requires', 'required', 'resolve', 'resolves', 'resolved'
-    ]
+    DEFAULT_QUALIFIERS = %w[
+      close closes closed fix fixes fixed need needs needed
+      require requires required resolve resolves resolved
+    ].freeze
 
     def initialize
       @qualifiers = DEFAULT_QUALIFIERS

data/lib/ping/issue_reference.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module Ping
   class IssueReference
     attr_accessor :qualifier, :repository, :number
 
-    REPOSITORY_NAME = /[a-z0-9][a-z0-9\-]*\/[a-z0-9][a-z0-9\-_]*/ix
+    REPOSITORY_NAME = %r{[a-z0-9][a-z0-9\-]*/[a-z0-9][a-z0-9\-_]*}ix.freeze
 
     class << self
       def qualifier_regex
@@ -40,12 +42,12 @@ module Ping
       # - needs https://github.com/codetree/feedback/issues/123
       # - etc...
       def url_pattern
-        /
+        %r{
           (?:^|\W)                         # beginning of string or non-word char
           (?:(#{qualifier_regex})(?:\s))?  # qualifier (optional)
-          https:\/\/github.com\/
+          https://github.com/
           (#{REPOSITORY_NAME})             # repository name
-          \/(?:issues|pulls)\/
+          /(?:issues|pulls)/
           (\d+)                            # issue number
           (?=
             \.+[ \t]|                      # dots followed by space or non-word character
@@ -53,7 +55,7 @@ module Ping
             [^0-9a-zA-Z_.]|                # non-word character except dot
             $                              # end of line
           )
-        /ix
+        }ix
       end
 
       def extract(text)
@@ -82,10 +84,9 @@ module Ping
         return replacement unless replacement.is_a?(IssueReference)
 
         # Reformat the given issue reference replacement to match
-        new_phrase = match[0] == ' ' ? ' ' : '' # fix leading space
-        new_phrase << replacement.qualifier + ' ' if replacement.qualifier
-        new_phrase << replacement.repository.to_s
-        new_phrase << '#' + replacement.number.to_s
+        lead_in = match[0] == ' ' ? ' ' : '' # fix leading space
+        lead_in += replacement.qualifier + ' ' if replacement.qualifier
+        lead_in + replacement.repository.to_s + '#' + replacement.number.to_s
       end
     end
 

data/lib/ping/mention.rb

@@ -1,18 +1,20 @@
+# frozen_string_literal: true
+
 module Ping
   class Mention
     attr_accessor :username
 
-    PATTERN = /
+    PATTERN = %r{
       (?:^|\W)                    # beginning of string or non-word char
       @((?>[a-z0-9][a-z0-9-]*))   # @username
-      (?!\/)                      # without a trailing slash
+      (?!/)                      # without a trailing slash
       (?=
         \.+[ \t]|                 # dots followed by space or non-word character
         \.+$|                     # dots at end of line
         [^0-9a-zA-Z_.]|           # non-word character except dot
         $                         # end of line
       )
-    /ix
+    }ix.freeze
 
     def initialize(username)
       @username = username
@@ -25,8 +27,8 @@ module Ping
           end
     end
 
-    def ==(username)
-      username == self.username
+    def ==(other)
+      other == username
     end
 
     def to_s

data/lib/ping/parser.rb

@@ -1,4 +1,6 @@
-require "ping/config"
+# frozen_string_literal: true
+
+require 'ping/config'
 require 'ping/mention'
 require 'ping/issue_reference'
 

data/lib/ping/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Ping
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

data/ping.gemspec

@@ -1,5 +1,6 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'ping/version'
 
@@ -12,16 +13,16 @@ Gem::Specification.new do |spec|
   spec.description            = 'A little library for parsing GitHub @mentions and issue references'
   spec.homepage               = 'https://github.com/codetree/ping'
   spec.license                = 'MIT'
-  spec.required_ruby_version  = '>= 2.2.2'
-  spec.files                  = `git ls-files -z`.split("\x0")
-  spec.executables            = spec.files.grep(/^bin\//) { |f| File.basename(f) }
-  spec.test_files             = spec.files.grep(/^(test|spec|features)\//)
+  spec.required_ruby_version  = '>= 2.4'
   spec.require_paths          = ['lib']
 
-  spec.add_development_dependency 'bundler', '~> 1.6'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.test_files = spec.files.grep(%r{^(test|spec|features)/})
+
+  spec.add_development_dependency 'bundler', '~> 2.1', '>= 2.1.4'
   spec.add_development_dependency 'minitest', '~> 5.0'
-  spec.add_development_dependency 'shoulda-context', '~> 1.2'
   spec.add_development_dependency 'mocha', '~> 1.0'
-  spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rubocop', '~> 0.80.1'
+  spec.add_development_dependency 'shoulda-context', '~> 1.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ping
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Codetree
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-15 00:00:00.000000000 Z
+date: 2020-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -17,84 +17,90 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '2.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '2.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.1.4
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: shoulda-context
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
-  name: mocha
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 0.80.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 0.80.1
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: shoulda-context
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.2'
 description: A little library for parsing GitHub @mentions and issue references
 email:
 - support@codetree.com
@@ -107,7 +113,6 @@ files:
 - ".circleci/setup-rubygems.sh"
 - ".gitignore"
 - ".rubocop.yml"
-- ".rubocop_todo.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -119,11 +124,6 @@ files:
 - lib/ping/parser.rb
 - lib/ping/version.rb
 - ping.gemspec
-- test/ping/config_test.rb
-- test/ping/issue_reference_test.rb
-- test/ping/mention_test.rb
-- test/ping/parser_test.rb
-- test/test_helper.rb
 homepage: https://github.com/codetree/ping
 licenses:
 - MIT
@@ -136,7 +136,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.2.2
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -147,9 +147,4 @@ rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Parse @mentions and issue references
-test_files:
-- test/ping/config_test.rb
-- test/ping/issue_reference_test.rb
-- test/ping/mention_test.rb
-- test/ping/parser_test.rb
-- test/test_helper.rb
+test_files: []

data/.rubocop_todo.yml

@@ -1,29 +0,0 @@
-# This configuration was generated by `rubocop --auto-gen-config`
-# on 2015-08-01 15:11:49 -0700 using RuboCop version 0.32.1.
-# The point is for the user to remove these configuration records
-# one by one as the offenses are removed from the code base.
-# Note that changes in the inspected code, or installation of new
-# versions of RuboCop, may require this file to be generated again.
-
-# Offense count: 1
-# Configuration parameters: CountComments.
-Metrics/ClassLength:
-  Max: 335
-
-# Offense count: 7
-# Configuration parameters: AllowURI, URISchemes.
-Metrics/LineLength:
-  Max: 93
-
-# Offense count: 4
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/ClassAndModuleChildren:
-  Enabled: false
-
-# Offense count: 9
-Style/Documentation:
-  Enabled: false
-
-# Offense count: 1
-Style/OpMethod:
-  Enabled: false

data/test/ping/config_test.rb

@@ -1,41 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper.rb'
-
-class Ping::ConfigTest < MiniTest::Test
-  def setup
-    Ping.reset
-  end
-
-  context '#default qualifiers' do
-    should "have default qualifiers" do
-      default_qualifiers = Ping::Config::DEFAULT_QUALIFIERS
-
-      Ping.config.qualifiers.each do |q|
-        assert default_qualifiers.include?(q)
-      end
-    end
-  end
-
-  context '#with custom configuration' do
-    setup do
-      @custom_qualifiers = %w(fix epic needed-by close)
-
-      Ping.configure do |config|
-        config.qualifiers = @custom_qualifiers
-      end
-    end
-
-    should 'include custom qualifiers' do
-      Ping.config.qualifiers.each do |q|
-        assert @custom_qualifiers.include?(q)
-      end
-    end
-
-    should 'not include unknown qualifiers' do
-      unknown_qualifiers = %w(resolve require done)
-
-      unknown_qualifiers.each do |q|
-        assert !Ping.config.qualifiers.include?(q)
-      end
-    end
-  end
-end

data/test/ping/issue_reference_test.rb

@@ -1,481 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper.rb'
-
-class Ping::IssueReferenceTest < MiniTest::Test
-  def setup
-    Ping.reset
-  end
-
-  def extract(text)
-    Ping::IssueReference.extract(text)
-  end
-
-  def extract_first(text)
-    Ping::IssueReference.extract(text).first
-  end
-
-  context '.extract with standard syntax' do
-    should 'extract single issue references' do
-      text = 'See #43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract single issue references followed by a period' do
-      text = 'See #43.'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract close qualifiers' do
-      %w(fix fixes fixed close closes closed resolve resolves resolved).each do |q|
-        text = "#{q} #55"
-        issue = extract_first(text)
-
-        assert_equal q, issue.qualifier
-        assert_nil issue.repository
-        assert_equal '55', issue.number
-      end
-    end
-
-    should 'extract dependency qualifiers' do
-      %w(need needs needed require requires required).each do |q|
-        text = "#{q} #123"
-        issue = extract_first(text)
-
-        assert_equal q, issue.qualifier
-        assert_nil issue.repository
-        assert_equal '123', issue.number
-      end
-    end
-
-    should 'extract repository' do
-      text = 'codetree/codetree#43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/codetree', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract repository with qualifier' do
-      text = 'Fixes codetree/codetree#43'
-      issue = extract_first(text)
-
-      assert_equal 'Fixes', issue.qualifier
-      assert_equal 'codetree/codetree', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'handle odd repository names' do
-      text = 'giant-sequoia-123/scaling_octokitten#43'
-      issue = extract_first(text)
-
-      assert_equal 'giant-sequoia-123/scaling_octokitten', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract multiple references' do
-      text = 'You should look at #2 and #4 because #5 fixes codetree/codetree#6'
-      refs = extract(text)
-
-      assert refs.include?(2)
-      assert refs.include?(4)
-      assert refs.include?(5)
-      assert refs.include?(6)
-    end
-
-    should 'not extract similar non-qualifiers' do
-      text = 'afixes #43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'not choke on case' do
-      text = 'FIxEs #43'
-      issue = extract_first(text)
-
-      assert_equal 'FIxEs', issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'require one space between qualifier and issue' do
-      text = 'fixes   #43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'handle variable casing in repo names' do
-      text = 'needs Liquid-Labs/rf-app-admin-web-app#2'
-      issue = extract_first(text)
-
-      assert_equal 'needs', issue.qualifier
-      assert_equal 'Liquid-Labs/rf-app-admin-web-app', issue.repository
-      assert_equal '2', issue.number
-    end
-
-    context '#with custom configuration' do
-      setup do
-        @custom_qualifiers = %w(epic needed-by)
-        Ping.configure do |config|
-          config.qualifiers = Ping::Config::DEFAULT_QUALIFIERS.push(*@custom_qualifiers)
-        end
-      end
-
-      should 'extract default qualifiers' do
-        Ping::Config::DEFAULT_QUALIFIERS.each do |q|
-          text = "#{q} #123"
-          issue = extract_first(text)
-
-          assert_equal q, issue.qualifier
-          assert_nil issue.repository
-          assert_equal '123', issue.number
-        end
-      end
-
-      should 'extract custom qualifiers' do
-        @custom_qualifiers.each do |q|
-          text = "#{q} #123"
-          issue = extract_first(text)
-
-          assert_equal q, issue.qualifier
-          assert_nil issue.repository
-          assert_equal '123', issue.number
-        end
-      end
-    end
-  end
-
-  context '.extract with GH-XXX syntax' do
-    should 'extract single issue references' do
-      text = 'See GH-43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract lower case issue references' do
-      text = 'See gh-43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract single issue references followed by a period' do
-      text = 'See GH-43.'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract close qualifiers' do
-      %w(fix fixes fixed close closes closed resolve resolves resolved).each do |q|
-        text = "#{q} GH-55"
-        issue = extract_first(text)
-
-        assert_equal q, issue.qualifier
-        assert_nil issue.repository
-        assert_equal '55', issue.number
-      end
-    end
-
-    should 'extract dependency qualifiers' do
-      %w(need needs needed require requires required).each do |q|
-        text = "#{q} GH-123"
-        issue = extract_first(text)
-
-        assert_equal q, issue.qualifier
-        assert_nil issue.repository
-        assert_equal '123', issue.number
-      end
-    end
-
-    should 'extract multiple references' do
-      text = 'You should look at GH-2 and GH-4 because GH-5 fixes codetree/codetree#6'
-      refs = extract(text)
-
-      assert refs.include?(2)
-      assert refs.include?(4)
-      assert refs.include?(5)
-      assert refs.include?(6)
-    end
-
-    should 'not extract similar non-qualifiers' do
-      text = 'afixes GH-43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'not choke on case' do
-      text = 'FIxEs GH-43'
-      issue = extract_first(text)
-
-      assert_equal 'FIxEs', issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'require only one space between qualifier and issue' do
-      text = 'fixes   GH-43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'require at least one space before GH' do
-      text = 'fixes codetree/codetreeGH-99 and fixes GH-43'
-      issue = extract_first(text)
-
-      assert_equal 'fixes', issue.qualifier
-      assert_nil issue.repository
-      assert_equal '43', issue.number
-    end
-  end
-
-  context '#issue_references with URL syntax' do
-    should 'extract single issue references' do
-      text = 'See https://github.com/codetree/feedback/issues/43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract single pull requests references' do
-      text = 'See https://github.com/codetree/feedback/pulls/43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract single issue references followed by a period' do
-      text = 'See https://github.com/codetree/feedback/issues/43.'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'extract close qualifiers' do
-      %w(fix fixes fixed close closes closed resolve resolves resolved).each do |q|
-        text = "#{q} https://github.com/codetree/feedback/issues/55"
-        issue = extract_first(text)
-        assert_equal q, issue.qualifier
-        assert_equal 'codetree/feedback', issue.repository
-        assert_equal '55', issue.number
-      end
-    end
-
-    should 'extract dependency qualifiers' do
-      %w(need needs needed require requires required).each do |q|
-        text = "#{q} https://github.com/codetree/feedback/issues/123"
-        issue = extract_first(text)
-
-        assert_equal q, issue.qualifier
-        assert_equal 'codetree/feedback', issue.repository
-        assert_equal '123', issue.number
-      end
-    end
-
-    should 'extract multiple references' do
-      text = 'You should look at https://github.com/codetree/feedback/issues/2 ' \
-        'and https://github.com/codetree/feedback/issues/4 because ' \
-        'https://github.com/codetree/feedback/issues/5 fixes codetree/codetree#6'
-
-      refs = extract(text)
-
-      assert refs.include?(2)
-      assert refs.include?(4)
-      assert refs.include?(5)
-      assert refs.include?(6)
-    end
-
-    should 'not extract similar non-qualifiers' do
-      text = 'afixes https://github.com/codetree/feedback/issues/43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'not choke on case' do
-      text = 'FIxEs https://github.com/codetree/feedback/issues/43'
-      issue = extract_first(text)
-
-      assert_equal 'FIxEs', issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    should 'require only one space between qualifier and issue' do
-      text = 'fixes   https://github.com/codetree/feedback/issues/43'
-      issue = extract_first(text)
-
-      assert_nil issue.qualifier
-      assert_equal 'codetree/feedback', issue.repository
-      assert_equal '43', issue.number
-    end
-
-    context '#with custom configuration' do
-      setup do
-        @custom_qualifiers = %w(epic needed-by)
-        Ping.configure do |config|
-          config.qualifiers = @custom_qualifiers
-        end
-      end
-
-      should 'extract custom qualifiers' do
-        @custom_qualifiers.each do |q|
-          text = "#{q} https://github.com/codetree/feedback/issues/123"
-          issue = extract_first(text)
-
-          assert_equal q, issue.qualifier
-          assert_equal 'codetree/feedback', issue.repository
-          assert_equal '123', issue.number
-        end
-      end
-    end
-  end
-
-  context '.replace' do
-    should 'yield the phrase and parsed reference' do
-      text = 'Fixes codetree/codetree#123   needs codetree/feedback#456'
-
-      expected = [
-        'Fixes codetree/codetree#123',
-        ' needs codetree/feedback#456'
-      ]
-
-      Ping::IssueReference.replace(text) do |phrase, reference|
-        expected_phrase = expected.shift
-        expected_reference = Ping::IssueReference.extract(expected_phrase).first
-
-        assert_equal expected_phrase, phrase
-        assert_equal expected_reference, reference
-      end
-    end
-
-    context 'given a IssueReference replacement' do
-      should 'handle qualifier + repo + number' do
-        text = 'Fixes a/b#123  fixes #456'
-
-        result = Ping::IssueReference.replace(text) do |_phrase, reference|
-          reference.tap do |r|
-            r.repository = 'codetree/feedback' unless r.repository
-          end
-        end
-
-        assert_equal 'Fixes a/b#123  fixes codetree/feedback#456', result
-      end
-
-      should 'handle qualifier + number' do
-        text = 'Fixes #123  fixes #456'
-
-        result = Ping::IssueReference.replace(text) do |_phrase, reference|
-          reference.tap do |r|
-            r.qualifier = 'needs'
-            r.repository = 'a/b'
-          end
-        end
-
-        assert_equal 'needs a/b#123  needs a/b#456', result
-      end
-
-      should 'handle repo + number' do
-        text = 'a/b#123  b/c#456'
-
-        result = Ping::IssueReference.replace(text) do |_phrase, reference|
-          reference.tap do |r|
-            r.qualifier = 'needs'
-            r.repository = 'd/e'
-          end
-        end
-
-        assert_equal 'needs d/e#123  needs d/e#456', result
-      end
-
-      should 'handle number only' do
-        text = '#123 #456'
-
-        result = Ping::IssueReference.replace(text) do |_phrase, reference|
-          reference.tap do |r|
-            r.repository = 'd/e'
-          end
-        end
-
-        assert_equal 'd/e#123 d/e#456', result
-      end
-    end
-
-    context 'given a string replacement' do
-      should 'replace references' do
-        text = 'Fixes a/b#123  fixes #456'
-
-        result = Ping::IssueReference.replace(text) do |phrase, _reference|
-          phrase + ' bar'
-        end
-
-        assert_equal 'Fixes a/b#123 bar  fixes #456 bar', result
-      end
-    end
-  end
-
-  context '#==' do
-    should 'compare with integers' do
-      issue = Ping::IssueReference.new('fixes', 'codetree/codetree', '123')
-      assert issue == 123
-    end
-
-    should 'compare with strings' do
-      issue = Ping::IssueReference.new('fixes', 'codetree/codetree', '123')
-      assert issue == '123'
-    end
-
-    should 'compare with issues' do
-      issue = Ping::IssueReference.new('fixes', 'codetree/codetree', '123')
-      assert issue == Ping::IssueReference.new(nil, 'codetree/codetree', '123')
-    end
-  end
-
-  context '#to_s' do
-    should 'return the issue number' do
-      issue = Ping::IssueReference.new('Fixes', 'codetree/codetree', '123')
-      assert_equal '123', issue.to_s
-    end
-  end
-
-  context '#to_i' do
-    should 'return the integer issue number' do
-      issue = Ping::IssueReference.new('Fixes', 'codetree/codetree', '123')
-      assert_equal 123, issue.to_i
-    end
-  end
-end

data/test/ping/mention_test.rb

@@ -1,45 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper.rb'
-
-class Ping::MentionTest < MiniTest::Test
-  def extract(text)
-    Ping::Mention.extract(text)
-  end
-
-  def extract_first(text)
-    Ping::Mention.extract(text).first
-  end
-
-  context '.extract' do
-    should 'extract @mentions' do
-      text = "Hey there, @djreimer. How's @defunkt?"
-      result = extract(text)
-      assert result.include?('djreimer')
-      assert result.include?('defunkt')
-    end
-
-    should 'de-dup mentions' do
-      text = "Hey there, @djreimer. How's @djreimer?"
-      result = extract(text)
-      assert_equal 1, result.length
-    end
-  end
-
-  context '#==' do
-    should 'compare with strings' do
-      mention = Ping::Mention.new('djreimer')
-      assert mention == 'djreimer'
-    end
-
-    should 'compare with mentions' do
-      mention = Ping::Mention.new('djreimer')
-      assert mention == Ping::Mention.new('djreimer')
-    end
-  end
-
-  context '#to_s' do
-    should 'return the username' do
-      mention = Ping::Mention.new('djreimer')
-      assert_equal 'djreimer', mention.to_s
-    end
-  end
-end

data/test/ping/parser_test.rb

@@ -1,4 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper.rb'
-
-class Ping::ParserTest < MiniTest::Test
-end

data/test/test_helper.rb

@@ -1,6 +0,0 @@
-$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
-
-require 'ping'
-require 'minitest/autorun'
-require 'shoulda-context'
-require 'mocha/api'