Commit 33ba8d4e authored by Furkan Ayhan's avatar Furkan Ayhan Committed by Jose Ivan Vargas

Remove FF new_pipeline_form_prefilled_vars

It was enabled by default
parent 779fbe05
...@@ -251,10 +251,6 @@ export default { ...@@ -251,10 +251,6 @@ export default {
return index < this.variables.length - 1; return index < this.variables.length - 1;
}, },
fetchConfigVariables(refValue) { fetchConfigVariables(refValue) {
if (!gon?.features?.newPipelineFormPrefilledVars) {
return Promise.resolve({ params: {}, descriptions: {} });
}
this.isLoading = true; this.isLoading = true;
return backOff((next, stop) => { return backOff((next, stop) => {
......
...@@ -17,7 +17,6 @@ class Projects::PipelinesController < Projects::ApplicationController ...@@ -17,7 +17,6 @@ class Projects::PipelinesController < Projects::ApplicationController
push_frontend_feature_flag(:graphql_pipeline_header, project, type: :development, default_enabled: false) push_frontend_feature_flag(:graphql_pipeline_header, project, type: :development, default_enabled: false)
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(:new_pipeline_form_prefilled_vars, project, type: :development, default_enabled: true)
end end
before_action :ensure_pipeline, only: [:show] before_action :ensure_pipeline, only: [:show]
before_action :push_experiment_to_gon, only: :index, if: :html_request? before_action :push_experiment_to_gon, only: :index, if: :html_request?
......
---
name: new_pipeline_form_prefilled_vars
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/44120
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/263276
milestone: '13.5'
type: development
group: group::continuous integration
default_enabled: true
...@@ -225,42 +225,29 @@ describe('Pipeline New Form', () => { ...@@ -225,42 +225,29 @@ describe('Pipeline New Form', () => {
}); });
}); });
describe('when feature flag new_pipeline_form_prefilled_vars is enabled', () => { describe('when yml defines a variable', () => {
let origGon;
const mockYmlKey = 'yml_var'; const mockYmlKey = 'yml_var';
const mockYmlValue = 'yml_var_val'; const mockYmlValue = 'yml_var_val';
const mockYmlDesc = 'A var from yml.'; const mockYmlDesc = 'A var from yml.';
beforeAll(() => { it('loading icon is shown when content is requested and hidden when received', async () => {
origGon = window.gon; createComponent('', mockParams, mount);
window.gon = { features: { newPipelineFormPrefilledVars: true } };
});
afterAll(() => {
window.gon = origGon;
});
describe('loading state', () => {
it('loading icon is shown when content is requested and hidden when received', async () => {
createComponent('', mockParams, mount);
mock.onGet(configVariablesPath).reply(httpStatusCodes.OK, { mock.onGet(configVariablesPath).reply(httpStatusCodes.OK, {
[mockYmlKey]: { [mockYmlKey]: {
value: mockYmlValue, value: mockYmlValue,
description: mockYmlDesc, description: mockYmlDesc,
}, },
}); });
expect(findLoadingIcon().exists()).toBe(true); expect(findLoadingIcon().exists()).toBe(true);
await waitForPromises(); await waitForPromises();
expect(findLoadingIcon().exists()).toBe(false); expect(findLoadingIcon().exists()).toBe(false);
});
}); });
describe('when yml defines a variable with description', () => { describe('with description', () => {
beforeEach(async () => { beforeEach(async () => {
createComponent('', mockParams, mount); createComponent('', mockParams, mount);
...@@ -302,7 +289,7 @@ describe('Pipeline New Form', () => { ...@@ -302,7 +289,7 @@ describe('Pipeline New Form', () => {
}); });
}); });
describe('when yml defines a variable without description', () => { describe('without description', () => {
beforeEach(async () => { beforeEach(async () => {
createComponent('', mockParams, mount); createComponent('', mockParams, mount);
......
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