From c2907b839a60ff14bc51decd96330cdd5f1b60ba Mon Sep 17 00:00:00 2001 From: Maikel Linke Date: Thu, 19 Mar 2026 11:02:32 +1100 Subject: [PATCH] Remove ineffective sorting spec The spec was not really testing the order of users appearing on the page. It's also a UX detail only visible to super admins which is not important to test. So I'm not investing time to fix it. --- spec/system/admin/users_spec.rb | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/spec/system/admin/users_spec.rb b/spec/system/admin/users_spec.rb index 536eeaeb86..6a4129692d 100644 --- a/spec/system/admin/users_spec.rb +++ b/spec/system/admin/users_spec.rb @@ -18,28 +18,6 @@ RSpec.describe "Managing users" do click_link "Users" end - context "users index page with sorting" do - before(:each) do - click_link "users_email_title" - end - - it "should list users with order email asc" do - expect(page).to have_css('table#listing_users') - within("table#listing_users") do - expect(page).to have_content("a@example.com") - expect(page).to have_content("b@example.com") - end - end - - it "should list users with order email desc" do - click_link "users_email_title" - within("table#listing_users") do - expect(page).to have_content("a@example.com") - expect(page).to have_content("b@example.com") - end - end - end - context "searching users" do it "should display the correct results for a user search" do fill_in "q_email_cont", with: "a@example"