Support pathlike objects in upload util

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

Author: allen-pattern

slack_sdk/web/async_client.py

@@ -3749,7 +3749,7 @@ async def files_upload_v2(
         *,
         # for sending a single file
         filename: Optional[str] = None,  # you can skip this only when sending along with content parameter
-        file: Optional[Union[str, bytes, IOBase]] = None,
+        file: Optional[Union[str, bytes, IOBase, os.PathLike]] = None,
         content: Optional[Union[str, bytes]] = None,
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,

slack_sdk/web/client.py

@@ -3739,7 +3739,7 @@ def files_upload_v2(
         *,
         # for sending a single file
         filename: Optional[str] = None,  # you can skip this only when sending along with content parameter
-        file: Optional[Union[str, bytes, IOBase]] = None,
+        file: Optional[Union[str, bytes, IOBase, os.PathLike]] = None,
         content: Optional[Union[str, bytes]] = None,
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,

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"
 

slack_sdk/web/legacy_client.py

@@ -3751,7 +3751,7 @@ def files_upload_v2(
         *,
         # for sending a single file
         filename: Optional[str] = None,  # you can skip this only when sending along with content parameter
-        file: Optional[Union[str, bytes, IOBase]] = None,
+        file: Optional[Union[str, bytes, IOBase, os.PathLike]] = None,
         content: Optional[Union[str, bytes]] = None,
         title: Optional[str] = None,
         alt_txt: Optional[str] = None,

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_to_v2_file_upload_item_can_accept_file_as_path(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