From 5aecff9e569957b9286c47e299e7adf85c3d9a65 Mon Sep 17 00:00:00 2001 From: Matthew Ross Date: Thu, 2 Feb 2017 05:58:02 -0500 Subject: [PATCH] Update API return and fix failing test --- src/api/controllers/AutoActions.php | 3 +-- test/api/controllers/AutoActionsTest.php | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/api/controllers/AutoActions.php b/src/api/controllers/AutoActions.php index b7a8fb0..49a9e15 100644 --- a/src/api/controllers/AutoActions.php +++ b/src/api/controllers/AutoActions.php @@ -107,10 +107,9 @@ class AutoActions extends BaseController { json_encode($before), '', 'action', $id); $autoActions = R::findAll('autoaction'); + $data = []; foreach ($autoActions as $action) { - $data = []; - if (Auth::HasBoardAccess($request, $action->board_id)) { $data[] = $action; } diff --git a/test/api/controllers/AutoActionsTest.php b/test/api/controllers/AutoActionsTest.php index e9f8cea..b79a33b 100644 --- a/test/api/controllers/AutoActionsTest.php +++ b/test/api/controllers/AutoActionsTest.php @@ -33,7 +33,7 @@ class AutoActionsTest extends PHPUnit_Framework_TestCase { $actual = $this->actions->getAllActions($request, new ResponseMock(), null); - $this->assertEquals(2, count($actual->data)); + $this->assertEquals(1, count($actual->data[1])); $this->assertEquals('success', $actual->status); DataMock::CreateStandardUser(); @@ -42,7 +42,7 @@ class AutoActionsTest extends PHPUnit_Framework_TestCase { $actual = $this->actions->getAllActions($request, new ResponseMock(), null); - $this->assertEquals(1, count($actual->data)); + $this->assertEquals(0, count($actual->data[1])); $this->assertEquals('success', $actual->status); }