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")

test/unit/github_hook/updater_test.rb

@@ -1,10 +1,9 @@
-require 'test_helper'
+require "test_helper"
 
-require 'test/unit'
-require 'mocha'
+require "test/unit"
+require "mocha"
 
 class GithubHookUpdaterTest < Test::Unit::TestCase
-
   def project
     return @project if @project
 
@@ -23,7 +22,7 @@ def repository
 
   def payload
     # Ruby hash with the parsed data from the JSON payload
-    {"before" => "5aef35982fb2d34e9d9d4502f6ede1072793222d", "repository"=>{"url" => "http://github.com/defunkt/github", "name" => "github", "description" => "You're lookin' at it.", "watchers"=>5, "forks"=>2, "private"=>1, "owner"=>{"email" => "[email protected]", "name" => "defunkt"}}, "commits"=>[{"id" => "41a212ee83ca127e3c8cf465891ab7216a705f59", "url" => "http://github.com/defunkt/github/commit/41a212ee83ca127e3c8cf465891ab7216a705f59", "author"=>{"email" => "[email protected]", "name" => "Chris Wanstrath"}, "message" => "okay i give in", "timestamp" => "2008-02-15T14:57:17-08:00", "added"=>["filepath.rb"]}, {"id" => "de8251ff97ee194a289832576287d6f8ad74e3d0", "url" => "http://github.com/defunkt/github/commit/de8251ff97ee194a289832576287d6f8ad74e3d0", "author"=>{"email" => "[email protected]", "name" => "Chris Wanstrath"}, "message" => "update pricing a tad", "timestamp" => "2008-02-15T14:36:34-08:00"}], "after" => "de8251ff97ee194a289832576287d6f8ad74e3d0", "ref" => "refs/heads/master"}
+    {"before" => "5aef35982fb2d34e9d9d4502f6ede1072793222d", "repository" => {"url" => "http://github.com/defunkt/github", "name" => "github", "description" => "You're lookin' at it.", "watchers" => 5, "forks" => 2, "private" => 1, "owner" => {"email" => "[email protected]", "name" => "defunkt"}}, "commits" => [{"id" => "41a212ee83ca127e3c8cf465891ab7216a705f59", "url" => "http://github.com/defunkt/github/commit/41a212ee83ca127e3c8cf465891ab7216a705f59", "author" => {"email" => "[email protected]", "name" => "Chris Wanstrath"}, "message" => "okay i give in", "timestamp" => "2008-02-15T14:57:17-08:00", "added" => ["filepath.rb"]}, {"id" => "de8251ff97ee194a289832576287d6f8ad74e3d0", "url" => "http://github.com/defunkt/github/commit/de8251ff97ee194a289832576287d6f8ad74e3d0", "author" => {"email" => "[email protected]", "name" => "Chris Wanstrath"}, "message" => "update pricing a tad", "timestamp" => "2008-02-15T14:36:34-08:00"}], "after" => "de8251ff97ee194a289832576287d6f8ad74e3d0", "ref" => "refs/heads/master"}
   end
 
   def build_updater(payload, options = {})
@@ -38,7 +37,7 @@ def updater
   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
@@ -50,7 +49,7 @@ def teardown
   end
 
   def test_uses_repository_name_as_project_identifier
-    Project.expects(:find_by_identifier).with('github').returns(project)
+    Project.expects(:find_by_identifier).with("github").returns(project)
     updater.call
   end
 
@@ -72,8 +71,8 @@ def test_resets_repository_when_fetch_origin_fails
   end
 
   def test_uses_project_identifier_from_request
-    Project.expects(:find_by_identifier).with('redmine').returns(project)
-    updater = build_updater(payload, {:project_id => 'redmine'})
+    Project.expects(:find_by_identifier).with("redmine").returns(project)
+    updater = build_updater(payload, :project_id => "redmine")
     updater.call
   end
 
@@ -92,7 +91,7 @@ def test_updates_only_the_specified_repository
     another_repository.expects(:fetch_changesets).never
     project.repositories << another_repository
 
-    updater = build_updater(payload, {:repository_id => 'redmine'})
+    updater = build_updater(payload, {:repository_id => "redmine"})
     updater.expects(:exec).with("git fetch origin", repository.url)
     updater.call
   end
@@ -102,7 +101,7 @@ def test_updates_all_repositories_if_specific_repository_is_not_found
     another_repository.expects(:fetch_changesets).returns(true)
     project.repositories << another_repository
 
-    updater = build_updater(payload, {:repository_id => 'redmine or something'})
+    updater = build_updater(payload, :repository_id => "redmine or something")
     updater.expects(:exec).with("git fetch origin", repository.url)
     updater.call
   end
@@ -116,23 +115,23 @@ def test_raises_record_not_found_if_project_identifier_not_found
 
   def test_raises_record_not_found_if_project_identifier_not_given
     assert_raises ActiveRecord::RecordNotFound do
-      updater = build_updater(payload.merge({"repository" => {}}))
+      updater = build_updater(payload.merge("repository" => {}))
       updater.call
     end
   end
 
   def test_raises_record_not_found_if_project_not_found
     assert_raises ActiveRecord::RecordNotFound do
-      Project.expects(:find_by_identifier).with('foobar').returns(nil)
-      updater = build_updater(payload, {:project_id => "foobar"})
+      Project.expects(:find_by_identifier).with("foobar").returns(nil)
+      updater = build_updater(payload, :project_id => "foobar")
       updater.call
     end
   end
 
   def test_downcases_identifier
     # Redmine project identifiers are always downcase
-    Project.expects(:find_by_identifier).with('redmine').returns(project)
-    updater = build_updater(payload, {:project_id => 'ReDmInE'})
+    Project.expects(:find_by_identifier).with("redmine").returns(project)
+    updater = build_updater(payload, :project_id => "ReDmInE")
     updater.call
   end
 
@@ -144,19 +143,19 @@ def test_fetches_changesets_into_the_repository
 
   def test_raises_type_error_if_project_has_no_repository
     assert_raises TypeError do
-      project = mock('project', :to_s => 'My Project', :identifier => 'github')
+      project = mock("project", :to_s => "My Project", :identifier => "github")
       project.expects(:repositories).returns([])
-      Project.expects(:find_by_identifier).with('github').returns(project)
+      Project.expects(:find_by_identifier).with("github").returns(project)
       updater.call
     end
   end
 
   def test_raises_type_error_if_repository_is_not_git
     assert_raises TypeError do
-      project = mock('project', :to_s => 'My Project', :identifier => 'github')
+      project = mock("project", :to_s => "My Project", :identifier => "github")
       repository = Repository::Subversion.new
       project.expects(:repositories).at_least(1).returns([repository])
-      Project.expects(:find_by_identifier).with('github').returns(project)
+      Project.expects(:find_by_identifier).with("github").returns(project)
       updater.call
     end
   end
@@ -165,7 +164,7 @@ def test_logs_if_a_logger_is_given
     updater = GithubHook::Updater.new(payload)
     updater.stubs(:exec).returns(true)
 
-    logger = stub('Logger')
+    logger = stub("Logger")
     logger.expects(:info).at_least_once
     updater.logger = logger