From 91eaa8a30874c3efa0307b7b10d1ef5e5eafe59d Mon Sep 17 00:00:00 2001 From: tcharding Date: Thu, 31 Aug 2017 16:36:17 +1000 Subject: [PATCH] Rename PluginsEnvVarPluginLoader to stop stutter --- pkg/kubectl/cmd/util/factory_builder.go | 2 +- pkg/kubectl/plugins/loader.go | 4 ++-- pkg/kubectl/plugins/loader_test.go | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/pkg/kubectl/cmd/util/factory_builder.go b/pkg/kubectl/cmd/util/factory_builder.go index e3a270fee30..c42df2699c4 100644 --- a/pkg/kubectl/cmd/util/factory_builder.go +++ b/pkg/kubectl/cmd/util/factory_builder.go @@ -180,7 +180,7 @@ func (f *ring2Factory) NewUnstructuredBuilder(allowRemoteCalls bool) (*resource. // system directory structure spec for the given platform. func (f *ring2Factory) PluginLoader() plugins.PluginLoader { if len(os.Getenv("KUBECTL_PLUGINS_PATH")) > 0 { - return plugins.PluginsEnvVarPluginLoader() + return plugins.KubectlPluginsPathPluginLoader() } return plugins.TolerantMultiPluginLoader{ plugins.XDGDataPluginLoader(), diff --git a/pkg/kubectl/plugins/loader.go b/pkg/kubectl/plugins/loader.go index aef346e0488..def8ae6e5ea 100644 --- a/pkg/kubectl/plugins/loader.go +++ b/pkg/kubectl/plugins/loader.go @@ -137,9 +137,9 @@ func PathFromEnvVarPluginLoader(envVarName string, subdirs ...string) PluginLoad return loader } -// PluginsEnvVarPluginLoader returns a PluginLoader that loads plugins from one or more +// KubectlPluginsPathPluginLoader returns a PluginLoader that loads plugins from one or more // directories specified by the KUBECTL_PLUGINS_PATH env var. -func PluginsEnvVarPluginLoader() PluginLoader { +func KubectlPluginsPathPluginLoader() PluginLoader { return PathFromEnvVarPluginLoader("KUBECTL_PLUGINS_PATH") } diff --git a/pkg/kubectl/plugins/loader_test.go b/pkg/kubectl/plugins/loader_test.go index b89ee671bfa..34f6b1c6abf 100644 --- a/pkg/kubectl/plugins/loader_test.go +++ b/pkg/kubectl/plugins/loader_test.go @@ -109,7 +109,7 @@ func TestUnexistentDirectoryPluginLoader(t *testing.T) { } } -func TestPluginsEnvVarPluginLoader(t *testing.T) { +func TestKubectlPluginsPathPluginLoader(t *testing.T) { tmp, err := setupValidPlugins(1, 0) if err != nil { t.Fatalf("unexpected error: %v", err) @@ -120,7 +120,7 @@ func TestPluginsEnvVarPluginLoader(t *testing.T) { os.Setenv(env, tmp) defer os.Unsetenv(env) - loader := PluginsEnvVarPluginLoader() + loader := KubectlPluginsPathPluginLoader() plugins, err := loader.Load() if err != nil {