Merge branch...

Merge branch '258630-environment-variables-are-not-getting-used-under-the-flag-new_pipeline_form' into 'master'

Fix variables payload format to create a pipeline

Closes #258630

See merge request gitlab-org/gitlab!43835

app/assets/javascripts/pipeline_new/components/pipeline_new_form.vue

@@ -170,13 +170,13 @@ export default {
         .map(({ variable_type, key, value }) => ({
           variable_type,
           key,
-          value,
+          secret_value: value,
         }));
 
       return axios
         .post(this.pipelinesPath, {
           ref: this.refValue,
-          variables: filteredVariables,
+          variables_attributes: filteredVariables,
         })
         .then(({ data }) => {
           redirectTo(`${this.pipelinesPath}/${data.id}`);

spec/frontend/pipeline_new/mock_data.js

@@ -14,9 +14,9 @@ export const mockProjectId = '21';
 
 export const mockPostParams = {
   ref: 'tag-1',
-  variables: [
-    { key: 'test_var', value: 'test_var_val', variable_type: 'env_var' },
-    { key: 'test_file', value: 'test_file_val', variable_type: 'file' },
+  variables_attributes: [
+    { key: 'test_var', secret_value: 'test_var_val', variable_type: 'env_var' },
+    { key: 'test_file', secret_value: 'test_file_val', variable_type: 'file' },
   ],
 };