Ignore StringProvoder in cli-bot Introspector

serpro69 · Apr 30, 2022 · f1c82f3 · f1c82f3
1 parent a6b3b3c
commit f1c82f3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/cli-bot/src/main/kotlin/io/github/serpro69/kfaker/app/cli/Introspector.kt b/cli-bot/src/main/kotlin/io/github/serpro69/kfaker/app/cli/Introspector.kt
@@ -2,6 +2,7 @@ package io.github.serpro69.kfaker.app.cli
 
 import io.github.serpro69.kfaker.Faker
 import io.github.serpro69.kfaker.provider.FakeDataProvider
+import io.github.serpro69.kfaker.provider.misc.StringProvider
 import kotlin.reflect.KFunction
 import kotlin.reflect.KProperty
 import kotlin.reflect.KVisibility
@@ -16,7 +17,9 @@ import kotlin.reflect.full.starProjectedType
 class Introspector(private val faker: Faker) {
     // Get a list of all publicly visible providers
     val providers: Sequence<KProperty<*>> = faker::class.declaredMemberProperties.asSequence().filter {
-        it.visibility == KVisibility.PUBLIC && it.returnType.isSubtypeOf(FakeDataProvider::class.starProjectedType)
+        it.visibility == KVisibility.PUBLIC
+            && it.returnType.isSubtypeOf(FakeDataProvider::class.starProjectedType)
+            && it.returnType.classifier != StringProvider::class // Ignore this one as it's "special"
     }
 
     // Get a list of all publicly visible functions in each provider

diff --git a/cli-bot/src/main/kotlin/io/github/serpro69/kfaker/app/cli/Renderer.kt b/cli-bot/src/main/kotlin/io/github/serpro69/kfaker/app/cli/Renderer.kt
@@ -82,7 +82,7 @@ fun renderProvider(
             Renderer("${it.name}() // => $value", emptyList())
         }
     } else {
-        functions.asSequence().map { Renderer("${it.name}()", emptyList()) }
+        functions.map { Renderer("${it.name}()", emptyList()) }
     }
 
     return if (options.javaSyntax) {

diff --git a/cli-bot/src/test/kotlin/io/github/serpro69/kfaker/app/cli/IntrospectorTest.kt b/cli-bot/src/test/kotlin/io/github/serpro69/kfaker/app/cli/IntrospectorTest.kt
@@ -166,7 +166,6 @@ class IntrospectorTest : DescribeSpec() {
                         "Stargate",
                         "StrangerThings",
                         "StreetFighter",
-                        "StringProvider",
                         "Stripe",
                         "Subscription",
                         "Suits",
@@ -346,7 +345,6 @@ class IntrospectorTest : DescribeSpec() {
                         "stargate",
                         "strangerThings",
                         "streetFighter",
-                        "string",
                         "stripe",
                         "subscription",
                         "suits",