Commit be6c4fef authored by Patricio Cano's avatar Patricio Cano

Removed unnecessary `id` from links and corrected tests to use the proper matcher.

parent 1867d0d5
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
%tr %tr
%td %td
- if user - if user
= link_to user.name, user, id: 'abuser_profile_path' = link_to user.name, user
.light.small .light.small
Joined #{time_ago_with_tooltip(user.created_at)} Joined #{time_ago_with_tooltip(user.created_at)}
- else - else
......
...@@ -31,7 +31,7 @@ ...@@ -31,7 +31,7 @@
= icon('rss') = icon('rss')
- if current_user.admin? - if current_user.admin?
   
= link_to [:admin, @user], id: 'admin_user_path', class: 'btn btn-gray', title: 'View user in admin area', = link_to [:admin, @user], class: 'btn btn-gray', title: 'View user in admin area',
data: {toggle: 'tooltip', placement: 'bottom', container: 'body'} do data: {toggle: 'tooltip', placement: 'bottom', container: 'body'} do
= icon('users') = icon('users')
......
...@@ -6,16 +6,15 @@ describe "Admin::AbuseReports", feature: true, js: true do ...@@ -6,16 +6,15 @@ describe "Admin::AbuseReports", feature: true, js: true do
context 'as an admin' do context 'as an admin' do
describe 'if a user has been reported for abuse' do describe 'if a user has been reported for abuse' do
before do before do
admin = create(:admin)
create(:abuse_report, user: user) create(:abuse_report, user: user)
login_as admin login_as :admin
end end
describe 'in the abuse report view' do describe 'in the abuse report view' do
it "should present a link to the user's profile" do it "should present a link to the user's profile" do
visit admin_abuse_reports_path visit admin_abuse_reports_path
expect(page).to have_selector '#abuser_profile_path' expect(page).to have_link user.name, href: user_path(user)
end end
end end
...@@ -23,7 +22,7 @@ describe "Admin::AbuseReports", feature: true, js: true do ...@@ -23,7 +22,7 @@ describe "Admin::AbuseReports", feature: true, js: true do
it 'should show a link to the admin view of the user' do it 'should show a link to the admin view of the user' do
visit user_path(user) visit user_path(user)
expect(page).to have_selector '#admin_user_path' expect(page).to have_link '', href: admin_user_path(user)
end end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment