Merge branch '65705-two-buttons' into 'master'

Removes duplicate button from job log page

Closes #65705

See merge request gitlab-org/gitlab-ce!31544
This commit is contained in:
Kushal Pandya 2019-08-07 15:37:03 +00:00
commit af2edf2825
3 changed files with 13 additions and 2 deletions

View File

@ -81,7 +81,7 @@ export default {
:variables-settings-url="variablesSettingsUrl"
/>
<div class="text-content">
<div v-if="action" class="text-center">
<div v-if="action && !shouldRenderManualVariables" class="text-center">
<gl-link
:href="action.path"
:data-method="action.method"

View File

@ -0,0 +1,5 @@
---
title: Prevent duplicated trigger action button
merge_request:
author:
type: fixed

View File

@ -105,7 +105,7 @@ describe('Empty State', () => {
});
describe('with playbale action and not scheduled job', () => {
it('renders manual variables form', () => {
beforeEach(() => {
vm = mountComponent(Component, {
...props,
content,
@ -117,9 +117,15 @@ describe('Empty State', () => {
method: 'post',
},
});
});
it('renders manual variables form', () => {
expect(vm.$el.querySelector('.js-manual-vars-form')).not.toBeNull();
});
it('does not render the empty state action', () => {
expect(vm.$el.querySelector('.js-job-empty-state-action')).toBeNull();
});
});
describe('with playbale action and scheduled job', () => {