about summary refs log tree commit diff
path: root/spec/workers/pubsubhubbub
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2017-05-14 03:22:48 +0200
committerGitHub <noreply@github.com>2017-05-14 03:22:48 +0200
commit657496b5a9488b904166c33764500b364e024679 (patch)
tree6776c7ca1f9878cf0d9e4b272a07b969bc89fe4d /spec/workers/pubsubhubbub
parentfd03a3d95731d479f19bc4894746027097b9e2a3 (diff)
Do not cancel PuSH subscriptions after encountering "permanent" error… (#3046)
* Do not cancel PuSH subscriptions after encountering "permanent" error response

After talking with MMN about it, turns out some servers/php setups do
return 4xx errors while rebooting, so this anti-feature that was meant
to take load off of the hub is doing more harm than good in terms of
breaking subscriptions

* Update delivery_worker.rb
Diffstat (limited to 'spec/workers/pubsubhubbub')
-rw-r--r--spec/workers/pubsubhubbub/delivery_worker_spec.rb9
1 files changed, 0 insertions, 9 deletions
diff --git a/spec/workers/pubsubhubbub/delivery_worker_spec.rb b/spec/workers/pubsubhubbub/delivery_worker_spec.rb
index ec1e319d5..081dfa41c 100644
--- a/spec/workers/pubsubhubbub/delivery_worker_spec.rb
+++ b/spec/workers/pubsubhubbub/delivery_worker_spec.rb
@@ -22,15 +22,6 @@ describe Pubsubhubbub::DeliveryWorker do
       expect(subscription.reload.last_successful_delivery_at).to be_within(2).of(2.days.ago)
     end
 
-    it 'destroys subscription when request fails permanently' do
-      subscription = Fabricate(:subscription)
-
-      stub_request_to_respond_with(subscription, 404)
-      subject.perform(subscription.id, payload)
-
-      expect { subscription.reload }.to raise_error(ActiveRecord::RecordNotFound)
-    end
-
     it 'raises when request fails' do
       subscription = Fabricate(:subscription)