diff --git a/app/controllers/admin/registrars_controller.rb b/app/controllers/admin/registrars_controller.rb index 8aecc8347..7479d53ac 100644 --- a/app/controllers/admin/registrars_controller.rb +++ b/app/controllers/admin/registrars_controller.rb @@ -3,7 +3,7 @@ require 'net/http' module Admin class RegistrarsController < BaseController # rubocop:disable Metrics/ClassLength load_and_authorize_resource - before_action :set_registrar, only: [:show, :edit, :update, :destroy] + before_action :set_registrar, only: %i[show edit update destroy] before_action :set_registrar_status_filter, only: [:index] helper_method :registry_vat_rate helper_method :iban_max_length @@ -39,6 +39,13 @@ module Admin def edit; end + def show + method = allowed_method(params[:records]) || 'api_users' + @result = @registrar.send(method.to_sym) + partial_name = "#{@registrar.name.parameterize}_#{method}" + render_by_format('admin/registrars/show', partial_name) + end + def update if @registrar.update(registrar_params) redirect_to [:admin, @registrar], notice: t('.updated') @@ -168,5 +175,10 @@ module Admin def iban_max_length Iban.max_length end + + def allowed_method(records_param) + allowed_methods = %w[api_users white_ips] + records_param if allowed_methods.include?(records_param) + end end end diff --git a/app/models/api_user.rb b/app/models/api_user.rb index a15b12a85..73e0f6c4e 100644 --- a/app/models/api_user.rb +++ b/app/models/api_user.rb @@ -91,6 +91,24 @@ class ApiUser < User another_api_user.identity_code == identity_code end + def as_csv_row + [ + username, + plain_text_password, + identity_code, + roles.join(', '), + active, + accredited?, + accreditation_expire_date, + created_at, updated_at + ] + end + + def self.csv_header + ['Username', 'Password', 'Identity Code', 'Role', 'Active', 'Accredited', + 'Accreditation Expire Date', 'Created', 'Updated'] + end + private def machine_readable_certificate(cert) diff --git a/app/models/white_ip.rb b/app/models/white_ip.rb index 7bbd8c18f..a90b8cdd2 100644 --- a/app/models/white_ip.rb +++ b/app/models/white_ip.rb @@ -13,6 +13,7 @@ class WhiteIp < ApplicationRecord def validate_ipv4_and_ipv6 return if ipv4.present? || ipv6.present? + errors.add(:base, I18n.t(:ipv4_or_ipv6_must_be_present)) end @@ -32,12 +33,12 @@ class WhiteIp < ApplicationRecord errors.add(:ipv6, :invalid) end - API = 'api' - REGISTRAR = 'registrar' - INTERFACES = [API, REGISTRAR] + API = 'api'.freeze + REGISTRAR = 'registrar'.freeze + INTERFACES = [API, REGISTRAR].freeze - scope :api, -> { where("interfaces @> ?::varchar[]", "{#{API}}") } - scope :registrar_area, -> { where("interfaces @> ?::varchar[]", "{#{REGISTRAR}}") } + scope :api, -> { where('interfaces @> ?::varchar[]', "{#{API}}") } + scope :registrar_area, -> { where('interfaces @> ?::varchar[]', "{#{REGISTRAR}}") } def interfaces=(interfaces) super(interfaces.reject(&:blank?)) @@ -72,5 +73,19 @@ class WhiteIp < ApplicationRecord rescue StandardError => _e nil end + + def csv_header + %w[IPv4 IPv6 Interfaces Created Updated] + end + end + + def as_csv_row + [ + ipv4, + ipv6, + interfaces.join(', ').upcase, + created_at, + updated_at, + ] end end diff --git a/app/services/csv_generator.rb b/app/services/csv_generator.rb index d92beeddc..7a8055680 100644 --- a/app/services/csv_generator.rb +++ b/app/services/csv_generator.rb @@ -22,7 +22,7 @@ class CsvGenerator def custom_csv?(class_name) [ Version::DomainVersion, Version::ContactVersion, Domain, - Contact, Invoice, Account, AccountActivity + Contact, Invoice, Account, AccountActivity, ApiUser, WhiteIp ].include?(class_name) end end diff --git a/app/views/admin/registrars/show/_api_users.html.erb b/app/views/admin/registrars/show/_api_users.html.erb index fddccd9b7..7e3cb287d 100644 --- a/app/views/admin/registrars/show/_api_users.html.erb +++ b/app/views/admin/registrars/show/_api_users.html.erb @@ -1,4 +1,4 @@ -
+
<%= t '.header' %>
@@ -15,7 +15,7 @@ <% registrar.api_users.each do |api_user| %> - <%= link_to api_user, admin_registrar_api_user_path(api_user.registrar, api_user) %> + <%= link_to api_user, admin_registrar_api_user_path(registrar, api_user) %> <%= api_user.active %> @@ -37,5 +37,7 @@
diff --git a/app/views/admin/registrars/show/_white_ips.html.erb b/app/views/admin/registrars/show/_white_ips.html.erb index c86c85eb9..5de3066c0 100644 --- a/app/views/admin/registrars/show/_white_ips.html.erb +++ b/app/views/admin/registrars/show/_white_ips.html.erb @@ -1,4 +1,4 @@ -
+
<%= t '.header' %>
@@ -34,5 +34,7 @@
diff --git a/config/routes.rb b/config/routes.rb index 7d11cb6a4..25d73043f 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -326,6 +326,11 @@ Rails.application.routes.draw do post 'set_test_date_to_api_user', to: 'api_users#set_test_date_to_api_user', as: 'set_test_date_to_api_user' post 'remove_test_date_to_api_user', to: 'api_users#remove_test_date_to_api_user', as: 'remove_test_date_to_api_user' end + + member do + get 'export/:records', to: 'registrars#show', constraints: { format: 'csv' }, + as: :export + end end resources :contacts do @@ -336,7 +341,7 @@ Rails.application.routes.draw do resources :admin_users # /admin/api_users is mainly for manual testing - resources :api_users, only: [:index, :show] do + resources :api_users, only: %i[index show] do resources :certificates do member do post 'sign' diff --git a/test/fixtures/files/api_users.csv b/test/fixtures/files/api_users.csv new file mode 100644 index 000000000..21afa2350 --- /dev/null +++ b/test/fixtures/files/api_users.csv @@ -0,0 +1,2 @@ +Username,Password,Identity Code,Role,Active,Accredited,Accreditation Expire Date,Created,Updated +test_bestnames,testtest,1234,super,true,false,,2010-07-05 10:30:00 +0300,2010-07-05 10:30:00 +0300 diff --git a/test/fixtures/files/white_ips.csv b/test/fixtures/files/white_ips.csv new file mode 100644 index 000000000..5c34cf160 --- /dev/null +++ b/test/fixtures/files/white_ips.csv @@ -0,0 +1,2 @@ +IPv4,IPv6,Interfaces,Created,Updated +127.0.0.1,,"REGISTRAR, API",2010-07-05 10:30:00 +0300,2010-07-05 10:30:00 +0300 diff --git a/test/system/admin_area/registrars/api_users_test.rb b/test/system/admin_area/registrars/api_users_test.rb index 5d833dde2..32de1451b 100644 --- a/test/system/admin_area/registrars/api_users_test.rb +++ b/test/system/admin_area/registrars/api_users_test.rb @@ -22,6 +22,26 @@ class AdminRegistrarsApiUsersSystemTest < ApplicationSystemTestCase assert_current_path admin_registrar_api_user_path(registrar, new_api_user) end + def test_downloads_api_users_list_as_csv + travel_to Time.zone.parse('2010-07-05 10:30') + registrar = registrars(:bestnames) + api_users = registrar.api_users + api_users.each do |u| + u.created_at = Time.zone.now + u.updated_at = Time.zone.now + u.save(validate: false) + end + + visit admin_registrar_path(registrar) + within('.api_users') do + click_on 'Export to CSV' + end + + assert_equal "attachment; filename=\"#{registrar.name.parameterize}_api_users_#{Time.zone.now.to_formatted_s(:number)}.csv\"; " \ + "filename*=UTF-8''#{registrar.name.parameterize}_api_users_#{Time.zone.now.to_formatted_s(:number)}.csv", response_headers['Content-Disposition'] + assert_equal file_fixture('api_users.csv').read, page.body + end + def test_shows_api_user_details api_user = users(:api_bestnames) diff --git a/test/system/admin_area/registrars/white_ips_test.rb b/test/system/admin_area/registrars/white_ips_test.rb new file mode 100644 index 000000000..f778d4dcd --- /dev/null +++ b/test/system/admin_area/registrars/white_ips_test.rb @@ -0,0 +1,27 @@ +require 'application_system_test_case' + +class AdminRegistrarsWhiteIpsSystemTest < ApplicationSystemTestCase + setup do + sign_in users(:admin) + end + + def test_downloads_whitelisted_ips_list_as_csv + travel_to Time.zone.parse('2010-07-05 10:30') + registrar = registrars(:bestnames) + white_ips = registrar.white_ips + white_ips.each do |ip| + ip.created_at = Time.zone.now + ip.updated_at = Time.zone.now + ip.save(validate: false) + end + + visit admin_registrar_path(registrar) + within('.white_ips') do + click_on 'Export to CSV' + end + + assert_equal "attachment; filename=\"#{registrar.name.parameterize}_white_ips_#{Time.zone.now.to_formatted_s(:number)}.csv\"; " \ + "filename*=UTF-8''#{registrar.name.parameterize}_white_ips_#{Time.zone.now.to_formatted_s(:number)}.csv", response_headers['Content-Disposition'] + assert_equal file_fixture('white_ips.csv').read, page.body + end +end