Full tests coverage

he110 · Aug 24, 2019 · 7ddf4dd · 7ddf4dd
1 parent 1abe486
commit 7ddf4dd
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/composer.json b/composer.json
@@ -14,7 +14,8 @@
   ],
   "require": {
     "php": ">=7.2",
-    "telegram-bot/api": "^2.3"
+    "telegram-bot/api": "^2.3",
+    "ext-curl": "^7.2"
   },
   "require-dev": {
     "phpunit/phpunit": "^8"

diff --git a/src/Telegram/Messenger.php b/src/Telegram/Messenger.php
@@ -182,22 +182,26 @@ public function sendScreen(MessengerScreen $screen): bool
         return $result;
     }
 
+    /**
+     * @param ScreenItemInterface $item
+     * @param array $buttons
+     * @return bool
+     * @throws AccessTokenException
+     * @throws AttachmentNotFoundException
+     * @throws TargetUserException
+     */
     private function workWithScreenItem(ScreenItemInterface $item, array $buttons = []): bool
     {
         $class = get_class($item);
         switch ($class) {
             case \He110\CommunicationTools\ScreenItems\Message::class:
                 return $this->sendMessage($item->getText(), $buttons);
-                break;
-            case File::class:
-                $method = $item->getType() == File::FILE_TYPE_IMAGE ? "sendImage" : "sendDocument";
-                return $this->{$method}($item->getPath(), $item->getDescription());
-                break;
             case Voice::class:
                 return $this->sendVoice($item->getPath());
-                break;
+            default:
+                $method = $item->getType() == File::FILE_TYPE_IMAGE ? "sendImage" : "sendDocument";
+                return $this->{$method}($item->getPath(), $item->getDescription());
         }
-        return false;
     }
 
     /**