From c497b512c92cae6b160fae2f4997bd0a3577dd19 Mon Sep 17 00:00:00 2001 From: MAEDA Go Date: Tue, 3 Nov 2026 17:23:12 +0900 Subject: [PATCH] Stop AttachmentTest from running without transactional fixtures attachment_transaction_test.rb reopened AttachmentTest instead of defining its own class, so `use_transactional_tests = false` and its `setup` applied to every test in attachment_test.rb when both files were loaded, as in a full test run. As a result, all AttachmentTest tests reloaded every fixture for each test (about 0.6 seconds per test), and the `Attachment.clear_markdownized_previews` call in AttachmentTest#setup was skipped. Running either file alone did not show the problem. Rename the class to AttachmentTransactionTest. --- test/unit/attachment_transaction_test.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/unit/attachment_transaction_test.rb b/test/unit/attachment_transaction_test.rb index 5b7543856..1a158083c 100644 --- a/test/unit/attachment_transaction_test.rb +++ b/test/unit/attachment_transaction_test.rb @@ -19,7 +19,7 @@ require_relative '../test_helper' -class AttachmentTest < ActiveSupport::TestCase +class AttachmentTransactionTest < ActiveSupport::TestCase self.use_transactional_tests = false def setup -- 2.55.0