Fix typecheking and simplify call(raw=True) (#183)

Author: leshchenko1979

README.md

@@ -128,12 +128,22 @@ b.call('crm.deal.update', tasks)
 ### call(raw=True)
 Вызов `call` с парамтером `raw=True` отправляет на сервер переданные ему параметры в оригинальном, необработанном виде (пропуская упаковку в батчи), и возвращает ответ сервера без какой-либо обработки.
 
-Подобный вызов можно использовать в отладочных целях, но кроме того, придется его использовать для отправки запросов, в параметрах которых есть `None` (None применяется для стирания значения полей, а упаковка в батчи мешает передавать `None`).
+Подобный вызов можно использовать в отладочных целях, но кроме того, придется его использовать для отправки запросов, которые:
+- в параметрах имеют `None` (None применяется для стирания значения полей, а упаковка в батчи мешает передавать `None`),
+- используют устревшие методы Битрикс24, которые принимают на вход список (см. [#157](https://github.com/leshchenko1979/fast_bitrix24/issues/157)).
+
 
 ```python
 # стереть DESCRIPTION в лиде 123
 params = {"ID": 123, "fields": {"DESCRIPTION": None}}
 b.call('crm.lead.update', params, raw=True)
+
+# добавить комментарий к задаче
+b.call(
+    'task.commentitem.add',
+    [123, {"POST_MESSAGE": "Комментарий к задаче"}],
+    raw=True
+)
 ```
 
 ### `call_batch()`

fast_bitrix24/server_response.py

@@ -55,7 +55,7 @@ def extract_results(self) -> Union[Dict, List[Dict]]:
             return self.extract_from_single_response(self.result)
 
     def is_batch(self) -> bool:
-        return "result" in self.response and "result" in self.response["result"]
+        return isinstance(self.result, dict) and "result" in self.result
 
     @staticmethod
     def extract_from_single_response(result: dict):

fast_bitrix24/user_request.py

@@ -260,7 +260,7 @@ def prepare_item_list(self):
         ]
 
 
-class RawCallUserRequest(UserRequestAbstract):
+class RawCallUserRequest():
     """Отправляем на сервер один элемент, не обрабатывая его и не заворачивая в батчи.
 
     Нужно для устревших методов, которые принимают на вход список
@@ -271,22 +271,13 @@ class RawCallUserRequest(UserRequestAbstract):
     """
 
     @beartype
-    def __init__(self, bitrix, method: str, item: Union[Dict, None]):
-        super().__init__(bitrix, method, item)
-
-    def standardized_params(self, p):
-        """Пропускаем все проверки и изменения параметров."""
-        return p
-
-    def standardized_method(self, method: str):
-        """Пропускаем все проверки и изменения методов."""
-        return method
-
-    def check_special_limitations(self):
-        return True
+    def __init__(self, bitrix, method: str, item):
+        self.srh = bitrix.srh
+        self.method = method
+        self.item = item
 
     async def run(self):
-        return await self.srh.single_request(self.method, self.params)
+        return await self.srh.single_request(self.method, self.item)
 
 
 class ListAndGetUserRequest:

tests/real_responses/single_add.py

@@ -0,0 +1,138 @@
+response = {
+    "result": {
+        "task": {
+            "id": "456",
+            "title": "Алм",
+            "description": "Курс на социально-ориентированный национальный проект станет частью наших традиций",
+            "descriptionInBbcode": "Y",
+            "declineReason": None,
+            "priority": "1",
+            "status": "2",
+            "notViewed": "N",
+            "statusComplete": "1",
+            "multitask": "N",
+            "stageId": "0",
+            "responsibleId": "1",
+            "responsibleName": "Korvin",
+            "responsibleLastName": "-Control",
+            "responsibleSecondName": None,
+            "responsibleLogin": "admin",
+            "responsibleWorkPosition": None,
+            "responsiblePhoto": None,
+            "dateStart": None,
+            "durationFact": None,
+            "timeEstimate": "0",
+            "timeSpentInLogs": None,
+            "replicate": "N",
+            "deadline": "2022-05-09T01:00:00+04:00",
+            "deadlineOrig": "2022-05-09 00:00:00",
+            "startDatePlan": None,
+            "endDatePlan": None,
+            "createdBy": "1",
+            "createdByName": "Korvin",
+            "createdByLastName": "-Control",
+            "createdBySecondName": None,
+            "createdByLogin": "admin",
+            "createdByWorkPosition": None,
+            "createdByPhoto": None,
+            "createdDate": "2022-06-08T21:54:00+04:00",
+            "changedBy": "1",
+            "changedDate": "2022-06-08T21:54:00+04:00",
+            "statusChangedBy": "1",
+            "statusChangedDate": "2022-06-08T21:54:00+04:00",
+            "closedBy": None,
+            "closedDate": None,
+            "activityDate": "2022-06-08T21:54:00+04:00",
+            "guid": "{bbe24f38-df7f-42d4-b172-9ebf432559ed}",
+            "xmlId": None,
+            "mark": None,
+            "allowChangeDeadline": "N",
+            "allowTimeTracking": "N",
+            "matchWorkTime": "N",
+            "taskControl": "N",
+            "addInReport": "N",
+            "groupId": "0",
+            "forumTopicId": None,
+            "parentId": None,
+            "commentsCount": None,
+            "serviceCommentsCount": None,
+            "forumId": None,
+            "siteId": "s1",
+            "subordinate": "N",
+            "exchangeModified": None,
+            "exchangeId": None,
+            "outlookVersion": "1",
+            "viewedDate": None,
+            "deadlineCounted": "0",
+            "forkedByTemplateId": None,
+            "favorite": "N",
+            "sorting": None,
+            "durationPlanSeconds": None,
+            "durationTypeAll": "days",
+            "durationPlan": None,
+            "durationType": "days",
+            "isMuted": "N",
+            "isPinned": "N",
+            "isPinnedInGroup": "N",
+            "ufCrmTask": False,
+            "ufTaskWebdavFiles": False,
+            "ufMailMessage": None,
+            "auditors": [],
+            "accomplices": [],
+            "tags": ["скпэ22-2"],
+            "checklist": [],
+            "files": [],
+            "dependsOn": [],
+            "group": [],
+            "creator": {
+                "id": "1",
+                "name": "Korvin -Control",
+                "link": "/company/personal/user/1/",
+                "icon": "/bitrix/images/tasks/default_avatar.png",
+            },
+            "responsible": {
+                "id": "1",
+                "name": "Korvin -Control",
+                "link": "/company/personal/user/1/",
+                "icon": "/bitrix/images/tasks/default_avatar.png",
+            },
+            "newCommentsCount": 0,
+            "action": {
+                "accept": False,
+                "decline": False,
+                "complete": True,
+                "approve": False,
+                "disapprove": False,
+                "start": True,
+                "pause": False,
+                "delegate": True,
+                "remove": True,
+                "edit": True,
+                "defer": True,
+                "renew": False,
+                "create": True,
+                "changeDeadline": True,
+                "checklistAddItems": True,
+                "addFavorite": True,
+                "deleteFavorite": False,
+                "rate": True,
+                "edit.originator": False,
+                "checklist.reorder": True,
+                "elapsedtime.add": True,
+                "dayplan.timer.toggle": False,
+                "edit.plan": True,
+                "checklist.add": True,
+                "favorite.add": True,
+                "favorite.delete": False,
+            },
+        }
+    },
+    "time": {
+        "start": 1654710840.774196,
+        "finish": 1654710840.852366,
+        "duration": 0.0781700611114502,
+        "processing": 0.06392693519592285,
+        "date_start": "2022-06-08T20:54:00+03:00",
+        "date_finish": "2022-06-08T20:54:00+03:00",
+    },
+}

tests/test_server_responses.py

@@ -140,3 +140,22 @@ def test_stagehistory(bx_dummy, recwarn):
     print(w.message for w in recwarn.list)
     with pytest.raises(Exception):
         recwarn.pop(RuntimeWarning)
+
+
+def test_single_add(bx_dummy):
+    from tests.real_responses.single_add import response
+
+    bx_dummy.srh = MockSRH(response)
+    results = bx_dummy.call(
+        "tasks.task.add",
+        {
+            "fields": {
+                "TITLE": "АГ",
+                "DESCRIPTION": "Сложно сказать, почему потускнели светлые лики икон",
+                "RESPONSIBLE_ID": 1,
+                "TAGS": "скпэ22-4",
+                "DEADLINE": "2022/07/30",
+            }
+        },
+    )
+    assert isinstance(results, dict)

tests/test_sync.py

@@ -2,6 +2,7 @@
 
 import pytest
 
+
 @pytest.mark.skipif(
     not os.getenv("FAST_BITRIX24_TEST_WEBHOOK"),
     reason="Нет аккаунта, на котором можно проверить",
@@ -137,7 +138,9 @@ def test_issue_132(self, create_100_tasks):
         def test_case(self, get_test):
             b = get_test
 
-            with pytest.raises(RuntimeError, match="Could not find value for parameter"):
+            with pytest.raises(
+                RuntimeError, match="Could not find value for parameter"
+            ):
                 b.call("disk.file.get", [{"ID": 1}])
 
             with pytest.raises(RuntimeError, match="Could not find entity with id"):
@@ -150,7 +153,6 @@ def test_long_task_description(self, get_test):
             )
             b.get_by_ID("crm.lead.delete", [lead_no])
 
-
     class TestParamsEncoding:
         def test_mobile_phone(self, get_test):
             b = get_test
@@ -204,6 +206,7 @@ def test_product_rows(self, create_a_lead):
 
             assert len(product_rows) == len(result_rows)
 
+
 class TestErrors:
     def test_get_all(self, bx_dummy):
         b = bx_dummy
@@ -242,3 +245,5 @@ async def stub(*args, **kwargs):
 
         with pytest.raises(Exception):
             b.call("_", {})
+
+        b.call("_", [1, {"a": 2}], raw=True)