More robust test for ParseConfig

config_test.go

@@ -4,20 +4,14 @@ import (
 	"testing"
 )
 
-var parseTests = []struct {
-	index        int
-	expectedPath string
-	expectedRun  string
-}{
-	{
-		0,
-		"/tmp/foo",
-		"/usr/local/bar/foobar",
+type expectedRule map[string]string
+
+var expectedRules = map[string]expectedRule{
+	"/tmp/foo": map[string]string{
+		"run": "/usr/local/bar/foobar",
 	},
-	{
-		1,
-		"/tmp/bar",
-		"/usr/local/bar/barfoo",
+	"/tmp/bar": map[string]string{
+		"run": "/usr/local/bar/barfoo",
 	},
 }
 
@@ -26,17 +20,16 @@ func TestParseConfig(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	if len(rules) != 2 {
-		t.Fatal("Did not include all rules")
-	}
 
-	for _, pt := range parseTests {
-		actualPath := rules[pt.index].Path
-		actualRun := rules[pt.index].Run
-
-		if actualPath != pt.expectedPath && actualRun != pt.expectedRun {
-			t.Errorf("ACTUALPATH %v - EXPECTEDPATH %v, ACTUALRUN %v - EXPECTEDRUN %v", actualPath, pt.expectedPath, actualRun, pt.expectedRun)
+	for _, rule := range rules {
+		expected, ok := expectedRules[rule.Path]
+		if !ok {
+			t.Error("Rule not found")
 			continue
 		}
+		if rule.Run != expected["run"] {
+			t.Error("Wrong Run. Expected: %v Actual: %v", expected["run"],
+				rule.Run)
+		}
 	}
 }