Browse Source

added extra field to test hidden content

pull/627/head
JG 3 years ago
parent
commit
7eedb74413
  1. 2
      app/helpers/tab_nav_helper.rb
  2. 2
      app/views/layouts/_footer.html.erb
  3. 7
      spec/requests/organisations_controller_spec.rb

2
app/helpers/tab_nav_helper.rb

@ -3,7 +3,7 @@ module TabNavHelper
def user_cell(user)
link_text = user.name.presence || user.email
[govuk_link_to(link_text, user), "<span class=\"govuk-!-font-weight-regular app-!-colour-muted\">#{user.email}</span>"].join("\n")
[govuk_link_to(link_text, user), "<span class=\"govuk-visually-hidden\">User </span><span class=\"govuk-!-font-weight-regular app-!-colour-muted\">#{user.email}</span>"].join("\n")
end
def org_cell(user)

2
app/views/layouts/_footer.html.erb

@ -29,7 +29,7 @@
</div>
</div>
<h2 class="govuk-visually-hidden">Helpful links<spec/requests/organisations_controller_spec.rb/h2>
<h2 class="govuk-visually-hidden">Helpful links</h2>
<ul class="govuk-footer__inline-list govuk-!-margin-top-6 govuk-!-margin-bottom-0">
<li class="govuk-footer__inline-list-item">
<%= govuk_link_to("Privacy notice", privacy_notice_path, class: "govuk-footer__link") %>

7
spec/requests/organisations_controller_spec.rb

@ -131,6 +131,13 @@ RSpec.describe OrganisationsController, type: :request do
expect(response.body).to include(user.email)
end
it "shows hidden accesibility fields only for active users in the current user's organisation" do
expected_case_row_log = "<span class=\"govuk-visually-hidden\">User </span><span class=\"govuk-!-font-weight-regular app-!-colour-muted\">#{user.email}</span>"
unauthorized_case_row_log = "<span class=\"govuk-visually-hidden\">User </span><span class=\"govuk-!-font-weight-regular app-!-colour-muted\">#{other_org_user.email}</span>"
expect(CGI.unescape_html(response.body)).to include(expected_case_row_log)
expect(CGI.unescape_html(response.body)).not_to include(unauthorized_case_row_log)
end
it "shows only active users in the current user's organisation" do
expect(page).to have_content(user.name)
expect(page).to have_content(other_user.name)

Loading…
Cancel
Save