puppet 6.24.0-x64-mingw32 → 7.0.0-x64-mingw32

data/spec/integration/util/windows/principal_spec.rb

@@ -7,7 +7,6 @@ describe Puppet::Util::Windows::SID::Principal, :if => Puppet::Util::Platform.wi
   let (:system_bytes) { [1, 1, 0, 0, 0, 0, 0, 5, 18, 0, 0, 0] }
   let (:null_sid_bytes) { [1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0] }
   let (:administrator_bytes) { [1, 2, 0, 0, 0, 0, 0, 5, 32, 0, 0, 0, 32, 2, 0, 0] }
-  let (:all_application_packages_bytes) { [1, 2, 0, 0, 0, 0, 0, 15, 2, 0, 0, 0, 1, 0, 0, 0] }
   let (:computer_sid) { Puppet::Util::Windows::SID.name_to_principal(Puppet::Util::Windows::ADSI.computer_name) }
   # BUILTIN is localized on German Windows, but not French
   # looking this up like this dilutes the values of the tests as we're comparing two mechanisms
@@ -122,26 +121,6 @@ describe Puppet::Util::Windows::SID::Principal, :if => Puppet::Util::Platform.wi
       expect(principal.to_s).to eq(builtin_localized)
     end
 
-    it "should always sanitize the account name first" do
-      expect(Puppet::Util::Windows::SID::Principal).to receive(:sanitize_account_name).with('NT AUTHORITY\\SYSTEM').and_call_original
-      Puppet::Util::Windows::SID::Principal.lookup_account_name('NT AUTHORITY\\SYSTEM')
-    end
-
-    it "should be able to create an instance from an account name prefixed by APPLICATION PACKAGE AUTHORITY" do
-      principal = Puppet::Util::Windows::SID::Principal.lookup_account_name('APPLICATION PACKAGE AUTHORITY\\ALL APPLICATION PACKAGES')
-      expect(principal.account).to eq('ALL APPLICATION PACKAGES')
-      expect(principal.sid_bytes).to eq(all_application_packages_bytes)
-      expect(principal.sid).to eq('S-1-15-2-1')
-      expect(principal.domain).to eq('APPLICATION PACKAGE AUTHORITY')
-      expect(principal.domain_account).to eq('APPLICATION PACKAGE AUTHORITY\\ALL APPLICATION PACKAGES')
-      expect(principal.account_type).to eq(:SidTypeWellKnownGroup)
-      expect(principal.to_s).to eq('APPLICATION PACKAGE AUTHORITY\\ALL APPLICATION PACKAGES')
-    end
-
-    it "should fail without proper account name sanitization when it is prefixed by APPLICATION PACKAGE AUTHORITY" do
-      given_account_name = 'APPLICATION PACKAGE AUTHORITY\\ALL APPLICATION PACKAGES'
-      expect { Puppet::Util::Windows::SID::Principal.lookup_account_name(nil, false, given_account_name) }.to raise_error(Puppet::Util::Windows::Error, /No mapping between account names and security IDs was done./)
-    end
   end
 
   describe ".lookup_account_sid" do

data/spec/integration/util/windows/registry_spec.rb

@@ -146,6 +146,16 @@ describe Puppet::Util::Windows::Registry do
         utf_8_bytes = ENDASH_UTF_8 + TM_UTF_8
         utf_8_str = utf_8_bytes.pack('c*').force_encoding(Encoding::UTF_8)
 
+        # this problematic Ruby codepath triggers a conversion of UTF-16LE to
+        # a local codepage which can totally break when that codepage has no
+        # conversion from the given UTF-16LE characters to local codepage
+        # a prime example is that IBM437 has no conversion from a Unicode en-dash
+        expect(Win32::Registry).not_to receive(:export_string)
+
+        # also, expect that we're using our variants of keys / values, not Rubys
+        expect(Win32::Registry).not_to receive(:each_key)
+        expect(Win32::Registry).not_to receive(:each_value)
+
         hklm.create("#{puppet_key}\\#{subkey_name}", Win32::Registry::KEY_ALL_ACCESS | regsam) do |reg|
           reg.write("#{guid}", Win32::Registry::REG_SZ, utf_16_str)
 
