Remove incompatible endpoints for retrieving privilege data

The multi-state license model does not generate privileges that exist
in the database, so these endpoints are not needed.

Author: landonshumway-ia

backend/compact-connect/bin/generate_privilege_test_data.py

@@ -470,8 +470,9 @@ def main():
 
         # Check if provider already has a privilege for this specific license type in the target state
         existing_privileges = provider_user_records.get_privilege_records(
-            filter_condition=lambda p, license_type=target_license_type: p.jurisdiction == args.privilege_state
-            and p.licenseType == license_type
+            filter_condition=lambda p, license_type=target_license_type: (
+                p.jurisdiction == args.privilege_state and p.licenseType == license_type
+            )
         )
         if existing_privileges:
             if args.provider_id:

backend/compact-connect/lambdas/python/common/cc_common/data_model/data_client.py

@@ -862,8 +862,9 @@ def end_military_affiliation(self, compact: str, provider_id: str) -> None:
 
         # Get all military affiliation records that are INITIALIZING or ACTIVE
         active_military_affiliation_records = provider_user_records.get_military_affiliation_records(
-            filter_condition=lambda record: record.status
-            in [MilitaryAffiliationStatus.INITIALIZING, MilitaryAffiliationStatus.ACTIVE]
+            filter_condition=lambda record: (
+                record.status in [MilitaryAffiliationStatus.INITIALIZING, MilitaryAffiliationStatus.ACTIVE]
+            )
         )
 
         # Create provider update record to track the removal of military status fields
@@ -2312,7 +2313,7 @@ def lift_privilege_encumbrance(
 
             # Get the privilege record
             privilege_records = provider_user_records.get_privilege_records(
-                filter_condition=lambda p: (p.jurisdiction == jurisdiction and p.licenseType == license_type_name)
+                filter_condition=lambda p: p.jurisdiction == jurisdiction and p.licenseType == license_type_name
             )
 
             if not privilege_records:
@@ -2611,8 +2612,9 @@ def update_provider_home_state_jurisdiction(
 
         # Get all privileges for the provider that were not deactivated previously
         all_active_privileges = provider_user_records.get_privilege_records(
-            filter_condition=lambda privilege: privilege.homeJurisdictionChangeStatus
-            != HomeJurisdictionChangeStatusEnum.INACTIVE
+            filter_condition=lambda privilege: (
+                privilege.homeJurisdictionChangeStatus != HomeJurisdictionChangeStatusEnum.INACTIVE
+            )
         )
 
         if not all_active_privileges:
@@ -2678,8 +2680,9 @@ def update_provider_home_state_jurisdiction(
 
                     # Get licenses from the current home state
                     current_home_state_licenses = provider_user_records.get_license_records(
-                        filter_condition=lambda license_data: license_data.jurisdiction
-                        == home_jurisdiction_before_update
+                        filter_condition=lambda license_data: (
+                            license_data.jurisdiction == home_jurisdiction_before_update
+                        )
                     )
 
                     # Get unique license types from all privileges
@@ -2827,8 +2830,10 @@ def _get_privilege_transaction_items_resulting_from_home_jurisdiction_move(
         privileges_for_license_type = [
             privilege
             for privilege in provider_user_records.get_privilege_records(
-                filter_condition=lambda p: p.licenseType == license_type
-                and p.homeJurisdictionChangeStatus != HomeJurisdictionChangeStatusEnum.INACTIVE
+                filter_condition=lambda p: (
+                    p.licenseType == license_type
+                    and p.homeJurisdictionChangeStatus != HomeJurisdictionChangeStatusEnum.INACTIVE
+                )
             )
         ]
 

backend/compact-connect/lambdas/python/common/cc_common/data_model/provider_record_util.py

@@ -712,8 +712,9 @@ def find_best_license_in_current_known_licenses(self, jurisdiction: str | None =
         else:
             # if jurisdiction is not provided, we filter by the user's current home jurisdiction
             current_home_jurisdiction_license_records = self.get_license_records(
-                filter_condition=lambda license_data: license_data.jurisdiction
-                == self.get_provider_record().currentHomeJurisdiction
+                filter_condition=lambda license_data: (
+                    license_data.jurisdiction == self.get_provider_record().currentHomeJurisdiction
+                )
             )
             # if there are no licenses for their current home jurisdiction, we will search through all licenses
             license_records = (

backend/compact-connect/lambdas/python/provider-data-v1/tests/function/__init__.py

@@ -378,8 +378,8 @@ def _generate_providers(
                 patch_kwargs['new_callable'] = lambda: date_of_update
             else:
                 # Use random dates for variation
-                patch_kwargs['new_callable'] = lambda: datetime.now(tz=UTC).replace(microsecond=0) - timedelta(
-                    days=randint(1, 365)
+                patch_kwargs['new_callable'] = lambda: (
+                    datetime.now(tz=UTC).replace(microsecond=0) - timedelta(days=randint(1, 365))
                 )
 
             with patch(

backend/compact-connect/lambdas/python/purchases/handlers/privileges.py

@@ -283,8 +283,9 @@ def post_purchase_privileges(event: dict, context: LambdaContext):  # noqa: ARG0
     # Check for any adverse actions that have been lifted 2 years ago
     latest_allowed_encumbrance_lift = _get_latest_allowed_encumbrance_lift()
     blocking_encumbrance_records = provider_user_records.get_adverse_action_records(
-        filter_condition=lambda record: record.effectiveLiftDate is None
-        or record.effectiveLiftDate > latest_allowed_encumbrance_lift
+        filter_condition=lambda record: (
+            record.effectiveLiftDate is None or record.effectiveLiftDate > latest_allowed_encumbrance_lift
+        )
     )
     if blocking_encumbrance_records:
         raise CCInvalidRequestException(
@@ -293,9 +294,11 @@ def post_purchase_privileges(event: dict, context: LambdaContext):  # noqa: ARG0
 
     # we now validate that the license type matches one of the license types from the home state license records
     matching_license_records = provider_user_records.get_license_records(
-        filter_condition=lambda record: record.licenseType == body['licenseType']
-        and record.jurisdiction == current_home_jurisdiction
-        and record.compactEligibility == CompactEligibilityStatus.ELIGIBLE,
+        filter_condition=lambda record: (
+            record.licenseType == body['licenseType']
+            and record.jurisdiction == current_home_jurisdiction
+            and record.compactEligibility == CompactEligibilityStatus.ELIGIBLE
+        ),
     )
 
     if not matching_license_records: