feat: misleading method name

Author: Numichi

src/main/java/io/github/numichi/reactive/logger/coroutine/ACoroutine.kt

@@ -40,8 +40,18 @@ abstract class ACoroutine<T : IReactorLogger>(
             return this
         }
 
-        fun withEnableError(enableError: Boolean): Builder<E, L, R> {
-            this.enableError = enableError
+        @Deprecated(
+            message = "It can be misleading due to enabling word in method name and boolean parameter. Use: withError(boolean); False by default",
+            replaceWith = ReplaceWith("withError(enable)"),
+            level = DeprecationLevel.WARNING
+        )
+        fun withEnableError(enable: Boolean): Builder<E, L, R> {
+            this.enableError = enable
+            return this
+        }
+
+        fun withError(enable: Boolean = true): Builder<E, L, R> {
+            this.enableError = enable
             return this
         }
 

src/main/java/io/github/numichi/reactive/logger/reactor/AReactive.kt

@@ -28,9 +28,19 @@ abstract class AReactive(
             return this;
         }
 
-        fun withEnableError(enableError: Boolean): Builder<L, R> {
-            this.enableError = enableError
-            return this;
+        @Deprecated(
+            message = "It can be misleading due to enabling word in method name and boolean parameter. Use: withError(boolean); False by default",
+            replaceWith = ReplaceWith("withError(enableError)"),
+            level = DeprecationLevel.WARNING
+        )
+        fun withEnableError(enable: Boolean): Builder<L, R> {
+            this.enableError = enable
+            return this
+        }
+
+        fun withError(enable: Boolean = true): Builder<L, R> {
+            this.enableError = enable
+            return this
         }
 
         fun withLogger(logger: L): Builder<L, R> {

src/test/java/io/github/numichi/reactive/logger/coroutine/CoroutineKLoggerTest.kt

@@ -33,7 +33,7 @@ internal class CoroutineKLoggerTest {
     private val imperativeLogger: KLogger = mockk(relaxed = true)
     private val logger: ICoroutineKLogger = CoroutineKLogger.reactorBuilder().withLogger(imperativeLogger).build()
     private val loggerScheduled: ICoroutineKLogger = CoroutineKLogger.reactorBuilder().withLogger(imperativeLogger).withScheduler(Schedulers.parallel()).build()
-    private val loggerWithError: ICoroutineKLogger = CoroutineKLogger.reactorBuilder().withLogger(imperativeLogger).withEnableError(true).build()
+    private val loggerWithError: ICoroutineKLogger = CoroutineKLogger.reactorBuilder().withLogger(imperativeLogger).withError().build()
 
     companion object {
         @JvmStatic

src/test/java/io/github/numichi/reactive/logger/coroutine/CoroutineLoggerTest.kt

@@ -32,7 +32,7 @@ internal class CoroutineLoggerTest {
     private val imperativeLogger: Logger = mockk(relaxed = true)
     private val logger = CoroutineLogger.reactorBuilder().withLogger(imperativeLogger).build()
     private val loggerScheduled = CoroutineLogger.reactorBuilder().withLogger(imperativeLogger).withScheduler(Schedulers.parallel()).build()
-    private val loggerWithError = CoroutineLogger.reactorBuilder().withLogger(imperativeLogger).withEnableError(true).build()
+    private val loggerWithError = CoroutineLogger.reactorBuilder().withLogger(imperativeLogger).withError(true).build()
 
     companion object {
         @JvmStatic

src/test/java/io/github/numichi/reactive/logger/coroutine/CoroutineOtherTest.kt

@@ -30,7 +30,6 @@ internal class CoroutineOtherTest {
                 .withContextKey(ReactorContext)
                 .withContextExtractive { coroutineContext[it]?.context }
                 .withScheduler(Schedulers.boundedElastic())
-                .withEnableError(false)
                 .withMDCContextKey("other-key")
                 .withLogger(mockKLogger)
                 .build()
@@ -45,7 +44,6 @@ internal class CoroutineOtherTest {
                 .withContextKey(ReactorContext)
                 .withContextExtractive { coroutineContext[it]?.context }
                 .withScheduler(Schedulers.boundedElastic())
-                .withEnableError(false)
                 .withMDCContextKey("other-key")
                 .withLogger(mockLogger)
                 .build()
@@ -181,7 +179,7 @@ internal class CoroutineOtherTest {
     @Test
     fun `should throw exception from snapshot`() {
         val logger = CoroutineLogger.reactorBuilder()
-            .withEnableError(true)
+            .withError(true)
             .withLogger(mockk(relaxed = true))
             .build()
 

src/test/java/io/github/numichi/reactive/logger/reactor/ReactiveKLoggerTest.kt

@@ -37,7 +37,7 @@ import java.util.*
 internal class ReactiveKLoggerTest {
     private val imperativeLogger: KLogger = mockk(relaxed = true)
     private val logger = builder().withLogger(imperativeLogger).build()
-    private val loggerWithError = builder().withLogger(imperativeLogger).withEnableError(true).build()
+    private val loggerWithError = builder().withLogger(imperativeLogger).withError().build()
 
     companion object {
         fun randomText(): String {

src/test/java/io/github/numichi/reactive/logger/reactor/ReactiveLoggerTest.kt

@@ -36,7 +36,7 @@ import java.util.*
 internal class ReactiveLoggerTest {
     private val imperativeLogger: Logger = mockk(relaxed = true)
     private val logger = builder().withLogger(imperativeLogger).build()
-    private val loggerWithError = builder().withLogger(imperativeLogger).withEnableError(true).build()
+    private val loggerWithError = builder().withLogger(imperativeLogger).withError().build()
 
     companion object {
         fun randomText(): String {