Fix tests that invoke findUsers
Signed-off-by: Julius Härtl <jus@bitgrid.net>
This commit is contained in:
committed by
Julius Härtl
parent
a56ceb348d
commit
ff2d96a1fb
@@ -103,7 +103,8 @@ class BoardService {
|
||||
'PERMISSION_MANAGE' => $permissions[Acl::PERMISSION_MANAGE],
|
||||
'PERMISSION_SHARE' => $permissions[Acl::PERMISSION_SHARE]
|
||||
]);
|
||||
$board->setUsers(array_values($this->permissionService->findUsers($boardId)));
|
||||
$boardUsers = $this->permissionService->findUsers($boardId);
|
||||
$board->setUsers(array_values($boardUsers));
|
||||
return $board;
|
||||
}
|
||||
|
||||
|
||||
@@ -97,6 +97,7 @@ class CardTest extends \PHPUnit_Framework_TestCase {
|
||||
'duedate' => null,
|
||||
'overdue' => 0,
|
||||
'archived' => false,
|
||||
'assignedUsers' => null,
|
||||
], $card->jsonSerialize());
|
||||
}
|
||||
|
||||
|
||||
@@ -105,6 +105,11 @@ class BoardServiceTest extends TestCase {
|
||||
->method('find')
|
||||
->with(1)
|
||||
->willReturn($b1);
|
||||
$this->permissionService->expects($this->once())
|
||||
->method('findUsers')
|
||||
->willReturn([
|
||||
'admin' => 'admin',
|
||||
]);
|
||||
$this->assertEquals($b1, $this->service->find(1));
|
||||
}
|
||||
|
||||
@@ -137,6 +142,11 @@ class BoardServiceTest extends TestCase {
|
||||
->method('update')
|
||||
->with($board)
|
||||
->willReturn($board);
|
||||
$this->permissionService->expects($this->once())
|
||||
->method('findUsers')
|
||||
->willReturn([
|
||||
'admin' => 'admin',
|
||||
]);
|
||||
$b = $this->service->update(123, 'MyNewNameBoard', 'ffffff', false);
|
||||
|
||||
$this->assertEquals($b->getTitle(), 'MyNewNameBoard');
|
||||
@@ -151,6 +161,11 @@ class BoardServiceTest extends TestCase {
|
||||
$this->boardMapper->expects($this->once())
|
||||
->method('find')
|
||||
->willReturn($board);
|
||||
$this->permissionService->expects($this->once())
|
||||
->method('findUsers')
|
||||
->willReturn([
|
||||
'admin' => 'admin',
|
||||
]);
|
||||
$this->assertEquals($board, $this->service->delete(123));
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user