formatting

Author: landonshumway-ia

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

@@ -2641,13 +2641,14 @@ def encumber_home_jurisdiction_license_privileges(
         adverse_action = provider_user_records.get_adverse_action_by_id(adverse_action_id)
 
         if not adverse_action:
-            logger.error("Adverse Action not found by id",
-                         provider_id=provider_id,
-                         encumbered_license_jurisdiction=jurisdiction,
-                         encumbered_license_type=license_type_abbreviation,
-                         adverse_action_id=adverse_action_id)
-            raise CCInternalException("Adverse Action not found by id")
-
+            logger.error(
+                'Adverse Action not found by id',
+                provider_id=provider_id,
+                encumbered_license_jurisdiction=jurisdiction,
+                encumbered_license_type=license_type_abbreviation,
+                adverse_action_id=adverse_action_id,
+            )
+            raise CCInternalException('Adverse Action not found by id')
 
         # Find privileges associated with the license that which was encumbered, which themselves are not currently
         # encumbered

backend/compact-connect/lambdas/python/common/cc_common/data_model/schema/adverse_action/record.py

@@ -49,7 +49,8 @@ class AdverseActionRecordSchema(BaseRecordSchema):
     # TODO - remove this hook once migration is complete to the new field  # noqa: FIX002
     @pre_load
     def migrate_clinical_privilege_action_category_on_load(self, in_data, **_kwargs):
-        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list when loading from database."""
+        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list
+        when loading from database."""
         # If the deprecated field exists and the new field doesn't, migrate it
         if 'clinicalPrivilegeActionCategory' in in_data and 'clinicalPrivilegeActionCategories' not in in_data:
             in_data['clinicalPrivilegeActionCategories'] = [in_data['clinicalPrivilegeActionCategory']]

backend/compact-connect/lambdas/python/common/cc_common/data_model/schema/privilege/record.py

@@ -56,25 +56,27 @@ class EncumbranceDetailsSchema(Schema):
     """
     Schema for tracking details about an encumbrance.
     """
+
     clinicalPrivilegeActionCategories = List(ClinicalPrivilegeActionCategoryField(), required=False, allow_none=False)
     adverseActionId = UUID(required=True, allow_none=False)
     # present if update is created by upstream license encumbrance
     licenseJurisdiction = Jurisdiction(required=False, allow_none=False)
     # TODO - remove this field as part of https://github.com/csg-org/CompactConnect/issues/1136  # noqa: FIX002
     clinicalPrivilegeActionCategory = ClinicalPrivilegeActionCategoryField(required=False, allow_none=False)
 
-
     # TODO - remove these hooks as part of https://github.com/csg-org/CompactConnect/issues/1136  # noqa: FIX002
     @pre_load
     def migrate_clinical_privilege_action_category_on_load(self, in_data, **_kwargs):
-        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list when loading from database."""
+        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list
+        when loading from database."""
         if 'clinicalPrivilegeActionCategory' in in_data and 'clinicalPrivilegeActionCategories' not in in_data:
             in_data['clinicalPrivilegeActionCategories'] = [in_data['clinicalPrivilegeActionCategory']]
         return in_data
 
     @pre_dump
     def migrate_clinical_privilege_action_category_on_dump(self, in_data, **_kwargs):
-        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list when dumping to database."""
+        """Migrate deprecated clinicalPrivilegeActionCategory to clinicalPrivilegeActionCategories list
+        when dumping to database."""
         if 'clinicalPrivilegeActionCategory' in in_data and 'clinicalPrivilegeActionCategories' not in in_data:
             in_data['clinicalPrivilegeActionCategories'] = [in_data['clinicalPrivilegeActionCategory']]
         return in_data

backend/compact-connect/lambdas/python/provider-data-v1/handlers/encumbrance.py

@@ -105,8 +105,8 @@ def _generate_adverse_action_for_record_type(
     if 'clinicalPrivilegeActionCategory' in adverse_action_request:
         # replicate data to both the deprecated and new fields
         adverse_action.clinicalPrivilegeActionCategory = ClinicalPrivilegeActionCategory(
-                adverse_action_request['clinicalPrivilegeActionCategory']
-            )
+            adverse_action_request['clinicalPrivilegeActionCategory']
+        )
         adverse_action.clinicalPrivilegeActionCategories = [
             ClinicalPrivilegeActionCategory(adverse_action_request['clinicalPrivilegeActionCategory'])
         ]

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

@@ -333,7 +333,7 @@ def test_privilege_encumbrance_handler_migrates_clinical_privilege_action_catego
         # Load the adverse action record from the database
         loaded_adverse_action = AdverseActionData.from_database_record(item)
 
-        # TODO - remove this assertion as part of https://github.com/csg-org/CompactConnect/issues/1136
+        # TODO - remove this assertion as part of https://github.com/csg-org/CompactConnect/issues/1136 # noqa: FIX002
         # Verify that the deprecated field is not present in the stored data
         self.assertIn('clinicalPrivilegeActionCategory', item)
         self.assertEqual('Unsafe Practice or Substandard Care', loaded_adverse_action.clinicalPrivilegeActionCategory)
@@ -615,7 +615,7 @@ def test_license_encumbrance_handler_migrates_clinical_privilege_action_category
         # Load the adverse action record from the database
         loaded_adverse_action = AdverseActionData.from_database_record(item)
 
-        # TODO - remove this assertion as part of https://github.com/csg-org/CompactConnect/issues/1136
+        # TODO - remove this assertion as part of https://github.com/csg-org/CompactConnect/issues/1136 # noqa: FIX002
         # Verify that the deprecated field is not present in the stored data
         self.assertIn('clinicalPrivilegeActionCategory', item)
         self.assertEqual('Unsafe Practice or Substandard Care', loaded_adverse_action.clinicalPrivilegeActionCategory)