Add integration test for CLI fact override being used over config file

Author: kpaulisse

spec/octocatalog-diff/fixtures/cli-configs/fact-file.rb

@@ -0,0 +1,14 @@
+# This is a configuration file for octocatalog-diff
+
+module OctocatalogDiff
+  class Config
+    def self.config
+      {
+        facts: OctocatalogDiff::Facts.new(
+          backend: :yaml,
+          fact_file_string: File.read(File.join(ENV['PUPPET_FACT_FILE_DIR'], 'valid-facts.yaml'))
+        )
+      }
+    end
+  end
+end

spec/octocatalog-diff/integration/fact_file_by_branch_spec.rb

@@ -142,3 +142,60 @@
     expect(@result.stderr).to match(/Diffs computed for rspec-node.github.net/)
   end
 end
+
+describe 'fact file from the configuration' do
+  before(:all) do
+    ENV['PUPPET_FACT_FILE_DIR'] = OctocatalogDiff::Spec.fixture_path('facts')
+    @result = OctocatalogDiff::Integration.integration_cli(
+      [
+        '-n', 'rspec-node.github.net',
+        '--bootstrapped-to-dir', OctocatalogDiff::Spec.fixture_path('repos/fact-overrides'),
+        '--catalog-only',
+        '--puppet-binary', OctocatalogDiff::Spec::PUPPET_BINARY,
+        '-d'
+      ],
+      OctocatalogDiff::Spec.fixture_path('cli-configs/fact-file.rb')
+    )
+  end
+
+  after(:all) do
+    ENV.delete('PUPPET_FACT_FILE_DIR')
+  end
+
+  it 'should exit with status 0' do
+    expect(@result.exitcode).to eq(0), @result.stderr
+  end
+
+  it 'should contain resources generated from the proper fact file' do
+    expect(@result.stdout).to match(/"10.20.30.40"/)
+  end
+end
+
+describe 'fact file overridden on the command line' do
+  before(:all) do
+    ENV['PUPPET_FACT_FILE_DIR'] = OctocatalogDiff::Spec.fixture_path('facts')
+    @result = OctocatalogDiff::Integration.integration_cli(
+      [
+        '-n', 'rspec-node.github.net',
+        '--bootstrapped-to-dir', OctocatalogDiff::Spec.fixture_path('repos/fact-overrides'),
+        '--catalog-only',
+        '--puppet-binary', OctocatalogDiff::Spec::PUPPET_BINARY,
+        '-d',
+        '--fact-file', OctocatalogDiff::Spec.fixture_path('facts/valid-facts-different-ip.yaml')
+      ],
+      OctocatalogDiff::Spec.fixture_path('cli-configs/fact-file.rb')
+    )
+  end
+
+  after(:all) do
+    ENV.delete('PUPPET_FACT_FILE_DIR')
+  end
+
+  it 'should exit with status 0' do
+    expect(@result.exitcode).to eq(0), @result.stderr
+  end
+
+  it 'should contain resources generated from the proper fact file' do
+    expect(@result.stdout).to match(/"10.30.50.70"/)
+  end
+end

spec/octocatalog-diff/integration/integration_helper.rb

@@ -31,10 +31,10 @@ def self.integration_with_puppetdb(server_opts, opts)
     end
 
     # For an integration test, run the full CLI and get results
-    def self.integration_cli(argv)
+    def self.integration_cli(argv, override_env = nil)
       script = File.expand_path('../../../bin/octocatalog-diff', File.dirname(__FILE__))
       cmdline = [script, argv].flatten.map { |x| Shellwords.escape(x) }.join(' ')
-      env = { 'OCTOCATALOG_DIFF_CONFIG_FILE' => OctocatalogDiff::Spec.fixture_path('cli-configs/no-op.rb') }
+      env = { 'OCTOCATALOG_DIFF_CONFIG_FILE' => override_env || OctocatalogDiff::Spec.fixture_path('cli-configs/no-op.rb') }
       stdout, stderr, status = Open3.capture3(env, cmdline)
       OpenStruct.new(
         stdout: stdout,