Skip to content

Commit 52dbdc7

Browse files
committed
Fix for rubocop
1 parent ea00b9d commit 52dbdc7

19 files changed

+25
-7
lines changed

.rubocop.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
inherit_from: .rubocop_todo.yml
22

33
AllCops:
4-
TargetRubyVersion: 2.3
4+
TargetRubyVersion: 2.2
55
DisplayCopNames: true
66

77
Performance/RedundantBlockCall:

Gemfile

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
source 'https://rubygems.org'
34

45
# Specify your gem's dependencies in switch_point.gemspec

Rakefile

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
# frozen_string_literal: true
2+
23
require 'bundler/gem_tasks'
34

4-
task :default => [:spec, :rubocop]
5+
task :default => %i[spec rubocop]
56

67
require 'rspec/core/rake_task'
78
RSpec::Core::RakeTask.new(:spec)

benchmark/proxy.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'benchmark/ips'
34
require 'switch_point'
45
require 'active_record'
@@ -33,7 +34,7 @@ class Proxy2 < ProxyBase
3334
ActiveRecord::Base.establish_connection(:default)
3435

3536
Plain.connection.execute('CREATE TABLE plains (id integer primary key autoincrement)')
36-
[:readonly, :writable].each do |mode|
37+
%i[readonly writable].each do |mode|
3738
ProxyBase.public_send("with_#{mode}") do
3839
%w[proxy1s proxy2s].each do |table|
3940
ProxyBase.connection.execute("CREATE TABLE #{table} (id integer primary key autoincrement)")

lib/switch_point.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'active_support/lazy_load_hooks'
34
require 'switch_point/config'
45
require 'switch_point/version'

lib/switch_point/config.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
module SwitchPoint
34
class Config
45
attr_accessor :auto_writable

lib/switch_point/connection.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
# frozen_string_literal: true
2+
23
require 'switch_point/error'
34
require 'switch_point/proxy_repository'
45

56
module SwitchPoint
67
module Connection
78
# See ActiveRecord::ConnectionAdapters::QueryCache
8-
DESTRUCTIVE_METHODS = [:insert, :update, :delete].freeze
9+
DESTRUCTIVE_METHODS = %i[insert update delete].freeze
910

1011
DESTRUCTIVE_METHODS.each do |method_name|
1112
define_method(method_name) do |*args, &block|

lib/switch_point/error.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
module SwitchPoint
34
class Error < StandardError
45
end

lib/switch_point/model.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'switch_point/error'
34
require 'switch_point/proxy_repository'
45

lib/switch_point/proxy.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
# frozen_string_literal: true
2+
23
require 'switch_point/error'
34

45
module SwitchPoint
56
class Proxy
67
attr_reader :initial_name
78

8-
AVAILABLE_MODES = [:writable, :readonly].freeze
9+
AVAILABLE_MODES = %i[writable readonly].freeze
910
DEFAULT_MODE = :readonly
1011

1112
def initialize(name)

lib/switch_point/proxy_repository.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'singleton'
34
require 'switch_point/proxy'
45

lib/switch_point/query_cache.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
module SwitchPoint
34
class QueryCache
45
def initialize(app, names = nil)

lib/switch_point/version.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
module SwitchPoint
3-
VERSION = '0.8.0'
4+
VERSION = '0.8.0'.freeze
45
end

spec/models.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
SwitchPoint.configure do |config|
34
config.define_switch_point :main,
45
readonly: :main_readonly,

spec/spec_helper.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'coveralls'
34
require 'simplecov'
45

spec/switch_point/model_spec.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
RSpec.describe SwitchPoint::Model do
34
describe '.use_switch_point' do
45
after do

spec/switch_point/query_cache_spec.rb

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
require 'spec_helper'
34
require 'rack'
45

@@ -46,7 +47,7 @@ def call(env)
4647
context 'when names are specified' do
4748
let(:app) do
4849
Rack::Builder.new do
49-
use SwitchPoint::QueryCache, [:main, :nanika1]
50+
use SwitchPoint::QueryCache, %i[main nanika1]
5051
run TestApp.new
5152
end
5253
end

spec/switch_point_spec.rb

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# frozen_string_literal: true
2+
23
RSpec.describe SwitchPoint do
34
describe '.writable_all!' do
45
after do

switch_point.gemspec

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# coding: utf-8
22
# frozen_string_literal: true
3+
34
lib = File.expand_path('../lib', __FILE__)
45
$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
56
require 'switch_point/version'

0 commit comments

Comments
 (0)