Merge pull request #1240 from Hestia-Homes/feature/sharepoint-renamer-no-images

Sharepoint renamer: ignore jpg and heic files. resolve filepath relatively

applications/sharepoint_renamer/handler.py

@@ -1,19 +1,24 @@
+import os
 from typing import Any
 
-from applications.sharepoint_renamer.sharepoint_renamer_request import SharepointRenamerRequest
+from applications.sharepoint_renamer.sharepoint_renamer_request import (
+    SharepointRenamerRequest,
+)
 from orchestration.sharepoint_renamer_orchestrator import SharepointRenamerOrchestrator
 from utils.sharepoint.domna_sharepoint_client import DomnaSharepointClient
 from utils.sharepoint.domna_sites import DomnaSites
 
-CSV_PATH = "applications/sharepoint_renamer/sero_address_list.csv"
+CSV_PATH = os.path.join(os.path.dirname(os.path.abspath(__file__)), "sero_address_list.csv")
 
 
 def handler(event: dict[str, Any], context: Any) -> None:
     request = SharepointRenamerRequest.model_validate(event)
     sp_client = DomnaSharepointClient(DomnaSites.SOCIAL_HOUSING_WAVE_3)
-    orchestrator = SharepointRenamerOrchestrator(sp_client, CSV_PATH, dry_run=request.dry_run)
+    orchestrator = SharepointRenamerOrchestrator(
+        sp_client, CSV_PATH, dry_run=request.dry_run
+    )
     orchestrator.run()
 
 
 if __name__ == "__main__":
-    handler({"dry_run": True}, None)
+    handler({"dry_run": False}, None)

orchestration/sharepoint_renamer_orchestrator.py

@@ -100,6 +100,8 @@ class SharepointRenamerOrchestrator:
                 )
             elif "file" in item:
                 original_name: str = item["name"]
+                if os.path.splitext(original_name)[1].lower() in {".jpg", ".heic"}:
+                    continue
                 new_name = build_canonical_filename(
                     uprn, address, postcode, original_name
                 )