Use builder pattern to create configuration

Author: OptimumCode

json-schema-validator-objects/src/commonMain/kotlin/io/github/optimumcode/json/schema/wrappers/objects/Wrappers.kt

@@ -13,38 +13,6 @@ import kotlin.jvm.JvmInline
 import kotlin.jvm.JvmName
 import kotlin.jvm.JvmOverloads
 
-@ExperimentalApi
-public class WrappingConfiguration internal constructor(
-  /**
-   * If set to `false` an exception is thrown when wrapping a [Set].
-   * If set to `true`, [Set] is wrapped the same way as [List]
-   */
-  public val allowSets: Boolean = false,
-  /**
-   * If set to `false` the [Char] is converted to [String].
-   * If set to `true` the [Char] is converted to a codepoint (and then to [Long])
-   */
-  public val charAsCodepoint: Boolean = false,
-  /**
-   * If set to `true` the [ByteArray] is encoded using Base64 encoding and wrapped as a [PrimitiveElement].
-   * Otherwise, the [ByteArray] is wrapped as an [ArrayElement].
-   */
-  public val byteArrayAsBase64String: Boolean = true,
-)
-
-@ExperimentalApi
-@JvmOverloads
-public fun wrappingConfiguration(
-  allowSets: Boolean = false,
-  charAsCodepoint: Boolean = false,
-  byteArrayAsBase64String: Boolean = true,
-): WrappingConfiguration =
-  WrappingConfiguration(
-    allowSets = allowSets,
-    charAsCodepoint = charAsCodepoint,
-    byteArrayAsBase64String = byteArrayAsBase64String,
-  )
-
 /**
  * Returns an [AbstractElement] produced by converting the [obj] value.
  * The [configuration] allows conversion customization.
@@ -80,7 +48,7 @@ public fun wrappingConfiguration(
 @ExperimentalApi
 public fun wrapAsElement(
   obj: Any?,
-  configuration: WrappingConfiguration = WrappingConfiguration(),
+  configuration: WrappingConfiguration = WrappingConfiguration.create(),
 ): AbstractElement {
   if (obj == null) {
     return NullWrapper

json-schema-validator-objects/src/commonMain/kotlin/io/github/optimumcode/json/schema/wrappers/objects/WrappingConfiguration.kt

@@ -0,0 +1,47 @@
+package io.github.optimumcode.json.schema.wrappers.objects
+
+import io.github.optimumcode.json.schema.ExperimentalApi
+import kotlin.jvm.JvmOverloads
+import kotlin.jvm.JvmStatic
+
+@ExperimentalApi
+public class WrappingConfiguration internal constructor(
+  /**
+   * If set to `false` an exception is thrown when wrapping a [Set].
+   * If set to `true`, [Set] is wrapped the same way as [List]
+   */
+  public val allowSets: Boolean,
+  /**
+   * If set to `false` the [Char] is converted to [String].
+   * If set to `true` the [Char] is converted to a codepoint (and then to [Long])
+   */
+  public val charAsCodepoint: Boolean,
+  /**
+   * If set to `true` the [ByteArray] is encoded using Base64 encoding and wrapped as a [io.github.optimumcode.json.schema.model.PrimitiveElement].
+   * Otherwise, the [ByteArray] is wrapped as an [io.github.optimumcode.json.schema.model.ArrayElement].
+   */
+  public val byteArrayAsBase64String: Boolean,
+) {
+  public companion object {
+    @ExperimentalApi
+    @JvmStatic
+    @JvmOverloads
+    public fun create(configuration: WrappingConfigurationBuilder.() -> Unit = {}): WrappingConfiguration =
+      WrappingConfigurationBuilder().apply(configuration).build()
+  }
+}
+
+@ExperimentalApi
+public class WrappingConfigurationBuilder internal constructor() {
+  public var allowSets: Boolean = false
+  public var charAsCodepoint: Boolean = false
+  public var byteArrayAsBase64String: Boolean = true
+
+  internal fun build(): WrappingConfiguration {
+    return WrappingConfiguration(
+      allowSets = allowSets,
+      charAsCodepoint = charAsCodepoint,
+      byteArrayAsBase64String = byteArrayAsBase64String,
+    )
+  }
+}

json-schema-validator-objects/src/commonTest/kotlin/io/github/optimumcode/json/schema/wrappers/objects/WrappersTest.kt

@@ -149,7 +149,7 @@ class WrappersTest : FunSpec() {
     }
 
     test("primitive wrapper for char as codepoint") {
-      wrapAsElement('4', wrappingConfiguration(charAsCodepoint = true))
+      wrapAsElement('4', WrappingConfiguration.create { charAsCodepoint = true })
         .shouldBeInstanceOf<PrimitiveElement> { el ->
           assertSoftly {
             "isString".asClue { el.isString.shouldBeFalse() }
@@ -220,9 +220,9 @@ class WrappersTest : FunSpec() {
         shouldNotThrowAny {
           wrapAsElement(
             setOf("a"),
-            wrappingConfiguration(
-              allowSets = true,
-            ),
+            WrappingConfiguration.create {
+              allowSets = true
+            },
           )
         }
       element.shouldBeInstanceOf<ArrayElement> {
@@ -309,9 +309,9 @@ class WrappersTest : FunSpec() {
     test("byte array can be wrapped as an array element") {
       wrapAsElement(
         byteArrayOf(42),
-        wrappingConfiguration(
-          byteArrayAsBase64String = false,
-        ),
+        WrappingConfiguration.create {
+          byteArrayAsBase64String = false
+        },
       ).shouldBeInstanceOf<ArrayElement> {
         it.size shouldBe 1
         it.single().shouldBeInstanceOf<PrimitiveElement>()