Fix warnings

JRuby-specific: fix 'warning: (...) interpreted as grouped expression'

lib/active_model/serializable_resource.rb

@@ -1,4 +1,4 @@
-require "set"
+require 'set'
 module ActiveModel
   class SerializableResource
 
@@ -76,7 +76,9 @@ module ActiveModel
 
     private
 
+    ActiveModelSerializers.silence_warnings do
     attr_reader :resource, :adapter_opts, :serializer_opts
+    end
 
   end
 end

lib/active_model/serializer.rb

@@ -9,6 +9,7 @@ module ActiveModel
     autoload :Adapter
     autoload :Lint
     autoload :Associations
+    autoload :Fieldset
     include Configuration
     include Associations
 
@@ -66,10 +67,10 @@ module ActiveModel
       @_attributes_keys[attr] = { key: key } if key != attr
       @_attributes << key unless @_attributes.include?(key)
 
-      unless respond_to?(key, false) || _fragmented.respond_to?(attr)
+      ActiveModelSerializers.silence_warnings do
         define_method key do
           object.read_attribute_for_serialization(attr)
-        end
+        end unless respond_to?(key, false) || _fragmented.respond_to?(attr)
       end
     end
 

lib/active_model/serializer/adapter.rb

@@ -1,13 +1,23 @@
-require 'active_model/serializer/adapter/fragment_cache'
-
 module ActiveModel
   class Serializer
     class Adapter
       extend ActiveSupport::Autoload
-      autoload :Json
+      require 'active_model/serializer/adapter/json'
+      require 'active_model/serializer/adapter/json_api'
       autoload :FlattenJson
       autoload :Null
-      autoload :JsonApi
+      autoload :FragmentCache
+
+      def self.create(resource, options = {})
+        override = options.delete(:adapter)
+        klass = override ? adapter_class(override) : ActiveModel::Serializer.adapter
+        klass.new(resource, options)
+      end
+
+      def self.adapter_class(adapter)
+        adapter_name = adapter.to_s.classify.sub("API", "Api")
+        "ActiveModel::Serializer::Adapter::#{adapter_name}".safe_constantize
+      end
 
       attr_reader :serializer
 
@@ -26,17 +36,6 @@ module ActiveModel
         hash
       end
 
-      def self.create(resource, options = {})
-        override = options.delete(:adapter)
-        klass = override ? adapter_class(override) : ActiveModel::Serializer.adapter
-        klass.new(resource, options)
-      end
-
-      def self.adapter_class(adapter)
-        adapter_name = adapter.to_s.classify.sub("API", "Api")
-        "ActiveModel::Serializer::Adapter::#{adapter_name}".safe_constantize
-      end
-
       def fragment_cache(*args)
         raise NotImplementedError, 'This is an abstract method. Should be implemented at the concrete adapter.'
       end

lib/active_model/serializer/adapter/json/fragment_cache.rb

@@ -1,3 +1,4 @@
+require 'active_model/serializer/adapter/fragment_cache'
 module ActiveModel
   class Serializer
     class Adapter

lib/active_model/serializer/adapter/json_api/fragment_cache.rb

@@ -1,3 +1,4 @@
+require 'active_model/serializer/adapter/fragment_cache'
 module ActiveModel
   class Serializer
     class Adapter

lib/active_model/serializer/associations.rb

@@ -73,11 +73,9 @@ module ActiveModel
         def associate(reflection)
           self._reflections = _reflections.dup
 
-          unless method_defined?(reflection.name)
           define_method reflection.name do
             object.send reflection.name
-            end
+          end unless method_defined?(reflection.name)
-          end
 
           self._reflections << reflection
         end

lib/active_model/serializer/fieldset.rb

@@ -18,7 +18,9 @@ module ActiveModel
 
     private
 
+      ActiveModelSerializers.silence_warnings do
       attr_reader :raw_fields, :root
+      end
 
       def parsed_fields
         if raw_fields.is_a?(Hash)

lib/active_model_serializers.rb

@@ -1,7 +1,18 @@
+module ActiveModelSerializers
+  module_function
+
+  def silence_warnings
+    verbose = $VERBOSE
+    $VERBOSE = nil
+    yield
+  ensure
+    $VERBOSE = verbose
+  end
+
+end
 require 'active_model'
 require 'active_model/serializer/version'
 require 'active_model/serializer'
-require 'active_model/serializer/fieldset'
 require 'active_model/serializable_resource'
 
 begin

test/action_controller/serialization_scope_name_test.rb

@@ -4,10 +4,12 @@ require 'pathname'
 class DefaultScopeNameTest < ActionController::TestCase
   class UserSerializer < ActiveModel::Serializer
     attributes :admin?
+    ActiveModelSerializers.silence_warnings do
       def admin?
         current_user.admin
       end
     end
+  end
 
   class UserTestController < ActionController::Base
     protect_from_forgery
@@ -34,10 +36,12 @@ end
 class SerializationScopeNameTest < ActionController::TestCase
   class AdminUserSerializer < ActiveModel::Serializer
     attributes :admin?
+    ActiveModelSerializers.silence_warnings do
       def admin?
         current_admin.admin
       end
     end
+  end
 
   class AdminUserTestController < ActionController::Base
     protect_from_forgery

test/action_controller/serialization_test.rb

@@ -129,7 +129,6 @@ module ActionController
         def render_fragment_changed_object_with_relationship
           comment = Comment.new({ id: 1, body: 'ZOMG A COMMENT' })
           comment2 = Comment.new({ id: 1, body: 'ZOMG AN UPDATED-BUT-NOT-CACHE-EXPIRED COMMENT' })