@@ -263,12 +273,6 @@ describe Puppet::Util::Windows::Registry do
           type: Win32::Registry::REG_EXPAND_SZ,
           value: "\0\0\0reg expand string",
           expected_value: ""
-        },
-        {
-          name: 'REG_EXPAND_SZ_2',
-          type: Win32::Registry::REG_EXPAND_SZ,
-          value: "1\x002\x003\x004\x00\x00\x00\x90\xD8UoY".force_encoding("UTF-16LE"),
-          expected_value: "1234"
         }
       ].each do |pair|
         it 'reads up to the first wide null' do

data/spec/integration/util/windows/security_spec.rb

@@ -822,7 +822,7 @@ describe "Puppet::Util::Windows::Security", :if => Puppet::Util::Platform.window
     end
 
     describe "locked files" do
-      let (:explorer) { File.join(Dir::WINDOWS, "explorer.exe") }
+      let (:explorer) { File.join(ENV['SystemRoot'], "explorer.exe") }
 
       it "should get the owner" do
         expect(winsec.get_owner(explorer)).to match(/^S-1-5-/)

data/spec/lib/matchers/include.rb

@@ -0,0 +1,27 @@
+module Matchers; module Include
+  extend RSpec::Matchers::DSL
+
+  matcher :include_in_any_order do |*matchers|
+    match do |enumerable|
+      @not_matched = []
+      expected_as_array.each do |matcher|
+        if enumerable.empty?
+          break
+        end
+
+        if found = enumerable.find { |elem| matcher.matches?(elem) }
+          enumerable = enumerable.reject { |elem| elem == found }
+        else
+          @not_matched << matcher
+        end
+      end
+
+
+      @not_matched.empty? && enumerable.empty?
+    end
+
+    failure_message do |enumerable|
+      "did not match #{@not_matched.collect(&:description).join(', ')} in #{enumerable.inspect}: <#{@not_matched.collect(&:failure_message).join('>, <')}>"
+    end
+  end
+end; end

data/spec/lib/matchers/include_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+require 'matchers/include'
+
+describe "include matchers" do
+  include Matchers::Include
+
+  context :include_in_any_order do
+    it "matches an empty list" do
+      expect([]).to include_in_any_order()
+    end
+
+    it "matches a list with a single element" do
+      expect([1]).to include_in_any_order(eq(1))
+    end
+
+    it "does not match when an expected element is missing" do
+      expect([1]).to_not include_in_any_order(eq(2))
+    end
+
+    it "matches a list with 2 elements in a different order from the expectation" do
+      expect([1, 2]).to include_in_any_order(eq(2), eq(1))
+    end
+
+    it "does not match when there are more than just the expected elements" do
+      expect([1, 2]).to_not include_in_any_order(eq(1))
+    end
+
+    it "matches multiple, equal elements when there are multiple, equal exepectations" do
+      expect([1, 1]).to include_in_any_order(eq(1), eq(1))
+    end
+  end
+end

data/spec/lib/puppet/test_ca.rb

@@ -30,7 +30,7 @@ module Puppet
     end
 
     def create_request(name)
-      key = OpenSSL::PKey::RSA.new(2048)
+      key = OpenSSL::PKey::RSA.new(1024)
       csr = OpenSSL::X509::Request.new
       csr.public_key = key.public_key
       csr.subject = OpenSSL::X509::Name.new([["CN", name]])
@@ -46,11 +46,6 @@ module Puppet
         ext = ef.create_extension(["subjectAltName", opts[:subject_alt_names], false])
         cert.add_extension(ext)
       end
-      if exts = opts[:extensions]
-        exts.each do |e|
-          cert.add_extension(OpenSSL::X509::Extension.new(*e))
-        end
-      end
       cert.sign(issuer_key, @digest)
       { private_key: key, cert: cert }
     end
@@ -132,7 +127,7 @@ module Puppet
       key = if opts[:key_type] == :ec
               key = OpenSSL::PKey::EC.generate('prime256v1')
             else
