Add more tests for resource semi-colons

Replaces existing fail cases

Author: alexjfisher

spec/fixtures/fail/semicolon_long.pp

@@ -1,4 +1,7 @@
 require 'spec_helper'
+# rubocop:disable Lint/RedundantRequireStatement
+require 'pp'
+# rubocop:enable Lint/RedundantRequireStatement
 
 describe 'strict_indent' do
   before do
@@ -90,4 +93,74 @@ class () {}
       expect(problems).to have(0).problems
     end
   end
+
+  context 'misplaced resource semicolons' do
+    before do
+      PuppetLint.configuration.fix = true
+    end
+
+    after do
+      PuppetLint.configuration.fix = false
+    end
+
+    let(:code) do
+      <<~EOF
+        file {
+          default:
+            ensure => file,
+            owner  => 'root',
+            group  => 'wheel',
+            mode   => '0600',
+            ;
+          ['ssh_host_dsa_key', 'ssh_host_key', 'ssh_host_rsa_key']:
+            # use all defaults
+            ;
+          ['ssh_config', 'ssh_host_dsa_key.pub', 'ssh_host_key.pub', 'ssh_host_rsa_key.pub', 'sshd_config']:
+            # override mode
+            mode => '0644',
+        ;
+        }
+      EOF
+    end
+
+    let(:fixed) do
+      <<~EOF
+        file {
+          default:
+            ensure => file,
+            owner  => 'root',
+            group  => 'wheel',
+            mode   => '0600',
+          ;
+          ['ssh_host_dsa_key', 'ssh_host_key', 'ssh_host_rsa_key']:
+            # use all defaults
+          ;
+          ['ssh_config', 'ssh_host_dsa_key.pub', 'ssh_host_key.pub', 'ssh_host_rsa_key.pub', 'sshd_config']:
+            # override mode
+            mode => '0644',
+          ;
+        }
+      EOF
+    end
+
+    it 'detects three problems' do
+      expect(problems).to have(3).problem
+    end
+
+    it 'fixes the first problem' do
+      expect(problems).to contain_fixed('indent should be 2 chars and is 4').on_line(7).in_column(1)
+    end
+
+    it 'fixes the second problem' do
+      expect(problems).to contain_fixed('indent should be 2 chars and is 4').on_line(10).in_column(1)
+    end
+
+    it 'fixes the third problem' do
+      expect(problems).to contain_fixed('indent should be 2 chars and is 0').on_line(14).in_column(1)
+    end
+
+    it 'moves the semicolons' do
+      expect(manifest).to eq fixed
+    end
+  end
 end