Test and cleaned up the logging library

randym · May 6, 2016 · e106aac · e106aac
1 parent 6217611
commit e106aac
Show file tree

Hide file tree

Showing 2 changed files with 409 additions and 317 deletions.
diff --git a/index.js b/index.js
@@ -48,12 +48,15 @@ class JsonApi {
       logger: true
     }
 
-    if (arguments.length === 2 || (arguments.length === 1 && _.isString(arguments[0]))) {
+    let deprecatedConstructos = (args) => {
+      return (args.length === 2 || (args.length === 1 && _.isString(args[0])))
+    }
+
+    if (deprecatedConstructos(arguments)) {
       defaults.apiUrl = arguments[0]
       if (arguments.length === 2) {
         defaults.middleware = arguments[1]
       }
-      console.error('Constructor (apiUrl, middleware) has been deprecated, initialize Devour with an object.')
     }
 
     options = _.assign(defaults, options)
@@ -69,9 +72,12 @@ class JsonApi {
     this.serialize = serialize
     this.builderStack = []
     this.logger = Minilog('devour')
-
     options.logger ? Minilog.enable() : MiniLog.disable()
 
+    if (deprecatedConstructos(arguments)) {
+      this.logger.warn('Constructor (apiUrl, middleware) has been deprecated, initialize Devour with an object.')
+    }
+
   }
 
   enableLogging(enabled = true){