Adding more coverage

config/config.go

@@ -1,6 +1,7 @@
 package config
 
 import (
+	"fmt"
 	"os"
 	"path/filepath"
 
@@ -38,19 +39,19 @@ func Load() {
 func readFile(cfg interface{}, path string) {
 	f, err := os.Open(filepath.Clean(path))
 	if err != nil {
-		panic(err)
+		fmt.Println(err.Error())
 	}
-	// defer the closing of our jsonFile so that we can parse it later on
+	// defer the closing of the file so that we can parse it later on
 	defer func() {
 		if err := f.Close(); err != nil {
-			panic(err)
+			fmt.Println(err.Error())
 		}
 	}()
 
 	decoder := yaml.NewDecoder(f)
-	err = decoder.Decode(cfg)
-	if err != nil {
-		panic(err)
+	fmt.Println(decoder)
+	if errD := decoder.Decode(cfg); errD != nil {
+		fmt.Println(errD.Error())
 	}
 }
 

config/config_test.go

@@ -40,35 +40,37 @@ func Test_readFile(t *testing.T) {
 		path string
 	}
 	tests := []struct {
-		name      string
-		args      args
-		wantPanic bool
+		name    string
+		args    args
+		wantErr bool
 	}{
 		{
 			name: "givenExistPath_success",
 			args: args{
 				cfg:  &AppConfig,
 				path: "prod.yml",
 			},
-			wantPanic: false,
+			wantErr: false,
 		},
 		{
 			name: "givenNotExistPath_getPanic_failure",
 			args: args{
 				cfg:  nil,
 				path: "dummy-file.yml",
 			},
-			wantPanic: true,
+			wantErr: true,
+		},
+		{
+			name: "givenTestPath_getPanic_failure",
+			args: args{
+				cfg:  3,
+				path: "test.yml",
+			},
+			wantErr: true,
 		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			defer func() {
-				r := recover()
-				if (r != nil) != tt.wantPanic {
-					t.Errorf("readFile(), wantPanic = %v", tt.wantPanic)
-				}
-			}()
 			readFile(tt.args.cfg, tt.args.path)
 		})
 	}

config/test.yml

@@ -0,0 +1 @@
+{"test":"foo"}