mirror of
https://github.com/discourse/discourse.git
synced 2026-03-03 23:54:20 +08:00
Introduces a minor accessibility improvement to the radio button group in the wizard component and adds a corresponding system test to ensure correct keyboard navigation behavior.
99 lines
3.5 KiB
Ruby
99 lines
3.5 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
describe "Wizard", type: :system do
|
|
fab!(:admin)
|
|
|
|
let(:wizard_page) { PageObjects::Pages::Wizard.new }
|
|
|
|
before { sign_in(admin) }
|
|
|
|
it "successfully completes the setup wizard" do
|
|
visit("/wizard")
|
|
expect(wizard_page).to be_on_step("setup")
|
|
wizard_page.fill_field("text", "title", "My Test Site")
|
|
wizard_page.select_dropdown_option("default-locale", "en")
|
|
wizard_page.click_jump_in
|
|
expect(page).to have_current_path("/")
|
|
end
|
|
|
|
describe "Setup step" do
|
|
it "lets user configure site settings including member access" do
|
|
wizard_page.go_to_step("setup")
|
|
expect(SiteSetting.login_required).to eq(false)
|
|
expect(SiteSetting.invite_only).to eq(false)
|
|
expect(SiteSetting.must_approve_users).to eq(false)
|
|
|
|
expect(wizard_page.privacy_step).to have_selected_choice("login-required", "public")
|
|
expect(wizard_page.privacy_step).to have_selected_choice("invite-only", "sign_up")
|
|
expect(wizard_page.privacy_step).to have_selected_choice("must-approve-users", "no")
|
|
|
|
wizard_page.fill_field("text", "title", "My Test Site")
|
|
wizard_page.privacy_step.select_access_option("login-required", "private")
|
|
wizard_page.privacy_step.select_access_option("invite-only", "invite_only")
|
|
wizard_page.privacy_step.select_access_option("must-approve-users", "yes")
|
|
|
|
wizard_page.click_jump_in
|
|
|
|
expect(SiteSetting.login_required).to eq(true)
|
|
expect(SiteSetting.invite_only).to eq(true)
|
|
expect(SiteSetting.must_approve_users).to eq(true)
|
|
end
|
|
|
|
it "redirects to homepage when completed" do
|
|
wizard_page.go_to_step("setup")
|
|
wizard_page.fill_field("text", "title", "My Test Site")
|
|
wizard_page.click_jump_in
|
|
|
|
expect(page).to have_current_path("/")
|
|
end
|
|
|
|
it "prevents submission when title is empty" do
|
|
wizard_page.go_to_step("setup")
|
|
wizard_page.fill_field("text", "title", "")
|
|
wizard_page.click_jump_in
|
|
|
|
expect(wizard_page).to be_on_step("setup")
|
|
expect(page).to have_css(".wizard-container__field.text-title.invalid")
|
|
end
|
|
|
|
it "prevents submission when title is 'Discourse'" do
|
|
wizard_page.go_to_step("setup")
|
|
wizard_page.fill_field("text", "title", "Discourse")
|
|
wizard_page.click_jump_in
|
|
|
|
expect(wizard_page).to be_on_step("setup")
|
|
expect(page).to have_css(".wizard-container__field.text-title.invalid")
|
|
end
|
|
|
|
it "allows keyboard navigation of radio inputs" do
|
|
wizard_page.go_to_step("setup")
|
|
|
|
radio_field = wizard_page.find_field("radio", "login-required")
|
|
first_radio = radio_field.find("input[type='radio']", match: :first, visible: :all)
|
|
choice_id = first_radio[:value]
|
|
|
|
page.execute_script("arguments[0].focus()", first_radio)
|
|
|
|
expect(radio_field).to have_css(".wizard-container__radio-choice:focus-within")
|
|
expect(page.evaluate_script("document.activeElement.type")).to eq("radio")
|
|
|
|
first_radio.send_keys(:space)
|
|
expect(radio_field).to have_css(
|
|
".wizard-container__radio-choice[data-choice-id='#{choice_id}'].--selected",
|
|
)
|
|
end
|
|
|
|
it "keeps arrow key navigation within same radio group" do
|
|
wizard_page.go_to_step("setup")
|
|
|
|
login_required_field = wizard_page.find_field("radio", "login-required")
|
|
private_radio =
|
|
login_required_field.find("[data-choice-id='private'] input[type='radio']", visible: :all)
|
|
|
|
private_radio.click
|
|
private_radio.send_keys(:right)
|
|
|
|
expect(login_required_field).to have_css("input[type='radio']:focus", visible: :all)
|
|
end
|
|
end
|
|
end
|