Follows inheritance with a namespace

Co-authored-by: Rafael Gaspar <[email protected]>
Co-authored-by: Darryl Pogue <[email protected]>
Co-authored-by: Artin Boghosian <[email protected]>

Author: 3 people

CHANGELOG.md

@@ -8,6 +8,8 @@ Features:
 
 Fixes:
 
+- [#2172](https://github.com/rails-api/active_model_serializers/pull/2172) Preserve the namespace when falling back to a superclass serializer
+
 Misc:
 
 ### [v0.10.9 (2019-02-08)](https://github.com/rails-api/active_model_serializers/compare/v0.10.8...v0.10.9)

lib/active_model/serializer.rb

@@ -94,7 +94,7 @@ def self.get_serializer_for(klass, namespace = nil)
         if serializer_class
           serializer_class
         elsif klass.superclass
-          get_serializer_for(klass.superclass)
+          get_serializer_for(klass.superclass, namespace)
         else
           nil # No serializer found
         end

test/serializers/serializer_for_with_namespace_test.rb

@@ -9,6 +9,7 @@ class Book < ::Model
         attributes :title, :author_name
         associations :publisher, :pages
       end
+      class Ebook < Book; end
       class Page < ::Model; attributes :number, :text end
       class Publisher < ::Model; attributes :name end
 
@@ -85,6 +86,11 @@ class BookSerializer < ActiveModel::Serializer
         }
         assert_equal expected, result
       end
+
+      test 'follows inheritance with a namespace' do
+        serializer = ActiveModel::Serializer.serializer_for(Ebook.new, namespace: Api::V3)
+        assert_equal Api::V3::BookSerializer, serializer
+      end
     end
   end
 end