refactor tests

LLMAgentBuilder · Oct 4, 2024 · 6799533 · 6799533
1 parent 236810b
commit 6799533
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 174 deletions.
diff --git a/core/pom.xml b/core/pom.xml
@@ -48,10 +48,6 @@
       <groupId>com.fasterxml.jackson.module</groupId>
       <artifactId>jackson-module-kotlin</artifactId>
     </dependency>
-    <dependency>
-      <groupId>commons-beanutils</groupId>
-      <artifactId>commons-beanutils</artifactId>
-    </dependency>
     <dependency>
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-text</artifactId>

diff --git a/core/src/main/kotlin/io/github/llmagentbuilder/core/tool/AgentToolFactory.kt b/core/src/main/kotlin/io/github/llmagentbuilder/core/tool/AgentToolFactory.kt
@@ -1,7 +1,5 @@
 package io.github.llmagentbuilder.core.tool
 
-import io.github.llmagentbuilder.core.MapToObject
-import org.apache.commons.beanutils.BeanUtils
 import java.util.function.Supplier
 
 /**
@@ -48,55 +46,3 @@ abstract class BaseConfigurableAgentToolFactory<out T : ConfigurableAgentTool<*,
         return create(configProvider.get())
     }
 }
-
-class MapConfigProvider<C>(
-    private val configClass: Class<C>,
-    private val config: Map<String, Any?>
-) : Supplier<C?> {
-    override fun get(): C? {
-        return MapToObject.toObject(configClass, config)
-    }
-}
-
-abstract class MapConfigurableAgentToolFactory<out T : ConfigurableAgentTool<*, *, CONFIG>, CONFIG>(
-    configClass: Class<CONFIG>,
-    config: Map<String, Any?>,
-) : BaseConfigurableAgentToolFactory<T, CONFIG>(
-    MapConfigProvider(
-        configClass,
-        config
-    )
-)
-
-abstract class EnvironmentVariableConfigurableAgentToolFactory<out T : ConfigurableAgentTool<*, *, CONFIG>, CONFIG>(
-    configClass: Class<CONFIG>,
-    environmentVariablePrefix: String
-) :
-    BaseConfigurableAgentToolFactory<T, CONFIG>(
-        EnvironmentVariableConfigProvider(
-            configClass,
-            environmentVariablePrefix
-        )
-    )
-
-open class EnvironmentVariableConfigProvider<C>(
-    private val configClass: Class<C>,
-    private val environmentVariablePrefix: String
-) :
-    Supplier<C?> {
-    override fun get(): C? {
-        val instance = configClass.getDeclaredConstructor().newInstance()
-        BeanUtils.populate(instance, getEnvironmentVariables())
-        return instance
-    }
-
-    private fun getEnvironmentVariables(): Map<String, String> {
-        val prefix = environmentVariablePrefix
-        return System.getenv()
-            .filterKeys {
-                it.startsWith(prefix)
-            }.mapKeys { entry ->
-                entry.key.removePrefix(prefix)
-            }
-    }
-}
diff --git a/.../test/kotlin/io/github/llmagentbuilder/core/tool/EnvironmentVariableConfigProviderTest.kt b/.../test/kotlin/io/github/llmagentbuilder/core/tool/EnvironmentVariableConfigProviderTest.kt
diff --git a/core/src/test/resources/output_parser/ReActJson/code.txt b/core/src/test/resources/output_parser/ReActJson/code.txt
diff --git a/core/src/test/resources/output_parser/ReActJson/double-quotes-escape.txt b/core/src/test/resources/output_parser/ReActJson/double-quotes-escape.txt
diff --git a/core/src/test/resources/output_parser/ReActJson/python-code.txt b/core/src/test/resources/output_parser/ReActJson/python-code.txt
diff --git a/core/src/test/resources/output_parser/ReActJson/triple-quotes-escape.txt b/core/src/test/resources/output_parser/ReActJson/triple-quotes-escape.txt
diff --git a/pom.xml b/pom.xml
@@ -256,11 +256,6 @@
         <artifactId>jsoup</artifactId>
         <version>1.17.2</version>
       </dependency>
-      <dependency>
-        <groupId>commons-beanutils</groupId>
-        <artifactId>commons-beanutils</artifactId>
-        <version>1.9.4</version>
-      </dependency>
       <dependency>
         <groupId>org.yaml</groupId>
         <artifactId>snakeyaml</artifactId>