Commit 3247bf14 authored by Miguel Rincon's avatar Miguel Rincon Committed by Olena Horal-Koretska

Remove ci_mini_pipeline_gl_dropdown feature flag

This change removes the ci_mini_pipeline_gl_dropdown feature flag
and related code.
parent 43698b7b
...@@ -12,11 +12,9 @@ ...@@ -12,11 +12,9 @@
* 4. Commit widget * 4. Commit widget
*/ */
import { GlDropdown, GlLoadingIcon, GlTooltipDirective, GlIcon } from '@gitlab/ui'; import { GlDropdown, GlLoadingIcon, GlTooltipDirective, GlIcon } from '@gitlab/ui';
import $ from 'jquery';
import { deprecatedCreateFlash as Flash } from '~/flash'; import { deprecatedCreateFlash as Flash } from '~/flash';
import axios from '~/lib/utils/axios_utils'; import axios from '~/lib/utils/axios_utils';
import { __ } from '~/locale'; import { __ } from '~/locale';
import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
import { PIPELINES_TABLE } from '../../constants'; import { PIPELINES_TABLE } from '../../constants';
import eventHub from '../../event_hub'; import eventHub from '../../event_hub';
import JobItem from '../graph/job_item.vue'; import JobItem from '../graph/job_item.vue';
...@@ -31,19 +29,16 @@ export default { ...@@ -31,19 +29,16 @@ export default {
directives: { directives: {
GlTooltip: GlTooltipDirective, GlTooltip: GlTooltipDirective,
}, },
mixins: [glFeatureFlagsMixin()],
props: { props: {
stage: { stage: {
type: Object, type: Object,
required: true, required: true,
}, },
updateDropdown: { updateDropdown: {
type: Boolean, type: Boolean,
required: false, required: false,
default: false, default: false,
}, },
type: { type: {
type: String, type: String,
required: false, required: false,
...@@ -57,11 +52,6 @@ export default { ...@@ -57,11 +52,6 @@ export default {
}; };
}, },
computed: { computed: {
isCiMiniPipelineGlDropdown() {
// Feature flag ci_mini_pipeline_gl_dropdown
// See more at https://gitlab.com/gitlab-org/gitlab/-/issues/300400
return this.glFeatures?.ciMiniPipelineGlDropdown;
},
triggerButtonClass() { triggerButtonClass() {
return `ci-status-icon-${this.stage.status.group}`; return `ci-status-icon-${this.stage.status.group}`;
}, },
...@@ -76,24 +66,12 @@ export default { ...@@ -76,24 +66,12 @@ export default {
} }
}, },
}, },
updated() {
if (!this.isCiMiniPipelineGlDropdown && this.dropdownContent.length) {
this.stopDropdownClickPropagation();
}
},
methods: { methods: {
onShowDropdown() { onShowDropdown() {
eventHub.$emit('clickedDropdown'); eventHub.$emit('clickedDropdown');
this.isLoading = true; this.isLoading = true;
this.fetchJobs(); this.fetchJobs();
}, },
onClickStage() {
if (!this.isDropdownOpen()) {
eventHub.$emit('clickedDropdown');
this.isLoading = true;
this.fetchJobs();
}
},
fetchJobs() { fetchJobs() {
axios axios
.get(this.stage.dropdown_path) .get(this.stage.dropdown_path)
...@@ -102,63 +80,30 @@ export default { ...@@ -102,63 +80,30 @@ export default {
this.isLoading = false; this.isLoading = false;
}) })
.catch(() => { .catch(() => {
if (this.isCiMiniPipelineGlDropdown) {
this.$refs.stageGlDropdown.hide(); this.$refs.stageGlDropdown.hide();
} else {
this.closeDropdown();
}
this.isLoading = false; this.isLoading = false;
Flash(__('Something went wrong on our end.')); Flash(__('Something went wrong on our end.'));
}); });
}, },
/**
* When the user right clicks or cmd/ctrl + click in the job name
* the dropdown should not be closed and the link should open in another tab,
* so we stop propagation of the click event inside the dropdown.
*
* Since this component is rendered multiple times per page we need to guarantee we only
* target the click event of this component.
*
* Note: This should be removed once ci_mini_pipeline_gl_dropdown FF is removed as true.
*/
stopDropdownClickPropagation() {
$(
'.js-builds-dropdown-list button, .js-builds-dropdown-list a.mini-pipeline-graph-dropdown-item',
this.$el,
).on('click', (e) => {
e.stopPropagation();
});
},
closeDropdown() {
if (this.isDropdownOpen()) {
$(this.$refs.dropdown).dropdown('toggle');
}
},
isDropdownOpen() { isDropdownOpen() {
return this.$el.classList.contains('show'); return this.$el.classList.contains('show');
}, },
pipelineActionRequestComplete() { pipelineActionRequestComplete() {
if (this.type === PIPELINES_TABLE) { if (this.type === PIPELINES_TABLE) {
// warn the table to update // warn the pipelines table to update
eventHub.$emit('refreshPipelinesTable'); eventHub.$emit('refreshPipelinesTable');
return; return;
} }
// close the dropdown in mr widget // close the dropdown in MR widget
if (this.isCiMiniPipelineGlDropdown) {
this.$refs.stageGlDropdown.hide(); this.$refs.stageGlDropdown.hide();
} else {
$(this.$refs.dropdown).dropdown('toggle');
}
}, },
}, },
}; };
</script> </script>
<template> <template>
<div class="dropdown">
<gl-dropdown <gl-dropdown
v-if="isCiMiniPipelineGlDropdown"
ref="stageGlDropdown" ref="stageGlDropdown"
v-gl-tooltip.hover v-gl-tooltip.hover
data-testid="mini-pipeline-graph-dropdown" data-testid="mini-pipeline-graph-dropdown"
...@@ -166,7 +111,7 @@ export default { ...@@ -166,7 +111,7 @@ export default {
variant="link" variant="link"
:lazy="true" :lazy="true"
:popper-opts="{ placement: 'bottom' }" :popper-opts="{ placement: 'bottom' }"
:toggle-class="['mini-pipeline-graph-gl-dropdown-toggle', triggerButtonClass]" :toggle-class="['mini-pipeline-graph-dropdown-toggle', triggerButtonClass]"
menu-class="mini-pipeline-graph-dropdown-menu" menu-class="mini-pipeline-graph-dropdown-menu"
@show="onShowDropdown" @show="onShowDropdown"
> >
...@@ -191,44 +136,4 @@ export default { ...@@ -191,44 +136,4 @@ export default {
</li> </li>
</ul> </ul>
</gl-dropdown> </gl-dropdown>
<template v-else>
<button
id="stageDropdown"
ref="dropdown"
v-gl-tooltip.hover
:class="triggerButtonClass"
:title="stage.title"
class="mini-pipeline-graph-dropdown-toggle"
data-testid="mini-pipeline-graph-dropdown-toggle"
data-toggle="dropdown"
data-display="static"
type="button"
aria-haspopup="true"
aria-expanded="false"
@click="onClickStage"
>
<span :aria-label="stage.title" aria-hidden="true" class="gl-pointer-events-none">
<gl-icon :name="borderlessIcon" />
</span>
</button>
<div
class="dropdown-menu mini-pipeline-graph-dropdown-menu js-builds-dropdown-container"
aria-labelledby="stageDropdown"
>
<gl-loading-icon v-if="isLoading" />
<ul v-else class="js-builds-dropdown-list scrollable-menu">
<li v-for="job in dropdownContent" :key="job.id">
<job-item
:dropdown-length="dropdownContent.length"
:job="job"
css-class-job-name="mini-pipeline-graph-dropdown-item"
@pipelineActionRequestComplete="pipelineActionRequestComplete"
/>
</li>
</ul>
</div>
</template>
</div>
</template> </template>
...@@ -67,8 +67,7 @@ ...@@ -67,8 +67,7 @@
// Mini Pipelines // Mini Pipelines
.stage-cell { .stage-cell {
.mini-pipeline-graph-dropdown-toggle, .mini-pipeline-graph-dropdown-toggle {
.mini-pipeline-graph-gl-dropdown-toggle {
svg { svg {
height: $ci-action-icon-size; height: $ci-action-icon-size;
width: $ci-action-icon-size; width: $ci-action-icon-size;
...@@ -138,14 +137,16 @@ ...@@ -138,14 +137,16 @@
} }
} }
// Dropdown button in mini pipeline graph // Commit mini pipeline (HAML)
button.mini-pipeline-graph-dropdown-toggle, button.mini-pipeline-graph-dropdown-toggle,
// GlDropdown mini pipeline (Vue)
// As the `mini-pipeline-item` mixin specificity is lower // As the `mini-pipeline-item` mixin specificity is lower
// than the toggle of dropdown with 'variant="link"' we add // than the toggle of dropdown with 'variant="link"' we add
// classes ".gl-button.btn-link" to make it more specific. // classes ".gl-button.btn-link" to make it more specific
// Once FF ci_mini_pipeline_gl_dropdown is removed, the `mini-pipeline-item` // and avoid having the size overriden
// itself could increase its specificity to simplify this selector //
button.gl-button.btn-link.mini-pipeline-graph-gl-dropdown-toggle { // See https://gitlab.com/gitlab-org/gitlab/-/issues/320737
button.gl-button.btn-link.mini-pipeline-graph-dropdown-toggle {
@include mini-pipeline-item(); @include mini-pipeline-item();
} }
......
...@@ -226,10 +226,6 @@ $tabs-holder-z-index: 250; ...@@ -226,10 +226,6 @@ $tabs-holder-z-index: 250;
} }
} }
.mini-pipeline-graph-dropdown-toggle {
vertical-align: top;
}
.normal { .normal {
flex: 1; flex: 1;
flex-basis: auto; flex-basis: auto;
...@@ -982,15 +978,15 @@ $tabs-holder-z-index: 250; ...@@ -982,15 +978,15 @@ $tabs-holder-z-index: 250;
line-height: initial; line-height: initial;
} }
.mini-pipeline-graph-dropdown-toggle, // GlDropdown mini pipeline (Vue)
.stage-cell .mini-pipeline-graph-dropdown-toggle svg,
// As the `mini-pipeline-item` mixin specificity is lower // As the `mini-pipeline-item` mixin specificity is lower
// than the toggle of dropdown with 'variant="link"' we add // than the toggle of dropdown with 'variant="link"' we add
// classes ".gl-button.btn-link" to make it more specific. // classes ".gl-button.btn-link" to make it more specific
// Once FF ci_mini_pipeline_gl_dropdown is removed, the `mini-pipeline-item` // and avoid having the size overriden
// itself could increase its specificity to simplify this selector //
button.gl-button.btn-link.mini-pipeline-graph-gl-dropdown-toggle, // See https://gitlab.com/gitlab-org/gitlab/-/issues/320737
.stage-cell button.gl-button.btn-link.mini-pipeline-graph-gl-dropdown-toggle svg { button.gl-button.btn-link.mini-pipeline-graph-dropdown-toggle,
.stage-cell button.gl-button.btn-link.mini-pipeline-graph-dropdown-toggle svg {
height: $ci-action-icon-size-lg; height: $ci-action-icon-size-lg;
width: $ci-action-icon-size-lg; width: $ci-action-icon-size-lg;
} }
......
...@@ -18,9 +18,6 @@ class Projects::CommitController < Projects::ApplicationController ...@@ -18,9 +18,6 @@ class Projects::CommitController < Projects::ApplicationController
before_action :define_commit_vars, only: [:show, :diff_for_path, :diff_files, :pipelines, :merge_requests] before_action :define_commit_vars, only: [:show, :diff_for_path, :diff_files, :pipelines, :merge_requests]
before_action :define_note_vars, only: [:show, :diff_for_path, :diff_files] before_action :define_note_vars, only: [:show, :diff_for_path, :diff_files]
before_action :authorize_edit_tree!, only: [:revert, :cherry_pick] before_action :authorize_edit_tree!, only: [:revert, :cherry_pick]
before_action only: [:pipelines] do
push_frontend_feature_flag(:ci_mini_pipeline_gl_dropdown, @project, type: :development, default_enabled: :yaml)
end
BRANCH_SEARCH_LIMIT = 1000 BRANCH_SEARCH_LIMIT = 1000
COMMIT_DIFFS_PER_PAGE = 75 COMMIT_DIFFS_PER_PAGE = 75
......
...@@ -44,7 +44,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo ...@@ -44,7 +44,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
push_frontend_feature_flag(:suggestions_custom_commit, @project, default_enabled: true) push_frontend_feature_flag(:suggestions_custom_commit, @project, default_enabled: true)
push_frontend_feature_flag(:local_file_reviews, default_enabled: :yaml) push_frontend_feature_flag(:local_file_reviews, default_enabled: :yaml)
push_frontend_feature_flag(:paginated_notes, @project, default_enabled: :yaml) push_frontend_feature_flag(:paginated_notes, @project, default_enabled: :yaml)
push_frontend_feature_flag(:ci_mini_pipeline_gl_dropdown, @project, type: :development, default_enabled: :yaml)
record_experiment_user(:invite_members_version_a) record_experiment_user(:invite_members_version_a)
record_experiment_user(:invite_members_version_b) record_experiment_user(:invite_members_version_b)
......
...@@ -17,7 +17,6 @@ class Projects::PipelinesController < Projects::ApplicationController ...@@ -17,7 +17,6 @@ class Projects::PipelinesController < Projects::ApplicationController
push_frontend_feature_flag(:new_pipeline_form, project, default_enabled: :yaml) push_frontend_feature_flag(:new_pipeline_form, project, default_enabled: :yaml)
push_frontend_feature_flag(:graphql_pipeline_details, project, type: :development, default_enabled: :yaml) push_frontend_feature_flag(:graphql_pipeline_details, project, type: :development, default_enabled: :yaml)
push_frontend_feature_flag(:graphql_pipeline_details_users, current_user, type: :development, default_enabled: :yaml) push_frontend_feature_flag(:graphql_pipeline_details_users, current_user, type: :development, default_enabled: :yaml)
push_frontend_feature_flag(:ci_mini_pipeline_gl_dropdown, project, type: :development, default_enabled: :yaml)
push_frontend_feature_flag(:jira_for_vulnerabilities, project, type: :development, default_enabled: :yaml) push_frontend_feature_flag(:jira_for_vulnerabilities, project, type: :development, default_enabled: :yaml)
end end
before_action :ensure_pipeline, only: [:show] before_action :ensure_pipeline, only: [:show]
......
---
name: ci_mini_pipeline_gl_dropdown
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/52821
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/300400
milestone: '13.9'
type: development
group: group::continuous integration
default_enabled: true
...@@ -18,7 +18,6 @@ RSpec.describe 'User adds a merge request to a merge train', :js do ...@@ -18,7 +18,6 @@ RSpec.describe 'User adds a merge request to a merge train', :js do
before do before do
stub_feature_flags(disable_merge_trains: false) stub_feature_flags(disable_merge_trains: false)
stub_feature_flags(ci_mini_pipeline_gl_dropdown: false)
stub_licensed_features(merge_pipelines: true, merge_trains: true) stub_licensed_features(merge_pipelines: true, merge_trains: true)
project.add_maintainer(user) project.add_maintainer(user)
project.update!(merge_pipelines_enabled: true, merge_trains_enabled: true) project.update!(merge_pipelines_enabled: true, merge_trains_enabled: true)
...@@ -70,11 +69,11 @@ RSpec.describe 'User adds a merge request to a merge train', :js do ...@@ -70,11 +69,11 @@ RSpec.describe 'User adds a merge request to a merge train', :js do
end end
it 'displays pipeline control' do it 'displays pipeline control' do
expect(page).to have_selector('[data-testid="mini-pipeline-graph-dropdown-toggle"]') expect(page).to have_selector('[data-testid="mini-pipeline-graph-dropdown"]')
end end
it 'does not allow retry for merge train pipeline' do it 'does not allow retry for merge train pipeline' do
find('[data-testid="mini-pipeline-graph-dropdown-toggle"]').click find('[data-testid="mini-pipeline-graph-dropdown"] .dropdown-toggle').click
page.within '.ci-job-component' do page.within '.ci-job-component' do
expect(page).to have_selector('.ci-status-icon') expect(page).to have_selector('.ci-status-icon')
expect(page).not_to have_selector('.retry') expect(page).not_to have_selector('.retry')
......
...@@ -9,23 +9,15 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do ...@@ -9,23 +9,15 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do
let(:pipeline) { create(:ci_empty_pipeline, project: project, ref: 'master', status: 'running', sha: project.commit.id) } let(:pipeline) { create(:ci_empty_pipeline, project: project, ref: 'master', status: 'running', sha: project.commit.id) }
let(:build) { create(:ci_build, pipeline: pipeline, stage: 'test') } let(:build) { create(:ci_build, pipeline: pipeline, stage: 'test') }
shared_examples 'mini pipeline renders' do |ci_mini_pipeline_gl_dropdown_enabled| dropdown_selector = '[data-testid="mini-pipeline-graph-dropdown"]'
before do before do
build.run build.run
build.trace.set('hello') build.trace.set('hello')
sign_in(user) sign_in(user)
stub_feature_flags(ci_mini_pipeline_gl_dropdown: ci_mini_pipeline_gl_dropdown_enabled)
visit_merge_request visit_merge_request
end end
let_it_be(:dropdown_toggle_selector) do
if ci_mini_pipeline_gl_dropdown_enabled
'[data-testid="mini-pipeline-graph-dropdown"] .dropdown-toggle'
else
'[data-testid="mini-pipeline-graph-dropdown-toggle"]'
end
end
def visit_merge_request(format: :html, serializer: nil) def visit_merge_request(format: :html, serializer: nil)
visit project_merge_request_path(project, merge_request, format: format, serializer: serializer) visit project_merge_request_path(project, merge_request, format: format, serializer: serializer)
end end
...@@ -61,24 +53,24 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do ...@@ -61,24 +53,24 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do
describe 'build list toggle' do describe 'build list toggle' do
let(:toggle) do let(:toggle) do
find(dropdown_toggle_selector) find(dropdown_selector)
first(dropdown_toggle_selector) first(dropdown_selector)
end end
# Status icon button styles should update as described in # Status icon button styles should update as described in
# https://gitlab.com/gitlab-org/gitlab-foss/issues/42769 # https://gitlab.com/gitlab-org/gitlab-foss/issues/42769
it 'has unique styles for default, :hover, :active, and :focus states' do it 'has unique styles for default, :hover, :active, and :focus states' do
default_background_color, default_foreground_color, default_box_shadow = get_toggle_colors(dropdown_toggle_selector) default_background_color, default_foreground_color, default_box_shadow = get_toggle_colors(dropdown_selector)
toggle.hover toggle.hover
hover_background_color, hover_foreground_color, hover_box_shadow = get_toggle_colors(dropdown_toggle_selector) hover_background_color, hover_foreground_color, hover_box_shadow = get_toggle_colors(dropdown_selector)
page.driver.browser.action.click_and_hold(toggle.native).perform page.driver.browser.action.click_and_hold(toggle.native).perform
active_background_color, active_foreground_color, active_box_shadow = get_toggle_colors(dropdown_toggle_selector) active_background_color, active_foreground_color, active_box_shadow = get_toggle_colors(dropdown_selector)
page.driver.browser.action.release(toggle.native).perform page.driver.browser.action.release(toggle.native).perform
page.driver.browser.action.click(toggle.native).move_by(100, 100).perform page.driver.browser.action.click(toggle.native).move_by(100, 100).perform
focus_background_color, focus_foreground_color, focus_box_shadow = get_toggle_colors(dropdown_toggle_selector) focus_background_color, focus_foreground_color, focus_box_shadow = get_toggle_colors(dropdown_selector)
expect(default_background_color).not_to eq(hover_background_color) expect(default_background_color).not_to eq(hover_background_color)
expect(hover_background_color).not_to eq(active_background_color) expect(hover_background_color).not_to eq(active_background_color)
...@@ -106,8 +98,8 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do ...@@ -106,8 +98,8 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do
describe 'builds list menu' do describe 'builds list menu' do
let(:toggle) do let(:toggle) do
find(dropdown_toggle_selector) find(dropdown_selector)
first(dropdown_toggle_selector) first(dropdown_selector)
end end
before do before do
...@@ -151,24 +143,15 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do ...@@ -151,24 +143,15 @@ RSpec.describe 'Merge request < User sees mini pipeline graph', :js do
end end
end end
end end
end
context 'with ci_mini_pipeline_gl_dropdown disabled' do
it_behaves_like "mini pipeline renders", false
end
context 'with ci_mini_pipeline_gl_dropdown enabled' do
it_behaves_like "mini pipeline renders", true
end
private private
def get_toggle_colors(selector) def get_toggle_colors(selector)
find(selector) find(selector)
[ [
evaluate_script("$('#{selector}:visible').css('background-color');"), evaluate_script("$('#{selector} button:visible').css('background-color');"),
evaluate_script("$('#{selector}:visible svg').css('fill');"), evaluate_script("$('#{selector} button:visible svg').css('fill');"),
evaluate_script("$('#{selector}:visible').css('box-shadow');") evaluate_script("$('#{selector} button:visible').css('box-shadow');")
] ]
end end
end end
...@@ -519,7 +519,6 @@ RSpec.describe 'Pipelines', :js do ...@@ -519,7 +519,6 @@ RSpec.describe 'Pipelines', :js do
end end
end end
shared_examples 'mini pipeline renders' do |ci_mini_pipeline_gl_dropdown_enabled|
context 'mini pipeline graph' do context 'mini pipeline graph' do
let!(:build) do let!(:build) do
create(:ci_build, :pending, pipeline: pipeline, create(:ci_build, :pending, pipeline: pipeline,
...@@ -527,33 +526,26 @@ RSpec.describe 'Pipelines', :js do ...@@ -527,33 +526,26 @@ RSpec.describe 'Pipelines', :js do
name: 'build') name: 'build')
end end
dropdown_selector = '[data-testid="mini-pipeline-graph-dropdown"]'
before do before do
stub_feature_flags(ci_mini_pipeline_gl_dropdown: ci_mini_pipeline_gl_dropdown_enabled)
visit_project_pipelines visit_project_pipelines
end end
let_it_be(:dropdown_toggle_selector) do
if ci_mini_pipeline_gl_dropdown_enabled
'[data-testid="mini-pipeline-graph-dropdown"] .dropdown-toggle'
else
'[data-testid="mini-pipeline-graph-dropdown-toggle"]'
end
end
it 'renders a mini pipeline graph' do it 'renders a mini pipeline graph' do
expect(page).to have_selector('[data-testid="widget-mini-pipeline-graph"]') expect(page).to have_selector('[data-testid="widget-mini-pipeline-graph"]')
expect(page).to have_selector(dropdown_toggle_selector) expect(page).to have_selector(dropdown_selector)
end end
context 'when clicking a stage badge' do context 'when clicking a stage badge' do
it 'opens a dropdown' do it 'opens a dropdown' do
find(dropdown_toggle_selector).click find(dropdown_selector).click
expect(page).to have_link build.name expect(page).to have_link build.name
end end
it 'is possible to cancel pending build' do it 'is possible to cancel pending build' do
find(dropdown_toggle_selector).click find(dropdown_selector).click
find('.js-ci-action').click find('.js-ci-action').click
wait_for_requests wait_for_requests
...@@ -569,7 +561,7 @@ RSpec.describe 'Pipelines', :js do ...@@ -569,7 +561,7 @@ RSpec.describe 'Pipelines', :js do
end end
it 'displays the failure reason' do it 'displays the failure reason' do
find(dropdown_toggle_selector).click find(dropdown_selector).click
within('.js-builds-dropdown-list') do within('.js-builds-dropdown-list') do
build_element = page.find('.mini-pipeline-graph-dropdown-item') build_element = page.find('.mini-pipeline-graph-dropdown-item')
...@@ -578,15 +570,6 @@ RSpec.describe 'Pipelines', :js do ...@@ -578,15 +570,6 @@ RSpec.describe 'Pipelines', :js do
end end
end end
end end
end
context 'with ci_mini_pipeline_gl_dropdown disabled' do
it_behaves_like "mini pipeline renders", false
end
context 'with ci_mini_pipeline_gl_dropdown enabled' do
it_behaves_like "mini pipeline renders", true
end
context 'with pagination' do context 'with pagination' do
before do before do
......
...@@ -69,7 +69,8 @@ describe('Pipelines', () => { ...@@ -69,7 +69,8 @@ describe('Pipelines', () => {
const findRunPipelineButton = () => wrapper.findByTestId('run-pipeline-button'); const findRunPipelineButton = () => wrapper.findByTestId('run-pipeline-button');
const findCiLintButton = () => wrapper.findByTestId('ci-lint-button'); const findCiLintButton = () => wrapper.findByTestId('ci-lint-button');
const findCleanCacheButton = () => wrapper.findByTestId('clear-cache-button'); const findCleanCacheButton = () => wrapper.findByTestId('clear-cache-button');
const findStagesDropdown = () => wrapper.findByTestId('mini-pipeline-graph-dropdown-toggle'); const findStagesDropdownToggle = () =>
wrapper.find('[data-testid="mini-pipeline-graph-dropdown"] .dropdown-toggle');
const findPipelineUrlLinks = () => wrapper.findAll('[data-testid="pipeline-url-link"]'); const findPipelineUrlLinks = () => wrapper.findAll('[data-testid="pipeline-url-link"]');
const createComponent = (props = defaultProps) => { const createComponent = (props = defaultProps) => {
...@@ -642,7 +643,7 @@ describe('Pipelines', () => { ...@@ -642,7 +643,7 @@ describe('Pipelines', () => {
// Mock init a polling cycle // Mock init a polling cycle
wrapper.vm.poll.options.notificationCallback(true); wrapper.vm.poll.options.notificationCallback(true);
findStagesDropdown().trigger('click'); findStagesDropdownToggle().trigger('click');
await waitForPromises(); await waitForPromises();
...@@ -652,7 +653,9 @@ describe('Pipelines', () => { ...@@ -652,7 +653,9 @@ describe('Pipelines', () => {
}); });
it('stops polling & restarts polling', async () => { it('stops polling & restarts polling', async () => {
findStagesDropdown().trigger('click'); findStagesDropdownToggle().trigger('click');
await waitForPromises();
expect(cancelMock).not.toHaveBeenCalled(); expect(cancelMock).not.toHaveBeenCalled();
expect(stopMock).toHaveBeenCalled(); expect(stopMock).toHaveBeenCalled();
......
...@@ -153,11 +153,10 @@ describe('Pipelines Table Row', () => { ...@@ -153,11 +153,10 @@ describe('Pipelines Table Row', () => {
}); });
it('should render an icon for each stage', () => { it('should render an icon for each stage', () => {
expect( const stages = wrapper.findAll(
wrapper.findAll( '.table-section:nth-child(5) [data-testid="mini-pipeline-graph-dropdown"]',
'.table-section:nth-child(5) [data-testid="mini-pipeline-graph-dropdown-toggle"]', );
).length, expect(stages).toHaveLength(pipeline.details.stages.length);
).toEqual(pipeline.details.stages.length);
}); });
}); });
......
import 'bootstrap/js/dist/dropdown';
import { GlDropdown } from '@gitlab/ui'; import { GlDropdown } from '@gitlab/ui';
import { mount } from '@vue/test-utils'; import { mount } from '@vue/test-utils';
import MockAdapter from 'axios-mock-adapter'; import MockAdapter from 'axios-mock-adapter';
import $ from 'jquery';
import axios from '~/lib/utils/axios_utils'; import axios from '~/lib/utils/axios_utils';
import StageComponent from '~/pipelines/components/pipelines_list/stage.vue'; import StageComponent from '~/pipelines/components/pipelines_list/stage.vue';
import eventHub from '~/pipelines/event_hub'; import eventHub from '~/pipelines/event_hub';
import { stageReply } from './mock_data'; import { stageReply } from './mock_data';
const dropdownPath = 'path.json';
describe('Pipelines stage component', () => { describe('Pipelines stage component', () => {
let wrapper; let wrapper;
let mock; let mock;
let glFeatures;
const defaultProps = { const createComponent = (props = {}) => {
wrapper = mount(StageComponent, {
attachTo: document.body,
propsData: {
stage: { stage: {
status: { status: {
group: 'success', group: 'success',
icon: 'status_success', icon: 'status_success',
title: 'success', title: 'success',
}, },
dropdown_path: 'path.json', dropdown_path: dropdownPath,
}, },
updateDropdown: false, updateDropdown: false,
};
const createComponent = (props = {}) => {
wrapper = mount(StageComponent, {
attachTo: document.body,
propsData: {
...defaultProps,
...props, ...props,
}, },
provide: {
glFeatures,
},
}); });
}; };
beforeEach(() => { beforeEach(() => {
mock = new MockAdapter(axios); mock = new MockAdapter(axios);
jest.spyOn(eventHub, '$emit'); jest.spyOn(eventHub, '$emit');
glFeatures = {};
}); });
afterEach(() => { afterEach(() => {
...@@ -52,175 +43,74 @@ describe('Pipelines stage component', () => { ...@@ -52,175 +43,74 @@ describe('Pipelines stage component', () => {
mock.restore(); mock.restore();
}); });
describe('when ci_mini_pipeline_gl_dropdown feature flag is disabled', () => { const findDropdown = () => wrapper.findComponent(GlDropdown);
const isDropdownOpen = () => wrapper.classes('show'); const findDropdownToggle = () => wrapper.find('button.dropdown-toggle');
describe('default', () => {
beforeEach(() => {
createComponent();
});
it('should render a dropdown with the status icon', () => {
expect(wrapper.attributes('class')).toEqual('dropdown');
expect(wrapper.find('svg').exists()).toBe(true);
expect(wrapper.find('button').attributes('data-toggle')).toEqual('dropdown');
});
});
describe('with successful request', () => {
beforeEach(() => {
mock.onGet('path.json').reply(200, stageReply);
createComponent();
});
it('should render the received data and emit `clickedDropdown` event', async () => {
wrapper.find('button').trigger('click');
await axios.waitForAll();
expect(wrapper.find('.js-builds-dropdown-container ul').text()).toContain(
stageReply.latest_statuses[0].name,
);
expect(eventHub.$emit).toHaveBeenCalledWith('clickedDropdown');
});
});
it('when request fails should close the dropdown', async () => {
mock.onGet('path.json').reply(500);
createComponent();
wrapper.find({ ref: 'dropdown' }).trigger('click');
expect(isDropdownOpen()).toBe(true);
wrapper.find('button').trigger('click');
await axios.waitForAll();
expect(isDropdownOpen()).toBe(false);
});
describe('update endpoint correctly', () => {
beforeEach(() => {
const copyStage = { ...stageReply };
copyStage.latest_statuses[0].name = 'this is the updated content';
mock.onGet('bar.json').reply(200, copyStage);
createComponent({
stage: {
status: {
group: 'running',
icon: 'status_running',
title: 'running',
},
dropdown_path: 'bar.json',
},
});
return axios.waitForAll();
});
it('should update the stage to request the new endpoint provided', async () => {
wrapper.find('button').trigger('click');
await axios.waitForAll();
expect(wrapper.find('.js-builds-dropdown-container ul').text()).toContain(
'this is the updated content',
);
});
});
describe('pipelineActionRequestComplete', () => {
beforeEach(() => {
mock.onGet('path.json').reply(200, stageReply);
mock.onPost(`${stageReply.latest_statuses[0].status.action.path}.json`).reply(200);
});
const clickCiAction = async () => {
wrapper.find('button').trigger('click');
await axios.waitForAll();
wrapper.find('.js-ci-action').trigger('click');
await axios.waitForAll();
};
describe('within pipeline table', () => {
it('emits `refreshPipelinesTable` event when `pipelineActionRequestComplete` is triggered', async () => {
createComponent({ type: 'PIPELINES_TABLE' });
await clickCiAction();
expect(eventHub.$emit).toHaveBeenCalledWith('refreshPipelinesTable');
});
});
describe('in MR widget', () => {
beforeEach(() => {
jest.spyOn($.fn, 'dropdown');
});
it('closes the dropdown when `pipelineActionRequestComplete` is triggered', async () => {
createComponent();
await clickCiAction();
expect($.fn.dropdown).toHaveBeenCalledWith('toggle');
});
});
});
});
describe('when ci_mini_pipeline_gl_dropdown feature flag is enabled', () => {
const findDropdown = () => wrapper.find(GlDropdown);
const findDropdownToggle = () => wrapper.find('button.gl-dropdown-toggle');
const findDropdownMenu = () => const findDropdownMenu = () =>
wrapper.find('[data-testid="mini-pipeline-graph-dropdown-menu-list"]'); wrapper.find('[data-testid="mini-pipeline-graph-dropdown-menu-list"]');
const findCiActionBtn = () => wrapper.find('.js-ci-action'); const findCiActionBtn = () => wrapper.find('.js-ci-action');
const openGlDropdown = () => { const openStageDropdown = () => {
findDropdownToggle().trigger('click'); findDropdownToggle().trigger('click');
return new Promise((resolve) => { return new Promise((resolve) => {
wrapper.vm.$root.$on('bv::dropdown::show', resolve); wrapper.vm.$root.$on('bv::dropdown::show', resolve);
}); });
}; };
beforeEach(() => { describe('default appearance', () => {
glFeatures = { ciMiniPipelineGlDropdown: true };
});
describe('default', () => {
beforeEach(() => { beforeEach(() => {
createComponent(); createComponent();
}); });
it('should render a dropdown with the status icon', () => { it('should render a dropdown with the status icon', () => {
expect(findDropdown().exists()).toBe(true); expect(findDropdown().exists()).toBe(true);
expect(findDropdownToggle().classes('gl-dropdown-toggle')).toEqual(true); expect(findDropdownToggle().exists()).toBe(true);
expect(wrapper.find('[data-testid="status_success_borderless-icon"]').exists()).toBe(true); expect(wrapper.find('[data-testid="status_success_borderless-icon"]').exists()).toBe(true);
}); });
}); });
describe('with successful request', () => { describe('when update dropdown is changed', () => {
beforeEach(() => { beforeEach(() => {
mock.onGet('path.json').reply(200, stageReply);
createComponent(); createComponent();
}); });
});
it('should render the received data and emit `clickedDropdown` event', async () => { describe('when user opens dropdown and stage request is successful', () => {
await openGlDropdown(); beforeEach(async () => {
mock.onGet(dropdownPath).reply(200, stageReply);
createComponent();
await openStageDropdown();
await axios.waitForAll(); await axios.waitForAll();
});
it('should render the received data and emit `clickedDropdown` event', async () => {
expect(findDropdownMenu().text()).toContain(stageReply.latest_statuses[0].name); expect(findDropdownMenu().text()).toContain(stageReply.latest_statuses[0].name);
expect(eventHub.$emit).toHaveBeenCalledWith('clickedDropdown'); expect(eventHub.$emit).toHaveBeenCalledWith('clickedDropdown');
}); });
});
it('when request fails should close the dropdown', async () => { it('should refresh when updateDropdown is set to true', async () => {
mock.onGet('path.json').reply(500); expect(mock.history.get).toHaveLength(1);
wrapper.setProps({ updateDropdown: true });
await axios.waitForAll();
expect(mock.history.get).toHaveLength(2);
});
});
describe('when user opens dropdown and stage request fails', () => {
beforeEach(async () => {
mock.onGet(dropdownPath).reply(500);
createComponent(); createComponent();
await openGlDropdown(); await openStageDropdown();
await axios.waitForAll(); await axios.waitForAll();
});
it('should close the dropdown', () => {
expect(findDropdown().classes('show')).toBe(false); expect(findDropdown().classes('show')).toBe(false);
}); });
});
describe('update endpoint correctly', () => { describe('update endpoint correctly', () => {
beforeEach(async () => { beforeEach(async () => {
...@@ -241,7 +131,7 @@ describe('Pipelines stage component', () => { ...@@ -241,7 +131,7 @@ describe('Pipelines stage component', () => {
}); });
it('should update the stage to request the new endpoint provided', async () => { it('should update the stage to request the new endpoint provided', async () => {
await openGlDropdown(); await openStageDropdown();
await axios.waitForAll(); await axios.waitForAll();
expect(findDropdownMenu().text()).toContain('this is the updated content'); expect(findDropdownMenu().text()).toContain('this is the updated content');
...@@ -250,12 +140,12 @@ describe('Pipelines stage component', () => { ...@@ -250,12 +140,12 @@ describe('Pipelines stage component', () => {
describe('pipelineActionRequestComplete', () => { describe('pipelineActionRequestComplete', () => {
beforeEach(() => { beforeEach(() => {
mock.onGet('path.json').reply(200, stageReply); mock.onGet(dropdownPath).reply(200, stageReply);
mock.onPost(`${stageReply.latest_statuses[0].status.action.path}.json`).reply(200); mock.onPost(`${stageReply.latest_statuses[0].status.action.path}.json`).reply(200);
}); });
const clickCiAction = async () => { const clickCiAction = async () => {
await openGlDropdown(); await openStageDropdown();
await axios.waitForAll(); await axios.waitForAll();
findCiActionBtn().trigger('click'); findCiActionBtn().trigger('click');
...@@ -276,7 +166,6 @@ describe('Pipelines stage component', () => { ...@@ -276,7 +166,6 @@ describe('Pipelines stage component', () => {
describe('in MR widget', () => { describe('in MR widget', () => {
beforeEach(() => { beforeEach(() => {
jest.spyOn($.fn, 'dropdown');
createComponent(); createComponent();
}); });
...@@ -293,5 +182,4 @@ describe('Pipelines stage component', () => { ...@@ -293,5 +182,4 @@ describe('Pipelines stage component', () => {
}); });
}); });
}); });
});
}); });
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