feat: improve panic stack trace (#96)

Signed-off-by: Tronje Krop <[email protected]>

.github/workflows/build.yaml

@@ -8,6 +8,7 @@ jobs:
       uses: actions/setup-go@v5
       with:
         go-version: 1.23.2
+        cache: false
 
     - name: Checkout code
       uses: actions/checkout@v4

Makefile

@@ -12,7 +12,7 @@ export GOPATH ?= $(shell $(GO) env GOPATH)
 export GOBIN ?= $(GOPATH)/bin
 
 # Setup go-make version to use desired build and config scripts.
-GOMAKE_DEP ?= github.com/tkrop/[email protected].106
+GOMAKE_DEP ?= github.com/tkrop/[email protected].108
 INSTALL_FLAGS ?= -mod=readonly -buildvcs=auto
 # Request targets from go-make targets target.
 TARGETS := $(shell command -v $(GOBIN)/go-make >/dev/null || \

VERSION

@@ -1 +1 @@
-0.0.21
+0.0.22

test/context.go

@@ -2,9 +2,9 @@ package test
 
 import (
 	"math"
+	"regexp"
 	"runtime"
 	"runtime/debug"
-	"strings"
 	gosync "sync"
 	"sync/atomic"
 	"testing"
@@ -450,10 +450,10 @@ func (t *Context) Failed() bool {
 	return t.failed.Load()
 }
 
-// Offset fr original stack in case of panic handling.
-//
-// TODO: check offset or/and find a better solution to handle panic stack.
-const panicOriginStackOffset = 10
+// regexPanic is a regular expression to extract the actual important panic
+// stack trace removing the distracting parts from the test framework.
+var regexPanic = regexp.MustCompile(`(?m)\nruntime\/debug\.Stack\(\)` +
+	`(\n|.)*runtime\/panic\.go:([0-9]+)[^\n]*\n`)
 
 // Panic handles failure notifications of panics that also abort the test
 // execution immediately.
@@ -469,10 +469,8 @@ func (t *Context) Panic(arg any) {
 	defer t.unlock()
 
 	if t.expect == Success {
-		stack := strings.SplitN(string(debug.Stack()),
-			"\n", panicOriginStackOffset)
-		t.Fatalf("panic: %v\n%s\n%s", arg, stack[0],
-			stack[panicOriginStackOffset-1])
+		stack := regexPanic.Split(string(debug.Stack()), -1)
+		t.Fatalf("panic: %v\n%s\n%s", arg, stack[0], stack[1])
 	} else if t.reporter != nil {
 		t.reporter.Panic(arg)
 	}

test/context_test.go

@@ -130,7 +130,6 @@ type TestDeadlineParam struct {
 	failure            test.Expect
 }
 
-// TODO: fix unit test for deadline!!
 var TestDeadlineParams = map[string]TestDeadlineParam{
 	"failed": {
 		time:    0,