Merge pull request #779 from cerebris/fix_related_resources_context

Fixes missing context on show_related_resources

Author: lgebhardt

lib/jsonapi/processor.rb

@@ -148,6 +148,7 @@ def show_related_resources
       source_resource ||= source_klass.find_by_key(source_id, context: context, fields: fields)
 
       related_resources = source_resource.public_send(relationship_type,
+                                                      context: context,
                                                       filters:  filters,
                                                       sort_criteria: sort_criteria,
                                                       paginator: paginator,

test/controllers/controller_test.rb

@@ -3448,6 +3448,23 @@ def test_get_related_resources
                        }, json_response)
   end
 
+  def test_get_related_resources_filtered
+    $test_user = Person.find(1)
+    get :get_related_resources, params: {moon_id: '1', relationship: 'craters', source: "api/v1/moons", filter: {description: 'Small crater'}}
+    assert_response :success
+    assert_hash_equals({
+                           data: [
+                               {
+                                   id:"A4D3",
+                                   type:"craters",
+                                   links:{self: "http://test.host/api/v1/craters/A4D3"},
+                                   attributes:{code: "A4D3", description: "Small crater"},
+                                   relationships:{moon: {links: {self: "http://test.host/api/v1/craters/A4D3/relationships/moon", related: "http://test.host/api/v1/craters/A4D3/moon"}}}
+                               }
+                           ]
+                       }, json_response)
+  end
+
   def test_show_relationship
     get :show_relationship, params: {crater_id: 'S56D', relationship: 'moon'}
 

test/fixtures/active_record.rb

@@ -662,15 +662,27 @@ class BreedsController < JSONAPI::ResourceController
     end
 
     class PlanetsController < JSONAPI::ResourceController
+      def context
+        {current_user: $test_user}
+      end
     end
 
     class PlanetTypesController < JSONAPI::ResourceController
+      def context
+        {current_user: $test_user}
+      end
     end
 
     class MoonsController < JSONAPI::ResourceController
+      def context
+        {current_user: $test_user}
+      end
     end
 
     class CratersController < JSONAPI::ResourceController
+      def context
+        {current_user: $test_user}
+      end
     end
 
     class LikesController < JSONAPI::ResourceController
@@ -1069,6 +1081,11 @@ class CraterResource < JSONAPI::Resource
 
   has_one :moon
 
+  filter :description, apply: -> (records, value, options) {
+    fail "context not set" unless options[:context][:current_user] != nil && options[:context][:current_user] == $test_user
+    records.where(:description => value)
+  }
+
   def self.verify_key(key, context = nil)
     key && String(key)
   end