diff options
author | Bernhard Posselt <nukeawhale@gmail.com> | 2013-08-05 17:38:34 +0200 |
---|---|---|
committer | Bernhard Posselt <nukeawhale@gmail.com> | 2013-08-05 17:38:34 +0200 |
commit | 1a74a758f9811a621e636f53d7393dae01199337 (patch) | |
tree | 9f301dbfdc4850e252ea3205aa5c95447146355e | |
parent | d0bce145e7a5be8f06c8875b2de7c41d164d9d57 (diff) |
also log errors when updating with update script
-rw-r--r-- | external/feedapi.php | 5 | ||||
-rw-r--r-- | tests/unit/external/FeedAPITest.php | 6 |
2 files changed, 9 insertions, 2 deletions
diff --git a/external/feedapi.php b/external/feedapi.php index 684769c6e..fa0f27f7c 100644 --- a/external/feedapi.php +++ b/external/feedapi.php @@ -214,7 +214,10 @@ class FeedAPI extends Controller { try { $this->feedBusinessLayer->update($feedId, $userId); // ignore update failure (feed could not be reachable etc, we dont care) - } catch(BusinessLayerException $ex) {} + } catch(\Exception $ex) { + $this->api->log('Could not update feed ' . $ex->getMessage(), + 'debug'); + } return new JSONResponse(); } diff --git a/tests/unit/external/FeedAPITest.php b/tests/unit/external/FeedAPITest.php index 9a092a84e..e490bbdf9 100644 --- a/tests/unit/external/FeedAPITest.php +++ b/tests/unit/external/FeedAPITest.php @@ -494,7 +494,11 @@ class FeedAPITest extends ControllerTestUtility { public function testUpdateError() { $this->feedBusinessLayer->expects($this->once()) ->method('update') - ->will($this->throwException(new BusinessLayerException($this->msg))); + ->will($this->throwException(new \Exception($this->msg))); + $this->api->expects($this->once()) + ->method('log') + ->with($this->equalTo('Could not update feed ' . $this->msg), + $this->equalTo('debug')); $response = $this->feedAPI->update(); |