ruboty-qiita-github 0.2.3 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ccd1e67858c99f728082ca9ae1123db3df278d74
-  data.tar.gz: a4d3b11e16ec956f31fa41f3d10d951cb05171cc
+SHA256:
+  metadata.gz: caf076de8647564c4cf76b70a6e79068e26a1cf64a9d317c0ff2954db458aca6
+  data.tar.gz: 13e155d2450644cd6277502e9b8024e4a0521573cc21655a22ad289b60575071
 SHA512:
-  metadata.gz: bb457fc3e32317405ed93e780f58b5536bde20d161008dc575b194e9338932b95ddf4332c42eb6a7366d4972348f0e53e009e5394133476b520eb4d43f522261
-  data.tar.gz: 19fafbc34effc2a1ea82cbe6ba8cdfd3a6aecc0776676b17c4e39bb8dd2982d641827bce4eadfc8f88a5c9268fb8c8205103bd157890a314bc589ece4108dff2
+  metadata.gz: dacf9494aa6a174e25a6ce677016d8d38462a23b92b7636f92aa8b5a6763fb6cd9729de624e7234a0c1b7969534d6c0e49c5c04b5bbda8c821658b4e6e5014a5
+  data.tar.gz: 74ea9024b5403dbba94ab085145b50dfadf4a439800884c6539cfc33e2f7d7afadf84c8d5787316917f2d9ff72d59c600dc568b52101858bb1d15cbfe77438ba

data/.github/dependabot.yml

@@ -0,0 +1,11 @@
+version: 2
+
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+  - package-ecosystem: "bundler"
+    directory: "/"
+    schedule:
+      interval: "weekly"

data/.github/workflows/test.yml

@@ -0,0 +1,37 @@
+name: Test
+
+on:
+  pull_request:
+  push:
+    branches:
+      - master
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  rspec:
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: ["2.7", "3.0", "3.1"]
+    runs-on: "ubuntu-latest"
+    continue-on-error: false
+    steps:
+      - uses: actions/checkout@v3
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - run: bundle exec rspec
+
+  rubocop:
+    runs-on: "ubuntu-latest"
+    steps:
+      - uses: actions/checkout@v3
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: "3.1"
+          bundler-cache: true
+      - run: bundle exec rubocop

data/.gitignore

@@ -20,3 +20,4 @@ tmp
 *.o
 *.a
 mkmf.log
+vendor/bundle/

data/.rubocop.yml

@@ -0,0 +1,9 @@
+inherit_from: .rubocop_todo.yml
+
+require:
+  - rubocop-performance
+  - rubocop-rspec
+  - rubocop-rake
+
+AllCops:
+  NewCops: enable

data/.rubocop_todo.yml

