(CAT-2484): Add integration test for Issue #254 YAML overwrite prevention

Verifies that --fix flag does not corrupt YAML files when processing
directories containing both .pp and .yaml files.

Signed-off-by: Gavin Didrichsen <[email protected]>

Author: gavindidrichsen

spec/fixtures/test/manifests/issue_254_overwriting_yaml/class_with_dash.pp

@@ -0,0 +1,3 @@
+class foo-bar {
+  notify { 'hello': }
+}

spec/fixtures/test/manifests/issue_254_overwriting_yaml/class_with_dash.yaml

@@ -0,0 +1,7 @@
+---
+classes:
+  foo-bar:
+    parameters: []
+    resources:
+      notify:
+        - title: hello

spec/unit/puppet-lint/bin_spec.rb

@@ -615,6 +615,24 @@ def initialize(args)
     end
   end
 
+  context 'when fixing a directory containing both .pp and .yaml files' do
+    let(:args) { ['--fix', 'spec/fixtures/test/manifests/issue_254_overwriting_yaml'] }
+
+    its(:exitstatus) { is_expected.to eq(1) }
+
+    it 'does not overwrite YAML files' do
+      yaml_file = 'spec/fixtures/test/manifests/issue_254_overwriting_yaml/class_with_dash.yaml'
+      original_yaml = File.read(yaml_file)
+
+      bin # Run the command
+
+      yaml_after = File.read(yaml_file)
+      expect(yaml_after).to eq(original_yaml)
+      expect(yaml_after).to include('classes:')
+      expect(yaml_after).not_to include('class foo-bar {')
+    end
+  end
+
   context 'when overriding config file options with command line options' do
     context 'and config file sets "--only-checks=variable_contains_dash"' do
       around(:context) do |example|