Address PR comments

Signed-off-by: Taylor Gray <tylgry@amazon.com>
opensearch-project · Mar 14, 2024 · a17ff35 · a17ff35
1 parent 0102a91
commit a17ff35
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/...epper/plugins/DefaultPluginFactoryIT.java → ...repper/plugin/DefaultPluginFactoryIT.java b/...epper/plugins/DefaultPluginFactoryIT.java → ...repper/plugin/DefaultPluginFactoryIT.java
@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-package org.opensearch.dataprepper.plugins;
+package org.opensearch.dataprepper.plugin;
 
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -15,11 +15,7 @@
 import org.opensearch.dataprepper.model.configuration.PipelinesDataFlowModel;
 import org.opensearch.dataprepper.model.configuration.PluginSetting;
 import org.opensearch.dataprepper.model.plugin.InvalidPluginConfigurationException;
-import org.opensearch.dataprepper.plugin.DefaultPluginFactory;
-import org.opensearch.dataprepper.plugin.ExtensionsConfiguration;
-import org.opensearch.dataprepper.plugin.PluginBeanFactoryProvider;
-import org.opensearch.dataprepper.plugin.TestPluggableInterface;
-import org.opensearch.dataprepper.plugin.TestPluginConfiguration;
+import org.opensearch.dataprepper.plugins.TestObjectPlugin;
 import org.opensearch.dataprepper.plugins.test.TestPlugin;
 import org.springframework.context.annotation.AnnotationConfigApplicationContext;
 

diff --git a/...-framework/src/main/java/org/opensearch/dataprepper/plugin/PluginBeanFactoryProvider.java b/...-framework/src/main/java/org/opensearch/dataprepper/plugin/PluginBeanFactoryProvider.java
@@ -25,7 +25,7 @@
  * <p><i>publicContext</i> is the root {@link ApplicationContext}</p>
  */
 @Named
-public class PluginBeanFactoryProvider implements Provider<BeanFactory> {
+class PluginBeanFactoryProvider implements Provider<BeanFactory> {
     private final GenericApplicationContext sharedPluginApplicationContext;
     private final GenericApplicationContext coreApplicationContext;