Use expect in Admin::ExportControllerConcern import params (#33745)

This commit is contained in:
Matt Jankowski 2025-01-27 09:29:25 -05:00 committed by GitHub
parent 65d9171c7d
commit d3b637c757
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 33 additions and 1 deletions

View File

@ -24,6 +24,6 @@ module Admin::ExportControllerConcern
end end
def import_params def import_params
params.require(:admin_import).permit(:data) params.expect(admin_import: [:data])
end end
end end

View File

@ -0,0 +1,16 @@
# frozen_string_literal: true
require 'rails_helper'
RSpec.describe 'Admin Export Domain Allows' do
describe 'POST /admin/export_domain_allows/import' do
before { sign_in Fabricate(:admin_user) }
it 'gracefully handles invalid nested params' do
post import_admin_export_domain_allows_path(admin_import: 'invalid')
expect(response)
.to redirect_to(admin_instances_path)
end
end
end

View File

@ -0,0 +1,16 @@
# frozen_string_literal: true
require 'rails_helper'
RSpec.describe 'Admin Export Domain Blocks' do
describe 'POST /admin/export_domain_blocks/import' do
before { sign_in Fabricate(:admin_user) }
it 'gracefully handles invalid nested params' do
post import_admin_export_domain_blocks_path(admin_import: 'invalid')
expect(response.body)
.to include(I18n.t('admin.export_domain_blocks.no_file'))
end
end
end