Merge pull request #1982 from bf4/better_ams_model_interface

Better AMS Model attributes interface

Author: bf4

CHANGELOG.md

@@ -6,6 +6,8 @@ Breaking changes:
 
 Features:
 
+- [#1982](https://github.com/rails-api/active_model_serializers/pull/1982) Add ActiveModelSerializers::Model.attributes to configure PORO attributes (@bf4).
+
 Fixes:
 
 Misc:

README.md

@@ -110,7 +110,7 @@ class SomeResource < ActiveRecord::Base
 end
 # or
 class SomeResource < ActiveModelSerializers::Model
-  attr_accessor :title, :body
+  attributes :title, :body
 end
 ```
 

docs/ARCHITECTURE.md

@@ -105,7 +105,7 @@ ActiveModelSerializers::Model may be used either as a template, or in production
 
 ```ruby
 class MyModel < ActiveModelSerializers::Model
-  attr_accessor :id, :name, :level
+  attributes :id, :name, :level
 end
 ```
 

docs/howto/serialize_poro.md

@@ -25,7 +25,7 @@ Fortunately, ActiveModelSerializers provides a [`ActiveModelSerializers::Model`]
 ```ruby
 # my_model.rb
 class MyModel < ActiveModelSerializers::Model
-  attr_accessor :id, :name, :level
+  attributes :id, :name, :level
 end
 ```
 

lib/active_model_serializers/model.rb

@@ -6,6 +6,10 @@ class Model
     include ActiveModel::Model
     include ActiveModel::Serializers::JSON
 
+    def self.attributes(*names)
+      attr_accessor(*names)
+    end
+
     attr_reader :attributes, :errors
 
     def initialize(attributes = {})

test/action_controller/serialization_scope_name_test.rb

@@ -2,16 +2,16 @@
 
 module SerializationScopeTesting
   class User < ActiveModelSerializers::Model
-    attr_accessor :id, :name, :admin
+    attributes :id, :name, :admin
     def admin?
       admin
     end
   end
   class Comment < ActiveModelSerializers::Model
-    attr_accessor :id, :body
+    attributes :id, :body
   end
   class Post < ActiveModelSerializers::Model
-    attr_accessor :id, :title, :body, :comments
+    attributes :id, :title, :body, :comments
   end
   class PostSerializer < ActiveModel::Serializer
     attributes :id, :title, :body, :comments

test/active_model_serializers/model_test.rb

@@ -10,7 +10,7 @@ def setup
 
     def test_initialization_with_string_keys
       klass = Class.new(ActiveModelSerializers::Model) do
-        attr_accessor :key
+        attributes :key
       end
       value = 'value'
 

test/adapter/json_api/relationship_test.rb

@@ -384,7 +384,7 @@ def build_serializer_and_serialize_relationship(model, relationship_name, &block
 
         def new_model(model_attributes)
           Class.new(ActiveModelSerializers::Model) do
-            attr_accessor(*model_attributes.keys)
+            attributes(*model_attributes.keys)
 
             def self.name
               'TestModel'

test/cache_test.rb

@@ -244,7 +244,7 @@ def test_uses_adapter_in_cache_key
     # rubocop:disable Metrics/AbcSize
     def test_a_serializer_rendered_by_two_adapter_returns_differently_fetch_attributes
       Object.const_set(:Alert, Class.new(ActiveModelSerializers::Model) do
-        attr_accessor :id, :status, :resource, :started_at, :ended_at, :updated_at, :created_at
+        attributes :id, :status, :resource, :started_at, :ended_at, :updated_at, :created_at
       end)
       Object.const_set(:UncachedAlertSerializer, Class.new(ActiveModel::Serializer) do
         attributes :id, :status, :resource, :started_at, :ended_at, :updated_at, :created_at

test/fixtures/poro.rb

@@ -31,7 +31,7 @@ def respond_to_missing?(method_name, _include_private = false)
 #   model.validate!            # => ["cannot be nil"]
 #   model.errors.full_messages # => ["name cannot be nil"]
 class ModelWithErrors < ::ActiveModelSerializers::Model
-  attr_accessor :name
+  attributes :name
 end
 
 class Profile < Model