fix errors after refactoring

Signed-off-by: mesilov <mesilov.maxim@gmail.com>
mesilov · Aug 6, 2023 · b4c54b5 · b4c54b5
1 parent dc8cb4e
commit b4c54b5
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 32 deletions.
diff --git a/src/Core/ApiLevelErrorHandler.php b/src/Core/ApiLevelErrorHandler.php
@@ -21,6 +21,8 @@ class ApiLevelErrorHandler
 {
     protected LoggerInterface $logger;
     protected const ERROR_KEY = 'error';
+    protected const RESULT_KEY = 'result';
+    protected const RESULT_ERROR_KEY = 'result_error';
     protected const ERROR_DESCRIPTION_KEY = 'error_description';
 
     /**
@@ -41,17 +43,18 @@ public function __construct(LoggerInterface $logger)
      */
     public function handle(array $responseBody): void
     {
-        //ошибка единичного запроса
+        // single query error response
         if (array_key_exists(self::ERROR_KEY, $responseBody) && array_key_exists(self::ERROR_DESCRIPTION_KEY, $responseBody)) {
             $this->handleError($responseBody);
         }
 
-        // ошибка в батче
-        if (!array_key_exists('result', $responseBody)) {
+        // error in batch response
+        if (!array_key_exists(self::RESULT_KEY, $responseBody) || (!is_array($responseBody[self::RESULT_KEY]))) {
             return;
         }
-        if (array_key_exists('result_error', $responseBody['result'])) {
-            foreach ($responseBody['result']['result_error'] as $cmdId => $errorData) {
+
+        if (array_key_exists(self::RESULT_ERROR_KEY, $responseBody[self::RESULT_KEY])) {
+            foreach ($responseBody[self::RESULT_KEY][self::RESULT_ERROR_KEY] as $cmdId => $errorData) {
                 $this->handleError($errorData, $cmdId);
             }
         }

diff --git a/.../Integration/Core/OperatingTimingTest.php → tests/Temp/OperatingTimingTest.php b/.../Integration/Core/OperatingTimingTest.php → tests/Temp/OperatingTimingTest.php
@@ -43,33 +43,33 @@ public function testOperatingTiming(): void
         //1510 |operating   104.5405356884  |cur_time  2023-04-15 16:57:21 |op_reset_at 1681567640    →   2023-04-15 14:07:20
 
 
-        $contactsToUpdate = $this->getContactsUpdateCommand(15000);
-
-
-        //todo считать количество контактов для обновления и считать количество контактов которые обновили, если не совпало, то падаем с ошибкой
-
-        // обновляем контакты в батч-режиме
-        $cnt = 0;
-        foreach ($this->contactService->batch->update($contactsToUpdate) as $b24ContactId => $contactUpdateResult) {
-            $cnt++;
-
-            $debugOperatingLog = sprintf(
-                    'cnt    %s |id %s |operating   %s  |cur_time  %s |op_reset_at %s    →   %s',
-                    $cnt,
-                    $b24ContactId,
-                    $contactUpdateResult->getResponseData()->getTime()->getOperating(),
-                    $contactUpdateResult->getResponseData()->getTime()->getDateFinish()->format('Y-m-d H:i:s'),
-                    $contactUpdateResult->getResponseData()->getTime()->getOperatingResetAt(),
-                    (new DateTime)->setTimestamp($contactUpdateResult->getResponseData()->getTime()->getOperatingResetAt())->format('Y-m-d H:i:s')
-                ) . PHP_EOL;
-            file_put_contents('operating.log', $debugOperatingLog);
-        }
-
-        $this->assertEquals(
-            count($contactsToUpdate),
-            $cnt,
-            sprintf('updated contacts count %s not equal to expected %s cmd items', $cnt, count($contactsToUpdate))
-        );
+//        $contactsToUpdate = $this->getContactsUpdateCommand(15000);
+//
+//
+//        //todo считать количество контактов для обновления и считать количество контактов которые обновили, если не совпало, то падаем с ошибкой
+//
+//        // обновляем контакты в батч-режиме
+//        $cnt = 0;
+//        foreach ($this->contactService->batch->update($contactsToUpdate) as $b24ContactId => $contactUpdateResult) {
+//            $cnt++;
+//
+//            $debugOperatingLog = sprintf(
+//                    'cnt    %s |id %s |operating   %s  |cur_time  %s |op_reset_at %s    →   %s',
+//                    $cnt,
+//                    $b24ContactId,
+//                    $contactUpdateResult->getResponseData()->getTime()->getOperating(),
+//                    $contactUpdateResult->getResponseData()->getTime()->getDateFinish()->format('Y-m-d H:i:s'),
+//                    $contactUpdateResult->getResponseData()->getTime()->getOperatingResetAt(),
+//                    (new DateTime)->setTimestamp($contactUpdateResult->getResponseData()->getTime()->getOperatingResetAt())->format('Y-m-d H:i:s')
+//                ) . PHP_EOL;
+//            file_put_contents('operating.log', $debugOperatingLog);
+//        }
+//
+//        $this->assertEquals(
+//            count($contactsToUpdate),
+//            $cnt,
+//            sprintf('updated contacts count %s not equal to expected %s cmd items', $cnt, count($contactsToUpdate))
+//        );
 
         // шаг 1 - выброс корректного исключения, что мол упали из за блокировки метода
         // проблемы: - можно потерять часть данных при обновлении, т.к. мы не знаем, какие контакты в клиентском коде обновились, а какие нет или знаем?