@@ -0,0 +1,84 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config --exclude-limit 99999`
+# on 2022-10-12 04:57:08 UTC using RuboCop version 1.36.0.
+# 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: Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'ruboty-github.gemspec'
+
+# Offense count: 1
+Lint/UnreachableCode:
+  Exclude:
+    - 'lib/ruboty/github/actions/close_issue.rb'
+
+# Offense count: 1
+# Configuration parameters: AllowedMethods, AllowedPatterns, IgnoredMethods, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 24
+
+# Offense count: 2
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods, AllowedMethods, AllowedPatterns, IgnoredMethods.
+Metrics/MethodLength:
+  Max: 14
+
+# Offense count: 2
+# Configuration parameters: NamePrefix, ForbiddenPrefixes, AllowedMethods, MethodDefinitionMacros.
+# NamePrefix: is_, has_, have_
+# ForbiddenPrefixes: is_, has_, have_
+# AllowedMethods: is_a?
+# MethodDefinitionMacros: define_method, define_singleton_method
+Naming/PredicateName:
+  Exclude:
+    - 'spec/**/*'
+    - 'lib/ruboty/github/actions/base.rb'
+    - 'lib/ruboty/github/actions/close_issue.rb'
+
+# Offense count: 3
+# Configuration parameters: .
+# SupportedStyles: have_received, receive
+RSpec/MessageSpies:
+  EnforcedStyle: receive
+
+# Offense count: 2
+RSpec/MultipleExpectations:
+  Max: 2
+
+# Offense count: 11
+# Configuration parameters: AllowSubject.
+RSpec/MultipleMemoizedHelpers:
+  Max: 13
+
+# Offense count: 1
+RSpec/NoExpectationExample:
+  Exclude:
+    - 'spec/ruboty/handlers/github_spec.rb'
+
+# Offense count: 9
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'
+    - 'lib/ruboty/github/actions/base.rb'
+    - 'lib/ruboty/github/actions/close_issue.rb'
+    - 'lib/ruboty/github/actions/create_deploy_pull_request.rb'
+    - 'lib/ruboty/github/actions/create_issue.rb'
+    - 'lib/ruboty/github/actions/create_pull_request.rb'
+    - 'lib/ruboty/github/actions/merge_pull_request.rb'
+    - 'lib/ruboty/github/actions/remember.rb'
+    - 'lib/ruboty/github/actions/search_issues.rb'
+    - 'lib/ruboty/handlers/github.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns, IgnoredPatterns.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 148

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+## Unreleased
+
+## 0.3.0
+- Fix a bug that deploy pr messages trigger other actions also
+- Warn when deployment pull request includes database migration
+- Refactor CreateDeployPullRequest
+- Add `search issues` command
+- Support test on GitHub Actions
+- Add rubocop, rubocop-rake, rubocop-performance, rubocop-rspec gems
+- Use rubocop autocorrect and modify the code
+- Added "search issues" description and status badge to README.md
+
 ## 0.2.3
 - Fix typo
 

data/Gemfile

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

data/README.md

@@ -1,4 +1,6 @@
 # Ruboty::Github
+[![Gem Version](https://badge.fury.io/rb/ruboty-qiita-github.svg)](https://badge.fury.io/rb/ruboty-qiita-github) [![Test](https://github.com/increments/ruboty-qiita-github/actions/workflows/test.yml/badge.svg?branch=master)](https://github.com/increments/ruboty-qiita-github/actions/workflows/test.yml) [![Maintainability](https://api.codeclimate.com/v1/badges/764bf9dc5796f0d3bef3/maintainability)](https://codeclimate.com/github/increments/ruboty-qiita-github/maintainability) [![Test Coverage](https://api.codeclimate.com/v1/badges/764bf9dc5796f0d3bef3/test_coverage)](https://codeclimate.com/github/increments/ruboty-qiita-github/test_coverage)
+
 Manage GitHub via Ruboty.
 This gem adds `deploy pull request` command to original ruboty-github plugin.
 
@@ -16,6 +18,7 @@ gem "ruboty-github"
 @ruboty pull request "<title>" from <from> to <to>        - Create a new Pull Request
 @ruboty deploy pull request "<title>" from <from> to <to> - Create a new Pull Request for Deploy
 @ruboty remember my github token <token>                  - Remember sender's GitHub access token
+@ruboty search issues "<query>"                           - Search an Issues/Pull Requests
 ```
 
 ## ENV

data/Rakefile

@@ -1,2 +1,3 @@
-require "bundler/gem_tasks"
+# frozen_string_literal: true
 
+require 'bundler/gem_tasks'

data/lib/ruboty/github/actions/base.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions
       class Base
-        NAMESPACE = "github"
+        NAMESPACE = 'github'
 
         attr_reader :message
 
@@ -17,7 +19,7 @@ module Ruboty
         end
 
         def body
-          message[:description] || ""
+          message[:description] || ''
         end
 
         def sender_name
@@ -45,14 +47,14 @@ module Ruboty
         end
 
         def client_options
-          client_options_with_nil_value.reject {|key, value| value.nil? }
+          client_options_with_nil_value.compact
         end
 
         def client_options_with_nil_value
           {
             access_token: access_token,
             api_endpoint: api_endpoint,
-            web_endpoint: web_endpoint,
+            web_endpoint: web_endpoint
           }
         end
 
@@ -66,11 +68,10 @@ module Ruboty
 
         # @note GITHUB_HOST will be deprecated on the next major version
         def github_base_url
