chore: add support for 'pending' to all formatters

Author: radim

regresql/formatter_console.go

@@ -45,6 +45,13 @@ func (f *ConsoleFormatter) AddResult(r TestResult, w io.Writer) error {
 		f.printWarnings(r.PlanWarnings, w)
 		fmt.Fprintln(w)
 
+	case "pending":
+		fmt.Fprintf(w, "? %s (%.2fs)\n", r.Name, r.Duration)
+		if r.Error != "" {
+			fmt.Fprintf(w, "  %s\n", r.Error)
+		}
+		fmt.Fprintln(w)
+
 	case "skipped":
 		return nil
 	}
@@ -224,11 +231,14 @@ func (f *ConsoleFormatter) printWarnings(warnings []PlanWarning, w io.Writer) {
 
 func (f *ConsoleFormatter) Finish(s *TestSummary, w io.Writer) error {
 	fmt.Fprintln(w)
-	if s.Failed > 0 || s.Skipped > 0 {
+	if s.Failed > 0 || s.Skipped > 0 || s.Pending > 0 {
 		fmt.Fprintf(w, "Results: %d passed, %d failed", s.Passed, s.Failed)
 		if s.Skipped > 0 {
 			fmt.Fprintf(w, ", %d skipped", s.Skipped)
 		}
+		if s.Pending > 0 {
+			fmt.Fprintf(w, ", %d pending", s.Pending)
+		}
 		fmt.Fprintf(w, " (%.2fs)\n", s.Duration)
 	} else {
 		fmt.Fprintf(w, "Results: %d passed (%.2fs)\n", s.Passed, s.Duration)

regresql/formatter_github.go

@@ -97,6 +97,8 @@ func (f *GitHubActionsFormatter) AddResult(r TestResult, w io.Writer) error {
 		}
 	case "skipped":
 		fmt.Fprintf(w, "::warning::%s skipped: %s\n", r.Name, r.Error)
+	case "pending":
+		fmt.Fprintf(w, "::notice::%s pending (no baseline): %s\n", r.Name, r.Error)
 	}
 	return nil
 }
@@ -109,12 +111,18 @@ func (f *GitHubActionsFormatter) Finish(s *TestSummary, w io.Writer) error {
 		if s.Skipped > 0 {
 			fmt.Fprintf(w, ", %d skipped", s.Skipped)
 		}
+		if s.Pending > 0 {
+			fmt.Fprintf(w, ", %d pending", s.Pending)
+		}
 		fmt.Fprintf(w, " (%.2fs)\n", s.Duration)
 	} else {
 		fmt.Fprintf(w, "::notice::All tests passed: %d passed", s.Passed)
 		if s.Skipped > 0 {
 			fmt.Fprintf(w, ", %d skipped", s.Skipped)
 		}
+		if s.Pending > 0 {
+			fmt.Fprintf(w, ", %d pending", s.Pending)
+		}
 		fmt.Fprintf(w, " (%.2fs)\n", s.Duration)
 	}
 

regresql/formatter_json.go

@@ -29,6 +29,7 @@ func (f *JSONFormatter) Finish(s *TestSummary, w io.Writer) error {
 			"passed":   s.Passed,
 			"failed":   s.Failed,
 			"skipped":  s.Skipped,
+			"pending":  s.Pending,
 			"duration": s.Duration,
 		},
 		"tests": formatTests(f.results),

regresql/formatter_junit.go

@@ -22,11 +22,11 @@ type (
 	}
 
 	JUnitTestCase struct {
-		Name      string              `xml:"name,attr"`
-		Classname string              `xml:"classname,attr"`
-		Time      float64             `xml:"time,attr"`
-		Failure   *JUnitFailure       `xml:"failure,omitempty"`
-		Skipped   *JUnitSkipped       `xml:"skipped,omitempty"`
+		Name      string        `xml:"name,attr"`
+		Classname string        `xml:"classname,attr"`
+		Time      float64       `xml:"time,attr"`
+		Failure   *JUnitFailure `xml:"failure,omitempty"`
+		Skipped   *JUnitSkipped `xml:"skipped,omitempty"`
 	}
 
 	JUnitFailure struct {
@@ -107,6 +107,11 @@ func (f *JUnitFormatter) Finish(s *TestSummary, w io.Writer) error {
 			tc.Skipped = &JUnitSkipped{
 				Message: r.Error,
 			}
+		} else if r.Status == "pending" {
+			// pending tests are included in skipped
+			tc.Skipped = &JUnitSkipped{
+				Message: "pending: " + r.Error,
+			}
 		}
 
 		cases = append(cases, tc)
@@ -116,7 +121,7 @@ func (f *JUnitFormatter) Finish(s *TestSummary, w io.Writer) error {
 		Name:     "regresql",
 		Tests:    s.Total,
 		Failures: s.Failed,
-		Skipped:  s.Skipped,
+		Skipped:  s.Skipped + s.Pending, // Include pending in skipped count
 		Time:     s.Duration,
 		Cases:    cases,
 	}

regresql/formatters.go

@@ -11,7 +11,7 @@ type (
 	TestResult struct {
 		Name     string
 		Type     string // "output", "cost", or "plan_quality"
-		Status   string // "passed", "failed", "skipped", "warning"
+		Status   string // "passed", "failed", "skipped", "warning", "pending"
 		Duration float64
 		Error    string