bug #42194 [RateLimiter] fix: sliding window policy to use microtime - fix test

jlekowski · jlekowski · commit e6169635d414 · 2021-10-25T09:35:12.000+01:00
diff --git a/src/Symfony/Component/RateLimiter/Tests/RateLimitTest.php b/src/Symfony/Component/RateLimiter/Tests/RateLimitTest.php
@@ -48,10 +48,10 @@ public function testEnsureAcceptedThrowsRateLimitExceptionIfNotAccepted()
     public function testWaitUsesMicrotime()
     {
         ClockMock::register(RateLimit::class);
-        $rateLimit = new RateLimit(10, new \DateTimeImmutable('+2500 ms'), true, 10);
+        $retryAfter = time() + 2.5; // get timestamp in the middle of a second (xxx.5)
+        $rateLimit = new RateLimit(10, \DateTimeImmutable::createFromFormat('U.u', $retryAfter), true, 10);
 
-        $start = microtime(true);
-        $rateLimit->wait(); // wait 2.5 seconds
-        $this->assertEqualsWithDelta($start + 2.5, microtime(true), 0.1);
+        $rateLimit->wait(); // wait until $retryAfter (~2.5 seconds)
+        $this->assertEqualsWithDelta($retryAfter, microtime(true), 0.49);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -48,10 +48,10 @@ public function testEnsureAcceptedThrowsRateLimitExceptionIfNotAccepted()`
`48`	`48`	`public function testWaitUsesMicrotime()`
`49`	`49`	`{`
`50`	`50`	`ClockMock::register(RateLimit::class);`
`51`		`- $rateLimit = new RateLimit(10, new \DateTimeImmutable('+2500 ms'), true, 10);`
	`51`	`+ $retryAfter = time() + 2.5; // get timestamp in the middle of a second (xxx.5)`
	`52`	`+ $rateLimit = new RateLimit(10, \DateTimeImmutable::createFromFormat('U.u', $retryAfter), true, 10);`
`52`	`53`
`53`		`- $start = microtime(true);`
`54`		`- $rateLimit->wait(); // wait 2.5 seconds`
`55`		`- $this->assertEqualsWithDelta($start + 2.5, microtime(true), 0.1);`
	`54`	`+ $rateLimit->wait(); // wait until $retryAfter (~2.5 seconds)`
	`55`	`+ $this->assertEqualsWithDelta($retryAfter, microtime(true), 0.49);`
`56`	`56`	`}`
`57`	`57`	`}`