Merge pull request #235 from exoego/factory

Use Factory macro where inheriting other trait (Follow-up of #234)

Author: exoego

app/current/src/main/scala/io/scalajs/nodejs/crypto/Cipher.scala

@@ -2,6 +2,7 @@ package io.scalajs.nodejs.crypto
 
 import io.scalajs.nodejs.buffer.Buffer
 import io.scalajs.nodejs.stream.{Transform, TransformOptions}
+import net.exoego.scalajs.types.util.Factory
 
 import scala.scalajs.js
 
@@ -85,8 +86,7 @@ sealed trait Cipher extends Transform {
   def update(data: BufferLike): Buffer                                            = js.native
 }
 
-// TODO: Use Factory macro
-class SetAADOptions(transform: js.UndefOr[js.Function] = js.undefined,
-                    flush: js.UndefOr[js.Function] = js.undefined,
-                    var plaintextLength: js.UndefOr[Int] = js.undefined
-) extends TransformOptions(transform, flush) {}
+@Factory
+trait SetAADOptions extends TransformOptions {
+  var plaintextLength: js.UndefOr[Int] = js.undefined
+}

app/current/src/main/scala/io/scalajs/nodejs/crypto/Hash.scala

@@ -3,6 +3,7 @@ package io.scalajs.nodejs.crypto
 import com.thoughtworks.enableIf
 import io.scalajs.nodejs.buffer.Buffer
 import io.scalajs.nodejs.stream.{Transform, TransformOptions}
+import net.exoego.scalajs.types.util.Factory
 
 import scala.scalajs.js
 
@@ -72,8 +73,7 @@ sealed trait Hash extends Transform {
   def update(data: BufferLike): Hash                    = js.native
 }
 
-// TODO: Use Fatory macro
-class CreateHashOptions(transform: js.UndefOr[js.Function] = js.undefined,
-                        flush: js.UndefOr[js.Function] = js.undefined,
-                        var outputLength: js.UndefOr[Int] = js.undefined
-) extends TransformOptions(transform, flush)
+@Factory
+trait CreateHashOptions extends TransformOptions {
+  var outputLength: js.UndefOr[Int] = js.undefined
+}

app/current/src/main/scala/io/scalajs/nodejs/stream/Stream.scala

@@ -356,7 +356,8 @@ trait DuplexOptions extends js.Object {
   var writableObjectMode: js.UndefOr[Boolean] = js.undefined
 }
 
-// TODO: Use Factory macro
-class TransformOptions(var transform: js.UndefOr[js.Function] = js.undefined,
-                       var flush: js.UndefOr[js.Function] = js.undefined
-) extends js.Object
+@Factory
+trait TransformOptions extends js.Object {
+  var transform: js.UndefOr[js.Function] = js.undefined
+  var flush: js.UndefOr[js.Function]     = js.undefined
+}