[Test] check optional parameters during dependency verification

luozejiaqun · luozejiaqun · commit 706cb15c918e · 2025-01-03T12:02:20.000+08:00
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
@@ -140,7 +140,8 @@ class Verification(val module: Module, extraTypes: List<KClass<*>>, injections:
             if (isWhiteList) {
                 println("| dependency '$ctorParamLabel' is whitelisted")
             }
-            val isDefinitionDeclared = hasDefinition || isParameterInjected || isWhiteList
+            val isOptionalParameter = constructorParameter.isOptional
+            val isDefinitionDeclared = hasDefinition || isParameterInjected || isWhiteList || isOptionalParameter
 
             val alreadyBoundFactory = verifiedFactories.keys.firstOrNull { ctorParamClass in listOf(it.beanDefinition.primaryType) + it.beanDefinition.secondaryTypes }
             val factoryDependencies = verifiedFactories[alreadyBoundFactory]

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,8 @@ class Verification(val module: Module, extraTypes: List<KClass<*>>, injections:`
`140`	`140`	`if (isWhiteList) {`
`141`	`141`	`println("\| dependency '$ctorParamLabel' is whitelisted")`
`142`	`142`	`}`
`143`		`- val isDefinitionDeclared = hasDefinition \|\| isParameterInjected \|\| isWhiteList`
	`143`	`+ val isOptionalParameter = constructorParameter.isOptional`
	`144`	`+ val isDefinitionDeclared = hasDefinition \|\| isParameterInjected \|\| isWhiteList \|\| isOptionalParameter`
`144`	`145`
`145`	`146`	`val alreadyBoundFactory = verifiedFactories.keys.firstOrNull { ctorParamClass in listOf(it.beanDefinition.primaryType) + it.beanDefinition.secondaryTypes }`
`146`	`147`	`val factoryDependencies = verifiedFactories[alreadyBoundFactory]`