(PUP-8766) add libdir option for puppet device

This allows overriding the per-device libdir to enable local
and testing scenarios where pluginsync doesn't make sense.

Author: DavidS

lib/puppet/application/device.rb

@@ -54,6 +54,10 @@ def preinit
     options[:detailed_exitcodes] = true
   end
 
+  option("--libdir LIBDIR") do |arg|
+    options[:libdir] = arg
+  end
+
   option("--apply MANIFEST") do |arg|
     options[:apply] = arg.to_s
   end
@@ -97,6 +101,7 @@ def help
   puppet device [-h|--help] [-v|--verbose] [-d|--debug]
                 [-l|--logdest syslog|<file>|console] [--detailed-exitcodes]
                 [--deviceconfig <file>] [-w|--waitforcert <seconds>]
+                [--libdir <directory>]
                 [-a|--apply <file>] [-f|--facts] [-r|--resource <type> [name]]
                 [-t|--target <device>] [--user=<user>] [-V|--version]
 
@@ -173,6 +178,10 @@ def help
   certificate request.  This is useful for the initial setup of a target.
   You can turn off waiting for certificates by specifying a time of 0.
 
+* --libdir:
+  Override the per-device libdir with a local directory. Specifying a libdir also
+  disables pluginsync. This is useful for testing.
+
 * --apply:
   Apply a manifest against a remote target. Target must be specified.
 
@@ -253,11 +262,11 @@ def main
 
           # override local $vardir and $certname
           Puppet[:confdir] = ::File.join(Puppet[:devicedir], device.name)
-          Puppet[:libdir] = ::File.join(Puppet[:devicedir], device.name, 'lib')
+          Puppet[:libdir] = options[:libdir] || ::File.join(Puppet[:devicedir], device.name, 'lib')
           Puppet[:vardir] = ::File.join(Puppet[:devicedir], device.name)
           Puppet[:certname] = device.name
 
-          unless options[:resource] || options[:facts] || options[:apply]
+          unless options[:resource] || options[:facts] || options[:apply] || options[:libdir]
             Puppet::Configurer::PluginHandler.new.download_plugins(env)
           end
           # this init the device singleton, so that the facts terminus

spec/unit/application/device_spec.rb

@@ -299,6 +299,7 @@
       @device.options.stubs(:[]).with(:facts).returns(false)
       @device.options.stubs(:[]).with(:resource).returns(false)
       @device.options.stubs(:[]).with(:to_yaml).returns(false)
+      @device.options.stubs(:[]).with(:libdir).returns(nil)
       @device.options.stubs(:[]).with(:client)
       @device.command_line.stubs(:args).returns([])
       Puppet::Util::NetworkDevice::Config.stubs(:devices).returns({})