Fix Style/PercentLiteralDelimiters

This commit is contained in:
Jared Beck 2016-03-05 17:22:29 -05:00
parent b8c9663e33
commit c7fe611907
5 changed files with 6 additions and 16 deletions

View File

@ -201,16 +201,6 @@ Style/ParallelAssignment:
- 'spec/models/gadget_spec.rb'
- 'test/test_helper.rb'
# Offense count: 6
# Cop supports --auto-correct.
# Configuration parameters: PreferredDelimiters.
Style/PercentLiteralDelimiters:
Exclude:
- 'lib/paper_trail/has_paper_trail.rb'
- 'spec/models/json_version_spec.rb'
- 'spec/models/version_spec.rb'
- 'test/unit/inheritance_column_test.rb'
# Offense count: 3
# Cop supports --auto-correct.
Style/RescueModifier:

View File

@ -116,7 +116,7 @@ module PaperTrail
# Record version before or after "destroy" event
def paper_trail_on_destroy(recording_order = "before")
unless %w[after before].include?(recording_order.to_s)
unless %w(after before).include?(recording_order.to_s)
raise ArgumentError, 'recording order can only be "after" or "before"'
end

View File

@ -71,9 +71,9 @@ if JsonVersion.table_exists?
end
context "valid arguments", versioning: true do
let(:color) { %w[red green] }
let(:color) { %w(red green) }
let(:fruit) { Fruit.create!(name: name[0]) }
let(:name) { %w[banana kiwi mango] }
let(:name) { %w(banana kiwi mango) }
before do
fruit.update_attributes!(name: name[1], color: color[0])

View File

@ -121,7 +121,7 @@ describe PaperTrail::Version, type: :model do
before do
if override
ActiveRecord::Base.connection.execute("SAVEPOINT pgtest;")
%w[object object_changes].each do |column|
%w(object object_changes).each do |column|
ActiveRecord::Base.connection.execute(
"ALTER TABLE versions DROP COLUMN #{column};"
)

View File

@ -31,13 +31,13 @@ class InheritanceColumnTest < ActiveSupport::TestCase
order(PaperTrail.timestamp_field)
assert_equal 4, dog_versions.count
assert_nil dog_versions.first.reify
assert_equal %w[NilClass Dog Dog Dog], dog_versions.map { |v| v.reify.class.name }
assert_equal %w(NilClass Dog Dog Dog), dog_versions.map { |v| v.reify.class.name }
cat_versions = PaperTrail::Version.where(item_id: @cat.id).
order(PaperTrail.timestamp_field)
assert_equal 4, cat_versions.count
assert_nil cat_versions.first.reify
assert_equal %w[NilClass Cat Cat Cat], cat_versions.map { |v| v.reify.class.name }
assert_equal %w(NilClass Cat Cat Cat), cat_versions.map { |v| v.reify.class.name }
end
end
end