mirror of
https://github.com/discourse/discourse.git
synced 2026-03-04 01:15:08 +08:00
43013b7b76
The admin email templates page currently uses a dropdown that lists all available templates for navigating between the templates. This is an odd way of navigation and inconsistent with how navigation is implemented in other admin pages. This commit replaces the dropdown with a dedicated index page that lists all templates in a table with an edit button for each row that navigates to the edit page for the respective template. Internal topic: t/162739.
65 lines
2.2 KiB
Ruby
65 lines
2.2 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
describe "Admin Email Templates Page", type: :system do
|
|
fab!(:admin)
|
|
|
|
let(:email_templates_page) { PageObjects::Pages::AdminEmailTemplatesIndex.new }
|
|
|
|
before { sign_in(admin) }
|
|
|
|
it "navigates to the email template edit page when clicking the Edit button" do
|
|
email_templates_page.visit
|
|
|
|
email_templates_page.template("user_notifications.account_exists").edit_button.click
|
|
|
|
expect(page).to have_current_path("/admin/email/templates/user_notifications.account_exists")
|
|
end
|
|
|
|
it "navigates to the email template edit page when clicking the template name" do
|
|
email_templates_page.visit
|
|
|
|
email_templates_page.template("user_notifications.account_exists").name_cell.click
|
|
|
|
expect(page).to have_current_path("/admin/email/templates/user_notifications.account_exists")
|
|
end
|
|
|
|
describe "filter controls" do
|
|
it "can find templates by name" do
|
|
email_templates_page.visit
|
|
|
|
email_templates_page.filter_controls.type_in_search("Account already")
|
|
|
|
expect(email_templates_page).to have_exact_count_templates_shown(1)
|
|
expect(
|
|
email_templates_page.template("user_notifications.account_exists").name_cell.text.strip,
|
|
).to eq("Account already exists")
|
|
end
|
|
|
|
it "can find templates by id" do
|
|
email_templates_page.visit
|
|
|
|
email_templates_page.filter_controls.type_in_search("user_notifications.account_exists")
|
|
|
|
expect(email_templates_page).to have_exact_count_templates_shown(1)
|
|
expect(
|
|
email_templates_page.template("user_notifications.account_exists").name_cell.text.strip,
|
|
).to eq("Account already exists")
|
|
end
|
|
|
|
it "can filter templates to only show overridden ones" do
|
|
TranslationOverride.upsert!(
|
|
I18n.locale,
|
|
"user_notifications.user_replied_pm.text_body_template",
|
|
"some new body",
|
|
)
|
|
email_templates_page.visit
|
|
|
|
expect(email_templates_page.only_overridden_checkbox.checked?).to be_falsey
|
|
|
|
email_templates_page.only_overridden_checkbox.click
|
|
|
|
expect(email_templates_page).to have_exact_count_templates_shown(1)
|
|
expect(email_templates_page.template("user_notifications.user_replied_pm")).to be_overridden
|
|
end
|
|
end
|
|
end
|
