From c5f682240fb913044eda2d6590151885182f692f Mon Sep 17 00:00:00 2001 From: Jason Del Ponte Date: Fri, 19 Jan 2018 11:33:05 -0800 Subject: [PATCH] aws/external: Fix EnvConfig misspelled container endpoint path getter Fixes EnvConfig's container endpoint path getter method was misspelled. This caused the type to not satsify the ContainerCredentialsEndpointPathProvider interface. The EnvConfig's tests also did not validate the type satisfied the interface. Fix #105 --- aws/external/env_config.go | 4 ++-- aws/external/env_config_test.go | 6 ++++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/aws/external/env_config.go b/aws/external/env_config.go index b96b1250fb1..902252c46da 100644 --- a/aws/external/env_config.go +++ b/aws/external/env_config.go @@ -194,9 +194,9 @@ func (c EnvConfig) GetCredentialsEndpoint() (string, error) { return c.CredentialsEndpoint, nil } -// GetContainerCredentailsEndpointPath returns the container credentails endpoint +// GetContainerCredentialsEndpointPath returns the container credentails endpoint // path string if set. -func (c EnvConfig) GetContainerCredentailsEndpointPath() (string, error) { +func (c EnvConfig) GetContainerCredentialsEndpointPath() (string, error) { return c.ContainerCredentialsEndpointPath, nil } diff --git a/aws/external/env_config_test.go b/aws/external/env_config_test.go index 80353e0a577..e0edcc7c773 100644 --- a/aws/external/env_config_test.go +++ b/aws/external/env_config_test.go @@ -9,11 +9,13 @@ import ( "github.com/aws/aws-sdk-go-v2/internal/awstesting" ) +var _ SharedConfigProfileProvider = (*EnvConfig)(nil) +var _ SharedConfigFilesProvider = (*EnvConfig)(nil) var _ CustomCABundleProvider = (*EnvConfig)(nil) var _ RegionProvider = (*EnvConfig)(nil) var _ CredentialsValueProvider = (*EnvConfig)(nil) -var _ SharedConfigProfileProvider = (*EnvConfig)(nil) -var _ SharedConfigFilesProvider = (*EnvConfig)(nil) +var _ CredentialsEndpointProvider = (*EnvConfig)(nil) +var _ ContainerCredentialsEndpointPathProvider = (*EnvConfig)(nil) func TestNewEnvConfig_Creds(t *testing.T) { env := awstesting.StashEnv()