From 5bea42412e41b1121af04dbe6ef5c9bed22219b8 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Mon, 8 May 2017 18:45:02 -0400 Subject: Services specs for subscribe and unsubscribe (#2928) * Add specs for unsubscribe service * Fix non existent methods in unsubscribe service * Clean up status handling in subscribe service --- spec/services/unsubscribe_service_spec.rb | 37 +++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100644 spec/services/unsubscribe_service_spec.rb (limited to 'spec/services/unsubscribe_service_spec.rb') diff --git a/spec/services/unsubscribe_service_spec.rb b/spec/services/unsubscribe_service_spec.rb new file mode 100644 index 000000000..c81772037 --- /dev/null +++ b/spec/services/unsubscribe_service_spec.rb @@ -0,0 +1,37 @@ +require 'rails_helper' + +RSpec.describe UnsubscribeService do + let(:account) { Fabricate(:account, username: 'bob', domain: 'example.com', hub_url: 'http://hub.example.com') } + subject { UnsubscribeService.new } + + it 'removes the secret and resets expiration on account' do + stub_request(:post, 'http://hub.example.com/').to_return(status: 204) + subject.call(account) + account.reload + + expect(account.secret).to be_blank + expect(account.subscription_expires_at).to be_blank + end + + it 'logs error on subscription failure' do + logger = stub_logger + stub_request(:post, 'http://hub.example.com/').to_return(status: 404) + subject.call(account) + + expect(logger).to have_received(:debug).with(/unsubscribe for bob@example.com failed/) + end + + it 'logs error on connection failure' do + logger = stub_logger + stub_request(:post, 'http://hub.example.com/').to_raise(HTTP::Error) + subject.call(account) + + expect(logger).to have_received(:debug).with(/PuSH subscription request for bob@example.com could not be made due to HTTP or SSL error/) + end + + def stub_logger + double(debug: nil).tap do |logger| + allow(Rails).to receive(:logger).and_return(logger) + end + end +end -- cgit