From 34b2157b8ca5997fee3c24c639eeddaf5e6d4930 Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Tue, 19 Feb 2019 12:29:50 -0500 Subject: [PATCH] FIX: Slow test (deadlock) It was getting caught in a `DistributedMutex` deadlock (twice!), which meant this test was taking 120s to run. I'm not sure why queue jobs was turned off here, because when I turn it on the test passes and takes <2s instead. --- spec/components/cooked_post_processor_spec.rb | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/spec/components/cooked_post_processor_spec.rb b/spec/components/cooked_post_processor_spec.rb index 938c9c8ed55..4f5e280f699 100644 --- a/spec/components/cooked_post_processor_spec.rb +++ b/spec/components/cooked_post_processor_spec.rb @@ -1228,12 +1228,10 @@ describe CookedPostProcessor do end it "works only on new posts" do - SiteSetting.queue_jobs = false - - hidden = Fabricate(:post, topic: topic, hidden: true, raw: "this is the second post after") - small_action = Fabricate(:post, topic: topic, post_type: Post.types[:small_action]) - + Fabricate(:post, topic: topic, hidden: true, raw: "this is the second post after") + Fabricate(:post, topic: topic, post_type: Post.types[:small_action]) reply = PostCreator.create!(topic.user, topic_id: topic.id, raw: raw) + CookedPostProcessor.new(reply).post_process expect(reply.raw).to eq(raw)