about summary refs log tree commit diff
path: root/spec
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2018-11-24 20:48:50 +0100
committerGitHub <noreply@github.com>2018-11-24 20:48:50 +0100
commit0eaf6d7693ba1f5568c9b857a306e01250f2f714 (patch)
tree256350b7083935d71287c72d6adb5dd09ffc9d1e /spec
parent87a43274f12bcdde3475b2c187cc203a42cb738d (diff)
Sort self-replies to the top of descendants (#9320)
Fix #6463
Diffstat (limited to 'spec')
-rw-r--r--spec/models/concerns/status_threading_concern_spec.rb10
1 files changed, 10 insertions, 0 deletions
diff --git a/spec/models/concerns/status_threading_concern_spec.rb b/spec/models/concerns/status_threading_concern_spec.rb
index e5736a307..94c2d5fc2 100644
--- a/spec/models/concerns/status_threading_concern_spec.rb
+++ b/spec/models/concerns/status_threading_concern_spec.rb
@@ -118,5 +118,15 @@ describe StatusThreadingConcern do
       viewer.block_domain!('example.com')
       expect(status.descendants(4, viewer)).to_not include(reply2)
     end
+
+    it 'promotes self-replies to the top while leaving the rest in order' do
+      a = Fabricate(:status, account: alice)
+      d = Fabricate(:status, account: jeff, thread: a)
+      e = Fabricate(:status, account: bob, thread: d)
+      c = Fabricate(:status, account: alice, thread: a)
+      f = Fabricate(:status, account: bob, thread: c)
+
+      expect(a.descendants(20)).to eq [c, d, e, f]
+    end
   end
 end