-          case
-          when ENV["GITHUB_BASE_URL"]
-            ENV["GITHUB_BASE_URL"]
-          when ENV["GITHUB_HOST"]
-            "https://#{ENV['GITHUB_HOST']}"
+          if ENV.fetch('GITHUB_BASE_URL', nil)
+            ENV.fetch('GITHUB_BASE_URL', nil)
+          elsif ENV.fetch('GITHUB_HOST', nil)
+            "https://#{ENV.fetch('GITHUB_HOST', nil)}"
           end
         end
       end

data/lib/ruboty/github/actions/close_issue.rb

@@ -1,23 +1,24 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions
       class CloseIssue < Base
         def call
-          case
-          when !has_access_token?
+          if !has_access_token?
             require_access_token
-          when has_closed_issue_number?
+          elsif has_closed_issue_number?
             reply_already_closed
           else
             close
           end
         rescue Octokit::Unauthorized
-          message.reply("Failed in authentication (401)")
+          message.reply('Failed in authentication (401)')
         rescue Octokit::NotFound
-          message.reply("Could not find that issue")
-        rescue => exception
-          raise exception
-          message.reply("Failed by #{exception.class}")
+          message.reply('Could not find that issue')
+        rescue StandardError => e
+          raise e
+          message.reply("Failed by #{e.class}")
         end
 
         private
@@ -32,7 +33,7 @@ module Ruboty
         end
 
         def has_closed_issue_number?
-          issue.state == "closed"
+          issue.state == 'closed'
         end
 
         def reply_already_closed

data/lib/ruboty/github/actions/create_deploy_pull_request.rb

@@ -1,40 +1,48 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions
       class CreateDeployPullRequest < CreatePullRequest
         private
 
+        def create
+          super
+
+          message.reply('@here :warning: This deployment includes some database migrations') if database_schema_changed?
+        end
+
         # e.g. master
         def to_branch
-          to.split(":").last
+          to.split(':').last
         end
 
         def body
-           body = "## Pull Requests to deploy\n\n"
-           pull_requests_to_deploy(repository, to_branch, from_branch).each do |pr|
-             body = body + [
-               "-",
-               "[##{pr[:number]}](#{pr[:html_url]}):",
-               pr[:title],
-               "by",
-               "@#{pr[:user][:login]}\n",
-             ].join(' ')
-           end
-           body
+          lines = included_pull_requests.map do |pr|
+            "- [##{pr[:number]}](#{pr[:html_url]}): #{pr[:title]} by @#{pr[:user][:login]}"
+          end
+          lines.unshift('## Pull Requests to deploy', '')
+          lines.join("\n")
         end
 
-        def pull_requests_to_deploy(repository, to_branch, from_branch)
-          numbers = compare(repository, to_branch, from_branch).commits.map do |commit|
+        def included_pull_requests
+          numbers = comparison.commits.map do |commit|
             /^Merge pull request #(\d+) from/ =~ commit[:commit][:message]
-            $1
+            ::Regexp.last_match(1)
           end
-          numbers.compact.map { |n| client.pull_request(repository, n.to_i) }
+          numbers.compact.map { |number| client.pull_request(repository, number.to_i) }
+        end
+
+        def database_schema_changed?
+          comparison.files.any? { |file| file.filename == 'db/schema.rb' }
         end
 
-        def compare(repository, to_branch, from_branch)
-          start = client.branch(repository, to_branch)[:commit][:sha]
-          endd  = client.branch(repository, from_branch)[:commit][:sha]
-          client.compare(repository, start, endd)
+        def comparison
+          @comparison ||= begin
+            start = client.branch(repository, to_branch)[:commit][:sha]
+            endd  = client.branch(repository, from_branch)[:commit][:sha]
+            client.compare(repository, start, endd)
+          end
         end
       end
     end

data/lib/ruboty/github/actions/create_issue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions
@@ -15,11 +17,11 @@ module Ruboty
         def create
           message.reply("Created #{issue.html_url}")
         rescue Octokit::Unauthorized
-          message.reply("Failed in authentication (401)")
+          message.reply('Failed in authentication (401)')
         rescue Octokit::NotFound
