Merge pull request #114 from exoego/util-inspect

Util.inspect object is now function

Author: exoego

app/current/src/main/scala/io/scalajs/nodejs/util/InspectOptions.scala

@@ -25,14 +25,14 @@ import scala.scalajs.js.|
   *
   * @see [[https://nodejs.org/api/util.html#util_util_inspect_object_options]]
   */
-class InspectOptions(var showHidden: js.UndefOr[Boolean] = js.undefined,
-                     var depth: js.UndefOr[Int] = js.undefined,
-                     var colors: js.UndefOr[Boolean] = js.undefined,
-                     var customInspect: js.UndefOr[Boolean] = js.undefined,
-                     var showProxy: js.UndefOr[Boolean] = js.undefined,
-                     var maxArrayLength: js.UndefOr[Int] = js.undefined,
-                     var breakLength: js.UndefOr[Int] = js.undefined,
-                     var compact: js.UndefOr[Boolean | Int] = js.undefined,
+class InspectOptions(var showHidden: js.UndefOr[Boolean] = false,
+                     var depth: Int = 2,
+                     var colors: Boolean = false,
+                     var customInspect: Boolean = true,
+                     var showProxy: Boolean = false,
+                     var maxArrayLength: js.UndefOr[Int] = 100,
+                     var breakLength: Int = 80,
+                     var compact: js.UndefOr[Boolean | Int] = 3,
                      var sorted: js.UndefOr[Boolean | js.Function2[String, String, Int]] = js.undefined,
-                     var getters: js.UndefOr[Boolean | String] = js.undefined)
+                     var getters: js.UndefOr[Boolean | String] = false)
     extends js.Object

app/current/src/main/scala/io/scalajs/nodejs/util/Util.scala

@@ -52,15 +52,6 @@ trait Util extends js.Object {
     */
   def inherits(constructor: js.Any, superConstructor: js.Any): js.Any = js.native
 
-  /**
-    * The util.inspect() method returns a string representation of object that is primarily useful for debugging.
-    * Additional options may be passed that alter certain aspects of the formatted string.
-    * @param `object` any JavaScript primitive or Object.
-    * @param options  the given [[InspectOptions inspect options]]
-    * @example util.inspect(object[, options])
-    */
-  def inspect(`object`: js.Any, options: InspectOptions = js.native): String = js.native
-
   val inspect: InspectObject = js.native
 
   def callbackify[T](original: js.Function): js.Function2[js.Any, T, Any] = js.native
@@ -83,7 +74,7 @@ trait Util extends js.Object {
 object Util extends Util
 
 @js.native
-trait InspectObject extends js.Object {
+trait InspectObject extends js.Function2[js.Any, InspectOptions, String] with js.Function1[js.Any, String] {
   var defaultOptions: InspectOptions = js.native
   var styles: js.Dictionary[String]  = js.native
 

app/nodejs-v8/src/test/scala/io/scalajs/nodejs/util/UtilTest.scala

@@ -2,10 +2,15 @@ package io.scalajs.nodejs.util
 
 import org.scalatest.FunSpec
 
+import scala.scalajs.js
+
 class UtilTest extends FunSpec {
 
   it("have inspect object") {
     assert(Util.inspect !== null)
+    assert(Util.inspect(js.Array(1, 2, 3)) === "[ 1, 2, 3 ]")
+    val inspectOptions = new InspectOptions(maxArrayLength = 1)
+    assert(Util.inspect(js.Array(1, 2, 3), inspectOptions) === "[ 1, ... 2 more items ]")
     assert(Util.inspect.defaultOptions !== null)
     assert(Util.inspect.styles !== null)
   }