Karavel/fixin.me
0
0
Fork 0
forked from fixin.me/fixin.me
Code Pull Requests Activity

Fix tests after routing/locale changes

Browse Source
This commit is contained in:
cryptogopher 2025-01-03 15:04:27 +01:00
parent af152c5e8b
commit d250119601
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
test/system/users_test.rb
View File

@ -47,7 +47,7 @@ class UsersTest < ApplicationSystemTestCase
assert_emails 1 do
click_on t("users.passwords.edit.update_password")
# Wait until redirected to make sure async request has been processed
assert_current_path /#{users_path}|#{edit_user_registration_path}/
assert_current_path units_path
end
assert_text t("devise.passwords.updated")
end
@ -99,7 +99,7 @@ class UsersTest < ApplicationSystemTestCase
test "show profile" do
sign_in user: users.select(&:admin?).select(&:confirmed?).sample
click_on t("layouts.application.users")
click_on t("users.navigation")
within all('tr').drop(1).sample do |tr|
email = first(:link).text
click_on email
@ -111,11 +111,12 @@ class UsersTest < ApplicationSystemTestCase
user = users.select(&:admin?).select(&:confirmed?).sample
sign_in user: user
click_on t("layouts.application.users")
all(:link_or_button, text: t("users.index.disguise")).sample.click
assert_current_path edit_user_registration_path
# TODO: test for "Profile" link button after root_url changed to different
# path then profile in routes.rb
click_on t("users.navigation")
link = all(:link_or_button, text: t("users.index.disguise")).sample
email = link.ancestor('tr').first(:link)[:text]
link.click
assert_current_path units_path
assert_link email
click_on t("layouts.application.revert")
assert_current_path users_path
@ -126,7 +127,7 @@ class UsersTest < ApplicationSystemTestCase
user = users.select(&:admin?).select(&:confirmed?).sample
sign_in user: user
click_on t("layouts.application.users")
click_on t("users.navigation")
text = t("users.index.disguise")
# Pick row without 'disguise' button
undisguisable = all(:xpath, "//tbody//tr[not(descendant::*[contains(text(),\"#{text}\")])]")