Merge pull request #6 from noplanman/add_loop_interval

Add loop interval as parameter.

Author: noplanman

src/BotManager.php

@@ -250,7 +250,7 @@ public function handleRequest(): self
     {
         if (empty($this->params->getBotParam('webhook'))) {
             if ($loop_time = $this->getLoopTime()) {
-                $this->handleGetUpdatesLoop($loop_time);
+                $this->handleGetUpdatesLoop($loop_time, $this->getLoopInterval());
             } else {
                 $this->handleGetUpdates();
             }
@@ -281,14 +281,32 @@ public function getLoopTime(): int
         return max(0, (int)$loop_time);
     }
 
+    /**
+     * Get the number of seconds the script should wait after each getUpdates request.
+     *
+     * @return int
+     */
+    public function getLoopInterval(): int
+    {
+        $interval_time = $this->params->getScriptParam('i');
+
+        if (null === $interval_time || (is_string($interval_time) && '' === trim($interval_time))) {
+            return 2;
+        }
+
+        // Minimum interval is 1 second.
+        return max(1, (int)$interval_time);
+    }
+
     /**
      * Loop the getUpdates method for the passed amount of seconds.
      *
      * @param int $loop_time_in_seconds
+     * @param int $loop_interval_in_seconds
      *
      * @return \NPM\TelegramBotManager\BotManager
      */
-    public function handleGetUpdatesLoop(int $loop_time_in_seconds): self
+    public function handleGetUpdatesLoop(int $loop_time_in_seconds, int $loop_interval_in_seconds = 2): self
     {
         // Remember the time we started this loop.
         $now = time();
@@ -299,7 +317,7 @@ public function handleGetUpdatesLoop(int $loop_time_in_seconds): self
             $this->handleGetUpdates();
 
             // Chill a bit.
-            sleep(2);
+            sleep($loop_interval_in_seconds);
         }
 
         return $this;

src/Params.php

@@ -19,6 +19,7 @@ class Params
         's',
         'a',
         'l',
+        'i',
     ];
 
     /**

tests/TelegramBotManager/Tests/BotManagerTest.php

@@ -288,6 +288,36 @@ public function testGetLoopTime()
         self::assertSame(12345, (new BotManager(ParamsTest::$demo_vital_params))->getLoopTime());
     }
 
+    public function testGetLoopInterval()
+    {
+        // Parameter not set.
+        self::assertSame(2, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => ''];
+        self::assertSame(2, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => '     '];
+        self::assertSame(2, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => 'text-string'];
+        self::assertSame(1, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => 0];
+        self::assertSame(1, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => -12345];
+        self::assertSame(1, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => 12345];
+        self::assertSame(12345, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => '-12345'];
+        self::assertSame(1, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+
+        $_GET = ['i' => '12345'];
+        self::assertSame(12345, (new BotManager(ParamsTest::$demo_vital_params))->getLoopInterval());
+    }
+
     public function testSetBotExtras()
     {
         $extras     = [