Set non-zero exit code for build/run errors when using -set-exit-code

Author: jstemmer

go-junit-report.go

@@ -102,7 +102,7 @@ func main() {
 	}
 	fmt.Fprintf(out, "\n")
 
-	if *setExitCode && report.HasFailures() {
+	if *setExitCode && !report.IsSuccessful() {
 		os.Exit(1)
 	}
 }

pkg/gtr/gtr.go

@@ -19,15 +19,18 @@ type Report struct {
 	Packages []Package
 }
 
-func (r *Report) HasFailures() bool {
+func (r *Report) IsSuccessful() bool {
 	for _, pkg := range r.Packages {
+		if pkg.BuildError.Name != "" || pkg.RunError.Name != "" {
+			return false
+		}
 		for _, t := range pkg.Tests {
-			if t.Result == Fail {
-				return true
+			if t.Result != Pass && t.Result != Skip {
+				return false
 			}
 		}
 	}
-	return false
+	return true
 }
 
 type Package struct {