-          author = Author.new(id: 1, name: 'Joao Moura.')
           like = Like.new({ id: 1, likeable: comment, time: 3.days.ago })
 
           generate_cached_serializer(like)
@@ -215,14 +214,16 @@ module ActionController
         get :render_json_object_without_serializer
 
         assert_equal 'application/json', @response.content_type
-        assert_equal ({error: 'Result is Invalid'}).to_json, @response.body
+        expected_body = {error: 'Result is Invalid'}
+        assert_equal expected_body.to_json, @response.body
       end
 
       def test_render_json_array_object_without_serializer
         get :render_json_array_object_without_serializer
 
         assert_equal 'application/json', @response.content_type
-        assert_equal ([{error: 'Result is Invalid'}]).to_json, @response.body
+        expected_body = [{error: 'Result is Invalid'}]
+        assert_equal expected_body.to_json, @response.body
       end
 
       def test_render_array_using_implicit_serializer
@@ -405,9 +406,9 @@ module ActionController
             false
           end
         }.new
-        assert_match /adapter: false/, (capture(:stderr) {
+        assert_match(/adapter: false/, (capture(:stderr) {
           controller.get_serializer(Profile.new)
-        })
+        }))
       end
 
       def test_dont_warn_overridding_use_adapter_as_truthy_on_controller_instance

test/adapter/json/belongs_to_test.rb

@@ -11,7 +11,6 @@ module ActiveModel
             @comment = Comment.new(id: 1, body: 'ZOMG A COMMENT')
             @post.comments = [@comment]
             @anonymous_post.comments = []
-            @post.author = @author
             @comment.post = @post
             @comment.author = nil
             @anonymous_post.author = nil

test/fixtures/poro.rb

@@ -1,3 +1,5 @@
+verbose = $VERBOSE
+$VERBOSE = nil
 class Model
   FILE_DIGEST = Digest::MD5.hexdigest(File.open(__FILE__).read)
 
@@ -260,9 +262,4 @@ Spam::UnrelatedLinkSerializer = Class.new(ActiveModel::Serializer) do
   cache only: [:id]
   attributes :id
 end
-
+$VERBOSE = verbose
-RaiseErrorSerializer = Class.new(ActiveModel::Serializer) do
-  def json_key
-    raise StandardError, 'Intentional error for rescue_from test'
-  end
-end

test/generators/serializer_generator_test.rb

@@ -47,9 +47,9 @@ class SerializerGeneratorTest < Rails::Generators::TestCase
     run_generator ["account"]
     assert_file "app/serializers/account_serializer.rb" do |content|
       if RUBY_PLATFORM =~ /mingw/
-        assert_no_match /\r\n\r\nend/, content
+        assert_no_match(/\r\n\r\nend/, content)
       else
-        assert_no_match /\n\nend/, content
+        assert_no_match(/\n\nend/, content)
       end
     end
   end

test/serializers/associations_test.rb

@@ -81,7 +81,7 @@ module ActiveModel
       def test_serializer_options_are_passed_into_associations_serializers
         association = @post_serializer
                         .associations
-                        .detect { |association| association.key == :comments }
+                        .detect { |assoc| assoc.key == :comments }
 
         assert association.serializer.first.custom_options[:custom_options]
       end

test/serializers/cache_test.rb

@@ -47,13 +47,13 @@ module ActiveModel
       end
 
       def test_cache_key_interpolation_with_updated_at
-        author = render_object_with_cache(@author)
+        render_object_with_cache(@author)
         assert_equal(nil, ActionController::Base.cache_store.fetch(@author.cache_key))
         assert_equal(@author_serializer.attributes.to_json, ActionController::Base.cache_store.fetch("#{@author_serializer.class._cache_key}/#{@author_serializer.object.id}-#{@author_serializer.object.updated_at.strftime("%Y%m%d%H%M%S%9N")}").to_json)
       end
 
       def test_default_cache_key_fallback
-        comment = render_object_with_cache(@comment)
+        render_object_with_cache(@comment)
         assert_equal(@comment_serializer.attributes.to_json, ActionController::Base.cache_store.fetch(@comment.cache_key).to_json)
       end
 
@@ -74,7 +74,7 @@ module ActiveModel
         assert_equal(nil, ActionController::Base.cache_store.fetch(@comment.cache_key))
 
         Timecop.freeze(Time.now) do
-          post = render_object_with_cache(@post)
+          render_object_with_cache(@post)
 
           assert_equal(@post_serializer.attributes, ActionController::Base.cache_store.fetch(@post.cache_key))
           assert_equal(@comment_serializer.attributes, ActionController::Base.cache_store.fetch(@comment.cache_key))
@@ -122,7 +122,7 @@ module ActiveModel
       end
 
       def test_uses_file_digest_in_cache_key
-        blog = render_object_with_cache(@blog)
+        render_object_with_cache(@blog)
         assert_equal(@blog_serializer.attributes, ActionController::Base.cache_store.fetch(@blog.cache_key_with_digest))
       end
 

test/test_helper.rb

@@ -15,11 +15,18 @@ Minitest::Test = MiniTest::Unit::TestCase unless defined?(Minitest::Test)
 
 
 require 'capture_warnings'
-@capture_warnings = CaptureWarnings.new(fail_build = false)
+@capture_warnings = CaptureWarnings.new(fail_build = true)
 @capture_warnings.before_tests
+if Minitest.respond_to?(:after_run)
   Minitest.after_run do
     @capture_warnings.after_tests
   end
+else
+  at_exit do
+    STDOUT.puts "Minitest.after_run not available."
+    @capture_warnings.after_tests
+  end
+end
 
 require 'active_model_serializers'