From 6acb4055c9ec68257b192ce4a3868383f2d6ffe4 Mon Sep 17 00:00:00 2001 From: Benjamin Fleischer Date: Fri, 6 Jan 2017 12:38:11 -0600 Subject: [PATCH] Fix MT6 assert_nil warnings --- .../serialization_scope_name_test.rb | 5 ++++- .../railtie_test_isolated.rb | 5 ++++- .../json_api/include_data_if_sideloaded_test.rb | 7 ++++++- test/cache_test.rb | 8 ++++---- .../caching_configuration_test_isolated.rb | 12 ++++++------ test/serializers/serializer_for_test.rb | 12 ++++++------ 6 files changed, 30 insertions(+), 19 deletions(-) diff --git a/test/action_controller/serialization_scope_name_test.rb b/test/action_controller/serialization_scope_name_test.rb index 33ac88f4..d4db3b63 100644 --- a/test/action_controller/serialization_scope_name_test.rb +++ b/test/action_controller/serialization_scope_name_test.rb @@ -75,7 +75,10 @@ module SerializationScopeTesting end def test_default_serialization_scope_object - assert_equal @controller.current_user, @controller.serialization_scope + expected = @controller.current_user + actual = @controller.serialization_scope + assert_nil expected + assert_nil actual end def test_default_scope_non_admin diff --git a/test/active_model_serializers/railtie_test_isolated.rb b/test/active_model_serializers/railtie_test_isolated.rb index 21f6c178..f0d1c7e5 100644 --- a/test/active_model_serializers/railtie_test_isolated.rb +++ b/test/active_model_serializers/railtie_test_isolated.rb @@ -32,7 +32,10 @@ class RailtieTest < ActiveSupport::TestCase test 'it is configured for caching' do assert_equal ActionController::Base.cache_store, ActiveModelSerializers.config.cache_store - assert_equal Rails.configuration.action_controller.perform_caching, ActiveModelSerializers.config.perform_caching + expected = Rails.configuration.action_controller.perform_caching + actual = ActiveModelSerializers.config.perform_caching + assert_nil expected + assert_nil actual end end diff --git a/test/adapter/json_api/include_data_if_sideloaded_test.rb b/test/adapter/json_api/include_data_if_sideloaded_test.rb index 728eae13..aabc3bc5 100644 --- a/test/adapter/json_api/include_data_if_sideloaded_test.rb +++ b/test/adapter/json_api/include_data_if_sideloaded_test.rb @@ -159,7 +159,12 @@ module ActiveModel def assert_relationship(relationship_name, expected, opts = {}) hash = result(opts) - assert_equal(expected, hash[:data][:relationships][relationship_name]) + actual = hash[:data][:relationships][relationship_name] + if expected.nil? + assert_nil(actual) + else + assert_equal(expected, actual) + end end end end diff --git a/test/cache_test.rb b/test/cache_test.rb index b2cb27eb..06043fb7 100644 --- a/test/cache_test.rb +++ b/test/cache_test.rb @@ -128,7 +128,7 @@ module ActiveModelSerializers def test_cache_key_definition assert_equal('post', @post_serializer.class._cache_key) assert_equal('writer', @author_serializer.class._cache_key) - assert_equal(nil, @comment_serializer.class._cache_key) + assert_nil(@comment_serializer.class._cache_key) end def test_cache_key_interpolation_with_updated_at_when_cache_key_is_not_defined_on_object @@ -171,7 +171,7 @@ module ActiveModelSerializers def test_cache_options_definition assert_equal({ expires_in: 0.1, skip_digest: true }, @post_serializer.class._cache_options) - assert_equal(nil, @blog_serializer.class._cache_options) + assert_nil(@blog_serializer.class._cache_options) assert_equal({ expires_in: 1.day, skip_digest: true }, @comment_serializer.class._cache_options) end @@ -182,8 +182,8 @@ module ActiveModelSerializers def test_associations_separately_cache cache_store.clear - assert_equal(nil, cache_store.fetch(@post.cache_key)) - assert_equal(nil, cache_store.fetch(@comment.cache_key)) + assert_nil(cache_store.fetch(@post.cache_key)) + assert_nil(cache_store.fetch(@comment.cache_key)) Timecop.freeze(Time.current) do render_object_with_cache(@post) diff --git a/test/serializers/caching_configuration_test_isolated.rb b/test/serializers/caching_configuration_test_isolated.rb index 82e497b2..b5698dd1 100644 --- a/test/serializers/caching_configuration_test_isolated.rb +++ b/test/serializers/caching_configuration_test_isolated.rb @@ -69,9 +69,9 @@ class CachingConfigurationTest < ActiveSupport::TestCase end test 'the non-cached serializer cache_store is nil' do - assert_equal nil, @non_cached_serializer._cache - assert_equal nil, @non_cached_serializer.cache_store - assert_equal nil, @non_cached_serializer._cache + assert_nil @non_cached_serializer._cache + assert_nil @non_cached_serializer.cache_store + assert_nil @non_cached_serializer._cache end test 'the non-cached serializer does not have cache_enabled?' do @@ -136,9 +136,9 @@ class CachingConfigurationTest < ActiveSupport::TestCase end test 'the non-cached serializer cache_store is nil' do - assert_equal nil, @non_cached_serializer._cache - assert_equal nil, @non_cached_serializer.cache_store - assert_equal nil, @non_cached_serializer._cache + assert_nil @non_cached_serializer._cache + assert_nil @non_cached_serializer.cache_store + assert_nil @non_cached_serializer._cache end test 'the non-cached serializer does not have cache_enabled?' do diff --git a/test/serializers/serializer_for_test.rb b/test/serializers/serializer_for_test.rb index b7607cfe..9f691708 100644 --- a/test/serializers/serializer_for_test.rb +++ b/test/serializers/serializer_for_test.rb @@ -59,7 +59,7 @@ module ActiveModel def test_serializer_for_non_ams_serializer serializer = ActiveModel::Serializer.serializer_for(@tweet) - assert_equal nil, serializer + assert_nil serializer end def test_serializer_for_existing_serializer @@ -71,12 +71,12 @@ module ActiveModel serializer = with_serializer_lookup_disabled do ActiveModel::Serializer.serializer_for(@profile) end - assert_equal nil, serializer + assert_nil serializer end def test_serializer_for_not_existing_serializer serializer = ActiveModel::Serializer.serializer_for(@model) - assert_equal nil, serializer + assert_nil serializer end def test_serializer_inherited_serializer @@ -88,7 +88,7 @@ module ActiveModel serializer = with_serializer_lookup_disabled do ActiveModel::Serializer.serializer_for(@my_profile) end - assert_equal nil, serializer + assert_nil serializer end def test_serializer_custom_serializer @@ -114,7 +114,7 @@ module ActiveModel serializer = with_serializer_lookup_disabled do ActiveModel::Serializer.serializer_for(post) end - assert_equal nil, serializer + assert_nil serializer end def test_serializer_for_nested_resource @@ -128,7 +128,7 @@ module ActiveModel serializer = with_serializer_lookup_disabled do ResourceNamespace::PostSerializer.serializer_for(comment) end - assert_equal nil, serializer + assert_nil serializer end end end