about summary refs log tree commit diff
path: root/spec
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2017-06-22 02:38:50 +0200
committerGitHub <noreply@github.com>2017-06-22 02:38:50 +0200
commit0190aac240fd804180a56b5fe174526a9e4c3f6d (patch)
tree82780606af6668bea6b72ee0672af358b8266588 /spec
parentcc382c5006ed5c4c18c3547150b214807b217c7b (diff)
Fix regression from #3842 (#3892)
* Fix regression from #3842

Simplify the query by omitting all direct statuses. Private statuses
are allowed because they are from accounts we are following (so
by definition)

Resolves #3887 (alternative)

* Adjust test
Diffstat (limited to 'spec')
-rw-r--r--spec/models/status_spec.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/spec/models/status_spec.rb b/spec/models/status_spec.rb
index a3ced1abc..0b90205ee 100644
--- a/spec/models/status_spec.rb
+++ b/spec/models/status_spec.rb
@@ -201,17 +201,17 @@ RSpec.describe Status, type: :model do
       expect(@results).to include(@self_status)
     end
 
-    it 'includes direct statuses from self' do
-      expect(@results).to include(@self_direct_status)
+    it 'does not include direct statuses from self' do
+      expect(@results).to_not include(@self_direct_status)
     end
 
     it 'includes statuses from followed' do
       expect(@results).to include(@followed_status)
     end
 
-    it 'includes direct statuses mentioning recipient from followed' do
+    it 'does not include direct statuses mentioning recipient from followed' do
       Fabricate(:mention, account: account, status: @followed_direct_status)
-      expect(@results).to include(@followed_direct_status)
+      expect(@results).to_not include(@followed_direct_status)
     end
 
     it 'does not include direct statuses not mentioning recipient from followed' do