return proper response for update method

2 files changed, 4 insertions, 2 deletions

diff --git a/external/feedapi.php b/external/feedapi.php
index fba43b249..3acd87a0e 100644
--- a/external/feedapi.php
+++ b/external/feedapi.php
@@ -198,7 +198,7 @@ class FeedAPI extends Controller {
 			));
 		}
 
-		return  new JSONResponse($result);
+		return new JSONResponse($result);
 	}
 
 
@@ -213,6 +213,7 @@ class FeedAPI extends Controller {
 
 		try {
 			$this->feedBusinessLayer->update($feedId, $userId);
+			return new JSONResponse();
 		} catch(BusinessLayerException $ex) {
 			return new JSONResponse(array('message' => $ex->getMessage()),
 				Http::STATUS_NOT_FOUND);
diff --git a/tests/unit/external/FeedAPITest.php b/tests/unit/external/FeedAPITest.php
index 670f5f942..2963125b2 100644
--- a/tests/unit/external/FeedAPITest.php
+++ b/tests/unit/external/FeedAPITest.php
@@ -486,7 +486,8 @@ class FeedAPITest extends ControllerTestUtility {
 			->method('update')
 			->with($this->equalTo($feedId), $this->equalTo($userId));
 
-		$this->feedAPI->update();
+		$response = $this->feedAPI->update();
+		$this->assertTrue($response instanceof JSONResponse);
 	}