puppet 5.5.22-x64-mingw32 → 6.0.0-x64-mingw32

data/spec/unit/network/http/webrick_spec.rb

@@ -1,277 +0,0 @@
-require 'spec_helper'
-require 'puppet/network/http'
-require 'puppet/network/http/webrick'
-
-describe Puppet::Network::HTTP::WEBrick, "after initializing" do
-  it "should not be listening" do
-    expect(Puppet::Network::HTTP::WEBrick.new).not_to be_listening
-  end
-end
-
-describe Puppet::Network::HTTP::WEBrick do
-  include PuppetSpec::Files
-
-  let(:address) { '127.0.0.1' }
-  let(:port) { 31337 }
-  let(:server) { Puppet::Network::HTTP::WEBrick.new }
-  let(:localcacert) { make_absolute("/ca/crt") }
-  let(:ssl_server_ca_auth) { make_absolute("/ca/ssl_server_auth_file") }
-  let(:key) { double('key', :content => "mykey") }
-  let(:cert) { double('cert', :content => "mycert") }
-  let(:host) { double('host', :key => key, :certificate => cert, :name => "yay", :ssl_store => "mystore") }
-
-  let(:mock_ssl_context) do
-    double('ssl_context', :ciphers= => nil)
-  end
-
-  let(:socket) { double('socket') }
-  let(:mock_webrick) do
-    server = double('webrick',
-                  :[] => {},
-                  :listeners => [],
-                  :status => :Running,
-                  :mount => nil,
-                  :shutdown => nil,
-                  :ssl_context => mock_ssl_context)
-    allow(server).to receive(:start).and_yield(socket)
-    allow(IO).to receive(:select).with([socket], nil, nil, anything).and_return(true)
-    allow(socket).to receive(:accept)
-    allow(server).to receive(:run).with(socket)
-    server
-  end
-
-  before :each do
-    allow(WEBrick::HTTPServer).to receive(:new).and_return(mock_webrick)
-    allow(Puppet::SSL::Certificate.indirection).to receive(:find).with('ca').and_return(cert)
-    allow(Puppet::SSL::Host).to receive(:localhost).and_return(host)
-  end
-
-  describe "when turning on listening" do
-    it "should fail if already listening" do
-      server.listen(address, port)
-      expect { server.listen(address, port) }.to raise_error(RuntimeError, /server is already listening/)
-    end
-
-    it "should tell webrick to listen on the specified address and port" do
-      expect(WEBrick::HTTPServer).to receive(:new).with(
-        hash_including(:Port => 31337, :BindAddress => "127.0.0.1")
-      ).and_return(mock_webrick)
-      server.listen(address, port)
-    end
-
-    it "should not perform reverse lookups" do
-      expect(WEBrick::HTTPServer).to receive(:new).with(
-        hash_including(:DoNotReverseLookup => true)
-      ).and_return(mock_webrick)
-      expect(BasicSocket).to receive(:do_not_reverse_lookup=).with(true)
-
-      server.listen(address, port)
-    end
-
-    it "should configure a logger for webrick" do
-      expect(server).to receive(:setup_logger).and_return(:Logger => :mylogger)
-
-      expect(WEBrick::HTTPServer).to receive(:new) do |args|
-        expect(args[:Logger]).to eq(:mylogger)
-      end.and_return(mock_webrick)
-
-      server.listen(address, port)
-    end
-
-    it "should configure SSL for webrick" do
-      expect(server).to receive(:setup_ssl).and_return(:Ssl => :testing, :Other => :yay)
-
-      expect(WEBrick::HTTPServer).to receive(:new).with(hash_including(:Ssl => :testing, :Other => :yay)).and_return(mock_webrick)
-
-      server.listen(address, port)
-    end
-
-    it "should be listening" do
-      server.listen(address, port)
-      expect(server).to be_listening
-    end
-
-    it "is passed a yet to be accepted socket" do
-      expect(socket).to receive(:accept)
-
-      server.listen(address, port)
-      server.unlisten
-    end
-
-    describe "when the REST protocol is requested" do
-      it "should register the REST handler at /" do
-        # We don't care about the options here.
-        expect(mock_webrick).to receive(:mount).with("/", Puppet::Network::HTTP::WEBrickREST)
-
-        server.listen(address, port)
-      end
-    end
-  end
-
-  describe "when turning off listening" do
-    it "should fail unless listening" do
-      expect { server.unlisten }.to raise_error(RuntimeError, /server is not listening/)
-    end
-
-    it "should order webrick server to stop" do
-      expect(mock_webrick).to receive(:shutdown)
-      server.listen(address, port)
-      server.unlisten
-    end
-
-    it "should no longer be listening" do
-      server.listen(address, port)
-      server.unlisten
-      expect(server).not_to be_listening
-    end
-  end
-
-  describe "when configuring an http logger" do
-    let(:server) { Puppet::Network::HTTP::WEBrick.new }
-
-    before :each do
-      allow(Puppet.settings).to receive(:use)
-      @filehandle = double('handle', :fcntl => nil, :sync= => nil)
-
-      allow(File).to receive(:open).and_return(@filehandle)
-    end
-
-    it "should use the settings for :main, :ssl, and :application" do
-      expect(Puppet.settings).to receive(:use).with(:main, :ssl, :application)
-
-      server.setup_logger
-    end
-
-    it "should use the masterhttplog" do
-      log = make_absolute("/master/log")
-      Puppet[:masterhttplog] = log
-
-      expect(File).to receive(:open).with(log, "a+:UTF-8").and_return(@filehandle)
-
-      server.setup_logger
-    end
-
-    describe "and creating the logging filehandle" do
-      it "should set the close-on-exec flag if supported" do
-        if defined? Fcntl::FD_CLOEXEC
-          expect(@filehandle).to receive(:fcntl).with(Fcntl::F_SETFD, Fcntl::FD_CLOEXEC)
-        else
-          expect(@filehandle).not_to receive(:fcntl)
-        end
-
-        server.setup_logger
-      end
-
-      it "should sync the filehandle" do
-        expect(@filehandle).to receive(:sync=).with(true)
-
-        server.setup_logger
-      end
-    end
-
-    it "should create a new WEBrick::Log instance with the open filehandle" do
-      expect(WEBrick::Log).to receive(:new).with(@filehandle)
-
-      server.setup_logger
-    end
-
-    it "should set debugging if the current loglevel is :debug" do
-      expect(Puppet::Util::Log).to receive(:level).and_return(:debug)
-
-      expect(WEBrick::Log).to receive(:new).with(anything, WEBrick::Log::DEBUG)
-
-      server.setup_logger
-    end
-
-    it "should return the logger as the main log" do
-      logger = double('logger')
-      expect(WEBrick::Log).to receive(:new).and_return(logger)
-
-      expect(server.setup_logger[:Logger]).to eq(logger)
-    end
-
-    it "should return the logger as the access log using both the Common and Referer log format" do
-      logger = double('logger')
-      expect(WEBrick::Log).to receive(:new).and_return(logger)
-
-      expect(server.setup_logger[:AccessLog]).to eq([
-        [logger, WEBrick::AccessLog::COMMON_LOG_FORMAT],
-        [logger, WEBrick::AccessLog::REFERER_LOG_FORMAT]
-      ])
-    end
-  end
-
-  describe "when configuring ssl" do
-    it "should use the key from the localhost SSL::Host instance" do
-      expect(Puppet::SSL::Host).to receive(:localhost).and_return(host)
-      expect(host).to receive(:key).and_return(key)
-
-      expect(server.setup_ssl[:SSLPrivateKey]).to eq("mykey")
-    end
-
-    it "should configure the certificate" do
-      expect(server.setup_ssl[:SSLCertificate]).to eq("mycert")
-    end
-
-    it "should fail if no CA certificate can be found" do
-      allow(Puppet::SSL::Certificate.indirection).to receive(:find).with('ca').and_return(nil)
-
-      expect { server.setup_ssl }.to raise_error(Puppet::Error, /Could not find CA certificate/)
-    end
-
-    it "should specify the path to the CA certificate" do
-      Puppet.settings[:hostcrl] = 'false'
-      Puppet.settings[:localcacert] = localcacert
-
-      expect(server.setup_ssl[:SSLCACertificateFile]).to eq(localcacert)
-    end
-
-    it "should specify the path to the CA certificate" do
-      Puppet.settings[:hostcrl] = 'false'
-      Puppet.settings[:localcacert] = localcacert
-      Puppet.settings[:ssl_server_ca_auth] = ssl_server_ca_auth
-
-      expect(server.setup_ssl[:SSLCACertificateFile]).to eq(ssl_server_ca_auth)
-    end
-
-    it "should not start ssl immediately" do
-      expect(server.setup_ssl[:SSLStartImmediately]).to eq(false)
-    end
-
-    it "should enable ssl" do
-      expect(server.setup_ssl[:SSLEnable]).to be_truthy
-    end
-
-    it "should reject SSLv2" do
-      options = server.setup_ssl[:SSLOptions]
-
-      expect(options & OpenSSL::SSL::OP_NO_SSLv2).to eq(OpenSSL::SSL::OP_NO_SSLv2)
-    end
-
-    it "should reject SSLv3" do
-      options = server.setup_ssl[:SSLOptions]
-
-      expect(options & OpenSSL::SSL::OP_NO_SSLv3).to eq(OpenSSL::SSL::OP_NO_SSLv3)
-    end
-
-    it "should configure the verification method as 'OpenSSL::SSL::VERIFY_PEER'" do
-      expect(server.setup_ssl[:SSLVerifyClient]).to eq(OpenSSL::SSL::VERIFY_PEER)
-    end
-
-    it "should add an x509 store" do
-      expect(host).to receive(:ssl_store).and_return("mystore")
-
-      expect(server.setup_ssl[:SSLCertificateStore]).to eq("mystore")
-    end
-
-    it "should set the certificate name to 'nil'" do
-      expect(server.setup_ssl[:SSLCertName]).to be_nil
-    end
-
-    it "specifies the allowable ciphers" do
-      expect(mock_ssl_context).to receive(:ciphers=).with(server.class::CIPHERS)
-
-      server.create_server('localhost', '8888')
-    end
-  end
-end

