[#2863] When retrieving eHerkenning-cases, filter on either vestiging…

…snummer or rsin/kvk, but not both

src/open_inwoner/cms/products/tests/test_plugin_categories.py

@@ -665,7 +665,6 @@ def test_categories_based_on_cases_for_eherkenning_user_with_vestigingsnummer(
                 furl(f"{ZAKEN_ROOT}zaken")
                 .add(
                     {
-                        "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": identifier,
                         "maximaleVertrouwelijkheidaanduiding": VertrouwelijkheidsAanduidingen.beperkt_openbaar,
                         "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": "1234",
                     }

src/open_inwoner/openzaak/clients.py

@@ -79,15 +79,19 @@ def fetch_cases(
             return self.fetch_cases_by_bsn(
                 user_bsn, max_requests=max_requests, identificatie=identificatie
             )
-
-        if user_kvk or user_rsin:
-            user_kvk_or_rsin = user_rsin if user_rsin else user_kvk
-            return self.fetch_cases_by_kvk_or_rsin(
-                user_kvk_or_rsin,
+        if vestigingsnummer:
+            return self.fetch_cases_for_company(
                 max_requests=max_requests,
                 zaak_identificatie=identificatie,
                 vestigingsnummer=vestigingsnummer,
             )
+        if user_kvk or user_rsin:
+            user_kvk_or_rsin = user_rsin if user_rsin else user_kvk
+            return self.fetch_cases_for_company(
+                kvk_or_rsin=user_kvk_or_rsin,
+                max_requests=max_requests,
+                zaak_identificatie=identificatie
+            )
         return []
 
     @cache_result(
@@ -142,36 +146,41 @@ def fetch_cases_by_bsn(
         "{self.base_url}:cases:{kvk_or_rsin}:{vestigingsnummer}:{max_requests}:{zaak_identificatie}",
         timeout=settings.CACHE_ZGW_ZAKEN_TIMEOUT,
     )
-    def fetch_cases_by_kvk_or_rsin(
+    def fetch_cases_for_company(
         self,
-        kvk_or_rsin: str | None,
+        kvk_or_rsin: str | None = None,
         max_requests: int | None = 4,
         zaak_identificatie: str | None = None,
         vestigingsnummer: str | None = None,
     ) -> list[Zaak]:
         """
         retrieve cases for particular company with allowed confidentiality level
 
+        :param kvk_or_rsin: - used to filter the cases by a KVK number or RSIN (configured via OpenZaakConfig)
         :param max_requests: - used to limit the number of requests to list_zaken resource.
         :param zaak_identificatie: - used to filter the cases by a unique Zaak identification number
         :param vestigingsnummer: - used to filter the cases by a vestigingsnummer
         """
-        if not kvk_or_rsin:
-            return []
 
         config = OpenZaakConfig.get_solo()
 
         params = {
-            "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": kvk_or_rsin,
             "maximaleVertrouwelijkheidaanduiding": config.zaak_max_confidentiality,
         }
-
         if vestigingsnummer:
             params.update(
                 {
                     "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": vestigingsnummer,
                 }
             )
+        elif kvk_or_rsin:
+            params.update(
+                {
+                    "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": kvk_or_rsin,
+                }
+            )
+        else:
+            return []
 
         if zaak_identificatie:
             params.update({"identificatie": zaak_identificatie})

src/open_inwoner/openzaak/tests/test_cases.py

@@ -422,7 +422,6 @@ def _setUpMocks(self, m):
                 furl(f"{self.zaken_root}zaken")
                 .add(
                     {
-                        "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": identifier,
                         "maximaleVertrouwelijkheidaanduiding": VertrouwelijkheidsAanduidingen.beperkt_openbaar,
                         "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": "1234",
                     }
@@ -840,13 +839,10 @@ def test_list_cases_for_eherkenning_user_with_vestigingsnummer(self, m):
                         else self.eherkenning_user.kvk
                     )
 
-                    self.assertEqual(len(list_zaken_req.qs), 3)
+                    self.assertEqual(len(list_zaken_req.qs), 2)
                     self.assertEqual(
                         list_zaken_req.qs,
                         {
-                            "rol__betrokkeneidentificatie__nietnatuurlijkpersoon__innnnpid": [
-                                identifier
-                            ],
                             "maximalevertrouwelijkheidaanduiding": [
                                 VertrouwelijkheidsAanduidingen.beperkt_openbaar
                             ],