-              key = OpenSSL::PKey::RSA.new(2048)
+              key = OpenSSL::PKey::RSA.new(1024)
             end
       cert = OpenSSL::X509::Certificate.new
       cert.public_key = if key.is_a?(OpenSSL::PKey::EC)

data/spec/lib/puppet_spec/puppetserver.rb

@@ -103,7 +103,7 @@ class PuppetSpec::Puppetserver
   end
 
   def register_mounts(mounts: {})
-    register_mount('/status/v1/simple/master', proc { |req, res|  }, nil)
+    register_mount('/status/v1/simple/server', proc { |req, res|  }, nil)
     register_mount('/puppet/v3/node', mounts[:node], NodeServlet)
     register_mount('/puppet/v3/catalog', mounts[:catalog], CatalogServlet)
     register_mount('/puppet/v3/file_metadatas', mounts[:file_metadatas], FileMetadatasServlet)

data/spec/lib/puppet_spec/settings.rb

@@ -11,6 +11,7 @@ module PuppetSpec::Settings
     :confdir      => { :type => :directory, :default => "test", :desc => "confdir" },
     :codedir      => { :type => :directory, :default => "test", :desc => "codedir" },
     :vardir       => { :type => :directory, :default => "test", :desc => "vardir" },
+    :publicdir    => { :type => :directory, :default => "test", :desc => "publicdir" },
     :rundir       => { :type => :directory, :default => "test", :desc => "rundir" },
   }.freeze
 
@@ -20,7 +21,6 @@ module PuppetSpec::Settings
   end.freeze
 
   def set_puppet_conf(confdir, settings)
-    FileUtils.mkdir_p(confdir)
     write_file(File.join(confdir, "puppet.conf"), settings)
   end
 

data/spec/spec_helper.rb

@@ -84,7 +84,10 @@ RSpec.configure do |config|
   config.filter_run_when_matching :focus
 
   config.mock_with :rspec do |mocks|
-    mocks.verify_partial_doubles = true
+    # We really should have this on, but it breaks a _lot_ of tests. We'll
+    # need to go through and fix those tests first before it can be enabled
+    # for real.
+    mocks.verify_partial_doubles = false
   end
 
   tmpdir = Puppet::FileSystem.expand_path(Dir.mktmpdir("rspecrun"))
@@ -145,6 +148,7 @@ RSpec.configure do |config|
 
     base = PuppetSpec::Files.tmpdir('tmp_settings')
     Puppet[:vardir] = File.join(base, 'var')
+    Puppet[:publicdir] = File.join(base, 'public')
     Puppet[:confdir] = File.join(base, 'etc')
     Puppet[:codedir] = File.join(base, 'code')
     Puppet[:logdir] = "$vardir/log"
@@ -152,6 +156,7 @@ RSpec.configure do |config|
     Puppet[:hiera_config] = File.join(base, 'hiera')
 
     FileUtils.mkdir_p Puppet[:statedir]
+    FileUtils.mkdir_p Puppet[:publicdir]
 
     Puppet::Test::TestHelper.before_each_test()
   end
