Merge branch 'fix-ci-variables-regression' into 'master'

Fix not being able to add more than one variable through the UI

Closes #227400

See merge request gitlab-org/gitlab!37001

app/assets/javascripts/ci_variable_list/store/mutations.js

@@ -74,7 +74,7 @@ export default {
       variable_type: displayText.variableText,
       key: '',
       secret_value: '',
-      protected: false,
+      protected_variable: false,
       masked: false,
       environment_scope: displayText.allEnvironmentsText,
     };
@@ -103,7 +103,7 @@ export default {
   },
 
   [types.SET_VARIABLE_PROTECTED](state) {
-    state.variable.protected = true;
+    state.variable.protected_variable = true;
   },
 
   [types.UPDATE_VARIABLE_KEY](state, key) {

changelogs/unreleased/fix-ci-variables-regression.yml

+---
+title: Fix not being able to add more than one CI variable through the UI
+merge_request: 37001
+author:
+type: fixed

spec/frontend/ci_variable_list/store/mutations_spec.js

@@ -41,7 +41,7 @@ describe('CI variable list mutations', () => {
         variable_type: 'Variable',
         key: '',
         secret_value: '',
-        protected: false,
+        protected_variable: false,
         masked: false,
         environment_scope: 'All (default)',
       };
@@ -85,7 +85,7 @@ describe('CI variable list mutations', () => {
     it('should set protected value to true', () => {
       mutations[types.SET_VARIABLE_PROTECTED](stateCopy);
 
-      expect(stateCopy.variable.protected).toBe(true);
+      expect(stateCopy.variable.protected_variable).toBe(true);
     });
   });