Ensure valid jsonapi when blank relationship

If you specify include_data false, and do not have any links for this
relationship, we would output something like:

`{ relationships: { comments: {} } }`

This is not valid jsonapi. We will now render

`{ relationships: { comments: { meta: {} } } }`

Instead.

Relevant jsonapi spec: http://jsonapi.org/format/#document-resource-object-relationships

Author: Lee Richmond

CHANGELOG.md

@@ -9,6 +9,7 @@ Fixes:
 - [#1887](https://github.com/rails-api/active_model_serializers/pull/1887) Make the comment reflect what the function does (@johnnymo87)
 - [#1890](https://github.com/rails-api/active_model_serializers/issues/1890) Ensure generator inherits from ApplicationSerializer when available (@richmolj)
 - [#1922](https://github.com/rails-api/active_model_serializers/pull/1922) Make railtie an optional dependency in runtime (@ggpasqualino)
+- [#1930](https://github.com/rails-api/active_model_serializers/pull/1930) Ensure valid jsonapi when relationship has no links or data (@richmolj)
 
 Features:
 

lib/active_model_serializers/adapter/json_api/relationship.rb

@@ -25,7 +25,7 @@ def as_json
           meta = meta_for(association)
           hash[:meta] = meta if meta
 
-          hash
+          ensure_valid!(hash)
         end
 
         protected
@@ -34,6 +34,11 @@ def as_json
 
         private
 
+        def ensure_valid!(json)
+          json[:meta] = {} if json.empty?
+          json
+        end
+
         def data_for(association)
           serializer = association.serializer
           if serializer.respond_to?(:each)

test/adapter/json_api/relationship_test.rb

@@ -54,7 +54,7 @@ def test_relationship_with_data_array
         end
 
         def test_relationship_data_not_included
-          test_relationship({}, options: { include_data: false })
+          test_relationship({ meta: {} }, options: { include_data: false })
         end
 
         def test_relationship_simple_link