Move live PHPUnit tests to the "live" group, to make it easier to exclude them while developing.

Add sleeps between live tests to prevent request limits.

Author: noplanman

src/BotManager.php

@@ -175,6 +175,8 @@ public function validateAndSetWebhook(): self
 
         if ($this->action->isAction(['unset', 'reset'])) {
             $this->handleOutput($this->telegram->unsetWebHook()->getDescription() . PHP_EOL);
+            // When resetting the webhook, sleep for a bit to prevent too many requests.
+            $this->action->isAction('reset') && sleep(1);
         }
         if ($this->action->isAction(['set', 'reset'])) {
             $this->handleOutput(

tests/TelegramBotManager/Tests/BotManagerTest.php

@@ -185,6 +185,9 @@ public function testValidateAndSetWebhookSuccess()
         self::assertSame('Webhook is already deleted' . PHP_EOL, $output);
     }
 
+    /**
+     * @group live
+     */
     public function testValidateAndSetWebhookSuccessLiveBot()
     {
         $botManager = new BotManager(array_merge(self::$live_params, [
@@ -203,26 +206,29 @@ public function testValidateAndSetWebhookSuccessLiveBot()
         // Make sure the webhook isn't set to start with.
         TestHelpers::setObjectProperty($botManager->getAction(), 'action', 'unset');
         $botManager->validateAndSetWebhook()->getOutput();
-
+        sleep(1);
         TestHelpers::setObjectProperty($botManager->getAction(), 'action', 'set');
         $output = $botManager->validateAndSetWebhook()->getOutput();
         self::assertSame('Webhook was set' . PHP_EOL, $output);
-
+        sleep(1);
         $output = $botManager->validateAndSetWebhook()->getOutput();
         self::assertSame('Webhook is already set' . PHP_EOL, $output);
-
+        sleep(1);
         TestHelpers::setObjectProperty($botManager->getAction(), 'action', 'reset');
         $output = $botManager->validateAndSetWebhook()->getOutput();
         self::assertSame('Webhook was deleted' . PHP_EOL . 'Webhook was set' . PHP_EOL, $output);
-
+        sleep(1);
         TestHelpers::setObjectProperty($botManager->getAction(), 'action', 'unset');
         $output = $botManager->validateAndSetWebhook()->getOutput();
         self::assertSame('Webhook was deleted' . PHP_EOL, $output);
-
+        sleep(1);
         $output = $botManager->validateAndSetWebhook()->getOutput();
         self::assertSame('Webhook is already deleted' . PHP_EOL, $output);
     }
 
+    /**
+     * @group live
+     */
     public function testUnsetWebhookViaRunLiveBot()
     {
         $_GET       = ['a' => 'unset'];
@@ -364,13 +370,19 @@ public function testGetOutput()
         self::assertEmpty($botManager->getOutput());
     }
 
+    /**
+     * @group live
+     */
     public function testGetUpdatesLiveBot()
     {
         $botManager = new BotManager(self::$live_params);
         $output     = $botManager->run()->getOutput();
         self::assertContains('Updates processed: 0', $output);
     }
 
+    /**
+     * @group live
+     */
     public function testGetUpdatesLoopLiveBot()
     {
         // Webhook must NOT be set for this to work!