Complete BoardServiceTest

lib/Service/BoardService.php

@@ -43,12 +43,14 @@ class BoardService {
 	private $logger;
 	private $l10n;
 
-    public function __construct(BoardMapper $boardMapper,
+	public function __construct(
+		BoardMapper $boardMapper,
 		ILogger $logger,
 		IL10N $l10n,
 		LabelMapper $labelMapper,
 		AclMapper $aclMapper,
-								IGroupManager $groupManager) {
+		IGroupManager $groupManager
+	) {
 		$this->boardMapper = $boardMapper;
 		$this->labelMapper = $labelMapper;
 		$this->aclMapper = $aclMapper;
@@ -81,7 +83,7 @@ class BoardService {
 			'FF7A66' => $this->l10n->t('Action needed'),
 			'F1DB50' => $this->l10n->t('Later')];
 		$labels = [];
-        foreach ($default_labels as $color=>$title) {
+		foreach ($default_labels as $color => $title) {
 			$label = new Label();
 			$label->setColor($color);
 			$label->setTitle($title);

tests/unit/Service/BoardServiceTest.php

@@ -112,6 +112,26 @@ class BoardServiceTest extends \PHPUnit_Framework_TestCase {
 		$this->assertCount(4, $b->getLabels());
 	}
 
+	public function testUpdate() {
+		$board = new Board();
+		$board->setTitle('MyBoard');
+		$board->setOwner('admin');
+		$board->setColor('00ff00');
+		$this->boardMapper->expects($this->once())
+			->method('find')
+			->with(123)
+			->willReturn($board);
+		$this->boardMapper->expects($this->once())
+			->method('update')
+			->with($board)
+			->willReturn($board);
+		$b = $this->service->update(123, 'MyNewNameBoard', 'ffffff');
+
+		$this->assertEquals($b->getTitle(), 'MyNewNameBoard');
+		$this->assertEquals($b->getOwner(), 'admin');
+		$this->assertEquals($b->getColor(), 'ffffff');
+	}
+
 	public function testDelete() {
 		$this->boardMapper->expects($this->once())
 			->method('find')