-          message.reply("Could not find that repository")
-        rescue => exception
-          message.reply("Failed by #{exception.class}")
+          message.reply('Could not find that repository')
+        rescue StandardError => e
+          message.reply("Failed by #{e.class}")
         end
 
         def issue

data/lib/ruboty/github/actions/create_pull_request.rb

@@ -1,25 +1,34 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions
       class CreatePullRequest < Base
         def call
           if has_access_token?
-            create
+            create_with_error_handling
           else
             require_access_token
           end
+          # Action handlers should return truthy value to tell ruboty that the given message has been handled.
+          # Otherwise, ruboty tries to execute other handlers.
+          true
         end
 
         private
 
-        def create
-          message.reply("Created #{pull_request.html_url}")
+        def create_with_error_handling
+          create
         rescue Octokit::Unauthorized
-          message.reply("Failed in authentication (401)")
+          message.reply('Failed in authentication (401)')
         rescue Octokit::NotFound
-          message.reply("Could not find that repository")
-        rescue => exception
-          message.reply("Failed by #{exception.class} #{exception}")
+          message.reply('Could not find that repository')
+        rescue StandardError => e
+          message.reply("Failed by #{e.class} #{e}")
+        end
+
+        def create
+          message.reply("Created #{pull_request.html_url}")
         end
 
         def pull_request
@@ -37,12 +46,12 @@ module Ruboty
 
         # e.g. alice
         def from_user
-          from.split("/").first
+          from.split('/').first
         end
 
         # e.g. test
         def from_branch
-          from.split(":").last
+          from.split(':').last
         end
 
         # e.g. bob/foo:master
@@ -52,7 +61,7 @@ module Ruboty
 
         # e.g. bob/foo
         def repository
-          to.split(":").first
+          to.split(':').first
         end
 
         # e.g. alice:test
@@ -62,7 +71,7 @@ module Ruboty
 
         # e.g. master
         def base
-          to.split(":").last
+          to.split(':').last
         end
       end
     end

data/lib/ruboty/github/actions/merge_pull_request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions

data/lib/ruboty/github/actions/remember.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
     module Actions

data/lib/ruboty/github/actions/search_issues.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+module Ruboty
+  module Github
+    module Actions
+      class SearchIssues < Base
+        def call
+          if has_access_token?
+            search
+          else
+            require_access_token
+          end
+        end
+
+        private
+
+        def search
+          results = search_issues.items.each_with_object(["Searched: '#{query}'"]) do |item, object|
+            repository_url = item[:repository_url].delete_prefix('https://api.github.com/repos/')
+
+            object << "[#{repository_url}##{item[:number]}] #{item[:title]} (#{item[:user][:login]})\n#{item[:html_url]}"
+          end
+
+          message.reply(results.join("\n"))
+        rescue Octokit::Unauthorized
+          message.reply('Failed in authentication (401)')
+        rescue Octokit::NotFound
+          message.reply('Could not find that repository')
+        rescue StandardError => e
+          message.reply("Failed by #{e.class}")
+        end
+
+        def query
+          message[:query]
+        end
+
+        def search_issues
+          client.search_issues(query)
+        end
+      end
+    end
+  end
+end

data/lib/ruboty/github/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Github
-    VERSION = "0.2.3"
+    VERSION = '0.3.0'
   end
 end

data/lib/ruboty/github.rb

@@ -1,13 +1,16 @@
-require "active_support/core_ext/string/strip"
-require "octokit"
+# frozen_string_literal: true
 
-require "ruboty"
-require "ruboty/github/actions/base"
-require "ruboty/github/actions/close_issue"
-require "ruboty/github/actions/create_issue"
-require "ruboty/github/actions/create_pull_request"
-require "ruboty/github/actions/create_deploy_pull_request"
-require "ruboty/github/actions/merge_pull_request"
-require "ruboty/github/actions/remember"
-require "ruboty/github/version"
-require "ruboty/handlers/github"
+require 'active_support/core_ext/string/strip'
+require 'octokit'
+
+require 'ruboty'
+require 'ruboty/github/actions/base'
+require 'ruboty/github/actions/close_issue'
+require 'ruboty/github/actions/create_issue'
+require 'ruboty/github/actions/create_pull_request'
+require 'ruboty/github/actions/create_deploy_pull_request'
+require 'ruboty/github/actions/merge_pull_request'
+require 'ruboty/github/actions/remember'
+require 'ruboty/github/actions/search_issues'
+require 'ruboty/github/version'
+require 'ruboty/handlers/github'

