Consistency

app/views/users/show.html.erb

@@ -12,53 +12,53 @@
 
     <%= govuk_summary_list do |summary_list| %>
       <%= summary_list.row do |row|
-        row.key { 'Name' }
+        row.key { "Name" }
         row.value { @user.name }
         if current_user == @user || current_user.data_coordinator?
-          row.action(visually_hidden_text: 'name', href: aliased_user_edit(@user, current_user), html_attributes: { 'data-qa': 'change-name' })
+          row.action(visually_hidden_text: "name", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "change-name" })
         else
           row.action()
         end
       end %>
 
       <%= summary_list.row() do |row|
-        row.key { 'Email address' }
+        row.key { "Email address" }
         row.value { @user.email }
         if current_user == @user || current_user.data_coordinator?
-          row.action(visually_hidden_text: 'email address', href: aliased_user_edit(@user, current_user), html_attributes: { 'data-qa': 'change-email' })
+          row.action(visually_hidden_text: "email address", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "change-email-address" })
         else
           row.action()
         end
       end %>
 
       <%= summary_list.row do |row|
-        row.key { 'Password' }
-        row.value { '••••••••' }
+        row.key { "Password" }
+        row.value { "••••••••" }
         if current_user == @user
-          row.action(visually_hidden_text: 'password', href: edit_password_account_path, html_attributes: { 'data-qa': 'change-password' })
+          row.action(visually_hidden_text: "password", href: edit_password_account_path, html_attributes: { "data-qa": "change-password" })
         else
           row.action()
         end
       end %>
 
       <%= summary_list.row do |row|
-        row.key { 'Organisation' }
+        row.key { "Organisation" }
         row.value { @user.organisation.name }
         row.action()
       end %>
 
       <%= summary_list.row do |row|
-        row.key { 'Role' }
+        row.key { "Role" }
         row.value { @user.role.humanize }
         if current_user.data_coordinator?
-          row.action(visually_hidden_text: "role", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "role" })
+          row.action(visually_hidden_text: "role", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "change-role" })
         else
           row.action()
         end
       end %>
 
       <%= summary_list.row do |row|
-        row.key { 'Data protection officer' }
+        row.key { "Data protection officer" }
         row.value { @user.is_data_protection_officer? ? "Yes" : "No" }
         if current_user.data_coordinator?
           row.action(visually_hidden_text: "are #{pronoun(@user, current_user)} a data protection officer?", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "change-are-#{pronoun(@user, current_user)}-a-data-protection-officer" })
@@ -68,7 +68,7 @@
       end %>
 
       <%= summary_list.row do |row|
-        row.key { 'Key contact' }
+        row.key { "Key contact" }
         row.value { @user.is_key_contact? ? "Yes" : "No" }
         if current_user.data_coordinator?
           row.action(visually_hidden_text: "are #{pronoun(@user, current_user)} a key contact?", href: aliased_user_edit(@user, current_user), html_attributes: { "data-qa": "change-are-#{pronoun(@user, current_user)}-a-key-contact" })