fixup: Add RecoverAndRethrow as per review feedback

Author: maru-ava

tests/antithesis/avalanchego/main.go

@@ -170,7 +170,7 @@ func (w *workload) run(ctx context.Context) {
 	tc := w.newTestContext(ctx)
 	// Any assertion failure from this test context will result in process exit due to the
 	// panic being rethrown. This ensures that failures in test setup are fatal.
-	defer tc.Recover(true /* rethrow */)
+	defer tc.RecoverAndRethrow()
 
 	xAVAX, pAVAX := e2e.GetWalletBalances(tc, w.wallet)
 	assert.Reachable("wallet starting", map[string]any{
@@ -215,7 +215,7 @@ func (w *workload) run(ctx context.Context) {
 func (w *workload) executeTest(ctx context.Context) {
 	tc := w.newTestContext(ctx)
 	// Panics will be recovered without being rethrown, ensuring that test failures are not fatal.
-	defer tc.Recover(false /* rethrow */)
+	defer tc.Recover()
 	require := require.New(tc)
 
 	val, err := rand.Int(rand.Reader, big.NewInt(5))

tests/antithesis/xsvm/main.go

@@ -151,7 +151,7 @@ func (w *workload) run(ctx context.Context) {
 	)
 	// Any assertion failure from this test context will result in process exit due to the
 	// panic being rethrown. This ensures that failures in test setup are fatal.
-	defer tc.Recover(true /* rethrow */)
+	defer tc.RecoverAndRethrow()
 	require := require.New(tc)
 
 	uri := w.uris[w.id%len(w.uris)]

tests/simple_test_context.go

@@ -102,11 +102,25 @@ func (tc *SimpleTestContext) RecoverAndExit() {
 	}
 }
 
+// Recover is intended to be deferred in a function executing a test whose
+// assertions may result in panics. Such a panic is intended to be recovered to
+// allow cleanup functions to be called before execution continues.
+func (tc *SimpleTestContext) Recover() {
+	tc.recover(false /* rethrow */)
+}
+
+// RecoverAndRethrow is intended to be deferred in a function executing a test
+// whose assertions may result in panics.  Such a panic is intended to be recovered
+// to allow cleanup functions to be called before the panic is rethrown.
+func (tc *SimpleTestContext) RecoverAndRethrow() {
+	tc.recover(true /* rethrow */)
+}
+
 // Recover is intended to be deferred in a function executing a test
 // whose assertions may result in panics. Such a panic is intended to
 // be recovered to allow cleanup functions to be called. A panic can
 // be optionally rethrown by setting `rethrow` to true.
-func (tc *SimpleTestContext) Recover(rethrow bool) {
+func (tc *SimpleTestContext) recover(rethrow bool) {
 	// Recover from test failure
 	var panicData any
 	if panicData = recover(); panicData != nil {