Merge pull request #1594 from prymitive/more-test

fix(tests): add more test coverage

internal/filters/filter_test.go

@@ -615,6 +615,20 @@ func TestFilters(t *testing.T) {
 				t.Errorf("[%s] GetRawText() returned %#v != %s passed as the expression", ft.Expression, f.GetRawText(), ft.Expression)
 			}
 		}
+		if !f.GetIsValid() {
+			func() {
+				didPanic := false
+				defer func() {
+					if r := recover(); r != nil {
+						didPanic = true
+					}
+				}()
+				f.Match(&alert, 0)
+				if !didPanic {
+					t.Errorf("[%s] Match() on invalid filter didn't cause panic", ft.Expression)
+				}
+			}()
+		}
 	}
 }
 

internal/slices/slices.go

@@ -29,14 +29,8 @@ func StringInSlice(stringArray []string, value string) bool {
 func StringSliceToSHA1(stringArray []string) (string, error) {
 	h := sha1.New()
 	for _, s := range stringArray {
-		_, err := h.Write([]byte(s))
-		if err != nil {
-			return "", err
-		}
-		_, err = h.Write([]byte("\n"))
-		if err != nil {
-			return "", err
-		}
+		_, _ = h.Write([]byte(s))
+		_, _ = h.Write([]byte("\n"))
 	}
 	return fmt.Sprintf("%x", h.Sum(nil)), nil
 }

internal/verprobe/verprobe_test.go

@@ -0,0 +1,54 @@
+package verprobe_test
+
+import (
+	"bytes"
+	"testing"
+
+	"github.com/prymitive/karma/internal/verprobe"
+)
+
+func TestDetect(t *testing.T) {
+	type testCaseT struct {
+		metrics string
+		version string
+		isError bool
+	}
+
+	testCases := []testCaseT{
+		{
+			metrics: "",
+			version: "",
+		},
+		{
+			metrics: "xxxx",
+			version: "",
+			isError: true,
+		},
+		{
+			metrics: "alertmanager_build_info 1\n",
+			version: "",
+		},
+		{
+			metrics: "alertmanager_build_info{foo=\"bar\"} 1\n",
+			version: "",
+		},
+		{
+			metrics: "alertmanager_build_info{version=\"1.2.3\"} 1\n",
+			version: "1.2.3",
+		},
+	}
+
+	for _, testCase := range testCases {
+		r := bytes.NewBufferString(testCase.metrics)
+		version, err := verprobe.Detect(r)
+		isError := err != nil
+		if isError != testCase.isError {
+			t.Errorf("Version probe error=%q, expected isError=%v", err, testCase.isError)
+		}
+		if !isError {
+			if version != testCase.version {
+				t.Errorf("Version mismatch, got %q, expected %q", version, testCase.version)
+			}
+		}
+	}
+}