From a84b978221e017399b515280a3f758e8f99eaf93 Mon Sep 17 00:00:00 2001 From: Bernhard Posselt Date: Mon, 9 Sep 2013 16:31:03 +0200 Subject: fix unit tests for phantomjs runner --- js/app/controllers/settingscontroller.coffee | 2 -- js/public/app.js | 2 -- js/tests/controllers/settingscontrollerSpec.coffee | 5 +++++ 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/js/app/controllers/settingscontroller.coffee b/js/app/controllers/settingscontroller.coffee index eaff0a274..b7cea8d21 100644 --- a/js/app/controllers/settingscontroller.coffee +++ b/js/app/controllers/settingscontroller.coffee @@ -34,7 +34,6 @@ angular.module('News').controller 'SettingsController', try FolderBusinessLayer.import(fileContent) catch error - console.error error $scope.error = true @@ -46,7 +45,6 @@ angular.module('News').controller 'SettingsController', parsedJSON = JSON.parse(fileContent) FeedBusinessLayer.importGoogleReader(parsedJSON) catch error - console.error error $scope.jsonError = true ] \ No newline at end of file diff --git a/js/public/app.js b/js/public/app.js index 79c08094c..7f0b6c8f4 100644 --- a/js/public/app.js +++ b/js/public/app.js @@ -793,7 +793,6 @@ License along with this library. If not, see . return FolderBusinessLayer["import"](fileContent); } catch (_error) { error = _error; - console.error(error); return $scope.error = true; } }; @@ -806,7 +805,6 @@ License along with this library. If not, see . return FeedBusinessLayer.importGoogleReader(parsedJSON); } catch (_error) { error = _error; - console.error(error); return $scope.jsonError = true; } }; diff --git a/js/tests/controllers/settingscontrollerSpec.coffee b/js/tests/controllers/settingscontrollerSpec.coffee index 39bc53371..8603e75e9 100644 --- a/js/tests/controllers/settingscontrollerSpec.coffee +++ b/js/tests/controllers/settingscontrollerSpec.coffee @@ -35,6 +35,8 @@ describe 'SettingsController', -> @scope = {} @replace = '$scope': @scope + 'FolderBusinessLayer': + import: jasmine.createSpy('import') @controller = $controller('SettingsController', @replace) @@ -44,9 +46,12 @@ describe 'SettingsController', -> it 'should show an error if the xml import failed', => xml = 'test' + @replace.FolderBusinessLayer.import.andCallFake -> + throw new Error() @scope.import(xml) + expect(@replace.FolderBusinessLayer.import).toHaveBeenCalledWith(xml); expect(@scope.error).toBe(true) -- cgit v1.2.3