Allow gotestsum to rerun tests that panic

cmd/main.go

@@ -237,7 +237,8 @@ func run(opts *options) error {
 	if exitErr == nil || opts.rerunFailsMaxAttempts == 0 {
 		return finishRun(opts, exec, exitErr)
 	}
-	if err := hasErrors(exitErr, exec); err != nil {
+	fmt.Printf("Executor has %#v", exec.FullPackages())
+	if err := hasErrors(exitErr, exec, true); err != nil {
 		return finishRun(opts, exec, err)
 	}
 

cmd/rerunfails.go

@@ -81,7 +81,7 @@ func rerunFailed(ctx context.Context, opts *options, scanConfig testjson.ScanCon
 			if exitErr != nil {
 				nextRec.lastErr = exitErr
 			}
-			if err := hasErrors(exitErr, scanConfig.Execution); err != nil {
+			if err := hasErrors(exitErr, scanConfig.Execution, true); err != nil {
 				return err
 			}
 		}
@@ -93,14 +93,17 @@ func rerunFailed(ctx context.Context, opts *options, scanConfig testjson.ScanCon
 // startGoTestFn is a shim for testing
 var startGoTestFn = startGoTest
 
-func hasErrors(err error, exec *testjson.Execution) error {
+func hasErrors(err error, exec *testjson.Execution, ignorePanic bool) error {
 	switch {
 	case len(exec.Errors()) > 0:
 		return fmt.Errorf("rerun aborted because previous run had errors")
 	// Exit code 0 and 1 are expected.
 	case ExitCodeWithDefault(err) > 1:
 		return fmt.Errorf("unexpected go test exit code: %v", err)
 	case exec.HasPanic():
+		if ignorePanic {
+			return nil
+		}
 		return fmt.Errorf("rerun aborted because previous run had a suspected panic and some test may not have run")
 	default:
 		return nil

testjson/execution.go

@@ -562,6 +562,21 @@ func (e *Execution) HasPanic() bool {
 	return false
 }
 
+func (e *Execution) FullPackages() []Package {
+	ps := []Package{}
+	for _, p := range e.packages {
+		ps = append(ps, *p)
+	}
+	return ps
+}
+
+func (e *Execution) PrintPackages() {
+	fmt.Println("Print packages")
+	for k, p := range e.packages {
+		fmt.Printf("Package '%s' has content %v\n", k, *p)
+	}
+}
+
 func (e *Execution) end() []TestEvent {
 	e.done = true
 	var result []TestEvent // nolint: prealloc