Добавить комментарии

Author: leshchenko1979

fast_bitrix24/user_request.py

@@ -330,13 +330,22 @@ async def run(self):
         return results[0] if results and is_single_item else results
 
     def prepare_item_list(self):
-        # добавим порядковый номер
+        # При отправке батчей на сервер результаты приходят не в том порядке,
+        # в котором они отправлялись. Для того, чтобы пользователь мог понять,
+        # какой результат пришел по каждому конкретному элементу списка,
+        # переданному в call(), нужно возвращать не результаты запроса общим списком,
+        # где может быть нарушен порядок, а словарь, содержащий на результаты запроса
+        # по каждому элементу списка.
+
+        # Для этого добавляем к каждому элементу списка ключ "__order",
+        # который при извелечении результатов запросов будет возвращен пользователю
+        # как ключ словаря с результатами.
+
         self.item_list = [
             ChainMap(item, {self.ID_field_name: f"order{i:010}"})
             for i, item in enumerate(self.item_list)
         ]
 
-
 class RawCallUserRequest:
     """Отправляем на сервер один элемент, не обрабатывая его и не заворачивая в батчи.
 

tests/test_server_responses.py

@@ -67,11 +67,17 @@ def test_call_several_success(bx_dummy):
     from tests.real_responses.call_several_success import response
 
     bx_dummy.srh = MockSRH(response)
-    results = bx_dummy.call("crm.lead.get", [{"ID": 35943}, {"ID": 161}, {"ID": 171}])
+
+    ID_list = [161, 35943, 171]
+    results = bx_dummy.call("crm.lead.get", [{"ID": ID} for ID in ID_list])
+
     assert isinstance(results, tuple)
     assert len(results) == 3
     assert isinstance(results[0], dict)
 
+    # нужен какой-то другой тест для контроля порядока результатов
+    # assert [result["ID"] for result in results] == ID_list, "Incorrect order of IDs"
+
 
 def test_call_list_empty(bx_dummy):
     from tests.real_responses.batch_list_empty import response