puppet 6.23.0 → 7.0.0

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