Support pathlike objects in upload util

Also switch to inbuilt filename encode helper, since it accepts paths

Author: allen-pattern

slack_sdk/web/internal_utils.py

@@ -317,8 +317,8 @@ def _to_v2_file_upload_item(upload_file: Dict[str, Any]) -> Dict[str, Optional[A
     content = upload_file.get("content")
     data: Optional[bytes] = None
     if file is not None:
-        if isinstance(file, str):  # filepath
-            with open(file.encode("utf-8", "ignore"), "rb") as readable:
+        if isinstance(file, (str, os.PathLike)):  # filepath
+            with open(os.fsencode(file), "rb") as readable:
                 data = readable.read()
         elif isinstance(file, bytes):
             data = file
@@ -339,8 +339,8 @@ def _to_v2_file_upload_item(upload_file: Dict[str, Any]) -> Dict[str, Optional[A
     filename = upload_file.get("filename")
     if filename is None:
         # use the local filename if filename is missing
-        if isinstance(file, str):
-            filename = file.split(os.path.sep)[-1]
+        if isinstance(file, (str, os.PathLike)):
+            filename = os.fspath(file).split(os.path.sep)[-1]
         else:
             filename = "Uploaded file"
 

tests/slack_sdk/web/test_internal_utils.py

@@ -1,6 +1,7 @@
 import json
 import unittest
 from io import BytesIO
+from pathlib import Path
 from typing import Dict, Sequence, Union
 
 import pytest
@@ -101,6 +102,13 @@ def test_files_upload_v2_issue_1356(self):
         file_io_item = _to_v2_file_upload_item({"file": file_io, "filename": "foo.txt"})
         assert file_io_item.get("filename") == "foo.txt"
 
+    def test_files_upload_v2_paths(self):
+        filepath = "tests/slack_sdk/web/test_internal_utils.py"
+        upload_item_str = _to_v2_file_upload_item({"file": filepath})
+        upload_item_path = _to_v2_file_upload_item({"file": Path(filepath)})
+        assert upload_item_path == upload_item_str
+        assert upload_item_str.get("filename") == "test_internal_utils.py"
+
     def test_next_cursor_is_present(self):
         assert _next_cursor_is_present({"next_cursor": "next-page"}) is True
         assert _next_cursor_is_present({"next_cursor": ""}) is False