Fixups

Author: lukemassa

server/core/config/raw/autodiscover_test.go

@@ -145,11 +145,11 @@ func TestAutoDiscover_Validate(t *testing.T) {
 			input: raw.AutoDiscover{
 				Mode: &autoDiscoverAuto,
 				IgnorePaths: []string{
-					"/foo",
 					"foo",
+					"foo[",
 				},
 			},
-			errContains: String("pattern must not begin with a slash '/'"),
+			errContains: String("invalid pattern: foo["),
 		},
 	}
 	for _, c := range cases {

server/core/config/raw/repo_cfg_test.go

@@ -162,7 +162,7 @@ allowed_regexp_prefixes:
 				Version: Int(3),
 				AutoDiscover: &raw.AutoDiscover{
 					Mode:        &autoDiscoverEnabled,
-					IgnorePaths: []string{"/foo.*/"},
+					IgnorePaths: []string{"foo/*"},
 				},
 				Automerge:     Bool(true),
 				ParallelApply: Bool(true),

server/core/config/valid/repo_cfg.go

@@ -121,8 +121,7 @@ func (r RepoCfg) IsPathIgnoredForAutoDiscover(path string) (bool, error) {
 		matches, err := doublestar.Match(r.AutoDiscover.IgnorePaths[i], path)
 		if err != nil {
 			// Per documentation https://pkg.go.dev/github.com/bmatcuk/doublestar, this only
-			// occurs if the pattern itself is invalid, and we already checked this when
-			// parsing raw config
+			// occurs if the pattern itself is invalid, and we already checked this when parsing raw config
 			return false, errors.Wrap(err, "unexpectedly found invalid ignore pattern (this is a bug, should have been validated at startup)")
 		}
 		if matches {