Rubocop

app/services/imports/user_import_service.rb

@@ -38,24 +38,21 @@ module Imports
     def role(field_value)
       return unless field_value
 
-      case field_value.downcase.strip
+      {
-      when "data provider"
+        "co-ordinator" => "data_coordinator",
-        "data_provider"
+        "data provider" => "data_provider",
-      when "co-ordinator"
+        "private data downloader" => "data_accessor",
-        "data_coordinator"
+      }[field_value.downcase.strip]
-      when "private data downloader"
-        "data_accessor"
-      end
     end
 
     def is_dpo?(field_value)
-      return false unless field_value.present?
+      return false if field_value.blank?
 
       field_value.downcase.strip == "data protection officer"
     end
 
     def is_key_contact?(field_value)
-      return false unless field_value.present?
+      return false if field_value.blank?
 
       ["ecore contact", "key performance contact"].include?(field_value.downcase.strip)
     end