data/spec/unit/network/server_spec.rb

@@ -1,94 +0,0 @@
-require 'spec_helper'
-require 'puppet/network/server'
-
-describe Puppet::Network::Server do
-  let(:port) { 8140 }
-  let(:address) { '0.0.0.0' }
-  let(:server) { Puppet::Network::Server.new(address, port) }
-
-  before do
-    @mock_http_server = double('http server')
-    allow(Puppet.settings).to receive(:use)
-    allow(Puppet::Network::HTTP::WEBrick).to receive(:new).and_return(@mock_http_server)
-  end
-
-  describe "when initializing" do
-    before do
-      Puppet[:masterport] = ''
-    end
-
-    it "should not be listening after initialization" do
-      expect(Puppet::Network::Server.new(address, port)).not_to be_listening
-    end
-
-    it "should use the :main setting section" do
-      expect(Puppet.settings).to receive(:use) { |*sections| expect(sections).to include(:main) }
-      Puppet::Network::Server.new(address, port)
-    end
-
-    it "should use the :application setting section" do
-      expect(Puppet.settings).to receive(:use) { |*sections| expect(sections).to include(:application) }
-
-      Puppet::Network::Server.new(address, port)
-    end
-  end
-
-  describe "when not yet started" do
-    before do
-      allow(@mock_http_server).to receive(:listen)
-    end
-
-    it "should indicate that it is not listening" do
-      expect(server).not_to be_listening
-    end
-
-    it "should not allow server to be stopped" do
-      expect { server.stop }.to raise_error(RuntimeError)
-    end
-
-    it "should allow server to be started" do
-      expect { server.start }.to_not raise_error
-    end
-  end
-
-  describe "when server is on" do
-    before do
-      allow(@mock_http_server).to receive(:listen)
-      allow(@mock_http_server).to receive(:unlisten)
-      server.start
-    end
-
-    it "should indicate that it is listening" do
-      expect(server).to be_listening
-    end
-
-    it "should not allow server to be started again" do
-      expect { server.start }.to raise_error(RuntimeError)
-    end
-
-    it "should allow server to be stopped" do
-      expect { server.stop }.to_not raise_error
-    end
-  end
-
-  describe "when server is being started" do
-    it "should cause the HTTP server to listen" do
-      server = Puppet::Network::Server.new(address, port)
-      expect(@mock_http_server).to receive(:listen).with(address, port)
-      server.start
-    end
-  end
-
-  describe "when server is being stopped" do
-    before do
-      allow(@mock_http_server).to receive(:listen)
-      allow(server).to receive(:http_server).and_return(@mock_http_server)
-      server.start
-    end
-
-    it "should cause the HTTP server to stop listening" do
-      expect(@mock_http_server).to receive(:unlisten)
-      server.stop
-    end
-  end
-end

data/spec/unit/provider/augeas/augeas_spec.rb