@@ -160,20 +165,10 @@ RSpec.configure do |config|
   PUPPET_FACTER_2_GCE_URL = %r{^http://metadata/computeMetadata/v1(beta1)?}.freeze
   PUPPET_FACTER_3_GCE_URL = "http://metadata.google.internal/computeMetadata/v1/?recursive=true&alt=json".freeze
 
-  # Facter azure metadata endpoint
-  PUPPET_FACTER_AZ_URL = "http://169.254.169.254/metadata/instance?api-version=2020-09-01"
-
-  # Facter EC2 endpoint
-  PUPPET_FACTER_EC2_METADATA_URL = 'http://169.254.169.254/latest/meta-data/'
-  PUPPET_FACTER_EC2_USERDATA_URL = 'http://169.254.169.254/latest/user-data/'
-
   config.around :each do |example|
-    # Ignore requests from Facter to external services
+    # Ignore requests from Facter GCE fact in Travis
     stub_request(:get, PUPPET_FACTER_2_GCE_URL)
     stub_request(:get, PUPPET_FACTER_3_GCE_URL)
-    stub_request(:get, PUPPET_FACTER_AZ_URL)
-    stub_request(:get, PUPPET_FACTER_EC2_METADATA_URL)
-    stub_request(:get, PUPPET_FACTER_EC2_USERDATA_URL)
 
     # Enable VCR if the example is tagged with `:vcr` metadata.
     if example.metadata[:vcr]

data/spec/unit/agent_spec.rb

@@ -3,13 +3,9 @@ require 'puppet/agent'
 require 'puppet/configurer'
 
 class AgentTestClient
-  def initialize(transaction_uuid = nil, job_id = nil)
-  end
-
-  def run(client_args)
+  def run
     # no-op
   end
-
   def stop
     # no-op
   end
@@ -24,8 +20,6 @@ end
 
 describe Puppet::Agent do
   before do
-    allow(Puppet::Status.indirection).to receive(:find).and_return(Puppet::Status.new("version" => Puppet.version))
-
     @agent = Puppet::Agent.new(AgentTestClient, false)
 
     # make Puppet::Application safe for stubbing; restore in an :after block; silence warnings for this.
@@ -55,10 +49,11 @@ describe Puppet::Agent do
 
   it "should create an instance of its client class and run it when asked to run" do
     client = double('client')
-    allow(AgentTestClient).to receive(:new).with(nil, nil).and_return(client)
+    expect(AgentTestClient).to receive(:new).and_return(client)
 
-    allow(@agent).to receive(:disabled?).and_return(false)
     expect(client).to receive(:run)
+
+    allow(@agent).to receive(:disabled?).and_return(false)
     @agent.run
   end
 
@@ -95,6 +90,7 @@ describe Puppet::Agent do
 
   describe "when being run" do
     before do
+      allow(AgentTestClient).to receive(:lockfile_path).and_return("/my/lock")
       allow(@agent).to receive(:disabled?).and_return(false)
     end
 
@@ -190,7 +186,7 @@ describe Puppet::Agent do
         allow(lockfile).to receive(:lock).and_return(false)
       end
 
-      it "should notify that a run is already in progress" do
+      it "should notify that a run is already in progres" do
         client = AgentTestClient.new
         expect(AgentTestClient).to receive(:new).and_return(client)
         expect(Puppet).to receive(:notice).with(/Run of .* already in progress; skipping .* exists/)

data/spec/unit/application/agent_spec.rb

@@ -202,6 +202,7 @@ describe Puppet::Application::Agent do
       allow(Puppet::Resource::Catalog.indirection).to receive(:terminus_class=)
       allow(Puppet::Resource::Catalog.indirection).to receive(:cache_class=)
       allow(Puppet::Node::Facts.indirection).to receive(:terminus_class=)
+      allow(Puppet).to receive(:settraps)
     end
 
     it "should not run with extra arguments" do
@@ -546,16 +547,11 @@ describe Puppet::Application::Agent do
         @puppetd.options[:digest] = :MD5
       end
 
-      def expected_fingerprint(name, x509)
-        digest = OpenSSL::Digest.new(name).hexdigest(x509.to_der)
-        digest.scan(/../).join(':').upcase
-      end
-
       it "should fingerprint the certificate if it exists" do
         cert = cert_fixture('signed.pem')
         allow_any_instance_of(Puppet::X509::CertProvider).to receive(:load_client_cert).and_return(cert)
 
-        expect(@puppetd).to receive(:puts).with("(MD5) #{expected_fingerprint('md5', cert)}")
+        expect(@puppetd).to receive(:puts).with('(MD5) E2:BA:9A:EF:20:A8:7D:10:8D:82:9A:61:5D:FD:5B:33')
 
         @puppetd.fingerprint
       end
@@ -565,7 +561,7 @@ describe Puppet::Application::Agent do
         allow_any_instance_of(Puppet::X509::CertProvider).to receive(:load_client_cert).and_return(nil)
         allow_any_instance_of(Puppet::X509::CertProvider).to receive(:load_request).and_return(request)
 
-        expect(@puppetd).to receive(:puts).with("(MD5) #{expected_fingerprint('md5', request)}")
+        expect(@puppetd).to receive(:puts).with('(MD5) B8:4C:FB:31:AE:17:86:E3:AD:53:97:CA:F6:3C:4A:CB')
 
         @puppetd.fingerprint
       end

data/spec/unit/application/facts_spec.rb

@@ -3,7 +3,7 @@ require 'puppet/application/facts'
 
 describe Puppet::Application::Facts do
   let(:app) { Puppet::Application[:facts] }
-  let(:values) { {"filesystems" => "apfs,autofs,devfs", "macaddress" => "64:52:11:22:03:2e"} }
+  let(:values) { {"filesystems" => "apfs,autofs,devfs", "macaddress" => "64:52:11:22:03:25"} }
 
   before :each do
     Puppet::Node::Facts.indirection.terminus_class = :memory
@@ -21,7 +21,7 @@ describe Puppet::Application::Facts do
       name: whatever
       values:
         filesystems: apfs,autofs,devfs
-        macaddress: "64:52:11:22:03:2e"
+        macaddress: "64:52:11:22:03:25"
     END
 
     expect {
@@ -42,7 +42,7 @@ describe Puppet::Application::Facts do
       name: #{Puppet[:certname]}
       values:
         filesystems: apfs,autofs,devfs
-        macaddress: "64:52:11:22:03:2e"
+        macaddress: "64:52:11:22:03:25"
     END
 
     expect {
@@ -52,12 +52,7 @@ describe Puppet::Application::Facts do
   end
 
   context 'when show action is called' do
-    let(:expected) { <<~END }
-      {
-        "filesystems": "apfs,autofs,devfs",
-        "macaddress": "64:52:11:22:03:2e"
-      }
-    END
+    let(:expected) { "{\n  \"filesystems\": \"apfs,autofs,devfs\",\n  \"macaddress\": \"64:52:11:22:03:25\"\n}\n" }
 
     before :each do
       Puppet::Node::Facts.indirection.terminus_class = :facter
@@ -69,464 +64,25 @@ describe Puppet::Application::Facts do
       expect {
         app.run
       }.to exit_with(0)
-       .and output(expected).to_stdout
-    end
-
-    it 'displays a single fact value' do
-      app.command_line.args << 'filesystems' << '--value-only'
-      expect {
-        app.run
-      }.to exit_with(0)
-       .and output("apfs,autofs,devfs\n").to_stdout
-    end
-
-    it "warns and ignores value-only when multiple fact names are specified" do
-      app.command_line.args << 'filesystems' << 'macaddress' << '--value-only'
-      expect {
-        app.run
-      }.to exit_with(0)
-       .and output(expected).to_stdout
-       .and output(/it can only be used when querying for a single fact/).to_stderr
-    end
-
-    {
-      "type_hash" => [{'a' => 2}, "{\n  \"a\": 2\n}"],
-      "type_array" => [[], "[\n\n]"],
-      "type_string" => ["str", "str"],
-      "type_int" => [1, "1"],
-      "type_float" => [1.0, "1.0"],
-      "type_true" => [true, "true"],
-      "type_false" => [false, "false"],
-      "type_nil" => [nil, ""],
-      "type_sym" => [:sym, "sym"]
-    }.each_pair do |name, values|
-      it "renders '#{name}' as '#{values.last}'" do
-        fact_value = values.first
-        fact_output = values.last
-
-        allow(Facter).to receive(:resolve).and_return({name => fact_value})
-
-        app.command_line.args << name << '--value-only'
-        expect {
-          app.run
-        }.to exit_with(0)
-         .and output("#{fact_output}\n").to_stdout
-      end
+               .and output(expected).to_stdout
     end
   end
 
-  context 'when diff action is called' do
-    let(:facter3_facts) { <<~END }
-{
-  "hypervisors": {
-    "vmware": {
-      "version": ""
-    }
-  },
-  "networking": {
-    "interfaces": {
-      "lo": {
-        "bindings6": [
-          {
-            "network": "::1"
-          }
-        ]
-      },
-      "em0.1": {
-        "ip": "0.0.0.0"
-      }
-    }
-  },
-  "macaddress": "64:52:11:22:03:2e",
-  "obsolete_fact": "true",
-  "mountpoints": {
-    "/var": {
-      "options": [
-        "noatime",
-        "journaled",
-        "nobrowse"
-      ]
-    }
-  }
-}
-END
-
-    let(:facter4_facts) { <<~END }
-{
-  "hypervisors": {
-    "vmware": {
-      "version": "ESXi 6.7"
-    }
-  },
-  "networking": {
-    "interfaces": {
-      "lo": {
-        "bindings6": [
-          {
-            "network": "::1",
-            "scope6": "host"
-          }
-        ]
-      },
-      "em0.1": {
-        "ip": "127.0.0.1"
-      }
-    }
-  },
-  "macaddress": "64:52:11:22:03:2e",
-  "mountpoints": {
-    "/var": {
-      "options": [
-        "noatime",
-        "nobrowse",
-        "journaled"
-      ]
-    }
-  }
-}
-END
+  context 'when default action is called' do
+    let(:expected) { "---\nfilesystems: apfs,autofs,devfs\nmacaddress: 64:52:11:22:03:25\n" }
 
     before :each do
       Puppet::Node::Facts.indirection.terminus_class = :facter
-      app.command_line.args = %w{diff}
-
-      allow(Facter).to receive(:value).with('facterversion').and_return('3.99.0')
-      allow(Puppet::Util::Execution).to receive(:execute).with(/puppet facts show --no-facterng/, combine:false).and_return(facter3_facts)
-      allow(Puppet::Util::Execution).to receive(:execute).with(/puppet facts show --facterng/, combine:false).and_return(facter4_facts)
-    end
-
-    # Workaround for YAML issue on Ubuntu where null values get space as key
-    let(:regex) { false }
-    shared_examples_for 'correctly rendering output' do |render_format|
-      it 'correctly displays output' do
-        app.command_line.args << '--structured' if structured
-        app.command_line.args << '--exclude' << exclude if exclude
-        app.command_line.args << '--render-as' << render_format if render_format
-        expect {
-          app.run
-        }.to exit_with(0)
-         .and output(regex ? /#{expected_output}/ : expected_output).to_stdout
-      end
-    end
-
-    context 'when no exclude regex is provided via CLI' do
-      let(:exclude) { nil }
-      context 'when structured is requested' do
-        let(:structured) { true }
-        context 'when rendering is set to default' do
-          let(:expected_output) { <<~END }
-{
-  "hypervisors": {
-    "vmware": {
-      "version": {
-        "new_value": "ESXi 6.7",
-        "old_value": ""
-      }
-    }
-  },
-  "mountpoints": {
-    "/var": {
-      "options": {
-        "1": {
-          "new_value": "nobrowse",
-          "old_value": "journaled"
-        },
-        "2": {
-          "new_value": "journaled",
-          "old_value": "nobrowse"
-        }
-      }
-    }
-  },
-  "networking": {
-    "interfaces": {
-      "em0.1": {
-        "ip": {
-          "new_value": "127.0.0.1",
-          "old_value": "0.0.0.0"
-        }
-      },
-      "lo": {
-        "bindings6": {
-          "0": {
-            "scope6": {
-              "new_value": "host",
-              "old_value": null
-            }
-          }
-        }
-      }
-    }
-  },
-  "obsolete_fact": {
-    "new_value": null,
-    "old_value": "true"
-  }
-}
-END
-
-          it_behaves_like 'correctly rendering output'
-        end
-
-        context 'when rendering is set to yaml' do
-          # Workaround for YAML issue on Ubuntu where null values get space as key
-          let(:regex) { true }
-          let(:expected_output) { <<~END }
----
-hypervisors:
-  vmware:
-    version:
-      :new_value: ESXi 6\.7
-      :old_value: ''
-mountpoints:
-  "/var":
-    options:
-      1:
-        :new_value: nobrowse
-        :old_value: journaled
-      2:
-        :new_value: journaled
-        :old_value: nobrowse
-networking:
-  interfaces:
-    em0\.1:
-      ip:
-        :new_value: 127\.0\.0\.1
-        :old_value: 0\.0\.0\.0
-    lo:
-      bindings6:
-        0:
-          scope6:
-            :new_value: host
-            :old_value:\s?
-obsolete_fact:
-  :new_value:\s?
-  :old_value: 'true'
-END
-
-          it_behaves_like 'correctly rendering output', 'yaml'
-        end
-
-        context 'when rendering is set to json' do
-          let(:expected_output) { <<~END }
-{"hypervisors":{"vmware":{"version":{"new_value":"ESXi 6.7","old_value":""}}},\
-"mountpoints":{"/var":{"options":{"1":{"new_value":"nobrowse","old_value":"journaled"},\
-"2":{"new_value":"journaled","old_value":"nobrowse"}}}},"networking":{"interfaces":\
-{"em0.1":{"ip":{"new_value":"127.0.0.1","old_value":"0.0.0.0"}},"lo":{"bindings6":\
-{"0":{"scope6":{"new_value":"host","old_value":null}}}}}},"obsolete_fact":\
-{"new_value":null,"old_value":"true"}}
-END
-
-          it_behaves_like 'correctly rendering output', 'json'
-        end
-      end
-
-      context 'when structured is not requested' do
-        let(:structured) { false }
-        context 'when rendering is set to default' do
-          let(:expected_output) { <<~END }
-{
-  "hypervisors.vmware.version": {
-    "new_value": "ESXi 6.7",
-    "old_value": ""
-  },
-  "mountpoints./var.options.1": {
-    "new_value": "nobrowse",
-    "old_value": "journaled"
-  },
-  "mountpoints./var.options.2": {
-    "new_value": "journaled",
-    "old_value": "nobrowse"
-  },
-  "networking.interfaces.\\"em0.1\\".ip": {
-    "new_value": "127.0.0.1",
-    "old_value": "0.0.0.0"
-  },
-  "networking.interfaces.lo.bindings6.0.scope6": {
-    "new_value": "host",
-    "old_value": null
-  },
-  "obsolete_fact": {
-    "new_value": null,
-    "old_value": "true"
-  }
-}
-END
-
-          it_behaves_like 'correctly rendering output'
-        end
-
-        context 'when rendering is set to yaml' do
-          # Workaround for YAML issue on Ubuntu where null values get space as key
-          let(:regex) { true }
-          let(:expected_output) { <<~END }
----
-hypervisors\.vmware\.version:
-  :new_value: ESXi 6\.7
-  :old_value: ''
-mountpoints\./var\.options\.1:
-  :new_value: nobrowse
-  :old_value: journaled
-mountpoints\./var\.options\.2:
-  :new_value: journaled
-  :old_value: nobrowse
-networking\.interfaces\."em0\.1"\.ip:
-  :new_value: 127\.0\.0\.1
-  :old_value: 0\.0\.0\.0
-networking\.interfaces\.lo\.bindings6\.0\.scope6:
-  :new_value: host
-  :old_value:\s?
-obsolete_fact:
-  :new_value:\s?
-  :old_value: 'true'
-END
-
-          it_behaves_like 'correctly rendering output', 'yaml'
-        end
-
-        context 'when rendering is set to json' do
-          let(:expected_output) { <<~END }
-{"hypervisors.vmware.version":{"new_value":"ESXi 6.7","old_value":""},\
-"mountpoints./var.options.1":{"new_value":"nobrowse","old_value":"journaled"},\
-"mountpoints./var.options.2":{"new_value":"journaled","old_value":"nobrowse"},\
-"networking.interfaces.\\"em0.1\\".ip":{"new_value":"127.0.0.1",\
-"old_value":"0.0.0.0"},"networking.interfaces.lo.bindings6.0.scope6":\
-{"new_value":"host","old_value":null},"obsolete_fact":{"new_value":null,\
-"old_value":"true"}}
-END
-
-          it_behaves_like 'correctly rendering output', 'json'
-        end
-      end
-    end
-
-    context 'when unwanted facts are excluded from diff via CLI option' do
-      let(:exclude) { '^mountpoints\./var\.options.*$|^obsolete_fact$|^hypervisors' }
-      context 'when structured is requested' do
-        let(:structured) { true }
-        context 'when rendering is set to default' do
-          let(:expected_output) { <<~END }
-{
-  "networking": {
-    "interfaces": {
-      "em0.1": {
-        "ip": {
-          "new_value": "127.0.0.1",
-          "old_value": "0.0.0.0"
-        }
-      },
-      "lo": {
-        "bindings6": {
-          "0": {
-            "scope6": {
-              "new_value": "host",
-              "old_value": null
-            }
-          }
-        }
-      }
-    }
-  }
-}
-END
-
-          it_behaves_like 'correctly rendering output'
-        end
-
-        context 'when rendering is set to yaml' do
-          # Workaround for YAML issue on Ubuntu where null values get space as key
-          let(:regex) { true }
-          let(:expected_output) { <<~END }
----
-networking:
-  interfaces:
-    em0\.1:
-      ip:
-        :new_value: 127\.0\.0\.1
-        :old_value: 0\.0\.0\.0
-    lo:
-      bindings6:
-        0:
-          scope6:
-            :new_value: host
-            :old_value:\s?
-END
-
-          it_behaves_like 'correctly rendering output', 'yaml'
-        end
-
-        context 'when rendering is set to json' do
-          let(:expected_output) { <<~END }
-{"networking":{"interfaces":{"em0.1":{"ip":{"new_value":"127.0.0.1",\
-"old_value":"0.0.0.0"}},"lo":{"bindings6":{"0":{"scope6":{"new_value":\
-"host","old_value":null}}}}}}}
-END
-
-          it_behaves_like 'correctly rendering output', 'json'
-        end
-      end
-
-      context 'when structured is not requested' do
-        let(:structured) { false }
-        context 'when rendering is set to default' do
-          let(:expected_output) { <<~END }
-{
-  "networking.interfaces.\\"em0.1\\".ip": {
-    "new_value": "127.0.0.1",
-    "old_value": "0.0.0.0"
-  },
-  "networking.interfaces.lo.bindings6.0.scope6": {
-    "new_value": "host",
-    "old_value": null
-  }
-}
-END
-
-          it_behaves_like 'correctly rendering output'
-        end
-
-        context 'when rendering is set to yaml' do
-          # Workaround for YAML issue on Ubuntu where null values get space as key
-          let(:regex) { true }
-          let(:expected_output) { <<~END }
----
-networking\.interfaces\."em0\.1"\.ip:
-  :new_value: 127\.0\.0\.1
-  :old_value: 0\.0\.0\.0
-networking\.interfaces\.lo\.bindings6\.0\.scope6:
-  :new_value: host
-  :old_value:\s?
-END
-
-          it_behaves_like 'correctly rendering output', 'yaml'
-        end
-
-        context 'when rendering is set to json' do
-          let(:expected_output) { <<~END }
-{"networking.interfaces.\\"em0.1\\".ip":{"new_value":"127.0.0.1",\
-"old_value":"0.0.0.0"},"networking.interfaces.lo.bindings6.0.scope6":\
-{"new_value":"host","old_value":null}}
-END
-
-          it_behaves_like 'correctly rendering output', 'json'
-        end
-      end
-    end
-  end
-
-  context 'when default action is called' do
-    before :each do
-      Puppet::Node::Facts.indirection.terminus_class = :memory
-      facts = Puppet::Node::Facts.new('whatever', values)
-      Puppet::Node::Facts.indirection.save(facts)
+      allow(Facter).to receive(:resolve).and_return(values)
+      app.command_line.args = %w{--render-as yaml}
     end
 
-    it 'calls find action' do
+    it 'calls show action' do
       expect {
         app.run
       }.to exit_with(0)
-       .and output(anything).to_stdout
-      expect(app.action.name).to eq(:find)
+               .and output(expected).to_stdout
+      expect(app.action.name).to eq(:show)
     end
   end
 end