api: Enable fetching Grants by non-primay OrgIDs

This commit enables fetching Grant data by the non-primary OrgID of a
Recipient of Funding Organisation.

datastore/api/org/api.py

@@ -110,12 +110,16 @@ def get_queryset(self):
         org_id = self.kwargs.get("org_id")
 
         # Raise 404 if the Org doesn't exist
-        if not models.Organisation.exists(org_id):
+        try:
+            org = models.Organisation.get(org_id)
+        except models.Organisation.DoesNotExist:
             raise rest_framework.exceptions.NotFound()
 
+        org_ids = [org.org_id] + [lo.org_id for lo in org.linked_orgs]
+
         return (
             db.Grant.objects.filter(source_file__latest__series=db.Latest.CURRENT)
-            .filter(funding_org_ids__contains=[org_id])
+            .filter(funding_org_ids__overlap=org_ids)
             .select_related("source_file")
         )
 
@@ -135,11 +139,15 @@ def get_queryset(self):
         org_id = self.kwargs.get("org_id")
 
         # Raise 404 if the Org doesn't exist
-        if not models.Organisation.exists(org_id):
+        try:
+            org = models.Organisation.get(org_id)
+        except models.Organisation.DoesNotExist:
             raise rest_framework.exceptions.NotFound()
 
+        org_ids = [org.org_id] + [lo.org_id for lo in org.linked_orgs]
+
         return (
             db.Grant.objects.filter(source_file__latest__series=db.Latest.CURRENT)
-            .filter(recipient_org_ids__contains=[org_id])
+            .filter(recipient_org_ids__overlap=org_ids)
             .select_related("source_file")
         )

datastore/tests/test_org_api.py

@@ -1,9 +1,8 @@
 import datetime
 
-from django.urls import reverse_lazy
-from django.test import TestCase
 from django.core.management import call_command
-
+from django.test import TestCase
+from django.urls import reverse_lazy
 
 current_year = datetime.date.today().year
 
@@ -21,6 +20,7 @@ def setUpTestData(cls):
     funder_grant_id = "360G-kahM5Ooc2u"
 
     recipient_org_id = "360G-example-a"
+    recipient_nonprimary_org_id = "360G-example-nonprimary"
     recipient_org_name = "Receive an example grant"
     recipient_grant_id = "360G-Eiz4Veij8o"
 
@@ -202,7 +202,7 @@ def test_recipient_org_detail(self):
         )
 
         expected_org_data = {
-            "linked_orgs": [],
+            "linked_orgs": [{"org_id": "360G-example-nonprimary"}],
             "self": "http://testserver" + expected_self_url,
             "grants_made": "http://testserver" + expected_grants_made_url,
             "grants_received": "http://testserver" + expected_received_made_url,
@@ -226,13 +226,22 @@ def test_recipient_org_detail(self):
             "name": self.recipient_org_name,
         }
 
+        # Lookup by primary org ID
         data = self.client.get(
             f"/api/v1/org/{self.recipient_org_id}/",
             headers={"accept": "application/json"},
         ).json()
 
         self.assertEqual(data, expected_org_data)
 
+        # Lookup by non-primary org ID
+        data = self.client.get(
+            f"/api/v1/org/{self.recipient_nonprimary_org_id}/",
+            headers={"accept": "application/json"},
+        ).json()
+
+        self.assertEqual(data, expected_org_data)
+
     def test_recipient_grants_received(self):
         """Assert that the Recipient gets the expected grant."""
 
@@ -318,6 +327,15 @@ def test_recipient_grants_received(self):
         self.assertIn(self.recipient_grant_id, grants)
         self.assertEqual(expected_grant_data, grants[self.recipient_grant_id])
 
+    def test_recipient_nonprimary_grants_received(self):
+        # Check that searching by a non-primary OrgID successfully returns results
+        data = self.client.get(
+            f"/api/v1/org/{self.recipient_nonprimary_org_id}/grants_received/",
+            headers={"accept": "application/json"},
+        ).json()
+
+        self.assertEqual(data["count"], 50)
+
     def test_recipient_grants_made(self):
         """A Recipient-only Org should not make any grants."""
         data = self.client.get(

datastore/tests/test_quality_data.py

@@ -32,7 +32,7 @@ def test_create_sourcefile_publisher_quality_data(self):
 
         expected_sourcefile_aggregate = {
             "count": 5,
-            "recipient_organisations": ["360G-example-a"],
+            "recipient_organisations": ["360G-example-a", "360G-example-nonprimary"],
             "recipient_individuals": 0,
             "funders": ["GB-example-b"],
             "max_award_date": "2019-10-03",