@@ -1,1096 +0,0 @@
-require 'spec_helper'
-require 'puppet/util/package'
-
-provider_class = Puppet::Type.type(:augeas).provider(:augeas)
-
-describe provider_class do
-  before(:each) do
-    @resource = Puppet::Type.type(:augeas).new(
-      :name     => "test",
-      :root     => my_fixture_dir,
-      :provider => :augeas
-    )
-    @provider = provider_class.new(@resource)
-  end
-
-  after(:each) do
-    @provider.close_augeas
-  end
-
-  describe "command parsing" do
-    it "should break apart a single line into three tokens and clean up the context" do
-      @resource[:context] = "/context"
-      tokens = @provider.parse_commands("set Jar/Jar Binks")
-      expect(tokens.size).to eq(1)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq("/context/Jar/Jar")
-      expect(tokens[0][2]).to eq("Binks")
-    end
-
-    it "should break apart a multiple line into six tokens" do
-      tokens = @provider.parse_commands("set /Jar/Jar Binks\nrm anakin")
-      expect(tokens.size).to eq(2)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[1].size).to eq(2)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq("/Jar/Jar")
-      expect(tokens[0][2]).to eq("Binks")
-      expect(tokens[1][0]).to eq("rm")
-      expect(tokens[1][1]).to eq("anakin")
-    end
-
-    it "should strip whitespace and ignore blank lines" do
-      tokens = @provider.parse_commands("  set /Jar/Jar Binks \t\n  \n\n  rm anakin ")
-      expect(tokens.size).to eq(2)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[1].size).to eq(2)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq("/Jar/Jar")
-      expect(tokens[0][2]).to eq("Binks")
-      expect(tokens[1][0]).to eq("rm")
-      expect(tokens[1][1]).to eq("anakin")
-    end
-
-    it "should handle arrays" do
-      @resource[:context] = "/foo/"
-      commands = ["set /Jar/Jar Binks", "rm anakin"]
-      tokens = @provider.parse_commands(commands)
-      expect(tokens.size).to eq(2)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[1].size).to eq(2)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq("/Jar/Jar")
-      expect(tokens[0][2]).to eq("Binks")
-      expect(tokens[1][0]).to eq("rm")
-      expect(tokens[1][1]).to eq("/foo/anakin")
-    end
-
-    # This is not supported in the new parsing class
-    #it "should concat the last values" do
-    #    provider = provider_class.new
-    #    tokens = provider.parse_commands("set /Jar/Jar Binks is my copilot")
-    #    tokens.size.should == 1
-    #    tokens[0].size.should == 3
-    #    tokens[0][0].should == "set"
-    #    tokens[0][1].should == "/Jar/Jar"
-    #    tokens[0][2].should == "Binks is my copilot"
-    #end
-
-    it "should accept spaces in the value and single ticks" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("set JarJar 'Binks is my copilot'")
-      expect(tokens.size).to eq(1)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq("/foo/JarJar")
-      expect(tokens[0][2]).to eq("Binks is my copilot")
-    end
-
-    it "should accept spaces in the value and double ticks" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands('set /JarJar "Binks is my copilot"')
-      expect(tokens.size).to eq(1)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq('/JarJar')
-      expect(tokens[0][2]).to eq('Binks is my copilot')
-    end
-
-    it "should accept mixed ticks" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands('set JarJar "Some \'Test\'"')
-      expect(tokens.size).to eq(1)
-      expect(tokens[0].size).to eq(3)
-      expect(tokens[0][0]).to eq("set")
-      expect(tokens[0][1]).to eq('/foo/JarJar')
-      expect(tokens[0][2]).to eq("Some \'Test\'")
-    end
-
-    it "should handle predicates with literals" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("rm */*[module='pam_console.so']")
-      expect(tokens).to eq([["rm", "/foo/*/*[module='pam_console.so']"]])
-    end
-
-    it "should handle whitespace in predicates" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("ins 42 before /files/etc/hosts/*/ipaddr[ . = '127.0.0.1' ]")
-      expect(tokens).to eq([["ins", "42", "before","/files/etc/hosts/*/ipaddr[ . = '127.0.0.1' ]"]])
-    end
-
-    it "should handle multiple predicates" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("clear pam.d/*/*[module = 'system-auth'][type = 'account']")
-      expect(tokens).to eq([["clear", "/foo/pam.d/*/*[module = 'system-auth'][type = 'account']"]])
-    end
-
-    it "should handle nested predicates" do
-      @resource[:context] = "/foo/"
-      args = ["clear", "/foo/pam.d/*/*[module[ ../type = 'type] = 'system-auth'][type[last()] = 'account']"]
-      tokens = @provider.parse_commands(args.join(" "))
-      expect(tokens).to eq([ args ])
-    end
-
-    it "should handle escaped doublequotes in doublequoted string" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("set /foo \"''\\\"''\"")
-      expect(tokens).to eq([[ "set", "/foo", "''\"''" ]])
-    end
-
-    it "should preserve escaped single quotes in double quoted strings" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("set /foo \"\\'\"")
-      expect(tokens).to eq([[ "set", "/foo", "\\'" ]])
-    end
-
-    it "should allow escaped spaces and brackets in paths" do
-      @resource[:context] = "/foo/"
-      args = [ "set", "/white\\ space/\\[section", "value" ]
-      tokens = @provider.parse_commands(args.join(" \t "))
-      expect(tokens).to eq([ args ])
-    end
-
-    it "should allow single quoted escaped spaces in paths" do
-      @resource[:context] = "/foo/"
-      args = [ "set", "'/white\\ space/key'", "value" ]
-      tokens = @provider.parse_commands(args.join(" \t "))
-      expect(tokens).to eq([[ "set", "/white\\ space/key", "value" ]])
-    end
-
-    it "should allow double quoted escaped spaces in paths" do
-      @resource[:context] = "/foo/"
-      args = [ "set", '"/white\\ space/key"', "value" ]
-      tokens = @provider.parse_commands(args.join(" \t "))
-      expect(tokens).to eq([[ "set", "/white\\ space/key", "value" ]])
-    end
-
-    it "should remove trailing slashes" do
-      @resource[:context] = "/foo/"
-      tokens = @provider.parse_commands("set foo/ bar")
-      expect(tokens).to eq([[ "set", "/foo/foo", "bar" ]])
-    end
-  end
-
-  describe "get filters" do
-    before do
-      augeas = double("augeas", :get => "value")
-      allow(augeas).to receive("close")
-      @provider.aug = augeas
-    end
-
-    it "should return false for a = nonmatch" do
-      command = ["get", "fake value", "==", "value"]
-      expect(@provider.process_get(command)).to eq(true)
-    end
-
-    it "should return true for a != match" do
-      command = ["get", "fake value", "!=", "value"]
-      expect(@provider.process_get(command)).to eq(false)
-    end
-
-    it "should return true for a =~ match" do
-      command = ["get", "fake value", "=~", "val*"]
-      expect(@provider.process_get(command)).to eq(true)
-    end
-
-    it "should return false for a == nonmatch" do
-      command = ["get", "fake value", "=~", "num*"]
-      expect(@provider.process_get(command)).to eq(false)
-    end
-  end
-
-  describe "values filters" do
-    before do
-      augeas = double("augeas", :match => ["set", "of", "values"])
-      allow(augeas).to receive(:get).and_return('set', 'of', 'values')
-      allow(augeas).to receive("close")
-      @provider = provider_class.new(@resource)
-      @provider.aug = augeas
-    end
-
-    it "should return true for includes match" do
-      command = ["values", "fake value", "include values"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return false for includes non match" do
-      command = ["values", "fake value", "include JarJar"]
-      expect(@provider.process_values(command)).to eq(false)
-    end
-
-    it "should return true for includes match" do
-      command = ["values", "fake value", "not_include JarJar"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return false for includes non match" do
-      command = ["values", "fake value", "not_include values"]
-      expect(@provider.process_values(command)).to eq(false)
-    end
-
-    it "should return true for an array match" do
-      command = ["values", "fake value", "== ['set', 'of', 'values']"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with double quotes and spaces" do
-      command = ["values", "fake value", "==   [  \"set\"  ,  \"of\" , \"values\"  ]  "]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with internally escaped single quotes" do
-      allow(@provider.aug).to receive(:match).and_return(["set", "o'values", "here"])
-      allow(@provider.aug).to receive(:get).and_return('set', "o'values", 'here')
-      command = ["values", "fake value", "== [ 'set', 'o\\'values', 'here']"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with octal character sequences" do
-      command = ["values", "fake value", "== [\"\\x73et\", \"of\", \"values\"]"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with hex character sequences" do
-      command = ["values", "fake value", "== [\"\\163et\", \"of\", \"values\"]"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with short unicode escape sequences" do
-      command = ["values", "fake value", "== [\"\\u0073et\", \"of\", \"values\"]"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with single character long unicode escape sequences" do
-      command = ["values", "fake value", "== [\"\\u{0073}et\", \"of\", \"values\"]"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with multi-character long unicode escape sequences" do
-      command = ["values", "fake value", "== [\"\\u{0073 0065 0074}\", \"of\", \"values\"]"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array match with literal backslashes" do
-      allow(@provider.aug).to receive(:match).and_return(["set", "o\\values", "here"])
-      allow(@provider.aug).to receive(:get).and_return('set', "o\\values", 'here')
-      command = ["values", "fake value", "== [ 'set', 'o\\\\values', 'here']"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return false for an array non match" do
-      command = ["values", "fake value", "== ['this', 'should', 'not', 'match']"]
-      expect(@provider.process_values(command)).to eq(false)
-    end
-
-    it "should return false for an array match with noteq" do
-      command = ["values", "fake value", "!= ['set', 'of', 'values']"]
-      expect(@provider.process_values(command)).to eq(false)
-    end
-
-    it "should return true for an array non match with noteq" do
-      command = ["values", "fake value", "!= ['this', 'should', 'not', 'match']"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an array non match with double quotes and spaces" do
-      command = ["values", "fake value", "!=   [  \"this\"  ,  \"should\" ,\"not\",  \"match\"  ]  "]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-
-    it "should return true for an empty array match" do
-      allow(@provider.aug).to receive(:match).and_return([])
-      allow(@provider.aug).to receive(:get)
-      command = ["values", "fake value", "== []"]
-      expect(@provider.process_values(command)).to eq(true)
-    end
-  end
-
-  describe "match filters" do
-    before do
-      augeas = double("augeas", :match => ["set", "of", "values"])
-      allow(augeas).to receive("close")
-      @provider = provider_class.new(@resource)
-      @provider.aug = augeas
-    end
-
-    it "should return true for size match" do
-      command = ["match", "fake value", "size == 3"]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return false for a size non match" do
-      command = ["match", "fake value", "size < 3"]
-      expect(@provider.process_match(command)).to eq(false)
-    end
-
-    it "should return true for includes match" do
-      command = ["match", "fake value", "include values"]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return false for includes non match" do
-      command = ["match", "fake value", "include JarJar"]
-      expect(@provider.process_match(command)).to eq(false)
-    end
-
-    it "should return true for includes match" do
-      command = ["match", "fake value", "not_include JarJar"]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return false for includes non match" do
-      command = ["match", "fake value", "not_include values"]
-      expect(@provider.process_match(command)).to eq(false)
-    end
-
-    it "should return true for an array match" do
-      command = ["match", "fake value", "== ['set', 'of', 'values']"]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return true for an array match with double quotes and spaces" do
-      command = ["match", "fake value", "==   [  \"set\"  ,  \"of\" , \"values\"  ]  "]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return false for an array non match" do
-      command = ["match", "fake value", "== ['this', 'should', 'not', 'match']"]
-      expect(@provider.process_match(command)).to eq(false)
-    end
-
-    it "should return false for an array match with noteq" do
-      command = ["match", "fake value", "!= ['set', 'of', 'values']"]
-      expect(@provider.process_match(command)).to eq(false)
-    end
-
-    it "should return true for an array non match with noteq" do
-      command = ["match", "fake value", "!= ['this', 'should', 'not', 'match']"]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-
-    it "should return true for an array non match with double quotes and spaces" do
-      command = ["match", "fake value", "!=   [  \"this\"  ,  \"should\" ,\"not\",  \"match\"  ]  "]
-      expect(@provider.process_match(command)).to eq(true)
-    end
-  end
-
-  describe "need to run" do
-    before(:each) do
-      @augeas = double("augeas")
-      allow(@augeas).to receive("close")
-      @provider.aug = @augeas
-
-      # These tests pretend to be an earlier version so the provider doesn't
-      # attempt to make the change in the need_to_run? method
-      allow(@provider).to receive(:get_augeas_version).and_return("0.3.5")
-    end
-
-    it "should handle no filters" do
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(true)
-    end
-
-    it "should return true when a get filter matches" do
-      @resource[:onlyif] = "get path == value"
-      allow(@augeas).to receive("get").and_return("value")
-      expect(@provider.need_to_run?).to eq(true)
-    end
-
-    describe "performing numeric comparisons (#22617)" do
-      it "should return true when a get string compare is true" do
-        @resource[:onlyif] = "get bpath > a"
-        allow(@augeas).to receive("get").and_return("b")
-        expect(@provider.need_to_run?).to eq(true)
-      end
-
-      it "should return false when a get string compare is false" do
-        @resource[:onlyif] = "get a19path > a2"
-        allow(@augeas).to receive("get").and_return("a19")
-        expect(@provider.need_to_run?).to eq(false)
-      end
-
-      it "should return true when a get int gt compare is true" do
-        @resource[:onlyif] = "get path19 > 2"
-        allow(@augeas).to receive("get").and_return("19")
-        expect(@provider.need_to_run?).to eq(true)
-      end
-
-      it "should return true when a get int ge compare is true" do
-        @resource[:onlyif] = "get path19 >= 2"
-        allow(@augeas).to receive("get").and_return("19")
-        expect(@provider.need_to_run?).to eq(true)
-      end
-
-      it "should return true when a get int lt compare is true" do
-        @resource[:onlyif] = "get path2 < 19"
-        allow(@augeas).to receive("get").and_return("2")
-        expect(@provider.need_to_run?).to eq(true)
-      end
-
-      it "should return false when a get int le compare is false" do
-        @resource[:onlyif] = "get path39 <= 4"
-        allow(@augeas).to receive("get").and_return("39")
-        expect(@provider.need_to_run?).to eq(false)
-      end
-    end
-    describe "performing is_numeric checks (#22617)" do
-      it "should return false for nil" do
-        expect(@provider.is_numeric?(nil)).to eq(false)
-      end
-      it "should return true for Integers" do
-        expect(@provider.is_numeric?(9)).to eq(true)
-      end
-      it "should return true for numbers in Strings" do
-        expect(@provider.is_numeric?('9')).to eq(true)
-      end
-      it "should return false for non-number Strings" do
-        expect(@provider.is_numeric?('x9')).to eq(false)
-      end
-      it "should return false for other types" do
-        expect(@provider.is_numeric?([true])).to eq(false)
-      end
-    end
-
-    it "should return false when a get filter does not match" do
-      @resource[:onlyif] = "get path == another value"
-      allow(@augeas).to receive("get").and_return("value")
-      expect(@provider.need_to_run?).to eq(false)
-    end
-
-    it "should return true when a match filter matches" do
-      @resource[:onlyif] = "match path size == 3"
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(true)
-    end
-
-    it "should return false when a match filter does not match" do
-      @resource[:onlyif] = "match path size == 2"
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(false)
-    end
-
-    # Now setting force to true
-    it "setting force should not change the above logic" do
-      @resource[:force] = true
-      @resource[:onlyif] = "match path size == 2"
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(false)
-    end
-
-    #Ticket 5211 testing
-    it "should return true when a size != the provided value" do
-      @resource[:onlyif] = "match path size != 17"
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(true)
-    end
-
-    #Ticket 5211 testing
-    it "should return false when a size does equal the provided value" do
-      @resource[:onlyif] = "match path size != 3"
-      allow(@augeas).to receive("match").and_return(["set", "of", "values"])
-      expect(@provider.need_to_run?).to eq(false)
-    end
-
-    [true, false].product([true, false]) do |cfg, param|
-      describe "and Puppet[:show_diff] is #{cfg} and show_diff => #{param}" do
-        let(:file) { "/some/random/file" }
-
-        before(:each) do
-          Puppet[:show_diff] = cfg
-          @resource[:show_diff] = param
-
-          @resource[:root] = ""
-          @resource[:context] = "/files"
-          @resource[:changes] = ["set #{file}/foo bar"]
-
-          allow(File).to receive(:delete)
-          allow(@provider).to receive(:get_augeas_version).and_return("0.10.0")
-          allow(@provider).to receive("diff").with("#{file}", "#{file}.augnew").and_return("diff")
-
-          allow(@augeas).to receive(:set).and_return(true)
-          allow(@augeas).to receive(:save).and_return(true)
-          allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved"])
-          allow(@augeas).to receive(:get).with("/augeas/events/saved").and_return("/files#{file}")
-          allow(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        end
-
-        if cfg && param
-          it "should display a diff" do
-            expect(@provider).to be_need_to_run
-
-            expect(@logs[0].message).to eq("\ndiff")
-          end
-        else
-          it "should not display a diff" do
-            expect(@provider).to be_need_to_run
-
-            expect(@logs).to be_empty
-          end
-        end
-      end
-    end
-
-    # Ticket 2728 (diff files)
-    describe "and configured to show diffs" do
-      before(:each) do
-        Puppet[:show_diff] = true
-        @resource[:show_diff] = true
-
-        @resource[:root] = ""
-        allow(@provider).to receive(:get_augeas_version).and_return("0.10.0")
-        allow(@augeas).to receive(:set).and_return(true)
-        allow(@augeas).to receive(:save).and_return(true)
-      end
-
-      it "should display a diff when a single file is shown to have been changed" do
-        file = "/etc/hosts"
-        allow(File).to receive(:delete)
-
-        @resource[:loglevel] = "crit"
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file}/foo bar"]
-
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved"])
-        allow(@augeas).to receive(:get).with("/augeas/events/saved").and_return("/files#{file}")
-        expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        expect(@provider).to receive("diff").with("#{file}", "#{file}.augnew").and_return("diff")
-
-        expect(@provider).to be_need_to_run
-
-        expect(@logs[0].message).to eq("\ndiff")
-        expect(@logs[0].level).to eq(:crit)
-      end
-
-      it "should display a diff for each file that is changed when changing many files" do
-        file1 = "/etc/hosts"
-        file2 = "/etc/resolv.conf"
-        allow(File).to receive(:delete)
-
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file1}/foo bar", "set #{file2}/baz biz"]
-
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved[1]", "/augeas/events/saved[2]"])
-        allow(@augeas).to receive(:get).with("/augeas/events/saved[1]").and_return("/files#{file1}")
-        allow(@augeas).to receive(:get).with("/augeas/events/saved[2]").and_return("/files#{file2}")
-        expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        expect(@provider).to receive(:diff).with("#{file1}", "#{file1}.augnew").and_return("diff #{file1}")
-        expect(@provider).to receive(:diff).with("#{file2}", "#{file2}.augnew").and_return("diff #{file2}")
-
-        expect(@provider).to be_need_to_run
-
-        expect(@logs.collect(&:message)).to include("\ndiff #{file1}", "\ndiff #{file2}")
-        expect(@logs.collect(&:level)).to eq([:notice, :notice])
-      end
-
-      describe "and resource[:root] is set" do
-        it "should call diff when a file is shown to have been changed" do
-          root = "/tmp/foo"
-          file = "/etc/hosts"
-          allow(File).to receive(:delete)
-
-          @resource[:context] = "/files"
-          @resource[:changes] = ["set #{file}/foo bar"]
-          @resource[:root] = root
-
-          allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved"])
-          allow(@augeas).to receive(:get).with("/augeas/events/saved").and_return("/files#{file}")
-          expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-          expect(@provider).to receive(:diff).with("#{root}#{file}", "#{root}#{file}.augnew").and_return("diff")
-
-          expect(@provider).to be_need_to_run
-
-          expect(@logs[0].message).to eq("\ndiff")
-          expect(@logs[0].level).to eq(:notice)
-        end
-      end
-
-      it "should not call diff if no files change" do
-        file = "/etc/hosts"
-
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file}/foo bar"]
-
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return([])
-        expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        expect(@augeas).not_to receive(:get).with("/augeas/events/saved")
-        expect(@augeas).to receive(:close)
-
-        expect(@provider).not_to receive(:diff)
-        expect(@provider).not_to be_need_to_run
-      end
-
-      it "should cleanup the .augnew file" do
-        file = "/etc/hosts"
-
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file}/foo bar"]
-
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved"])
-        allow(@augeas).to receive(:get).with("/augeas/events/saved").and_return("/files#{file}")
-        expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        expect(@augeas).to receive(:close)
-
-        expect(File).to receive(:delete).with(file + ".augnew")
-
-        expect(@provider).to receive(:diff).with("#{file}", "#{file}.augnew").and_return("")
-        expect(@provider).to be_need_to_run
-      end
-
-      # Workaround for Augeas bug #264 which reports filenames twice
-      it "should handle duplicate /augeas/events/saved filenames" do
-        file = "/etc/hosts"
-
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file}/foo bar"]
-
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return(["/augeas/events/saved[1]", "/augeas/events/saved[2]"])
-        allow(@augeas).to receive(:get).with("/augeas/events/saved[1]").and_return("/files#{file}")
-        allow(@augeas).to receive(:get).with("/augeas/events/saved[2]").and_return("/files#{file}")
-        expect(@augeas).to receive(:set).with("/augeas/save", "newfile")
-        expect(@augeas).to receive(:close)
-
-        expect(File).to receive(:delete).with(file + ".augnew").once()
-
-        expect(@provider).to receive(:diff).with("#{file}", "#{file}.augnew").and_return("").once()
-        expect(@provider).to be_need_to_run
-      end
-
-      it "should fail with an error if saving fails" do
-        file = "/etc/hosts"
-
-        @resource[:context] = "/files"
-        @resource[:changes] = ["set #{file}/foo bar"]
-
-        allow(@augeas).to receive(:save).and_return(false)
-        allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return([])
-        expect(@augeas).to receive(:close)
-
-        expect(@provider).not_to receive(:diff)
-        expect(@provider).to receive(:print_put_errors)
-        expect { @provider.need_to_run? }.to raise_error(Puppet::Error)
-      end
-    end
-  end
-
-  describe "augeas execution integration" do
-    before do
-      @augeas = double("augeas", load: nil)
-      allow(@augeas).to receive("close")
-      allow(@augeas).to receive(:match).with("/augeas/events/saved").and_return([])
-
-      @provider.aug = @augeas
-      allow(@provider).to receive(:get_augeas_version).and_return("0.3.5")
-    end
-
-    it "should handle set commands" do
-      @resource[:changes] = "set JarJar Binks"
-      @resource[:context] = "/some/path/"
-      expect(@augeas).to receive(:set).with("/some/path/JarJar", "Binks").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle rm commands" do
-      @resource[:changes] = "rm /Jar/Jar"
-      expect(@augeas).to receive(:rm).with("/Jar/Jar")
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle remove commands" do
-      @resource[:changes] = "remove /Jar/Jar"
-      expect(@augeas).to receive(:rm).with("/Jar/Jar")
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle clear commands" do
-      @resource[:changes] = "clear Jar/Jar"
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:clear).with("/foo/Jar/Jar").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    describe "touch command" do
-      it "should clear missing path" do
-        @resource[:changes] = "touch Jar/Jar"
-        @resource[:context] = "/foo/"
-        expect(@augeas).to receive(:match).with("/foo/Jar/Jar").and_return([])
-        expect(@augeas).to receive(:clear).with("/foo/Jar/Jar").and_return(true)
-        expect(@augeas).to receive(:save).and_return(true)
-        expect(@augeas).to receive(:close)
-        expect(@provider.execute_changes).to eq(:executed)
-      end
-
-      it "should not change on existing path" do
-        @resource[:changes] = "touch Jar/Jar"
-        @resource[:context] = "/foo/"
-        expect(@augeas).to receive(:match).with("/foo/Jar/Jar").and_return(["/foo/Jar/Jar"])
-        expect(@augeas).not_to receive(:clear)
-        expect(@augeas).to receive(:save).and_return(true)
-        expect(@augeas).to receive(:close)
-        expect(@provider.execute_changes).to eq(:executed)
-      end
-    end
-
-    it "should handle ins commands with before" do
-      @resource[:changes] = "ins Binks before Jar/Jar"
-      @resource[:context] = "/foo"
-      expect(@augeas).to receive(:insert).with("/foo/Jar/Jar", "Binks", true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle ins commands with after" do
-      @resource[:changes] = "ins Binks after /Jar/Jar"
-      @resource[:context] = "/foo"
-      expect(@augeas).to receive(:insert).with("/Jar/Jar", "Binks", false)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle ins with no context" do
-      @resource[:changes] = "ins Binks after /Jar/Jar"
-      expect(@augeas).to receive(:insert).with("/Jar/Jar", "Binks", false)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle multiple commands" do
-      @resource[:changes] = ["ins Binks after /Jar/Jar", "clear Jar/Jar"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:insert).with("/Jar/Jar", "Binks", false)
-      expect(@augeas).to receive(:clear).with("/foo/Jar/Jar").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle defvar commands" do
-      @resource[:changes] = "defvar myjar Jar/Jar"
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:defvar).with("myjar", "/foo/Jar/Jar").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should pass through augeas variables without context" do
-      @resource[:changes] = ["defvar myjar Jar/Jar","set $myjar/Binks 1"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:defvar).with("myjar", "/foo/Jar/Jar").and_return(true)
-      # this is the important bit, shouldn't be /foo/$myjar/Binks
-      expect(@augeas).to receive(:set).with("$myjar/Binks", "1").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle defnode commands" do
-      @resource[:changes] = "defnode newjar Jar/Jar[last()+1] Binks"
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:defnode).with("newjar", "/foo/Jar/Jar[last()+1]", "Binks").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle mv commands" do
-      @resource[:changes] = "mv Jar/Jar Binks"
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:mv).with("/foo/Jar/Jar", "/foo/Binks").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle rename commands" do
-      @resource[:changes] = "rename Jar/Jar Binks"
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:rename).with("/foo/Jar/Jar", "Binks").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should handle setm commands" do
-      @resource[:changes] = ["set test[1]/Jar/Jar Foo","set test[2]/Jar/Jar Bar","setm test Jar/Jar Binks"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:respond_to?).with("setm").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[1]/Jar/Jar", "Foo").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[2]/Jar/Jar", "Bar").and_return(true)
-      expect(@augeas).to receive(:setm).with("/foo/test", "Jar/Jar", "Binks").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should throw error if setm command not supported" do
-      @resource[:changes] = ["set test[1]/Jar/Jar Foo","set test[2]/Jar/Jar Bar","setm test Jar/Jar Binks"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:respond_to?).with("setm").and_return(false)
-      expect(@augeas).to receive(:set).with("/foo/test[1]/Jar/Jar", "Foo").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[2]/Jar/Jar", "Bar").and_return(true)
-      expect { @provider.execute_changes }.to raise_error RuntimeError, /command 'setm' not supported/
-    end
-
-    it "should handle clearm commands" do
-      @resource[:changes] = ["set test[1]/Jar/Jar Foo","set test[2]/Jar/Jar Bar","clearm test Jar/Jar"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:respond_to?).with("clearm").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[1]/Jar/Jar", "Foo").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[2]/Jar/Jar", "Bar").and_return(true)
-      expect(@augeas).to receive(:clearm).with("/foo/test", "Jar/Jar").and_return(true)
-      expect(@augeas).to receive(:save).and_return(true)
-      expect(@augeas).to receive(:close)
-      expect(@provider.execute_changes).to eq(:executed)
-    end
-
-    it "should throw error if clearm command not supported" do
-      @resource[:changes] = ["set test[1]/Jar/Jar Foo","set test[2]/Jar/Jar Bar","clearm test Jar/Jar"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:respond_to?).with("clearm").and_return(false)
-      expect(@augeas).to receive(:set).with("/foo/test[1]/Jar/Jar", "Foo").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[2]/Jar/Jar", "Bar").and_return(true)
-      expect { @provider.execute_changes }.to raise_error(RuntimeError, /command 'clearm' not supported/)
-    end
-
-    it "should throw error if saving failed" do
-      @resource[:changes] = ["set test[1]/Jar/Jar Foo","set test[2]/Jar/Jar Bar","clearm test Jar/Jar"]
-      @resource[:context] = "/foo/"
-      expect(@augeas).to receive(:respond_to?).with("clearm").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[1]/Jar/Jar", "Foo").and_return(true)
-      expect(@augeas).to receive(:set).with("/foo/test[2]/Jar/Jar", "Bar").and_return(true)
-      expect(@augeas).to receive(:clearm).with("/foo/test", "Jar/Jar").and_return(true)
-      expect(@augeas).to receive(:save).and_return(false)
-      expect(@provider).to receive(:print_put_errors)
-      expect(@augeas).to receive(:match).and_return([])
-      expect { @provider.execute_changes }.to raise_error(Puppet::Error)
-    end
-  end
-
-  describe "when making changes", :if => Puppet.features.augeas? do
-    include PuppetSpec::Files
-
-    it "should not clobber the file if it's a symlink" do
-      allow(Puppet::Util::Storage).to receive(:store)
-
-      link = tmpfile('link')
-      target = tmpfile('target')
-      FileUtils.touch(target)
-      Puppet::FileSystem.symlink(target, link)
-
-      resource = Puppet::Type.type(:augeas).new(
-        :name => 'test',
-        :incl => link,
-        :lens => 'Sshd.lns',
-        :changes => "set PermitRootLogin no"
-      )
-
-      catalog = Puppet::Resource::Catalog.new
-      catalog.add_resource resource
-
-      catalog.apply
-
-      expect(File.ftype(link)).to eq('link')
-      expect(Puppet::FileSystem.readlink(link)).to eq(target)
-      expect(File.read(target)).to match(/PermitRootLogin no/)
-    end
-  end
-
-  describe "load/save failure reporting" do
-    before do
-      @augeas = double("augeas")
-      allow(@augeas).to receive("close")
-      @provider.aug = @augeas
-    end
-
-    describe "should find load errors" do
-      before do
-        expect(@augeas).to receive(:match).with("/augeas//error").and_return(["/augeas/files/foo/error"])
-        expect(@augeas).to receive(:match).with("/augeas/files/foo/error/*").and_return(["/augeas/files/foo/error/path", "/augeas/files/foo/error/message"])
-        expect(@augeas).to receive(:get).with("/augeas/files/foo/error").and_return("some_failure")
-        expect(@augeas).to receive(:get).with("/augeas/files/foo/error/path").and_return("/foo")
-        expect(@augeas).to receive(:get).with("/augeas/files/foo/error/message").and_return("Failed to...")
-      end
-
-      it "and output only to debug when no path supplied" do
-        expect(@provider).to receive(:debug).exactly(5).times()
-        expect(@provider).not_to receive(:warning)
-        @provider.print_load_errors(nil)
-      end
-
-      it "and output a warning and to debug when path supplied" do
-        expect(@augeas).to receive(:match).with("/augeas/files/foo//error").and_return(["/augeas/files/foo/error"])
-        expect(@provider).to receive(:warning).once()
-        expect(@provider).to receive(:debug).exactly(4).times()
-        @provider.print_load_errors('/augeas/files/foo//error')
-      end
-
-      it "and output only to debug when path doesn't match" do
-        expect(@augeas).to receive(:match).with("/augeas/files/foo//error").and_return([])
-        expect(@provider).not_to receive(:warning)
-        expect(@provider).to receive(:debug).exactly(5).times()
-        @provider.print_load_errors('/augeas/files/foo//error')
-      end
-    end
-
-    it "should find load errors from lenses" do
-      expect(@augeas).to receive(:match).with("/augeas//error").twice.and_return(["/augeas/load/Xfm/error"])
-      expect(@augeas).to receive(:match).with("/augeas/load/Xfm/error/*").and_return([])
-      expect(@augeas).to receive(:get).with("/augeas/load/Xfm/error").and_return(["Could not find lens php.aug"])
-      expect(@provider).to receive(:warning).once()
-      expect(@provider).to receive(:debug).twice()
-      @provider.print_load_errors('/augeas//error')
-    end
-
-    it "should find save errors and output to debug" do
-      expect(@augeas).to receive(:match).with("/augeas//error[. = 'put_failed']").and_return(["/augeas/files/foo/error"])
-      expect(@augeas).to receive(:match).with("/augeas/files/foo/error/*").and_return(["/augeas/files/foo/error/path", "/augeas/files/foo/error/message"])
-      expect(@augeas).to receive(:get).with("/augeas/files/foo/error").and_return("some_failure")
-      expect(@augeas).to receive(:get).with("/augeas/files/foo/error/path").and_return("/foo")
-      expect(@augeas).to receive(:get).with("/augeas/files/foo/error/message").and_return("Failed to...")
-      expect(@provider).to receive(:debug).exactly(5).times()
-      @provider.print_put_errors
-    end
-  end
-
-  # Run initialisation tests of the real Augeas library to test our open_augeas
-  # method.  This relies on Augeas and ruby-augeas on the host to be
-  # functioning.
-  describe "augeas lib initialisation", :if => Puppet.features.augeas? do
-    # Expect lenses for fstab and hosts
-    it "should have loaded standard files by default" do
-      aug = @provider.open_augeas
-      expect(aug).not_to eq(nil)
-      expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-      expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      expect(aug.match("/files/etc/test")).to eq([])
-    end
-
-    it "should report load errors to debug only" do
-      expect(@provider).to receive(:print_load_errors).with(nil)
-      aug = @provider.open_augeas
-      expect(aug).not_to eq(nil)
-    end
-
-    # Only the file specified should be loaded
-    it "should load one file if incl/lens used" do
-      @resource[:incl] = "/etc/hosts"
-      @resource[:lens] = "Hosts.lns"
-
-      expect(@provider).to receive(:print_load_errors).with('/augeas//error')
-      aug = @provider.open_augeas
-      expect(aug).not_to eq(nil)
-      expect(aug.match("/files/etc/fstab")).to eq([])
-      expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      expect(aug.match("/files/etc/test")).to eq([])
-    end
-
-    it "should also load lenses from load_path" do
-      @resource[:load_path] = my_fixture_dir
-
-      aug = @provider.open_augeas
-      expect(aug).not_to eq(nil)
-      expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-      expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      expect(aug.match("/files/etc/test")).to eq(["/files/etc/test"])
-    end
-
-    it "should also load lenses from pluginsync'd path" do
-      Puppet[:libdir] = my_fixture_dir
-
-      aug = @provider.open_augeas
-      expect(aug).not_to eq(nil)
-      expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-      expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      expect(aug.match("/files/etc/test")).to eq(["/files/etc/test"])
-    end
-
-    # Optimisations added for Augeas 0.8.2 or higher is available, see #7285
-    describe ">= 0.8.2 optimisations", :if => Puppet.features.augeas? && Facter.value(:augeasversion) && Puppet::Util::Package.versioncmp(Facter.value(:augeasversion), "0.8.2") >= 0 do
-      it "should only load one file if relevant context given" do
-        @resource[:context] = "/files/etc/fstab"
-
-        expect(@provider).to receive(:print_load_errors).with('/augeas/files/etc/fstab//error')
-        aug = @provider.open_augeas
-        expect(aug).not_to eq(nil)
-        expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-        expect(aug.match("/files/etc/hosts")).to eq([])
-      end
-
-      it "should only load one lens from load_path if context given" do
-        @resource[:context] = "/files/etc/test"
-        @resource[:load_path] = my_fixture_dir
-
-        expect(@provider).to receive(:print_load_errors).with('/augeas/files/etc/test//error')
-        aug = @provider.open_augeas
-        expect(aug).not_to eq(nil)
-        expect(aug.match("/files/etc/fstab")).to eq([])
-        expect(aug.match("/files/etc/hosts")).to eq([])
-        expect(aug.match("/files/etc/test")).to eq(["/files/etc/test"])
-      end
-
-      it "should load standard files if context isn't specific" do
-        @resource[:context] = "/files/etc"
-
-        expect(@provider).to receive(:print_load_errors).with(nil)
-        aug = @provider.open_augeas
-        expect(aug).not_to eq(nil)
-        expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-        expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      end
-
-      it "should not optimise if the context is a complex path" do
-        @resource[:context] = "/files/*[label()='etc']"
-
-        expect(@provider).to receive(:print_load_errors).with(nil)
-        aug = @provider.open_augeas
-        expect(aug).not_to eq(nil)
-        expect(aug.match("/files/etc/fstab")).to eq(["/files/etc/fstab"])
-        expect(aug.match("/files/etc/hosts")).to eq(["/files/etc/hosts"])
-      end
-    end
-  end
-
-  describe "get_load_path" do
-    it "should offer no load_path by default" do
-      expect(@provider.get_load_path(@resource)).to eq("")
-    end
-
-    it "should offer one path from load_path" do
-      @resource[:load_path] = "/foo"
-      expect(@provider.get_load_path(@resource)).to eq("/foo")
-    end
-
-    it "should offer multiple colon-separated paths from load_path" do
-      @resource[:load_path] = "/foo:/bar:/baz"
-      expect(@provider.get_load_path(@resource)).to eq("/foo:/bar:/baz")
-    end
-
-    it "should offer multiple paths in array from load_path" do
-      @resource[:load_path] = ["/foo", "/bar", "/baz"]
-      expect(@provider.get_load_path(@resource)).to eq("/foo:/bar:/baz")
-    end
-
-    it "should offer pluginsync augeas/lenses subdir" do
-      Puppet[:libdir] = my_fixture_dir
-      expect(@provider.get_load_path(@resource)).to eq("#{my_fixture_dir}/augeas/lenses")
-    end
-
-    it "should offer both pluginsync and load_path paths" do
-      Puppet[:libdir] = my_fixture_dir
-      @resource[:load_path] = ["/foo", "/bar", "/baz"]
-      expect(@provider.get_load_path(@resource)).to eq("/foo:/bar:/baz:#{my_fixture_dir}/augeas/lenses")
-    end
-  end
-end