Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't call discourse hub during user destroy if hub integration is disabled

This commit is contained in:
Neil Lalonde 2013-04-29 11:38:43 -04:00
parent dc07563c0d
commit eb151d440b
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/user_destroyer.rb
View File

@ -20,7 +20,7 @@ class UserDestroyer
user.destroy.tap do |u| user.destroy.tap do |u|
if u if u
AdminLogger.new(@admin).log_user_deletion(user) AdminLogger.new(@admin).log_user_deletion(user)
DiscourseHub.unregister_nickname(user.username) DiscourseHub.unregister_nickname(user.username) if SiteSetting.call_discourse_hub?
MessageBus.publish "/file-change", ["refresh"], user_ids: [user.id] MessageBus.publish "/file-change", ["refresh"], user_ids: [user.id]
end end
end end

9
spec/components/user_destroyer_spec.rb
View File

@ -86,10 +86,17 @@ describe UserDestroyer do
destroy destroy
end end
it 'should unregister the nickname as the discourse hub' do it 'should unregister the nickname as the discourse hub if hub integration is enabled' do
SiteSetting.stubs(:call_discourse_hub?).returns(true)
DiscourseHub.expects(:unregister_nickname).with(@user.username) DiscourseHub.expects(:unregister_nickname).with(@user.username)
destroy destroy
end end
it 'should not try to unregister the nickname as the discourse hub if hub integration is disabled' do
SiteSetting.stubs(:call_discourse_hub?).returns(false)
DiscourseHub.expects(:unregister_nickname).never
destroy
end
end end
context 'and destroy fails' do context 'and destroy fails' do