Commit 34bef34f authored by Patrick Steinhardt's avatar Patrick Steinhardt

Fix tests failing because of Chrome 84 upgrade

parent 24fc15cf
...@@ -22,7 +22,7 @@ RSpec.describe 'Projects > Show > User manages notifications', :js do ...@@ -22,7 +22,7 @@ RSpec.describe 'Projects > Show > User manages notifications', :js do
click_notifications_button click_notifications_button
expect(find('.update-notification.is-active')).to have_content('On mention') expect(find('.update-notification.is-active')).to have_content('On mention')
expect(find('.notifications-icon use')[:'xlink:href']).to end_with('#notifications') expect(page).to have_css('.notifications-icon[data-testid="notifications-icon"]')
end end
it 'changes the notification setting to disabled' do it 'changes the notification setting to disabled' do
...@@ -32,7 +32,7 @@ RSpec.describe 'Projects > Show > User manages notifications', :js do ...@@ -32,7 +32,7 @@ RSpec.describe 'Projects > Show > User manages notifications', :js do
wait_for_requests wait_for_requests
expect(find('.notifications-icon use')[:'xlink:href']).to end_with('#notifications-off') expect(page).to have_css('.notifications-icon[data-testid="notifications-off-icon"]')
end end
context 'custom notification settings' do context 'custom notification settings' do
......
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