add some more checks

Signed-off-by: Lamont Granquist <[email protected]>

Author: lamont-granquist

spec/unit/health_check_spec.rb

@@ -109,6 +109,32 @@ def mkdump(base, size, x64 = false)
         )
       end
 
+      let(:empty_list) do
+        double("Mixlib::Shellout",
+          error!: false,
+          stdout: <<~EOH
+          EOH
+        )
+      end
+
+      let(:failed_list) do
+        double("Mixlib::Shellout",
+          error!: true,
+          stdout: <<~EOH
+            /opt/chefdk/shouldnt/matter
+          EOH
+        )
+      end
+
+      let(:bad_list) do
+        double("Mixlib::Shellout",
+          error!: false,
+          stdout: <<~EOH
+            /somewhere/other/than/install/dir
+          EOH
+        )
+      end
+
       let(:bad_healthcheck) do
         double("Mixlib::Shellout",
           error!: false,
@@ -174,6 +200,30 @@ def mkdump(base, size, x64 = false)
       it "will not perform dll base relocation checks" do
         expect(subject.relocation_checkable?).to be false
       end
+
+      it "raises an exception if there's nothing in the file list" do
+        allow(subject).to receive(:shellout)
+          .with("find /opt/chefdk/ -type f")
+          .and_return(empty_list)
+
+        expect { subject.run! }.to raise_error(HealthCheckFailed)
+      end
+
+      it "raises an exception if the file list command raises" do
+        allow(subject).to receive(:shellout)
+          .with("find /opt/chefdk/ -type f")
+          .and_return(failed_list)
+
+        expect { subject.run! }.to raise_error(HealthCheckFailed)
+      end
+
+      it "raises an exception if the file list command has no entries in the install_dir" do
+        allow(subject).to receive(:shellout)
+          .with("find /opt/chefdk/ -type f")
+          .and_return(bad_list)
+
+        expect { subject.run! }.to raise_error(HealthCheckFailed)
+      end
     end
   end
 end