From 602aa37f06418017fad78738cd3832c163c08d99 Mon Sep 17 00:00:00 2001
From: Cody Soyland <codysoyland@gmail.com>
Date: Fri, 13 Sep 2019 11:13:08 -0500
Subject: [PATCH] Generate uppercase env var names in LoadEnv (#8)

---
 com.go      |  2 +-
 com_test.go | 16 ++++++++--------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/com.go b/com.go
index 64df573..6ab282d 100644
--- a/com.go
+++ b/com.go
@@ -67,7 +67,7 @@ func Run(main interface{}) error {
 var replacer *strings.Replacer = strings.NewReplacer("-", "_", ".", "_")
 
 func envNorm(name string) string {
-	return replacer.Replace(name)
+	return strings.ToUpper(replacer.Replace(name))
 }
 
 // loadEnv visits each flag in the FlagSet and sets its value based on
diff --git a/com_test.go b/com_test.go
index 1ab7611..967ed18 100644
--- a/com_test.go
+++ b/com_test.go
@@ -36,11 +36,11 @@ func TestLoadEnv(t *testing.T) {
 
 	// set up environment
 	prefix := "COMMANDEER_"
-	err = os.Setenv("COMMANDEER_one", "z")
+	err = os.Setenv("COMMANDEER_ONE", "z")
 	if err != nil {
 		t.Fatalf("setting up environment for set: %v", err)
 	}
-	defer os.Unsetenv("COMMANDEER_one")
+	defer os.Unsetenv("COMMANDEER_ONE")
 
 	// change values on instance by reading environment
 	err = loadEnv(fs, prefix)
@@ -71,17 +71,17 @@ func TestLoadEnv(t *testing.T) {
 	mm.Five = 55
 
 	// reload environment
-	err = os.Setenv("COMMANDEER_five", "56")
+	err = os.Setenv("COMMANDEER_FIVE", "56")
 	if err != nil {
 		t.Fatalf("setting up environment for set: %v", err)
 	}
-	defer os.Unsetenv("COMMANDEER_five")
+	defer os.Unsetenv("COMMANDEER_FIVE")
 
-	err = os.Setenv("COMMANDEER_two", "23")
+	err = os.Setenv("COMMANDEER_TWO", "23")
 	if err != nil {
 		t.Fatalf("setting up environment for set: %v", err)
 	}
-	defer os.Unsetenv("COMMANDEER_two")
+	defer os.Unsetenv("COMMANDEER_TWO")
 
 	err = loadEnv(fs, prefix)
 	if err != nil {
@@ -113,8 +113,8 @@ func TestLoadEnv(t *testing.T) {
 
 func TestLoadArgsEnv(t *testing.T) {
 	mm := test.NewSimpleMain()
-	mustSetenv(t, "COMMANDEER_one", "envone")
-	mustSetenv(t, "COMMANDEER_two", "32")
+	mustSetenv(t, "COMMANDEER_ONE", "envone")
+	mustSetenv(t, "COMMANDEER_TWO", "32")
 
 	fs := flag.NewFlagSet("", flag.ExitOnError)
 	err := LoadArgsEnv(fs, mm, []string{"-two=24", "-seven", "7.3", "-nine", "a,b,c"}, "COMMANDEER_", nil)