skip regression check unit tests if octo-diff absent

Author: mschuchard

spec/puppet-check/regression_check_spec.rb

@@ -1,37 +1,41 @@
 require_relative '../spec_helper'
-require_relative '../../lib/puppet-check/regression_check'
+begin
+  require_relative '../../lib/puppet-check/regression_check'
 
-# once a good config is loaded, bad configs would have no effect so failures are tested first; config is also tested before compilation so that the good config can be loaded at the end and used for compilation and regression tests
-describe RegressionCheck do
-  context '.config' do
-    # json gem got messed up for the EOL Ruby versions
-    it 'raise an appropriate error if the file is malformed' do
-      expect { RegressionCheck.config("#{fixtures_dir}metadata.json") }.to raise_error(OctocatalogDiff::Errors::ConfigurationFileContentError, 'Configuration must define OctocatalogDiff::Config!')
-    end
-    it 'loads in a good octocatalog-diff config file' do
-      expect { RegressionCheck.config("#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.not_to raise_exception
-    end
-    it 'loads in the settings from the file correctly' do
-      # TODO
+  # once a good config is loaded, bad configs would have no effect so failures are tested first; config is also tested before compilation so that the good config can be loaded at the end and used for compilation and regression tests
+  describe RegressionCheck do
+    context '.config' do
+      # json gem is messed up for the EOL Ruby versions
+      it 'raise an appropriate error if the file is malformed' do
+        expect { RegressionCheck.config("#{fixtures_dir}metadata.json") }.to raise_error(OctocatalogDiff::Errors::ConfigurationFileContentError, 'Configuration must define OctocatalogDiff::Config!')
+      end
+      it 'loads in a good octocatalog-diff config file' do
+        expect { RegressionCheck.config("#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.not_to raise_exception
+      end
+      it 'loads in the settings from the file correctly' do
+        # TODO
+      end
     end
-  end
 
-  context '.smoke' do
-    # octocatalog-diff is returning a blank error for these tests
-    unless ENV['CIRCLECI'] == 'true' || ENV['GITHUB_ACTIONS'] == 'true'
-      it 'returns a pass for a successful catalog compilation' do
-        expect { RegressionCheck.smoke(['good.example.com'], "#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.not_to raise_exception
+    context '.smoke' do
+      # octocatalog-diff is returning a blank error for these tests
+      unless ENV['CIRCLECI'] == 'true' || ENV['GITHUB_ACTIONS'] == 'true'
+        it 'returns a pass for a successful catalog compilation' do
+          expect { RegressionCheck.smoke(['good.example.com'], "#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.not_to raise_exception
+        end
+        it 'returns a failure for a catalog with an error' do
+          expect { RegressionCheck.smoke(['does_not_exist.example.com'], "#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.to raise_error(OctocatalogDiff::Errors::CatalogError)
+        end
       end
-      it 'returns a failure for a catalog with an error' do
-        expect { RegressionCheck.smoke(['does_not_exist.example.com'], "#{octocatalog_diff_dir}octocatalog_diff.cfg.rb") }.to raise_error(OctocatalogDiff::Errors::CatalogError)
+      it 'returns a failure for a good and bad catalog' do
+        # RegressionCheck.smoke(['good.example.com', 'syntax_error.example.com'], "#{fixtures_dir}octocatalog_diff.cfg.rb")
       end
     end
-    it 'returns a failure for a good and bad catalog' do
-      # RegressionCheck.smoke(['good.example.com', 'syntax_error.example.com'], "#{fixtures_dir}octocatalog_diff.cfg.rb")
-    end
-  end
 
-  context '.regression' do
-    # TODO
+    context '.regression' do
+      # TODO
+    end
   end
+rescue NameError
+  puts 'octocatalog-diff is not installed, and its unit tests will be skipped'
 end