Rubocop automated fixes

Author: koppen

Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Use the gems defined in the plugins gemspec
 gemspec(:path => File.dirname(__FILE__))

Rakefile

@@ -1,9 +1,9 @@
 require "bundler/gem_tasks"
-require 'rake/testtask'
+require "rake/testtask"
 
 Rake::TestTask.new do |t|
   t.libs << "test"
-  files = FileList['test/**/*test.rb']
+  files = FileList["test/**/*test.rb"]
   t.test_files = files
   t.verbose = true
 end

app/controllers/github_hook_controller.rb

@@ -1,4 +1,4 @@
-require 'json'
+require "json"
 
 class GithubHookController < ApplicationController
   skip_before_filter :verify_authenticity_token, :check_if_login_required

app/services/github_hook/null_logger.rb

@@ -1,8 +1,11 @@
 module GithubHook
   class NullLogger
     def debug(*_); end
+
     def info(*_); end
+
     def warn(*_); end
+
     def error(*_); end
   end
 end

app/services/github_hook/updater.rb

@@ -1,6 +1,6 @@
 module GithubHook
   class Updater
-    GIT_BIN = Redmine::Configuration['scm_git_command'] || "git"
+    GIT_BIN = Redmine::Configuration["scm_git_command"] || "git"
 
     attr_writer :logger
 
@@ -23,7 +23,7 @@ def call
         repository.fetch_changesets
         tr2 = Time.now
 
-        logger.info { "  GithubHook: Redmine repository updated: #{repository.identifier} (Git: #{time_diff_milli(tg1,tg2)}ms, Redmine: #{time_diff_milli(tr1,tr2)}ms)" }
+        logger.info { "  GithubHook: Redmine repository updated: #{repository.identifier} (Git: #{time_diff_milli(tg1, tg2)}ms, Redmine: #{time_diff_milli(tr1, tr2)}ms)" }
       end
     end
 
@@ -40,7 +40,7 @@ def exec(command, directory)
       logger.debug { "  GithubHook: Executing command: '#{command}'" }
 
       # Get a path to a temp file
-      logfile = Tempfile.new('github_hook_exec')
+      logfile = Tempfile.new("github_hook_exec")
       logfile.close
 
       full_command = "#{command} > #{logfile.path} 2>&1"
@@ -54,9 +54,9 @@ def exec(command, directory)
 
       output_from_command = File.readlines(logfile.path)
       if success
-        logger.debug { "  GithubHook: Command output: #{output_from_command.inspect}"}
+        logger.debug { "  GithubHook: Command output: #{output_from_command.inspect}" }
       else
-        logger.error { "  GithubHook: Command '#{command}' didn't exit properly. Full output: #{output_from_command.inspect}"}
+        logger.error { "  GithubHook: Command '#{command}' didn't exit properly. Full output: #{output_from_command.inspect}" }
       end
 
       return success
@@ -68,8 +68,8 @@ def exec(command, directory)
     def find_project
       identifier = get_identifier
       project = Project.find_by_identifier(identifier.downcase)
-      raise ActiveRecord::RecordNotFound, "No project found with identifier '#{identifier}'" if project.nil?
-      return project
+      fail ActiveRecord::RecordNotFound, "No project found with identifier '#{identifier}'" if project.nil?
+      project
     end
 
     # Returns the Redmine Repository object we are trying to update
@@ -79,41 +79,41 @@ def find_repositories
         repo.is_a?(Repository::Git)
       end
 
-      if repositories.nil? or repositories.length == 0
-        raise TypeError, "Project '#{project.to_s}' ('#{project.identifier}') has no repository"
+      if repositories.nil? || repositories.length == 0
+        fail TypeError, "Project '#{project}' ('#{project.identifier}') has no repository"
       end
 
       # if a specific repository id is passed in url parameter "repository_id", then try to find it in
       # the list of current project repositories and use only this and not all to pull changes from
       # (issue #54)
-      if params.has_key?(:repository_id)
+      if params.key?(:repository_id)
         param_repo = repositories.select do |repo|
           repo.identifier == params[:repository_id]
         end
 
-        if param_repo.nil? or param_repo.length == 0
+        if param_repo.nil? || param_repo.length == 0
           logger.info { "  GithubHook: The repository '#{params[:repository_id]}' isn't in the list of projects repos. Updating all repos instead." }
 
         else
           repositories = param_repo
         end
       end
 
