Fix datetime equal tests

andrewmy · andrewmy · commit 63aaebff90e1 · 2021-01-24T21:22:57.000+02:00
diff --git a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php
@@ -102,7 +102,10 @@ public function testShouldCalculateRootJobStatusAndSetStoppedAtTimeIfGotStopStat
         $case->calculate($childJob);
 
         $this->assertEquals($stopStatus, $rootJob->getStatus());
-        $this->assertEquals(new \DateTime(), $rootJob->getStoppedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $rootJob->getStoppedAt()->getTimestamp()
+        );
     }
 
     public function testShouldSetStoppedAtOnlyIfWasNotSet()
@@ -129,7 +132,10 @@ public function testShouldSetStoppedAtOnlyIfWasNotSet()
         $case = new CalculateRootJobStatusService($em);
         $case->calculate($childJob);
 
-        $this->assertEquals(new \DateTime('2012-12-12 12:12:12'), $rootJob->getStoppedAt());
+        $this->assertEquals(
+            (new \DateTime('2012-12-12 12:12:12'))->getTimestamp(),
+            $rootJob->getStoppedAt()->getTimestamp()
+        );
     }
 
     public function testShouldThrowIfInvalidStatus()
diff --git a/pkg/job-queue/Tests/JobProcessorTest.php b/pkg/job-queue/Tests/JobProcessorTest.php
@@ -60,8 +60,14 @@ public function testShouldCreateRootJobAndReturnIt()
 
         $this->assertSame($job, $result);
         $this->assertEquals(Job::STATUS_NEW, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getCreatedAt(), '', 1);
-        $this->assertEquals(new \DateTime(), $job->getStartedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getCreatedAt()->getTimestamp()
+        );
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStartedAt()->getTimestamp()
+        );
         $this->assertNull($job->getStoppedAt());
         $this->assertEquals('job-name', $job->getName());
         $this->assertEquals('owner-id', $job->getOwnerId());
@@ -183,7 +189,10 @@ public function testCreateChildJobShouldCreateAndSaveJobAndPublishRecalculateRoo
 
         $this->assertSame($job, $result);
         $this->assertEquals(Job::STATUS_NEW, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getCreatedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getCreatedAt()->getTimestamp()
+        );
         $this->assertNull($job->getStartedAt());
         $this->assertNull($job->getStoppedAt());
         $this->assertEquals('job-name', $job->getName());
@@ -256,7 +265,10 @@ public function testStartJobShouldUpdateJobWithRunningStatusAndStartAtTime()
         $processor->startChildJob($job);
 
         $this->assertEquals(Job::STATUS_RUNNING, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getStartedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStartedAt()->getTimestamp()
+        );
     }
 
     public function testSuccessChildJobShouldThrowIfRootJob()
@@ -325,7 +337,10 @@ public function testSuccessJobShouldUpdateJobWithSuccessStatusAndStopAtTime()
         $processor->successChildJob($job);
 
         $this->assertEquals(Job::STATUS_SUCCESS, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getStoppedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStoppedAt()->getTimestamp()
+        );
     }
 
     public function testFailChildJobShouldThrowIfRootJob()
@@ -394,7 +409,10 @@ public function testFailJobShouldUpdateJobWithFailStatusAndStopAtTime()
         $processor->failChildJob($job);
 
         $this->assertEquals(Job::STATUS_FAILED, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getStoppedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStoppedAt()->getTimestamp()
+        );
     }
 
     public function testCancelChildJobShouldThrowIfRootJob()
@@ -463,8 +481,14 @@ public function testCancelJobShouldUpdateJobWithCancelStatusAndStoppedAtTimeAndS
         $processor->cancelChildJob($job);
 
         $this->assertEquals(Job::STATUS_CANCELLED, $job->getStatus());
-        $this->assertEquals(new \DateTime(), $job->getStoppedAt(), '', 1);
-        $this->assertEquals(new \DateTime(), $job->getStartedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStoppedAt()->getTimestamp()
+        );
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $job->getStartedAt()->getTimestamp()
+        );
     }
 
     public function testInterruptRootJobShouldThrowIfNotRootJob()
@@ -536,7 +560,10 @@ public function testInterruptRootJobShouldUpdateJobAndSetInterruptedTrueAndStopp
         $processor->interruptRootJob($rootJob, true);
 
         $this->assertTrue($rootJob->isInterrupted());
-        $this->assertEquals(new \DateTime(), $rootJob->getStoppedAt(), '', 1);
+        $this->assertEquals(
+            (new \DateTime())->getTimestamp(),
+            $rootJob->getStoppedAt()->getTimestamp()
+        );
     }
 
     /**
