Apply suggestions from code review

Cleanup `failFunc` to make it easier to read.

Co-authored-by: ka3de <daniel.jimenez@grafana.com>

k6ext/panic.go

@@ -19,21 +19,25 @@ import (
 // to be used when an error will occur in all iterations,
 // so it's permanent.
 func Abort(ctx context.Context, format string, a ...any) {
-	sharedPanic(ctx, func(rt *goja.Runtime, a ...any) {
+	failFunc := func(rt *goja.Runtime, a ...any) {
 		reason := fmt.Errorf(format, a...).Error()
 		rt.Interrupt(&errext.InterruptError{Reason: reason})
-	}, a...)
+	}
+	sharedPanic(ctx, failFunc, a...)
 }
 
 // Panic will cause a panic with the given error which will stop
 // the current iteration. Before panicking, it will find the
 // browser process from the context and kill it if it still exists.
 // TODO: test.
 func Panic(ctx context.Context, format string, a ...any) {
-	sharedPanic(ctx, func(rt *goja.Runtime, a ...any) { k6common.Throw(rt, fmt.Errorf(format, a...)) }, a...)
+	failFunc := func(rt *goja.Runtime, a ...any) {
+		k6common.Throw(rt, fmt.Errorf(format, a...))
+	}
+	sharedPanic(ctx, failFunc, a...)
 }
 
-func sharedPanic(ctx context.Context, fail func(rt *goja.Runtime, a ...any), a ...any) {
+func sharedPanic(ctx context.Context, failFunc func(rt *goja.Runtime, a ...any), a ...any) {
 	rt := Runtime(ctx)
 	if rt == nil {
 		// this should never happen unless a programmer error
@@ -49,7 +53,7 @@ func sharedPanic(ctx context.Context, fail func(rt *goja.Runtime, a ...any), a .
 			a[len(a)-1] = &UserFriendlyError{Err: err}
 		}
 	}
-	defer fail(rt, a...)
+	defer failFunc(rt, a...)
 
 	// TODO: Remove this after moving k6ext.Panic into the mapping layer.
 	pidder, ok := GetVU(ctx).(interface {