about summary refs log tree commit diff
path: root/spec/services
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2017-06-11 17:13:43 +0200
committerGitHub <noreply@github.com>2017-06-11 17:13:43 +0200
commitce812466c749c6abb45e57efe24b01e0f511e0b5 (patch)
treefa4f58c7fc672fbe973227ee19fede42717f93b3 /spec/services
parent47bf7a8047ce59b899d147e4483168f9852eeb7c (diff)
Fix removal of status sending the original status to mentioned users instead of delete Salmon (#3672)
* Fix removal of status sending the original status to mentioned users instead
of delete Salmon, add test

* Create remove_status_service_spec.rb
Diffstat (limited to 'spec/services')
-rw-r--r--spec/services/remove_status_service_spec.rb40
1 files changed, 40 insertions, 0 deletions
diff --git a/spec/services/remove_status_service_spec.rb b/spec/services/remove_status_service_spec.rb
new file mode 100644
index 000000000..a3bce7613
--- /dev/null
+++ b/spec/services/remove_status_service_spec.rb
@@ -0,0 +1,40 @@
+require 'rails_helper'
+
+RSpec.describe RemoveStatusService do
+  subject { RemoveStatusService.new }
+
+  let!(:alice)  { Fabricate(:account) }
+  let!(:bob)    { Fabricate(:account, username: 'bob', domain: 'example.com', salmon_url: 'http://example.com/salmon') }
+  let!(:jeff)   { Fabricate(:account) }
+
+  before do
+    stub_request(:post, 'http://example.com/push').to_return(status: 200, body: '', headers: {})
+    stub_request(:post, 'http://example.com/salmon').to_return(status: 200, body: '', headers: {})
+
+    Fabricate(:subscription, account: alice, callback_url: 'http://example.com/push', confirmed: true, expires_at: 30.days.from_now)
+    jeff.follow!(alice)
+    @status = PostStatusService.new.call(alice, 'Hello @bob@example.com')
+    subject.call(@status)
+  end
+
+  it 'removes status from author\'s home feed' do
+    expect(Feed.new(:home, alice).get(10)).to_not include(@status.id)
+  end
+
+  it 'removes status from local follower\'s home feed' do
+    expect(Feed.new(:home, jeff).get(10)).to_not include(@status.id)
+  end
+
+  it 'sends PuSH update to PuSH subscribers' do
+    expect(a_request(:post, 'http://example.com/push').with { |req|
+      req.body.match(TagManager::VERBS[:delete])
+    }).to have_been_made
+  end
+
+  it 'sends Salmon slap to previously mentioned users' do
+    expect(a_request(:post, "http://example.com/salmon").with { |req|
+      xml = OStatus2::Salmon.new.unpack(req.body)
+      xml.match(TagManager::VERBS[:delete])
+    }).to have_been_made.once
+  end
+end