[Maintenance] replace listOf() with emptyList()

Refactored code to use `emptyList()` instead of `listOf()` for empty list initialization to improve readability.
InsertKoinIO · Oct 27, 2024 · f78659b · f78659b
1 parent 2eb9163
commit f78659b
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/...cts/android/koin-android-test/src/main/java/org/koin/android/test/verify/AndroidVerify.kt b/...cts/android/koin-android-test/src/main/java/org/koin/android/test/verify/AndroidVerify.kt
@@ -21,8 +21,8 @@ import kotlin.reflect.KClass
  * @param extraTypes - allow to declare extra type, to be bound above the existing definitions
  * @throws MissingKoinDefinitionException
  */
-fun Module.verify(extraTypes: List<KClass<*>> = listOf(), injections: List<ParameterTypeInjection>? = null) {
-    org.koin.test.verify.Verify.verify(this,extraTypes + androidTypes, injections)
+fun Module.verify(extraTypes: List<KClass<*>> = emptyList(), injections: List<ParameterTypeInjection>? = null) {
+    org.koin.test.verify.Verify.verify(this, extraTypes + androidTypes, injections)
 }
 
 /**
@@ -36,8 +36,8 @@ fun Module.verify(extraTypes: List<KClass<*>> = listOf(), injections: List<Param
  * @param extraTypes - allow to declare extra type, to be bound above the existing definitions
  * @throws MissingKoinDefinitionException
  */
-fun Module.androidVerify(extraTypes: List<KClass<*>> = listOf(), injections: List<ParameterTypeInjection>? = null) {
-    org.koin.test.verify.Verify.verify(this,extraTypes + androidTypes, injections)
+fun Module.androidVerify(extraTypes: List<KClass<*>> = emptyList(), injections: List<ParameterTypeInjection>? = null) {
+    org.koin.test.verify.Verify.verify(this, extraTypes + androidTypes, injections)
 }
 
 object AndroidVerify {
@@ -49,4 +49,4 @@ object AndroidVerify {
         SavedStateHandle::class,
         WorkerParameters::class
     )
-}
+}
diff --git a/projects/core/koin-core/src/commonMain/kotlin/org/koin/core/definition/BeanDefinition.kt b/projects/core/koin-core/src/commonMain/kotlin/org/koin/core/definition/BeanDefinition.kt
@@ -36,7 +36,7 @@ class BeanDefinition<T>(
     var qualifier: Qualifier? = null,
     val definition: Definition<T>,
     val kind: Kind,
-    var secondaryTypes: List<KClass<*>> = listOf(),
+    var secondaryTypes: List<KClass<*>> = emptyList(),
 ) {
     var callbacks: Callbacks<T> = Callbacks()
 

diff --git a/projects/core/koin-test/src/jvmMain/kotlin/org/koin/test/verify/Verification.kt b/projects/core/koin-test/src/jvmMain/kotlin/org/koin/test/verify/Verification.kt
@@ -90,7 +90,7 @@ class Verification(val module: Module, extraTypes: List<KClass<*>>, injections:
         return verificationByStatus[VerificationStatus.OK]?.map {
             println("|- dependency '${it.name}' - ${it.type.qualifiedName} found!")
             it.type
-        } ?: emptyList()
+        }.orEmpty()
     }
 
     private fun generateFixDefinition(first: VerifiedParameter): String {
@@ -172,4 +172,4 @@ data class VerifiedParameter(val name : String, val type : KClass<*>, val status
 
 enum class VerificationStatus {
     OK, MISSING, CIRCULAR
-}
+}
diff --git a/projects/core/koin-test/src/jvmMain/kotlin/org/koin/test/verify/VerifyModule.kt b/projects/core/koin-test/src/jvmMain/kotlin/org/koin/test/verify/VerifyModule.kt
@@ -27,15 +27,15 @@ import kotlin.time.measureTime
  * @param injections - Experimental - defines parameters injection types to allow (normally used in parametersOf)
  * @throws MissingKoinDefinitionException
  */
-fun Module.verify(extraTypes: List<KClass<*>> = listOf(), injections: List<ParameterTypeInjection>? = emptyList()) = Verify.verify(this, extraTypes,injections)
+fun Module.verify(extraTypes: List<KClass<*>> = emptyList(), injections: List<ParameterTypeInjection>? = emptyList()) = Verify.verify(this, extraTypes, injections)
 
 /**
  * Verify a list of Modules
  *
  * @see Module.verify
  */
-fun List<Module>.verifyAll(extraTypes: List<KClass<*>> = listOf(), injections: List<ParameterTypeInjection>? = emptyList()) {
-    forEach { module -> module.verify(extraTypes,injections) }
+fun List<Module>.verifyAll(extraTypes: List<KClass<*>> = emptyList(), injections: List<ParameterTypeInjection>? = emptyList()) {
+    forEach { module -> module.verify(extraTypes, injections) }
 }
 
 /**
@@ -72,7 +72,7 @@ object Verify {
      * @param extraTypes allow to declare extra type, to be bound above the existing definitions
      * @throws MissingKoinDefinitionException
      */
-    fun verify(module: Module, extraTypes: List<KClass<*>> = listOf(), injections: List<ParameterTypeInjection>? = null) {
+    fun verify(module: Module, extraTypes: List<KClass<*>> = emptyList(), injections: List<ParameterTypeInjection>? = null) {
 
         val verification = Verification(module, extraTypes, injections)
         println("Verifying module '$module' ...")