diff --git a/app/assets/javascripts/environments/components/environment_stop.js.es6 b/app/assets/javascripts/environments/components/environment_stop.js.es6 index b39af2efb57..779d16937ec 100644 --- a/app/assets/javascripts/environments/components/environment_stop.js.es6 +++ b/app/assets/javascripts/environments/components/environment_stop.js.es6 @@ -13,6 +13,12 @@ }, }, + computed: { + stopUrl() { + return `${this.stop_url}/stop`; + }, + }, + methods: { openConfirmDialog() { return window.confirm('Are you sure you want to stop this environment?'); // eslint-disable-line @@ -22,9 +28,9 @@ template: ` + :href="stopUrl" + data-method="post" + data-rel="nofollow"> `, diff --git a/spec/features/environments_spec.rb b/spec/features/environments_spec.rb index b932d672548..fe5a783a386 100644 --- a/spec/features/environments_spec.rb +++ b/spec/features/environments_spec.rb @@ -59,6 +59,7 @@ feature 'Environments', feature: true, js: true do given(:deployment) { create(:deployment, environment: environment) } scenario 'does show deployment SHA' do + # TODO: Fix me! expect(page).to have_link(deployment.short_sha) end @@ -73,11 +74,15 @@ feature 'Environments', feature: true, js: true do given(:manual) { create(:ci_build, :manual, pipeline: pipeline, name: 'deploy to production') } scenario 'does show a play button' do + # TODO: Fix me! + binding.pry expect(page).to have_link(manual.name.humanize) end scenario 'does allow to play manual action' do expect(manual).to be_skipped + # TODO: Fix me! + binding.pry expect{ click_link(manual.name.humanize) }.not_to change { Ci::Pipeline.count } expect(page).to have_content(manual.name) expect(manual.reload).to be_pending @@ -114,7 +119,7 @@ feature 'Environments', feature: true, js: true do end scenario 'starts build when stop button clicked' do - page.find('.stop-env-link').click + find('.stop-env-link').click expect(page).to have_content('close_app') end