Merge branch '439346-yaml-editor-extra-characters' into 'master'

Fix adding extra characters for pipeline execution policy

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/143068



Merged-by: Alexander Turinske <aturinske@gitlab.com>
Approved-by: Martin Čavoj <mcavoj@gitlab.com>
Approved-by: Alexander Turinske <aturinske@gitlab.com>
Reviewed-by: Martin Čavoj <mcavoj@gitlab.com>
Co-authored-by: Artur Fedorov <afedorov@gitlab.com>

ee/app/assets/javascripts/security_orchestration/components/policy_editor/scan_execution/action/code_block_action.vue

@@ -9,10 +9,7 @@ import { THOUSAND } from '~/lib/utils/constants';
 import CodeBlockSourceSelector from 'ee/security_orchestration/components/policy_editor/scan_execution/action/code_block_source_selector.vue';
 import PolicyPopover from 'ee/security_orchestration/components/policy_popover.vue';
 import { parseCustomFileConfiguration } from 'ee/security_orchestration/components/policy_editor/utils';
-import {
-  buildCustomCodeAction,
-  toYaml,
-} from 'ee/security_orchestration/components/policy_editor/scan_execution/lib';
+import { buildCustomCodeAction } from 'ee/security_orchestration/components/policy_editor/scan_execution/lib';
 import SectionLayout from '../../section_layout.vue';
 import { ACTION_AND_LABEL } from '../../constants';
 import {
@@ -140,10 +137,9 @@ export default {
     },
     updateYaml(val) {
       this.yamlEditorValue = val;
-      const yaml = toYaml(val);
 
       this.triggerChanged({
-        ci_configuration: yaml,
+        ci_configuration: val,
       });
     },
     setSelectedRef(ref) {

ee/spec/frontend/security_orchestration/components/policy_editor/scan_execution/action/code_block_action_spec.js

@@ -2,10 +2,7 @@ import { GlSprintf } from '@gitlab/ui';
 import { shallowMount } from '@vue/test-utils';
 import waitForPromises from 'helpers/wait_for_promises';
 import Api from 'ee/api';
-import {
-  buildCustomCodeAction,
-  toYaml,
-} from 'ee/security_orchestration/components/policy_editor/scan_execution/lib';
+import { buildCustomCodeAction } from 'ee/security_orchestration/components/policy_editor/scan_execution/lib';
 import CodeBlockSourceSelector from 'ee/security_orchestration/components/policy_editor/scan_execution/action/code_block_source_selector.vue';
 import CodeBlockAction from 'ee/security_orchestration/components/policy_editor/scan_execution/action/code_block_action.vue';
 import CodeBlockFilePath from 'ee/security_orchestration/components/policy_editor/scan_execution/action/code_block_file_path.vue';
@@ -75,7 +72,7 @@ describe('CodeBlockAction', () => {
       await findYamlEditor().vm.$emit('input', fileContents);
       expect(findCodeBlockImport().props('hasExistingCode')).toBe(true);
       expect(wrapper.emitted('changed')).toEqual([
-        [{ ...buildCustomCodeAction(), ci_configuration: toYaml(fileContents) }],
+        [{ ...buildCustomCodeAction(), ci_configuration: fileContents }],
       ]);
     });
 
@@ -86,7 +83,7 @@ describe('CodeBlockAction', () => {
       expect(findYamlEditor().props('value')).toBe(fileContents);
       expect(findCodeBlockImport().props('hasExistingCode')).toBe(true);
       expect(wrapper.emitted('changed')).toEqual([
-        [{ ...buildCustomCodeAction(), ci_configuration: toYaml(fileContents) }],
+        [{ ...buildCustomCodeAction(), ci_configuration: fileContents }],
       ]);
     });