From 5a569d4ed1cecd6cee41fce9c954c0ab79486fce Mon Sep 17 00:00:00 2001 From: Mathieu Fenniak Date: Thu, 21 Aug 2025 09:21:20 -0600 Subject: [PATCH] adopt t.Context() now that we're on go1.24; remove per-test explicit timeout --- internal/app/run/runner_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/app/run/runner_test.go b/internal/app/run/runner_test.go index 9db43b4e..92e96452 100644 --- a/internal/app/run/runner_test.go +++ b/internal/app/run/runner_test.go @@ -192,7 +192,7 @@ func TestRunnerCacheConfiguration(t *testing.T) { } t.Run("Cache accessible", func(t *testing.T) { - ctx, cancel := context.WithTimeout(context.Background(), 3*time.Minute) + ctx, cancel := context.WithCancel(t.Context()) defer cancel() // Step 1: Populate shared cache with push workflow @@ -236,7 +236,7 @@ jobs: }) t.Run("PR cache pollution prevented", func(t *testing.T) { - ctx, cancel := context.WithTimeout(context.Background(), 3*time.Minute) + ctx, cancel := context.WithCancel(t.Context()) defer cancel() // Step 1: Populate shared cache with push workflow