[Notifier] Extended TelegramOptionsTest to make sure cache time is provided when valid number is provided

alexsoft · alexsoft · commit 901abe488e2e · 2022-11-30T21:27:38.000+01:00
diff --git a/src/Symfony/Component/Notifier/Bridge/Telegram/Tests/TelegramOptionsTest.php b/src/Symfony/Component/Notifier/Bridge/Telegram/Tests/TelegramOptionsTest.php
@@ -18,20 +18,56 @@
 
 final class TelegramOptionsTest extends TestCase
 {
-    public function testAnswerCallbackQuery()
+    /**
+     * @dataProvider validCacheTimeDataProvider
+     */
+    public function testAnswerCallbackQueryWithCacheTime(int $cacheTime)
     {
         $options = new TelegramOptions();
 
-        $returnedOptions = $options->answerCallbackQuery('123', true, 1);
+        $returnedOptions = $options->answerCallbackQuery('123', true, $cacheTime);
 
         $this->assertSame($options, $returnedOptions);
         $this->assertEquals(
             [
                 'callback_query_id' => '123',
                 'show_alert' => true,
-                'cache_time' => 1,
+                'cache_time' => $cacheTime,
             ],
             $options->toArray(),
         );
     }
+
+    public function validCacheTimeDataProvider(): iterable
+    {
+        yield 'cache time equals 1' => [1];
+        yield 'cache time equals 2' => [2];
+        yield 'cache time equals 10' => [10];
+    }
+
+    /**
+     * @dataProvider invalidCacheTimeDataProvider
+     */
+    public function testAnswerCallbackQuery(int $cacheTime)
+    {
+        $options = new TelegramOptions();
+
+        $returnedOptions = $options->answerCallbackQuery('123', true, $cacheTime);
+
+        $this->assertSame($options, $returnedOptions);
+        $this->assertEquals(
+            [
+                'callback_query_id' => '123',
+                'show_alert' => true,
+            ],
+            $options->toArray(),
+        );
+    }
+
+    public function invalidCacheTimeDataProvider(): iterable
+    {
+        yield 'cache time equals 0' => [0];
+        yield 'cache time equals -1' => [-1];
+        yield 'cache time equals -10' => [-10];
+    }
 }