-      return repositories
+      repositories
     end
 
     # Gets the project identifier from the querystring parameters and if that's not supplied, assume
     # the Github repository name is the same as the project identifier.
     def get_identifier
       identifier = get_project_name
-      raise ActiveRecord::RecordNotFound, "Project identifier not specified" if identifier.nil?
-      return identifier
+      fail ActiveRecord::RecordNotFound, "Project identifier not specified" if identifier.nil?
+      identifier
     end
 
     # Attempts to find the project name. It first looks in the params, then in the
     # payload if params[:project_id] isn't given.
     def get_project_name
-      params[:project_id] || (payload['repository'] ? payload['repository']['name'] : nil)
+      params[:project_id] || (payload["repository"] ? payload["repository"]["name"] : nil)
     end
 
     def git_command(command)
@@ -134,12 +134,11 @@ def time_diff_milli(start, finish)
 
     # Fetches updates from the remote repository
     def update_repository(repository)
-      command = git_command('fetch origin')
+      command = git_command("fetch origin")
       if exec(command, repository.url)
         command = git_command("fetch --prune origin \"+refs/heads/*:refs/heads/*\"")
         exec(command, repository.url)
       end
     end
-
   end
 end

config/routes.rb

@@ -1,4 +1,4 @@
 RedmineApp::Application.routes.draw do
-  match 'github_hook' => 'github_hook#index', :via => [:post]
-  match 'github_hook' => 'github_hook#welcome', :via => [:get]
+  match "github_hook" => 'github_hook#index', :via => [:post]
+  match "github_hook" => 'github_hook#welcome', :via => [:get]
 end

init.rb

@@ -1,10 +1,10 @@
-require 'redmine'
+require "redmine"
 
 Redmine::Plugin.register :redmine_github_hook do
-  name 'Redmine Github Hook plugin'
-  author 'Jakob Skjerning'
-  description 'This plugin allows your Redmine installation to receive Github post-receive notifications'
-  url 'https://github.com/koppen/redmine_github_hook'
-  author_url 'http://mentalized.net'
+  name "Redmine Github Hook plugin"
+  author "Jakob Skjerning"
+  description "This plugin allows your Redmine installation to receive Github post-receive notifications"
+  url "https://github.com/koppen/redmine_github_hook"
+  author_url "http://mentalized.net"
   version RedmineGithubHook::VERSION
 end

redmine_github_hook.gemspec

@@ -1,7 +1,7 @@
 # coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'redmine_github_hook/version'
+require "redmine_github_hook/version"
 
 Gem::Specification.new do |spec|
   spec.name          = "redmine_github_hook"

test/functional/github_hook_controller_test.rb

@@ -1,7 +1,7 @@
-require 'test_helper'
+require "test_helper"
 
-require 'test/unit'
-require 'mocha'
+require "test/unit"
+require "mocha"
 
 class GithubHookControllerTest < ActionController::TestCase
   def json
@@ -65,7 +65,7 @@ def project
   end
 
   def setup
-    Project.stubs(:find_by_identifier).with('github').returns(project)
+    Project.stubs(:find_by_identifier).with("github").returns(project)
 
     # Make sure we don't run actual commands in test
     GithubHook::Updater.any_instance.expects(:system).never
@@ -88,9 +88,9 @@ def test_should_render_error_message
     do_post
     assert_response :not_found
     assert_equal({
-      "title" => "ActiveRecord::RecordNotFound",
-      "message" => "Repository not found"
-    }, JSON.parse(@response.body))
+                   "title" => "ActiveRecord::RecordNotFound",
+                   "message" => "Repository not found"
+                 }, JSON.parse(@response.body))
   end
 
   def test_should_not_require_login
@@ -115,5 +115,4 @@ def test_should_respond_to_get
     get :index
     assert_response :success
   end
-
 end

test/test_helper.rb

@@ -1,2 +1,2 @@
 # Load the normal Rails helper from the Redmine host app
-require File.expand_path(File.dirname(__FILE__) + '/../../../test/test_helper')
+require File.expand_path(File.dirname(__FILE__) + "/../../../test/test_helper")