From dd38749081e8323d1c7fb96f98c60eeca3671d38 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20B=C3=B6hmer?= Date: Sat, 23 Jul 2022 23:07:51 +0200 Subject: [PATCH] Fixed test which heavily depended on timing of the assertation functions This caused a lot of false positive test failures. --- tests/Entity/UserSystem/UserTest.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/Entity/UserSystem/UserTest.php b/tests/Entity/UserSystem/UserTest.php index 4ac24c80..38ca5420 100644 --- a/tests/Entity/UserSystem/UserTest.php +++ b/tests/Entity/UserSystem/UserTest.php @@ -88,10 +88,12 @@ class UserTest extends TestCase public function testSetBackupCodes(): void { $user = new User(); + $this->assertNull($user->getBackupCodesGenerationDate()); + $codes = ['test', 'invalid', 'test']; $user->setBackupCodes($codes); // Backup Codes generation date must be changed! - $this->assertEqualsWithDelta(new DateTime(), $user->getBackupCodesGenerationDate(), 0.1); + $this->assertInstanceOf(\DateTime::class, $user->getBackupCodesGenerationDate()); $this->assertSame($codes, $user->getBackupCodes()); //Test what happens if we delete the backup keys