Commit 869ace0d

mo <mo.khan@gmail.com>
2017-09-04 01:37:36
remove extra constructor parameter.
1 parent 11cd3ab
Changed files (2)
app
assets
javascripts
spec
javascripts
app/assets/javascripts/initializers/auto_view_setup.js.coffee
@@ -4,9 +4,6 @@ class CakeSide.AutoViewSetup extends CakeSide.Behaviour
   @views = {}
   @on "ready"
 
-  constructor: (logger = console) ->
-    @logger = logger
-
   execute: ->
     for element in $('[data-autoview]')
       #CakeSide.AutoView.install(element)
@@ -17,7 +14,7 @@ class CakeSide.AutoViewSetup extends CakeSide.Behaviour
     constructor = CakeSide.Proxy.create(CakeSide.Views, viewName)
 
     if _.isUndefined(constructor)
-      @logger.error("Could not find autoview for #{viewName}")
+      console.error("Could not find autoview for #{viewName}")
       return
 
     view = new constructor
spec/javascripts/initializers/auto_view_setup_spec.js.coffee
@@ -3,10 +3,8 @@ describe "CakeSide.AutoViewSetup", ->
   logger = null
 
   beforeEach ->
-    logger =
-      error: () -> {}
-    spyOn(logger, 'error')
-    subject = new CakeSide.AutoViewSetup(logger)
+    spyOn(console, 'error')
+    subject = new CakeSide.AutoViewSetup()
 
   describe "#execute", ->
     beforeEach ->
@@ -23,4 +21,4 @@ describe "CakeSide.AutoViewSetup", ->
       subject.execute()
 
       expect(_.isEmpty(CakeSide.AutoViewSetup.views)).toEqual(true)
-      expect(logger.error).toHaveBeenCalled()
+      expect(console.error).toHaveBeenCalled()