upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-05-28 04:32:30 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 25

Fix DummyJobList class used by tests

Browse source 
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
This commit is contained in:
Côme Chilliet 2022-06-27 12:06:23 +02:00 committed by Côme Chilliet (Rebase PR Action)
parent 868d748dbf
commit 50a7f612d9
1 changed files with 27 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

44
tests/lib/BackgroundJob/DummyJobList.php
View file

@ -19,20 +19,20 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
/**
* @var IJob[]
*/
private $jobs = [];
private array $jobs = [];
private $last = 0;
private int $last = 0;
public function __construct() {
}
/**
* @param IJob|string $job
* @param IJob|class-string<IJob> $job
* @param mixed $argument
*/
public function add($job, $argument = null) {
public function add($job, $argument = null): void {
if (is_string($job)) {
/** @var \OC\BackgroundJob\Job $job */
/** @var IJob $job */
$job = new $job;
}
$job->setArgument($argument);
@ -45,7 +45,7 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
* @param IJob|string $job
* @param mixed $argument
*/
public function remove($job, $argument = null) {
public function remove($job, $argument = null): void {
$index = array_search($job, $this->jobs);
if ($index !== false) {
unset($this->jobs[$index]);
@ -59,7 +59,7 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
* @param mixed $argument
* @return bool
*/
public function has($job, $argument) {
public function has($job, $argument): bool {
return array_search($job, $this->jobs) !== false;
}
@ -72,6 +72,22 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
return $this->jobs;
}
public function getJobs($job, ?int $limit, int $offset): array {
if ($job instanceof IJob) {
$jobClass = get_class($job);
} else {
$jobClass = $job;
}
return array_slice(
array_filter(
$this->jobs,
fn ($job) => ($jobClass === null) || (get_class($job) == $jobClass)
),
$offset,
$limit
);
}
/**
* get the next job in the list
*
@ -96,7 +112,7 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
*
* @param \OC\BackgroundJob\Job $job
*/
public function setLastJob(IJob $job) {
public function setLastJob(IJob $job): void {
$i = array_search($job, $this->jobs);
if ($i !== false) {
$this->last = $i;
@ -107,9 +123,8 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
/**
* @param int $id
* @return IJob
*/
public function getById($id) {
public function getById($id): IJob {
foreach ($this->jobs as $job) {
if ($job->getId() === $id) {
return $job;
@ -122,16 +137,11 @@ class DummyJobList extends \OC\BackgroundJob\JobList {
return null;
}
/**
* set the lastRun of $job to now
*
* @param IJob $job
*/
public function setLastRun(IJob $job) {
public function setLastRun(IJob $job): void {
$job->setLastRun(time());
}
public function setExecutionTime(IJob $job, $timeTaken) {
public function setExecutionTime(IJob $job, $timeTaken): void {
}
public function resetBackgroundJob(IJob $job): void {