diff options
author | Michael Herold <michael.j.herold@gmail.com> | 2014-08-23 10:35:23 -0500 |
---|---|---|
committer | Michael Herold <michael.j.herold@gmail.com> | 2014-08-23 10:35:23 -0500 |
commit | e8561ef55b2cfc39dbfe3b8b727596d948510b59 (patch) | |
tree | 091c399565779430209577dd1a68220ae3ca3340 /spec | |
parent | 7ca3fa43342fb61b24cf8f3f45d80a06d0b31254 (diff) | |
download | hashie-e8561ef55b2cfc39dbfe3b8b727596d948510b59.tar.gz |
Address TODO about the Lambda style cop
Diffstat (limited to 'spec')
-rw-r--r-- | spec/hashie/extensions/coercion_spec.rb | 8 | ||||
-rw-r--r-- | spec/hashie/extensions/dash/indifferent_access_spec.rb | 2 | ||||
-rw-r--r-- | spec/hashie/trash_spec.rb | 16 |
3 files changed, 13 insertions, 13 deletions
diff --git a/spec/hashie/extensions/coercion_spec.rb b/spec/hashie/extensions/coercion_spec.rb index 7392ba0..20cd706 100644 --- a/spec/hashie/extensions/coercion_spec.rb +++ b/spec/hashie/extensions/coercion_spec.rb @@ -245,7 +245,7 @@ describe Hashie::Extensions::Coercion do end it 'can coerce via a proc' do - subject.coerce_key :foo, ->(v) do + subject.coerce_key(:foo, lambda do |v| case v when String return !!(v =~ /^(true|t|yes|y|1)$/i) @@ -254,7 +254,7 @@ describe Hashie::Extensions::Coercion do else return v == true end - end + end) true_values = [true, 'true', 't', 'yes', 'y', '1', 1, -1] false_values = [false, 'false', 'f', 'no', 'n', '0', 0] @@ -506,9 +506,9 @@ describe Hashie::Extensions::Coercion do end it 'can coerce via a proc' do - subject.coerce_value String, ->(v) do + subject.coerce_value(String, lambda do |v| return !!(v =~ /^(true|t|yes|y|1)$/i) - end + end) true_values = %w(true t yes y 1) false_values = %w(false f no n 0) diff --git a/spec/hashie/extensions/dash/indifferent_access_spec.rb b/spec/hashie/extensions/dash/indifferent_access_spec.rb index 2342b95..f18036c 100644 --- a/spec/hashie/extensions/dash/indifferent_access_spec.rb +++ b/spec/hashie/extensions/dash/indifferent_access_spec.rb @@ -3,7 +3,7 @@ require 'spec_helper' describe Hashie::Extensions::Dash::IndifferentAccess do class TrashWithIndifferentAccess < Hashie::Trash include Hashie::Extensions::Dash::IndifferentAccess - property :per_page, transform_with: lambda { |v| v.to_i } + property :per_page, transform_with: ->(v) { v.to_i } property :total, from: :total_pages end diff --git a/spec/hashie/trash_spec.rb b/spec/hashie/trash_spec.rb index e1d35fb..e844093 100644 --- a/spec/hashie/trash_spec.rb +++ b/spec/hashie/trash_spec.rb @@ -110,7 +110,7 @@ describe Hashie::Trash do describe 'translating properties using a proc' do class TrashLambdaTest < Hashie::Trash - property :first_name, from: :firstName, with: lambda { |value| value.reverse } + property :first_name, from: :firstName, with: ->(value) { value.reverse } end let(:lambda_trash) { TrashLambdaTest.new } @@ -136,7 +136,7 @@ describe Hashie::Trash do describe 'uses with or transform_with interchangeably' do class TrashLambdaTestTransformWith < Hashie::Trash - property :first_name, from: :firstName, transform_with: lambda { |value| value.reverse } + property :first_name, from: :firstName, transform_with: ->(value) { value.reverse } end let(:lambda_trash) { TrashLambdaTestTransformWith.new } @@ -154,7 +154,7 @@ describe Hashie::Trash do describe 'translating properties without from option using a proc' do class TrashLambdaTestWithProperties < Hashie::Trash - property :first_name, transform_with: lambda { |value| value.reverse } + property :first_name, transform_with: ->(value) { value.reverse } end let(:lambda_trash) { TrashLambdaTestWithProperties.new } @@ -170,7 +170,7 @@ describe Hashie::Trash do context 'when :from option is given' do class TrashLambdaTest3 < Hashie::Trash - property :first_name, from: :firstName, transform_with: lambda { |value| value.reverse } + property :first_name, from: :firstName, transform_with: ->(value) { value.reverse } end it 'does not override the :from option in the constructor' do @@ -188,15 +188,15 @@ describe Hashie::Trash do describe 'inheritable transforms' do class TransformA < Hashie::Trash - property :some_value, transform_with: lambda { |v| v.to_i } + property :some_value, transform_with: ->(v) { v.to_i } end class TransformB < TransformA - property :some_other_value, transform_with: lambda { |v| v.to_i } + property :some_other_value, transform_with: ->(v) { v.to_i } end class TransformC < TransformB - property :some_value, transform_with: lambda { |v| -v.to_i } + property :some_value, transform_with: ->(v) { -v.to_i } end it 'inherit properties transforms' do @@ -210,7 +210,7 @@ describe Hashie::Trash do describe 'inheritable translations' do class TranslationA < Hashie::Trash - property :some_value, from: 'someValue', with: lambda { |v| v.to_i } + property :some_value, from: 'someValue', with: ->(v) { v.to_i } end class TranslationB < TranslationA |