Commit 10178af1 authored by Filipa Lacerda's avatar Filipa Lacerda

Fixes after review

parent 7db05c4d
...@@ -16,5 +16,5 @@ ...@@ -16,5 +16,5 @@
- if status.has_action? - if status.has_action?
= link_to status.action_path, class: 'ci-action-icon-container has-tooltip', method: status.action_method, data: { toggle: 'tooltip', title: status.action_title } do = link_to status.action_path, class: 'ci-action-icon-container has-tooltip', method: status.action_method, data: { toggle: 'tooltip', title: status.action_title } do
%i.ci-action-icon-wrapper{ class: "js-#{status.action_icon}" } %i.ci-action-icon-wrapper{ class: "js-#{status.action_icon.dasherize}" }
= custom_icon(status.action_icon) = custom_icon(status.action_icon)
...@@ -67,7 +67,7 @@ describe 'Pipeline', :feature, :js do ...@@ -67,7 +67,7 @@ describe 'Pipeline', :feature, :js do
it 'shows a running icon and a cancel action for the running build' do it 'shows a running icon and a cancel action for the running build' do
page.within('#ci-badge-deploy') do page.within('#ci-badge-deploy') do
expect(page).to have_selector('.js-ci-status-icon-running') expect(page).to have_selector('.js-ci-status-icon-running')
expect(page).to have_selector('.js-icon_action_cancel') expect(page).to have_selector('.js-icon-action-cancel')
expect(page).to have_content('deploy') expect(page).to have_content('deploy')
end end
end end
...@@ -87,7 +87,7 @@ describe 'Pipeline', :feature, :js do ...@@ -87,7 +87,7 @@ describe 'Pipeline', :feature, :js do
end end
page.within('#ci-badge-build .ci-action-icon-container') do page.within('#ci-badge-build .ci-action-icon-container') do
expect(page).to have_selector('.js-icon_action_retry') expect(page).to have_selector('.js-icon-action-retry')
end end
end end
...@@ -106,7 +106,7 @@ describe 'Pipeline', :feature, :js do ...@@ -106,7 +106,7 @@ describe 'Pipeline', :feature, :js do
end end
page.within('#ci-badge-test .ci-action-icon-container') do page.within('#ci-badge-test .ci-action-icon-container') do
expect(page).to have_selector('.js-icon_action_retry') expect(page).to have_selector('.js-icon-action-retry')
end end
end end
...@@ -125,7 +125,7 @@ describe 'Pipeline', :feature, :js do ...@@ -125,7 +125,7 @@ describe 'Pipeline', :feature, :js do
end end
page.within('#ci-badge-manual-build .ci-action-icon-container') do page.within('#ci-badge-manual-build .ci-action-icon-container') do
expect(page).to have_selector('.js-icon_action_play') expect(page).to have_selector('.js-icon-action-play')
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