Add spec test to cover the deletion of the cluster id state

Author: hgreebe

cookbooks/aws-parallelcluster-slurm/spec/unit/recipes/clear_slurm_accounting_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe 'aws-parallelcluster-slurm::clear_slurm_accounting' do
+  for_all_oses do |platform, version|
+    context "on #{platform}#{version}" do
+        cached(:chef_run) do
+          runner = runner(platform: platform, version: version) do |node|
+            mock_file_exists("/var/spool/slurm.state/clustername", true)
+            node.override['cluster']['slurmdbd_service_enabled'] = true
+          end
+          runner.converge(described_recipe)
+        end
+        cached(:node) { chef_run.node }
+
+        it 'stops the slurm database daemon' do
+          is_expected.to disable_service("slurmdbd")
+        end
+
+        it 'deletes the Slurm database password update script' do
+          is_expected.to delete_file("#{node['cluster']['scripts_dir']}/slurm/update_slurm_database_password.sh")
+        end
+
+        it 'Removes existing cluster name state file' do
+          is_expected.to run_bash("Remove existing cluster name state file")
+        end
+    end
+  end
+end

cookbooks/aws-parallelcluster-slurm/spec/unit/recipes/config_slurm_accounting_spec.rb

@@ -10,6 +10,7 @@
               allow_any_instance_of(Object).to receive(:are_mount_or_unmount_required?).and_return(false)
               allow_any_instance_of(Object).to receive(:dig).and_return(true)
               RSpec::Mocks.configuration.allow_message_expectations_on_nil = true
+              mock_file_exists("/var/spool/slurm.state/clustername", true)
               node.override['cluster']['slurmdbd_service_enabled'] = enable_service
             end
             runner.converge(described_recipe)
@@ -70,6 +71,9 @@
             )
           end
           if enable_service == "true"
+            it 'Removes existing cluster name state file' do
+              is_expected.to run_bash("Remove existing cluster name state file")
+            end
             it 'starts the slurm database daemon' do
               is_expected.to enable_service("slurmdbd")
               is_expected.to start_service("slurmdbd")