add exclusion filter and toString, fixes #40

Author: jangalinski

_itest/spi-itest/src/test/kotlin/SpiITest.kt

@@ -6,20 +6,34 @@ import com.squareup.kotlinpoet.ExperimentalKotlinPoetApi
 import com.tschuchort.compiletesting.KotlinCompilation
 import io.toolisticon.kotlin.generation.KotlinCodeGeneration
 import io.toolisticon.kotlin.generation.KotlinCodeGeneration.className
+import io.toolisticon.kotlin.generation.KotlinCodeGeneration.spi.registry
 import io.toolisticon.kotlin.generation.spi.strategy.executeSingle
 import io.toolisticon.kotlin.generation.test.KotlinCodeGenerationTest
 import io.toolisticon.kotlin.generation.test.model.KotlinCompilationCommand
 import org.assertj.core.api.Assertions.assertThat
+import org.assertj.core.api.Assertions.assertThatThrownBy
 import org.jetbrains.kotlin.compiler.plugin.ExperimentalCompilerApi
 import org.junit.jupiter.api.Test
 
 internal class SpiITest {
 
+  @Test
+  fun `init fails when only strategy is excluded`() {
+    assertThatThrownBy {
+      registry(
+        contextTypeUpperBound = TestContext::class,
+        exclusions = setOf("io.toolisticon.kotlin.generation.itest.spi.TestDataClassStrategy")
+      )
+    }.isInstanceOf(IllegalStateException::class.java)
+      .hasMessage("No serviceInstances found, configure `META-INF/services/io.toolisticon.kotlin.generation.spi.KotlinCodeGenerationSpi`, and/or check your exclusions filter.")
+  }
+
   @Test
   fun `use spi defined strategies and processors to generate code`() {
-    val registry = KotlinCodeGeneration.spi.registry(TestContext::class)
+    val registry = registry(contextTypeUpperBound = TestContext::class)
     val context = TestContext(registry)
 
+
     val input = MapInput(
       className = className(packageName = "foo.bar", simpleName = "ExampleDataClass"),
       fields = mapOf(

kotlin-code-generation/src/main/kotlin/KotlinCodeGeneration.kt

@@ -396,8 +396,9 @@ object KotlinCodeGeneration : KLogging() {
      */
     fun registry(
       contextTypeUpperBound: KClass<*> = Any::class,
-      classLoader: ClassLoader = defaultClassLoader()
-    ): KotlinCodeGenerationSpiRegistry = KotlinCodeGenerationServiceLoader(contextTypeUpperBound = contextTypeUpperBound, classLoader = classLoader).invoke()
+      classLoader: ClassLoader = defaultClassLoader(),
+      exclusions: Set<String> = emptySet()
+    ): KotlinCodeGenerationSpiRegistry = KotlinCodeGenerationServiceLoader(contextTypeUpperBound = contextTypeUpperBound, classLoader = classLoader, exclusions = exclusions).invoke()
   }
 
   /**

kotlin-code-generation/src/main/kotlin/spi/processor/KotlinCodeGenerationProcessorList.kt

@@ -27,6 +27,8 @@ value class KotlinCodeGenerationProcessorList(val list: List<UnboundKotlinCodeGe
   ): List<PROCESSOR> {
     return list.filterIsInstance(processorType.java)
   }
+
+  override fun toString(): String = "KotlinCodeGenerationProcessorList(processors=${list.map { it.name }})"
 }
 
 

kotlin-code-generation/src/main/kotlin/spi/registry/KotlinCodeGenerationServiceLoader.kt

@@ -21,12 +21,14 @@ import kotlin.reflect.full.isSubclassOf
 @ExperimentalKotlinPoetApi
 class KotlinCodeGenerationServiceLoader(
   val contextTypeUpperBound: KClass<*> = Any::class,
-  val classLoader: ClassLoader = KotlinCodeGeneration.spi.defaultClassLoader()
+  val classLoader: ClassLoader = KotlinCodeGeneration.spi.defaultClassLoader(),
+  val exclusions: Set<String> = emptySet()
 ) : () -> KotlinCodeGenerationSpiRegistry {
 
   override fun invoke(): KotlinCodeGenerationSpiRegistry {
     val serviceInstances: List<KotlinCodeGenerationSpi<*, *>> = ServiceLoader.load(KotlinCodeGenerationSpi::class.java, classLoader).toList()
-    require(serviceInstances.isNotEmpty()) { "No serviceInstances found, configure `${KotlinCodeGenerationSpi.metaInfServices}`." }
+      .filterNot { exclusions.contains(it::class.java.name) }
+    check(serviceInstances.isNotEmpty()) { "No serviceInstances found, configure `${KotlinCodeGenerationSpi.metaInfServices}`, and/or check your exclusions filter." }
 
     val withIllegalContextType = serviceInstances.filterNot { it.contextType.isSubclassOf(contextTypeUpperBound) }
 

kotlin-code-generation/src/main/kotlin/spi/registry/KotlinCodeGenerationServiceRepository.kt

@@ -23,10 +23,12 @@ class KotlinCodeGenerationServiceRepository(
 ) : KotlinCodeGenerationSpiRegistry {
   companion object : KLogging() {
 
-    fun load(contextTypeUpperBound: KClass<*>, classLoader: ClassLoader = KotlinCodeGeneration.spi.defaultClassLoader()): KotlinCodeGenerationSpiRegistry {
-      return KotlinCodeGenerationServiceLoader(contextTypeUpperBound, classLoader)()
+    fun load(contextTypeUpperBound: KClass<*>,
+             classLoader: ClassLoader = KotlinCodeGeneration.spi.defaultClassLoader(),
+             exclusions: Set<String> = emptySet()
+    ): KotlinCodeGenerationSpiRegistry {
+      return KotlinCodeGenerationServiceLoader(contextTypeUpperBound, classLoader, exclusions)()
     }
-
   }
 
   init {

kotlin-code-generation/src/main/kotlin/spi/strategy/KotlinCodeGenerationStrategyList.kt

@@ -26,6 +26,8 @@ value class KotlinCodeGenerationStrategyList(private val list: List<UnboundKotli
   ): List<STRATEGY> {
     return list.filterIsInstance(strategyType.java)
   }
+
+  override fun toString(): String = "KotlinCodeGenerationStrategyList(strategies=${list.map { it.name }})"
 }
 
 @ExperimentalKotlinPoetApi