From f48cb3eb170314cba1938522c0da44af21d47e83 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Sat, 29 Apr 2017 18:25:38 -0400 Subject: More coverage yes more even more (#2627) * Add coverage for admin/confirmations controller * Coverage for statuses controller show action * Add coverage for admin/domain_blocks controller * Add coverage for settings/profiles#update --- app/controllers/admin/domain_blocks_controller.rb | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'app/controllers/admin/domain_blocks_controller.rb') diff --git a/app/controllers/admin/domain_blocks_controller.rb b/app/controllers/admin/domain_blocks_controller.rb index d40ec829b..72bf1cd0d 100644 --- a/app/controllers/admin/domain_blocks_controller.rb +++ b/app/controllers/admin/domain_blocks_controller.rb @@ -27,7 +27,7 @@ module Admin def destroy @domain_block = DomainBlock.find(params[:id]) - UnblockDomainService.new.call(@domain_block, resource_params[:retroactive]) + UnblockDomainService.new.call(@domain_block, retroactive_unblock?) redirect_to admin_domain_blocks_path, notice: I18n.t('admin.domain_blocks.destroyed_msg') end @@ -36,5 +36,9 @@ module Admin def resource_params params.require(:domain_block).permit(:domain, :severity, :reject_media, :retroactive) end + + def retroactive_unblock? + resource_params[:retroactive] == '1' + end end end -- cgit