data/lib/ruboty/handlers/github.rb

@@ -1,50 +1,62 @@
+# frozen_string_literal: true
+
 module Ruboty
   module Handlers
     class Github < Base
-      ISSUE_PATTERN = %r<(?:https?://[^/]+/)?(?<repo>.+)(?:#|/pull/|/issues/)(?<number>\d+) ?>
+      ISSUE_PATTERN = %r{(?:https?://[^/]+/)?(?<repo>.+)(?:#|/pull/|/issues/)(?<number>\d+) ?}.freeze
 
-      env :GITHUB_BASE_URL, "Pass GitHub URL if needed (e.g. https://github.example.com)", optional: true
+      env :GITHUB_BASE_URL, 'Pass GitHub URL if needed (e.g. https://github.example.com)', optional: true
 
       on(
         /create issue "(?<title>.+)" on (?<repo>.+)(?:\n(?<description>[\s\S]+))?\z/,
-        name: "create_issue",
-        description: "Create a new issue",
+        name: 'create_issue',
+        description: 'Create a new issue'
+      )
+
+      on(
+        /search issues "(?<query>.+)"/,
+        name: 'search_issues',
+        description: 'Search an issue'
       )
 
       on(
         /remember my github token (?<token>.+)\z/,
-        name: "remember",
-        description: "Remember sender's GitHub access token",
+        name: 'remember',
+        description: "Remember sender's GitHub access token"
       )
 
       on(
-        /close(?: issue)? #{ISSUE_PATTERN}\z/,
-        name: "close_issue",
-        description: "Close an issue",
+        /close(?: issue)? #{ISSUE_PATTERN}\z/o,
+        name: 'close_issue',
+        description: 'Close an issue'
       )
 
       on(
         /pull request "(?<title>.+)" from (?<from>.+) to (?<to>.+)(?:\n(?<description>[\s\S]+))?\z/,
-        name: "create_pull_request",
-        description: "Create a pull request",
+        name: 'create_pull_request',
+        description: 'Create a pull request'
       )
 
       on(
         /deploy pull request "(?<title>.+)" from (?<from>.+) to (?<to>.+)(?:\n(?<description>[\s\S]+))?\z/,
-        name: "create_deploy_pull_request",
-        description: "Create a pull request to deploy",
+        name: 'create_deploy_pull_request',
+        description: 'Create a pull request to deploy'
       )
 
       on(
-        /merge #{ISSUE_PATTERN}\z/,
-        name: "merge_pull_request",
-        description: "Merge pull request",
+        /merge #{ISSUE_PATTERN}\z/o,
+        name: 'merge_pull_request',
+        description: 'Merge pull request'
       )
 
       def create_issue(message)
         Ruboty::Github::Actions::CreateIssue.new(message).call
       end
 
+      def search_issues(message)
+        Ruboty::Github::Actions::SearchIssues.new(message).call
+      end
+
       def close_issue(message)
         Ruboty::Github::Actions::CloseIssue.new(message).call
       end

data/ruboty-github.gemspec

@@ -1,27 +1,33 @@
-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 'ruboty/github/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "ruboty-qiita-github"
+  spec.name          = 'ruboty-qiita-github'
   spec.version       = Ruboty::Github::VERSION
-  spec.authors       = ["Ryo Nakamura", "Seigo Uchida"]
-  spec.email         = ["r7kamura@gmail.com", "spesnova@gmail.com"]
-  spec.summary       = "Manage GitHub via Ruboty."
-  spec.homepage      = "https://github.com/increments/ruboty-qiita-github"
-  spec.license       = "MIT"
+  spec.authors       = ['Ryo Nakamura', 'Seigo Uchida']
+  spec.email         = ['r7kamura@gmail.com', 'spesnova@gmail.com']
+  spec.summary       = 'Manage GitHub via Ruboty.'
+  spec.homepage      = 'https://github.com/increments/ruboty-qiita-github'
+  spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "activesupport"
-  spec.add_dependency "ruboty"
-  spec.add_dependency "octokit"
-  spec.add_development_dependency "bundler", "~> 1.6"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "rspec", "2.14.1"
-  spec.add_development_dependency "simplecov"
-  spec.add_development_dependency "webmock"
+  spec.add_dependency 'activesupport'
+  spec.add_dependency 'octokit'
+  spec.add_dependency 'ruboty'
+  spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rubocop-performance'
+  spec.add_development_dependency 'rubocop-rake'
+  spec.add_development_dependency 'rubocop-rspec'
+  spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'webmock'
+  spec.metadata['rubygems_mfa_required'] = 'true'
 end

data/spec/ruboty/handlers/github_spec.rb

@@ -1,5 +1,7 @@
-require "spec_helper"
-require "json"
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'json'
 
 describe Ruboty::Handlers::Github do
   before do
@@ -15,23 +17,23 @@ describe Ruboty::Handlers::Github do
   end
 
   let(:github_access_token) do
-    "dummy"
+    'dummy'
   end
 
   let(:sender) do
-    "bob"
+    'bob'
   end
 
   let(:channel) do
-    "#general"
+    '#general'
   end
 
   let(:user) do
-    "alice"
+    'alice'
   end
 
   let(:repository) do
-    "test"
+    'test'
   end
 
   let(:access_tokens) do
@@ -42,78 +44,112 @@ describe Ruboty::Handlers::Github do
     robot.receive(body: body, from: sender, to: channel)
   end
 
-  shared_examples_for "requires access token without access token" do
-    context "without access token" do
+  shared_examples_for 'requires access token without access token' do
+    context 'without access token' do
       let(:stored_access_token) do
         nil
       end
 
-      it "requires access token" do
-        Ruboty.logger.should_receive(:info).with("I don't know your github access token")
+      it 'requires access token' do
+        expect(robot).to receive(:say).with(hash_including(body: "I don't know your github access token"))
         call
-        a_request(:any, //).should_not have_been_made
+        expect(a_request(:any, //)).not_to have_been_made
       end
     end
   end
 
-  describe "#create_issue" do
+  describe '#create_issue' do
     before do
-      stub_request(:post, "https://api.github.com/repos/#{user}/#{repository}/issues").
-        with(
+      stub_request(:post, "https://api.github.com/repos/#{user}/#{repository}/issues")
+        .with(
           body: {
             labels: [],
             title: title,
-            body: "",
+            body: ''
           }.to_json,
           headers: {
             Authorization: "token #{github_access_token}"
-          },
+          }
         )
     end
 
     let(:title) do
-      "This is a test issue"
+      'This is a test issue'
     end
 
     let(:body) do
-      %<ruboty create issue "#{title}" on #{user}/#{repository}>
+      %(ruboty create issue "#{title}" on #{user}/#{repository})
     end
 
-    include_examples "requires access token without access token"
+    include_examples 'requires access token without access token'
 
-    context "with access token" do
-      it "creates a new issue with given title on given repository" do
+    context 'with access token' do
+      it 'creates a new issue with given title on given repository' do
         call
-        a_request(:any, //).should have_been_made
+        expect(a_request(:any, //)).to have_been_made
       end
     end
   end
 
-  describe "#close_issue" do
+  describe '#search_issues' do
     before do
-      stub_request(:get, "https://api.github.com/repos/#{user}/#{repository}/issues/#{issue_number}").
-        with(
+      stub_request(:get, "https://api.github.com/search/issues?q=#{query}")
+        .with(
           headers: {
             Authorization: "token #{github_access_token}"
-          },
-        ).
-        to_return(
+          }
+        )
+        .to_return(
+          body: '',
+          headers: {
+            'Content-Type' => 'application/json'
+          }
+        )
+    end
+
+    let(:query) do
+      'org:increments is:open is:public is:pr'
+    end
+
+    let(:body) do
+      %(ruboty search issues "#{query}")
+    end
+
+    include_examples 'requires access token without access token'
+
+    context 'with access token' do
+      it 'search an issue with given query' do
+        call
+        expect(a_request(:any, //)).to have_been_made
+      end
+    end
+  end
+
+  describe '#close_issue' do
+    before do
+      stub_request(:get, "https://api.github.com/repos/#{user}/#{repository}/issues/#{issue_number}")
+        .with(
+          headers: {
+            Authorization: "token #{github_access_token}"
+          }
+        )
+        .to_return(
           body: {
             state: issue_status,
-            html_url: html_url,
+            html_url: html_url
           }.to_json,
           headers: {
-            "Content-Type" => "application/json",
-          },
+            'Content-Type' => 'application/json'
+          }
         )
-      stub_request(:patch, "https://api.github.com/repos/#{user}/#{repository}/issues/#{issue_number}").
-        with(
+      stub_request(:patch, "https://api.github.com/repos/#{user}/#{repository}/issues/#{issue_number}")
+        .with(
           body: {
-            state: "closed",
+            state: 'closed'
           }.to_json,
           headers: {
             Authorization: "token #{github_access_token}"
-          },
+          }
         )
     end
 
@@ -122,7 +158,7 @@ describe Ruboty::Handlers::Github do
     end
 
     let(:issue_status) do
-      "open"
+      'open'
     end
 
     let(:body) do
@@ -133,31 +169,31 @@ describe Ruboty::Handlers::Github do
       1
     end
 
-    include_examples "requires access token without access token"
+    include_examples 'requires access token without access token'
 
-    context "with closed issue" do
-      it "replies so" do
-        Ruboty.logger.should_receive(:info).with("Closed #{html_url}")
+    context 'with closed issue' do
+      it 'replies so' do
+        expect(robot).to receive(:say).with(hash_including(body: "Closed #{html_url}"))
         call
       end
     end
 
-    context "with access token" do
-      it "closes specified issue" do
+    context 'with access token' do
+      it 'closes specified issue' do
         call
       end
     end
   end
 
-  describe "#remember" do
+  describe '#remember' do
     let(:body) do
       "@ruboty remember my github token #{github_access_token}"
     end
 
     it "remembers sender's access token in its brain" do
-      Ruboty.logger.should_receive(:info).with("Remembered #{sender}'s github access token")
+      expect(robot).to receive(:say).with(hash_including(body: "Remembered #{sender}'s github access token"))
       call
-      access_tokens[sender].should == github_access_token
+      expect(access_tokens[sender]).to eq(github_access_token)
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,11 +1,12 @@
-require "simplecov"
+# frozen_string_literal: true
+
+require 'simplecov'
 SimpleCov.start
 
-require "ruboty/github"
-require "webmock/rspec"
+require 'ruboty/github'
+require 'webmock/rspec'
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
   config.filter_run :focus
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: ruboty-qiita-github
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.3.0
 platform: ruby
 authors:
 - Ryo Nakamura
 - Seigo Uchida
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-01-06 00:00:00.000000000 Z
+date: 2022-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -26,7 +26,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: ruboty
+  name: octokit
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -40,7 +40,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: octokit
+  name: ruboty
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -57,16 +57,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -85,16 +85,72 @@ dependencies:
   name: rspec
   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: rubocop
+  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: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-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: rubocop-rspec
+  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: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -123,7 +179,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - r7kamura@gmail.com
 - spesnova@gmail.com
@@ -131,7 +187,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
+- ".github/workflows/test.yml"
 - ".gitignore"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -146,6 +206,7 @@ files:
 - lib/ruboty/github/actions/create_pull_request.rb
 - lib/ruboty/github/actions/merge_pull_request.rb
 - lib/ruboty/github/actions/remember.rb
+- lib/ruboty/github/actions/search_issues.rb
 - lib/ruboty/github/version.rb
 - lib/ruboty/handlers/github.rb
 - ruboty-github.gemspec
@@ -154,8 +215,9 @@ files:
 homepage: https://github.com/increments/ruboty-qiita-github
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -170,12 +232,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Manage GitHub via Ruboty.
-test_files:
-- spec/ruboty/handlers/github_spec.rb
-- spec/spec_helper.rb
-has_rdoc: 
+test_files: []