Add CLI option for override script directory

Author: Kevin Paulisse

lib/octocatalog-diff/cli/options/override_script_path.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+# Provide an optional directory to override default built-in scripts such as git checkout
+# and puppet version determination.
+# @param parser [OptionParser object] The OptionParser argument
+# @param options [Hash] Options hash being constructed; this is modified in this method.
+OctocatalogDiff::Cli::Options::Option.newoption(:override_script_path) do
+  has_weight 385
+
+  def parse(parser, options)
+    parser.on('--override-script-path DIRNAME', 'Directory with scripts to override built-ins') do |dir|
+      unless dir.start_with?('/')
+        raise ArgumentError, 'Absolute path is required for --override-script-path'
+      end
+
+      unless File.directory?(dir)
+        raise Errno::ENOENT, 'Invalid --override-script-path'
+      end
+
+      options[:override_script_path] = dir
+    end
+  end
+end

spec/octocatalog-diff/tests/cli/options/override_script_path_spec.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require_relative '../options_helper'
+
+describe OctocatalogDiff::Cli::Options do
+  describe '#opt_override_script_path' do
+    context 'with relative path' do
+      it 'should raise ArgumentError' do
+        expect do
+          run_optparse(['--override-script-path', '../foo/bar'])
+        end.to raise_error(ArgumentError, 'Absolute path is required for --override-script-path')
+      end
+    end
+
+    context 'with non-existing directory' do
+      it 'should raise Errno::ENOENT' do
+        expect(File).to receive(:'directory?').with('/foo/bar').and_return(false)
+        expect do
+          run_optparse(['--override-script-path', '/foo/bar'])
+        end.to raise_error(Errno::ENOENT, /Invalid --override-script-path/)
+      end
+    end
+
+    context 'with existing directory' do
+      it 'should establish option' do
+        expect(File).to receive(:'directory?').with('/foo/bar').and_return(true)
+        result = run_optparse(['--override-script-path', '/foo/bar'])
+        expect(result[:override_script_path]).to eq('/foo/bar')
+      end
+    end
+  end
+end