small fixes

tellerops · May 7, 2021 · a364438 · a364438
1 parent 1eb99a4
commit a364438
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 8 deletions.
diff --git a/main.go b/main.go
@@ -49,7 +49,7 @@ var CLI struct {
 
 	Drift struct {
 		Providers []string `arg optional name:"providers" help:"A list of providers to check for drift"`
-	} `cmd help:"Scans your codebase for sensitive keys"`
+	} `cmd help:"Detect secret and value drift between providers"`
 }
 
 var (

diff --git a/pkg/core/types.go b/pkg/core/types.go
@@ -92,7 +92,7 @@ type EnvEntryLookup struct {
 func (ee *EnvEntryLookup) EnvBy(key, provider, path, dflt string) string {
 	for i := range ee.Entries {
 		e := ee.Entries[i]
-		if e.Key == key && e.Provider == provider && e.ResolvedPath == path {
+		if e.Key == key && e.ProviderName == provider && e.ResolvedPath == path {
 			return e.Value
 		}
 	}
@@ -112,7 +112,7 @@ func (ee *EnvEntryLookup) EnvByKey(key, dflt string) string {
 func (ee *EnvEntryLookup) EnvByKeyAndProvider(key, provider, dflt string) string {
 	for i := range ee.Entries {
 		e := ee.Entries[i]
-		if e.Key == key && e.Provider == provider {
+		if e.Key == key && e.ProviderName == provider {
 			return e.Value
 		}
 

diff --git a/pkg/teller_test.go b/pkg/teller_test.go
@@ -228,11 +228,6 @@ func TestTellerPorcelainNonInteractive(t *testing.T) {
 
 func TestTellerDrift(t *testing.T) {
 	tl := Teller{
-		Entries:   []core.EnvEntry{},
-		Providers: &BuiltinProviders{},
-	}
-
-	tl = Teller{
 		Entries: []core.EnvEntry{
 			{Key: "k", Value: "v", Source: "s1", Provider: "test-provider", ProviderName: "test-provider", ResolvedPath: "path/kv"},
 			{Key: "k", Value: "v", Sink: "s1", Provider: "test-provider", ProviderName: "test-provider2", ResolvedPath: "path/kv"},