Add config test to mergelists

config/BUILD.bazel

@@ -53,6 +53,7 @@ filegroup(
         "//config/prow-staging:all-srcs",
         "//config/tests/jobs:all-srcs",
         "//config/tests/lint:all-srcs",
+        "//config/tests/mergelists:all-srcs",
         "//config/tests/testgrids:all-srcs",
     ],
     tags = ["automanaged"],

config/tests/mergelists/BUILD.bazel

@@ -0,0 +1,26 @@
+package(default_visibility = ["//visibility:public"])
+
+load("@io_bazel_rules_go//go:def.bzl", "go_test")
+
+go_test(
+    name = "go_default_test",
+    srcs = ["mergelist_test.go"],
+    args = ["-list='$(locations //config/mergelists)'"],
+    data = ["//config/mergelists"],
+    rundir = ".",
+    deps = ["@com_github_googlecloudplatform_testgrid//pkg/merger:go_default_library"],
+)
+
+filegroup(
+    name = "package-srcs",
+    srcs = glob(["**"]),
+    tags = ["automanaged"],
+    visibility = ["//visibility:private"],
+)
+
+filegroup(
+    name = "all-srcs",
+    srcs = [":package-srcs"],
+    tags = ["automanaged"],
+    visibility = ["//visibility:public"],
+)

config/tests/mergelists/mergelist_test.go

@@ -0,0 +1,46 @@
+/*
+Copyright 2021 The Kubernetes Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package mergelist_test
+
+import (
+	"flag"
+	"io/ioutil"
+	"strings"
+	"testing"
+
+	"github.com/GoogleCloudPlatform/testgrid/pkg/merger"
+)
+
+var lists = flag.String("list", "", "Space-delimited list of mergelists to test")
+
+func TestMergelist(t *testing.T) {
+	files := strings.Split(*lists, " ")
+
+	for _, filename := range files {
+		t.Run(filename, func(t *testing.T) {
+			file, err := ioutil.ReadFile(filename)
+			if err != nil {
+				t.Errorf("Can't read file: %v", err)
+			}
+
+			_, err = merger.ParseAndCheck(file)
+			if err != nil {
+				t.Error(err)
+			}
+		})
+	}
+}

prow/spyglass/lenses/junit/lens.go

@@ -158,7 +158,7 @@ func (lens Lens) getJvd(artifacts []api.Artifact) JVD {
 				resultChan <- result
 				return
 			}
-			var suites junit.Suites
+			var suites *junit.Suites
 			suites, result.err = junit.Parse(contents)
 			if result.err != nil {
 				logrus.WithError(result.err).WithField("artifact", artifact.CanonicalLink()).Info("Error parsing junit file.")

testgrid/cmd/configurator/main.go

@@ -198,7 +198,7 @@ func write(ctx context.Context, client *storage.Client, path string, bytes []byt
 func doOneshot(ctx context.Context, client *storage.Client, opt options, prowConfigAgent *prowConfig.Agent) error {
 
 	// Read Data Sources: Default, YAML configs, Prow Annotations
-	c, err := yamlcfg.ReadConfig(opt.inputs, opt.defaultYAML)
+	c, err := yamlcfg.ReadConfig(opt.inputs, opt.defaultYAML, false)
 	if err != nil {
 		return fmt.Errorf("could not read testgrid config: %v", err)
 	}