Adjust timing in testRateLimit() so less likely to fail.

Also, use CakePHP Time() when doing rate limiting for consistency.

Author: Alex Mayfield

src/Model/Table/QueuedTasksTable.php

@@ -266,7 +266,7 @@ public function requestJob(array $capabilities, $group = null) {
 			$this->save($data, ['fieldList' => ['id', 'failed', 'failure_message']]);
 		}
 		//save last fetch by type for Rate Limiting.
-		$this->rateHistory[$data['jobtype']] = time();
+		$this->rateHistory[$data['jobtype']] = (new Time())->toUnixString() ;
 		return $data;
 	}
 

tests/TestCase/Model/Table/QueuedTasksTableTest.php

@@ -292,7 +292,7 @@ public function testRateLimit() {
 				'name' => 'task1',
 				'timeout' => 101,
 				'retries' => 2,
-				'rate' => 1,
+				'rate' => 2,
 			],
 			'dummytask' => [
 				'name' => 'dummytask',
@@ -324,15 +324,15 @@ public function testRateLimit() {
 		$this->assertEquals('dummytask', $tmp['jobtype']);
 		$this->assertNull(unserialize($tmp['data']));
 
-		usleep(500000);
+		usleep(100000);
 		//and again.
 		$this->QueuedTasks->clearKey();
 		$tmp = $this->QueuedTasks->requestJob($capabilities);
 		$this->assertEquals('dummytask', $tmp['jobtype']);
 		$this->assertNull(unserialize($tmp['data']));
 
 		//Then some time passes
-		sleep(1);
+		sleep(2);
 
 		//Now we should get task1-2
 		$this->QueuedTasks->clearKey();
@@ -347,7 +347,7 @@ public function testRateLimit() {
 		$this->assertNull(unserialize($tmp['data']));
 
 		//Then some more time passes
-		sleep(1);
+		sleep(2);
 
 		//Now we should get task1-3
 		$this->QueuedTasks->clearKey();