From 96b8499af0d0c3c3fef9b8456bae62322da78dd7 Mon Sep 17 00:00:00 2001 From: Damodar Lohani Date: Wed, 26 Nov 2025 15:36:46 +0545 Subject: [PATCH] Refactor: assertions - replace assertEquals with assertSame --- tests/Abuse/Base.php | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/tests/Abuse/Base.php b/tests/Abuse/Base.php index f0bc337..81a4d5e 100644 --- a/tests/Abuse/Base.php +++ b/tests/Abuse/Base.php @@ -17,9 +17,9 @@ public function testStaticKey(): void { $adapter = $this->getAdapter('static-key', 2, 1); $abuse = new Abuse($adapter); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), true); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), true); } /** @@ -30,9 +30,9 @@ public function testDynamicKey(): void $adapter = $this->getAdapter('dynamic-key-{{ip}}', 2, 1); $adapter->setParam('{{ip}}', '0.0.0.10'); $abuse = new Abuse($adapter); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), true); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), true); } /** @@ -44,9 +44,9 @@ public function testDynamicKeyWith2Params(): void $adapter->setParam('{{ip}}', '0.0.0.10'); $adapter->setParam('{{email}}', 'test@test.com'); $abuse = new Abuse($adapter); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), false); - $this->assertEquals($abuse->check(), true); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), false); + $this->assertSame($abuse->check(), true); } /** @@ -58,9 +58,9 @@ public function testDynamicKeyFastRequests(): void $adapter->setParam('{{ip}}', '0.0.0.10'); $abuse = new Abuse($adapter); for ($i = 0; $i < 10; $i++) { - $this->assertEquals($abuse->check(), false); + $this->assertSame($abuse->check(), false); } - $this->assertEquals($abuse->check(), true); + $this->assertSame($abuse->check(), true); } /** @@ -72,9 +72,9 @@ public function testLimitReset(): void $adapter->setParam('{{ip}}', '127.0.0.1'); $abuse = new Abuse($adapter); for ($i = 0; $i < 10; $i++) { - $this->assertEquals($abuse->check(), false); + $this->assertSame($abuse->check(), false); } - $this->assertEquals($abuse->check(), true); + $this->assertSame($abuse->check(), true); // Wait for the limit to reset sleep(2); @@ -83,7 +83,7 @@ public function testLimitReset(): void $adapter = $this->getAdapter('limit-reset-{{ip}}', 10, 1); $adapter->setParam('{{ip}}', '127.0.0.1'); $abuse = new Abuse($adapter); - $this->assertEquals($abuse->check(), false); + $this->assertSame($abuse->check(), false); } /** @@ -93,7 +93,7 @@ public function testTimeFormat(): void { $now = time(); $adapter = $this->getAdapter('', 1, 1); - $this->assertEquals($adapter->time(), $now); - $this->assertEquals(true, \is_int($adapter->time())); + $this->assertSame($adapter->time(), $now); + $this->assertSame(true, \is_int($adapter->time())); } }