From 1200ef700dd9ca3e4c7c59792cbad1db8a17b823 Mon Sep 17 00:00:00 2001
From: claw <claw@weiker.me>
Date: Tue, 12 May 2026 15:22:27 -0700
Subject: [PATCH] test: use per-subtest TempDir in TestYAMLEmptyFileRejection

Move t.TempDir() inside each subtest for idiomatic test isolation,
as suggested by reviewers.
---
 review/persona_test.go | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/review/persona_test.go b/review/persona_test.go
index 6154831..c928d74 100644
--- a/review/persona_test.go
+++ b/review/persona_test.go
@@ -490,8 +490,6 @@ func TestYAMLDeeplyNestedRejection(t *testing.T) {
 }
 
 func TestYAMLEmptyFileRejection(t *testing.T) {
-	dir := t.TempDir()
-
 	tests := []struct {
 		name    string
 		content string
@@ -503,6 +501,7 @@ func TestYAMLEmptyFileRejection(t *testing.T) {
 
 	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {
+			dir := t.TempDir()
 			path := filepath.Join(dir, tc.name+".yaml")
 			if err := os.WriteFile(path, []byte(tc.content), 0644); err != nil {
 				t.Fatalf("failed to write test file: %v", err)