From 375a469af45d44d70b9374dc0da8af717a107cf4 Mon Sep 17 00:00:00 2001 From: Finn Bacall Date: Tue, 21 Jul 2026 10:40:54 +0100 Subject: [PATCH 1/2] Put test results in a directory So they can be symlinked if necessary --- app/models/concerns/has_test_job.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/concerns/has_test_job.rb b/app/models/concerns/has_test_job.rb index a50e6ce70..8cea2aa6f 100644 --- a/app/models/concerns/has_test_job.rb +++ b/app/models/concerns/has_test_job.rb @@ -53,7 +53,7 @@ def test_job_id_key def test_results_path path_id = Rails.env.test? ? "fakeid_#{model_name.singular}_#{id}" : test_job_id - File.join(Rails.root, 'tmp', "test_results_#{path_id}.yml") + Rails.root.join('tmp', 'source_test_results').tap { |d| d.mkdir unless d.exist? }.join("test_results_#{path_id}.yml") end def deep_hashify(params) From 8d4205ab4a236361f8d87adeebcd09db47c471e1 Mon Sep 17 00:00:00 2001 From: Finn Bacall Date: Fri, 24 Jul 2026 17:35:24 +0100 Subject: [PATCH 2/2] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- app/models/concerns/has_test_job.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/models/concerns/has_test_job.rb b/app/models/concerns/has_test_job.rb index 8cea2aa6f..cc3769a11 100644 --- a/app/models/concerns/has_test_job.rb +++ b/app/models/concerns/has_test_job.rb @@ -53,7 +53,9 @@ def test_job_id_key def test_results_path path_id = Rails.env.test? ? "fakeid_#{model_name.singular}_#{id}" : test_job_id - Rails.root.join('tmp', 'source_test_results').tap { |d| d.mkdir unless d.exist? }.join("test_results_#{path_id}.yml") + dir = Rails.root.join('tmp', 'source_test_results') + dir.mkpath + dir.join("test_results_#{path_id}.yml") end def deep_hashify(params)