Add full_file_info_required to files_upload_v2 method (#1282)

Author: seratch

integration_tests/web/test_files_upload_v2.py

@@ -190,3 +190,50 @@ async def test_uploading_file_with_token_param_async(self):
             file=upload["file"]["id"],
         )
         self.assertIsNotNone(deletion)
+
+    def test_request_file_info_false(self):
+        client = self.sync_client
+        upload = client.files_upload_v2(
+            channels=self.channel_id,
+            title="Foo",
+            filename="foo.txt",
+            content="foo",
+        )
+        self.assertIsNotNone(upload)
+        self.assertIsNotNone(upload.get("files")[0].get("id"))
+        self.assertIsNotNone(upload.get("files")[0].get("name"))
+
+        upload = client.files_upload_v2(
+            channels=self.channel_id,
+            title="Foo",
+            filename="foo.txt",
+            content="foo",
+            request_file_info=False,
+        )
+        self.assertIsNotNone(upload)
+        self.assertIsNotNone(upload.get("files")[0].get("id"))
+        self.assertIsNone(upload.get("files")[0].get("name"))
+
+    @async_test
+    async def test_request_file_info_false_async(self):
+        client = self.async_client
+        upload = await client.files_upload_v2(
+            channels=self.channel_id,
+            title="Foo",
+            filename="foo.txt",
+            content="foo",
+        )
+        self.assertIsNotNone(upload)
+        self.assertIsNotNone(upload.get("files")[0].get("id"))
+        self.assertIsNotNone(upload.get("files")[0].get("name"))
+
+        upload = await client.files_upload_v2(
+            channels=self.channel_id,
+            title="Foo",
+            filename="foo.txt",
+            content="foo",
+            request_file_info=False,
+        )
+        self.assertIsNotNone(upload)
+        self.assertIsNotNone(upload.get("files")[0].get("id"))
+        self.assertIsNone(upload.get("files")[0].get("name"))

slack_sdk/web/async_client.py

@@ -3015,6 +3015,7 @@ async def files_upload_v2(
         channel: Optional[str] = None,
         initial_comment: Optional[str] = None,
         thread_ts: Optional[str] = None,
+        request_file_info: bool = True,
         **kwargs,
     ) -> AsyncSlackResponse:
         """This wrapper method provides an easy way to upload files using the following endpoints:
@@ -3113,11 +3114,12 @@ async def files_upload_v2(
             thread_ts=thread_ts,
             **kwargs,
         )
-        await _attach_full_file_metadata_async(
-            client=self,
-            token_as_arg=kwargs.get("token"),
-            completion=completion,
-        )
+        if request_file_info is True:
+            await _attach_full_file_metadata_async(
+                client=self,
+                token_as_arg=kwargs.get("token"),
+                completion=completion,
+            )
         return completion
 
     async def files_getUploadURLExternal(

slack_sdk/web/client.py

@@ -3006,6 +3006,7 @@ def files_upload_v2(
         channel: Optional[str] = None,
         initial_comment: Optional[str] = None,
         thread_ts: Optional[str] = None,
+        request_file_info: bool = True,
         **kwargs,
     ) -> SlackResponse:
         """This wrapper method provides an easy way to upload files using the following endpoints:
@@ -3104,11 +3105,12 @@ def files_upload_v2(
             thread_ts=thread_ts,
             **kwargs,
         )
-        _attach_full_file_metadata(
-            client=self,
-            token_as_arg=kwargs.get("token"),
-            completion=completion,
-        )
+        if request_file_info is True:
+            _attach_full_file_metadata(
+                client=self,
+                token_as_arg=kwargs.get("token"),
+                completion=completion,
+            )
         return completion
 
     def files_getUploadURLExternal(

slack_sdk/web/legacy_client.py

@@ -3017,6 +3017,7 @@ def files_upload_v2(
         channel: Optional[str] = None,
         initial_comment: Optional[str] = None,
         thread_ts: Optional[str] = None,
+        request_file_info: bool = True,
         **kwargs,
     ) -> Union[Future, SlackResponse]:
         """This wrapper method provides an easy way to upload files using the following endpoints:
@@ -3115,11 +3116,12 @@ def files_upload_v2(
             thread_ts=thread_ts,
             **kwargs,
         )
-        _attach_full_file_metadata(
-            client=self,
-            token_as_arg=kwargs.get("token"),
-            completion=completion,
-        )
+        if request_file_info is True:
+            _attach_full_file_metadata(
+                client=self,
+                token_as_arg=kwargs.get("token"),
+                completion=completion,
+            )
         return completion
 
     def files_getUploadURLExternal(