Allow nil value for JSONAPI::Renderer params include/fields. (#22)

Author: beauby

lib/jsonapi/renderer/document.rb

@@ -7,11 +7,11 @@ class Document
       def initialize(params = {})
         @data    = params.fetch(:data,    :no_data)
         @errors  = params.fetch(:errors,  [])
-        @meta    = params.fetch(:meta,    nil)
-        @links   = params.fetch(:links,   {})
-        @fields  = _symbolize_fields(params.fetch(:fields, {}))
-        @jsonapi = params.fetch(:jsonapi, nil)
-        @include = JSONAPI::IncludeDirective.new(params.fetch(:include, {}))
+        @meta    = params[:meta]
+        @links   = params[:links] || {}
+        @fields  = _symbolize_fields(params[:fields] || {})
+        @jsonapi = params[:jsonapi]
+        @include = JSONAPI::IncludeDirective.new(params[:include] || {})
       end
 
       def to_hash

spec/renderer_spec.rb

@@ -346,6 +346,47 @@ def as_jsonapi(options = {})
     expect(actual).to eq(expected)
   end
 
+  context 'when fields option is nil' do
+    it 'does not filter out fields' do
+      actual = subject.render(data: @users[0], fields: nil)
+
+      expected = {
+        data: {
+          type: 'users',
+          id: '1',
+          attributes: {
+            name: 'User 1',
+            address: '123 Example st.'
+          },
+          relationships: {
+            posts: {
+              links: {
+                self: 'http://api.example.com/users/1/relationships/posts',
+                related: {
+                  href: 'http://api.example.com/users/1/posts',
+                  meta: {
+                    do_not_use: true
+                  }
+                }
+              },
+              meta: {
+                deleted_posts: 5
+              }
+            }
+          },
+          links: {
+            self: 'http://api.example.com/users/1'
+          },
+          meta: {
+            user_meta: 'is_meta'
+          }
+        }
+      }
+
+      expect(actual).to eq(expected)
+    end
+  end
+
   it 'renders a toplevel meta' do
     actual = subject.render(data: nil,
                             meta: { this: 'is_meta' })