fix updater cleanup route

3 files changed, 4 insertions, 2 deletions

diff --git a/appinfo/routes.php b/appinfo/routes.php
index 6ceaf94e1..617b63c53 100644
--- a/appinfo/routes.php
+++ b/appinfo/routes.php
@@ -233,7 +233,7 @@ $this->create('news_api_version', '/api/v1-2/version')->get()->action(
 	}
 );
 
-$this->create('news_api_cleanup', '/api/v1-2/cleanUp')->get()->action(
+$this->create('news_api_cleanup', '/api/v1-2/cleanup')->get()->action(
 	function($params) {
 		return App::main('NewsAPI', 'cleanUp', $params, new DIContainer());
 	}
diff --git a/external/newsapi.php b/external/newsapi.php
index c5fd5e567..f0f49f9da 100644
--- a/external/newsapi.php
+++ b/external/newsapi.php
@@ -67,6 +67,7 @@ class NewsAPI extends Controller {
 	 */
 	public function cleanUp() {
 		$this->updater->cleanUp();
+		return new JSONResponse();
 	}
 
 
diff --git a/tests/unit/external/NewsAPITest.php b/tests/unit/external/NewsAPITest.php
index 9a319a378..116418fc6 100644
--- a/tests/unit/external/NewsAPITest.php
+++ b/tests/unit/external/NewsAPITest.php
@@ -88,7 +88,8 @@ class NewsAPITest extends ControllerTestUtility {
 	public function testCleanUp(){
 		$this->updater->expects($this->once())
 			->method('cleanUp');
-		$this->newsAPI->cleanUp();
+		$response = $this->newsAPI->cleanUp();
+		$this->assertTrue($response instanceof JSONResponse);
 	}