[fix] linter errcheck

Author: bmf-san

cmdutil/when.go

@@ -1,6 +1,7 @@
 package cmdutil
 
 import (
+	"fmt"
 	"os"
 	"strings"
 
@@ -46,8 +47,10 @@ func IsAllowedToExecute(when string) (bool, error) {
 	}
 	if got, err := expr.Run(program, whenEnv); err != nil {
 		return false, errors.WithStack(err)
+	} else if got, ok := got.(bool); !ok {
+		return false, fmt.Errorf("expected bool, but got %T", got)
 	} else {
-		return got.(bool), nil
+		return got, nil
 	}
 }
 

cmdutil/when_test.go

@@ -113,25 +113,39 @@ func TestIsAllowedToExecute(t *testing.T) {
 			want:          false,
 			errorContains: "unknown name NoneSuchVariable",
 		},
+		{
+			name:          "Expression produces an integer",
+			envset:        map[string]string{},
+			when:          "123",
+			want:          false,
+			errorContains: "expected bool, but got int",
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			NewWhenEnv = func() *WhenEnv { return &WhenEnv{Env: tt.envset} }
 			got, err := IsAllowedToExecute(tt.when)
+
 			if err != nil {
 				if tt.errorContains != nil {
-					if !strings.Contains(err.Error(), tt.errorContains.(string)) {
-						t.Errorf("Error %v does not contain %s", err, tt.errorContains)
+					if errStr, ok := tt.errorContains.(string); ok {
+						if !strings.Contains(err.Error(), errStr) {
+							t.Errorf("Error %v does not contain %s", err, errStr)
+						}
+					} else {
+						t.Errorf("errorContains should be a string, but got %T", tt.errorContains)
 					}
 				} else {
 					t.Error(err)
 				}
 			} else if tt.errorContains != nil {
 				t.Errorf("Expected an error containing %v", tt.errorContains)
 			}
+
 			if got != tt.want {
 				t.Errorf("got %v\nwant %v", got, tt.want)
 			}
 		})
 	}
+
 }

config/yaml.go

@@ -72,7 +72,9 @@ func (f *Format) UnmarshalYAML(data []byte) error {
 	case []interface{}:
 		values := []string{}
 		for _, vv := range v {
-			values = append(values, vv.(string))
+			if str, ok := vv.(string); ok {
+				values = append(values, str)
+			}
 		}
 		f.HideColumnsWithoutValues = values
 	}

dict/dict.go

@@ -18,7 +18,9 @@ func New() Dict {
 
 func (d *Dict) Lookup(k string) string {
 	if v, ok := d.s.Load(k); ok {
-		return v.(string)
+		if str, ok := v.(string); ok {
+			return str
+		}
 	}
 	return k
 }