rbvmomi2 3.0.0 → 3.0.1

data/lib/rbvmomi/vim/PerformanceManager.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2012-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -20,18 +21,18 @@ class RbVmomi::VIM::PerformanceManager
   def perfcounter_cached
     @perfcounter ||= perfCounter
   end
-  
+
   def perfcounter_hash
-    @perfcounter_hash ||= Hash[perfcounter_cached.map{|x| [x.name, x]}]
+    @perfcounter_hash ||= Hash[perfcounter_cached.map{ |x| [x.name, x] }]
   end
-  
-  def perfcounter_idhash 
-    @perfcounter_idhash ||= Hash[perfcounter_cached.map{|x| [x.key, x]}]
+
+  def perfcounter_idhash
+    @perfcounter_idhash ||= Hash[perfcounter_cached.map{ |x| [x.key, x] }]
   end
-  
+
   def provider_summary obj
     @provider_summary ||= {}
-    @provider_summary[obj.class] ||= QueryPerfProviderSummary(:entity => obj)
+    @provider_summary[obj.class] ||= QueryPerfProviderSummary(entity: obj)
   end
 
   def retrieve_stats objects, metrics, opts = {}
@@ -44,45 +45,41 @@ class RbVmomi::VIM::PerformanceManager
       realtime = true
     else
       provider = provider_summary objects.first
-      if opts[:interval] == provider.refreshRate
-        realtime = true
-      end
+      realtime = true if opts[:interval] == provider.refreshRate
     end
-      
+
     instances = opts[:instance] || '*'
-    if !instances.is_a?(Array)
-      instances = [instances]
-    end
+    instances = [instances] if !instances.is_a?(Array)
     metric_ids = []
-    metrics.each do |x| 
+    metrics.each do |x|
       counter = perfcounter_hash[x]
       if !counter
         pp perfcounter_hash.keys
-        fail "Counter for #{x} couldn't be found"
+        raise "Counter for #{x} couldn't be found"
       end
       instances.each do |instance|
-        metric_ids << RbVmomi::VIM::PerfMetricId(:counterId => counter.key, 
-                                                 :instance => instance)
+        metric_ids << RbVmomi::VIM::PerfMetricId(counterId: counter.key,
+                                                 instance: instance)
       end
     end
     query_specs = objects.map do |obj|
       RbVmomi::VIM::PerfQuerySpec({
-        :maxSample => max_samples, 
-        :entity => obj, 
-        :metricId => metric_ids, 
-        :intervalId => opts[:interval],
-        :startTime => (realtime == false ? opts[:start_time].to_datetime : nil),
+        maxSample: max_samples,
+        entity: obj,
+        metricId: metric_ids,
+        intervalId: opts[:interval],
+        startTime: (realtime == false ? opts[:start_time].to_datetime : nil),
       })
     end
-    stats = QueryPerf(:querySpec => query_specs)
-    
+    stats = QueryPerf(querySpec: query_specs)
+
     if !opts[:multi_instance]
       Hash[stats.map do |res|
         [
-          res.entity, 
+          res.entity,
           {
-            :sampleInfo => res.sampleInfo,
-            :metrics => Hash[res.value.map do |metric|
+            sampleInfo: res.sampleInfo,
+            metrics: Hash[res.value.map do |metric|
               metric_name = perfcounter_idhash[metric.id.counterId].name
               [metric_name, metric.value]
             end]
@@ -92,10 +89,10 @@ class RbVmomi::VIM::PerformanceManager
     else
       Hash[stats.map do |res|
         [
-          res.entity, 
+          res.entity,
           {
-            :sampleInfo => res.sampleInfo,
-            :metrics => Hash[res.value.map do |metric|
+            sampleInfo: res.sampleInfo,
+            metrics: Hash[res.value.map do |metric|
               metric_name = perfcounter_idhash[metric.id.counterId].name
               [[metric_name, metric.id.instance], metric.value]
             end]

data/lib/rbvmomi/vim/PropertyCollector.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -5,22 +6,22 @@ class RbVmomi::VIM::PropertyCollector
   def collectMultiple objs, *pathSet
     return {} if objs.empty?
 
-    klasses = objs.map{|x| x.class}.uniq 
+    klasses = objs.map{ |x| x.class }.uniq
     klass = if klasses.length > 1
       # common superclass
-      klasses.map(&:ancestors).inject(&:&)[0]
+              klasses.map(&:ancestors).inject(&:&)[0]
     else
       klasses.first
     end
 
     spec = {
-      :objectSet => objs.map{|x| { :obj => x }},
-      :propSet => [{
-        :pathSet => pathSet,
-        :type => klass.wsdl_name
+      objectSet: objs.map{ |x| { obj: x } },
+      propSet: [{
+        pathSet: pathSet,
+        type: klass.wsdl_name
       }]
     }
-    res = RetrieveProperties(:specSet => [spec])
+    res = RetrieveProperties(specSet: [spec])
     Hash[res.map do |x|
       [x.obj, x.to_hash]
     end]

data/lib/rbvmomi/vim/ReflectManagedMethodExecuter.rb

@@ -1,33 +1,34 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
 module RbVmomi
 
-class VIM::ReflectManagedMethodExecuter
-  def fetch moid, prop
-    result = FetchSoap(:moid => moid, :version => 'urn:vim25/6.5', :prop => prop)
-    xml = Nokogiri(result.response)
-    _connection.deserializer.deserialize xml.root, nil
-  end
+  class VIM::ReflectManagedMethodExecuter
+    def fetch moid, prop
+      result = FetchSoap(moid: moid, version: 'urn:vim25/6.5', prop: prop)
+      xml = Nokogiri(result.response)
+      _connection.deserializer.deserialize xml.root, nil
+    end
 
-  def execute moid, method, args
-    soap_args = args.map do |k,v|
-      VIM::ReflectManagedMethodExecuterSoapArgument.new.tap do |soap_arg|
-        soap_arg.name = k
-        xml = Builder::XmlMarkup.new :indent => 0
-        _connection.obj2xml xml, k, :anyType, false, v
-        soap_arg.val = xml.target!
+    def execute moid, method, args
+      soap_args = args.map do |k, v|
+        VIM::ReflectManagedMethodExecuterSoapArgument.new.tap do |soap_arg|
+          soap_arg.name = k
+          xml = Builder::XmlMarkup.new indent: 0
+          _connection.obj2xml xml, k, :anyType, false, v
+          soap_arg.val = xml.target!
+        end
+      end
+      result = ExecuteSoap(moid: moid, version: 'urn:vim25/6.5',
+                           method: method, argument: soap_args)
+      if result
+        _connection.deserializer.deserialize Nokogiri(result.response).root, nil
+      else
+        nil
       end
-    end
-    result = ExecuteSoap(:moid => moid, :version => 'urn:vim25/6.5',
-                         :method => method, :argument => soap_args)
-    if result
-      _connection.deserializer.deserialize Nokogiri(result.response).root, nil
-    else
-      nil
     end
   end
-end
 
 end
 

data/lib/rbvmomi/vim/ResourcePool.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -6,7 +7,7 @@ class RbVmomi::VIM::ResourcePool
   # @param name [String] Name of the child.
   # @return [VIM::ResourcePool]
   def find name
-    _connection.searchIndex.FindChild(:entity => self, :name => name)
+    _connection.searchIndex.FindChild(entity: self, name: name)
   end
 
   # Retrieve a descendant of this ResourcePool.
@@ -14,7 +15,7 @@ class RbVmomi::VIM::ResourcePool
   # @return [VIM::ResourcePool]
   def traverse path
     es = path.split('/').reject(&:empty?)
-    es.inject(self) do |f,e|
+    es.inject(self) do |f, e|
       f.find(e) || return
     end
   end
@@ -22,35 +23,35 @@ class RbVmomi::VIM::ResourcePool
   def resourcePoolSubTree fields = []
     self.class.resourcePoolSubTree [self], fields
   end
-  
+
   def self.resourcePoolSubTree objs, fields = []
     fields = (fields + ['name', 'resourcePool']).uniq
     filterSpec = RbVmomi::VIM.PropertyFilterSpec(
-      :objectSet => objs.map do |obj|
+      objectSet: objs.map do |obj|
         RbVmomi::VIM.ObjectSpec(
-          :obj => obj,
-          :selectSet => [
+          obj: obj,
+          selectSet: [
             RbVmomi::VIM.TraversalSpec(
-              :name => "tsRP",
-              :type => 'ResourcePool',
-              :path => 'resourcePool',
-              :skip => false,
-              :selectSet => [
-                RbVmomi::VIM.SelectionSpec(:name => "tsRP")
+              name: 'tsRP',
+              type: 'ResourcePool',
+              path: 'resourcePool',
+              skip: false,
+              selectSet: [
+                RbVmomi::VIM.SelectionSpec(name: 'tsRP')
               ]
             )
           ]
         )
       end,
-      :propSet => [{
-        :pathSet => fields,
-        :type => 'ResourcePool'
+      propSet: [{
+        pathSet: fields,
+        type: 'ResourcePool'
       }]
     )
-  
+
     propCollector = objs.first._connection.propertyCollector
-    result = propCollector.RetrieveProperties(:specSet => [filterSpec])
-    
+    result = propCollector.RetrieveProperties(specSet: [filterSpec])
+
     Hash[result.map do |x|
       [x.obj, x.to_hash]
     end]

data/lib/rbvmomi/vim/ServiceInstance.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -24,16 +25,16 @@ class RbVmomi::VIM::ServiceInstance
   def wait_for_multiple_tasks interested, tasks
     version = ''
     interested = (interested + ['info.state']).uniq
-    task_props = Hash.new { |h,k| h[k] = {} }
+    task_props = Hash.new { |h, k| h[k] = {} }
 
-    filter = _connection.propertyCollector.CreateFilter :spec => {
-      :propSet => [{ :type => 'Task', :all => false, :pathSet => interested }],
-      :objectSet => tasks.map { |x| { :obj => x } },
-    }, :partialUpdates => false
+    filter = _connection.propertyCollector.CreateFilter spec: {
+      propSet: [{ type: 'Task', all: false, pathSet: interested }],
+      objectSet: tasks.map { |x| { obj: x } },
+    }, partialUpdates: false
 
     begin
-      until task_props.size == tasks.size and task_props.all? { |k,h| %w(success error).member? h['info.state'] }
-        result = _connection.propertyCollector.WaitForUpdates(:version => version)
+      until task_props.size == tasks.size and task_props.all? { |k, h| %w(success error).member? h['info.state'] }
+        result = _connection.propertyCollector.WaitForUpdates(version: version)
         version = result.version
         os = result.filterSet[0].objectSet
 

data/lib/rbvmomi/vim/Task.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -14,7 +15,7 @@ class RbVmomi::VIM::Task
       raise info.error
     end
   end
-  
+
   # Wait for all child tasks to finish. If any one child task failed,
   # the exception of the first failing task is thrown.
   # @return [Hash] Map of tasks to their +info.result+ on success.
@@ -51,17 +52,17 @@ class RbVmomi::VIM::Task
       raise info.error
     end
   end
-  
+
   # Get child tasks of this task.
   # @return [Array] List of VIM::Task objects
   def child_tasks
     tm = _connection.serviceContent.taskManager
-    col = tm.CreateCollectorForTasks(:filter => {
-      :rootTaskKey => [self.info.key],
+    col = tm.CreateCollectorForTasks(filter: {
+      rootTaskKey: [self.info.key],
     })
     # XXX: Likely this is not enough and we need to collect pages other
     #      than the latest.
-    tasks = col.latestPage.map{|x| x.task}
+    tasks = col.latestPage.map{ |x| x.task }
     col.DestroyCollector()
     tasks
   end

data/lib/rbvmomi/vim/VirtualMachine.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
@@ -18,7 +19,7 @@ class RbVmomi::VIM::VirtualMachine
   # @return [String] Current IP reported (as per VMware Tools) or nil
   def guest_ip
     g = self.guest
-    if g.ipAddress && (g.toolsStatus == "toolsOk" || g.toolsStatus == "toolsOld")
+    if g.ipAddress && (g.toolsStatus == 'toolsOk' || g.toolsStatus == 'toolsOld')
       g.ipAddress
     else
       nil
@@ -36,7 +37,7 @@ class RbVmomi::VIM::VirtualMachine
   # share the base disk.
   def add_delta_disk_layer_on_all_disks
     spec = update_spec_add_delta_disk_layer_on_all_disks
-    self.ReconfigVM_Task(:spec => spec).wait_for_completion
+    self.ReconfigVM_Task(spec: spec).wait_for_completion
   end
 
   # Updates a passed in spec to perform the task of adding a delta disk layer
@@ -50,13 +51,13 @@ class RbVmomi::VIM::VirtualMachine
     device_change = []
     disks.each do |disk|
       device_change << {
-        :operation => :remove,
-        :device => disk
+        operation: :remove,
+        device: disk
       }
       device_change << {
-        :operation => :add,
-        :fileOperation => :create,
-        :device => disk.dup.tap { |x|
+        operation: :add,
+        fileOperation: :create,
+        device: disk.dup.tap { |x|
           x.backing = x.backing.dup
           x.backing.fileName = "[#{disk.backing.datastore.name}]"
           x.backing.parent = disk.backing

data/lib/rbvmomi/vim.rb

@@ -1,157 +1,140 @@
+# frozen_string_literal: true
 # Copyright (c) 2011-2017 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT
 
-# Win32::SSPI is part of core on Windows
-begin
-  require 'win32/sspi'
-rescue LoadError
-end
-WIN32 = (defined? Win32::SSPI)
-
 module RbVmomi
 
 # A connection to one vSphere SDK endpoint.
 # @see #serviceInstance
-class VIM < Connection
-  # Connect to a vSphere SDK endpoint
-  #
-  # @param [Hash] opts The options hash.
-  # @option opts [String]  :host Host to connect to.
-  # @option opts [Numeric] :port (443) Port to connect to.
-  # @option opts [Boolean] :ssl (true) Whether to use SSL.
-  # @option opts [Boolean] :insecure (false) If true, ignore SSL certificate errors.
-  # @option opts [String]  :cookie If set, use cookie to connect instead of user/password
-  # @option opts [String]  :user (root) Username.
-  # @option opts [String]  :password Password.
-  # @option opts [String]  :path (/sdk) SDK endpoint path.
-  # @option opts [Boolean] :debug (false) If true, print SOAP traffic to stderr.
-  # @option opts [String]  :operation_id If set, use for operationID
-  # @option opts [Boolean] :close_on_exit (true) If true, will close connection with at_exit
-  # @option opts [RbVmomi::SSO] :sso (nil) Use SSO token to login if set
-  def self.connect opts
-    fail unless opts.is_a? Hash
-    fail "host option required" unless opts[:host]
-    opts[:cookie] ||= nil
-    opts[:user] ||= (WIN32 ? ENV['USERNAME'].dup : 'root')
-    opts[:password] ||= ''
-    opts[:ssl] = true unless opts.member? :ssl or opts[:"no-ssl"]
-    opts[:insecure] ||= false
-    opts[:port] ||= (opts[:ssl] ? 443 : 80)
-    opts[:path] ||= '/sdk'
-    opts[:ns] ||= 'urn:vim25'
-    opts[:rev] = '6.7' if opts[:rev].nil?
-    opts[:debug] = (!ENV['RBVMOMI_DEBUG'].empty? rescue false) unless opts.member? :debug
-
-    conn = new(opts).tap do |vim|
-      unless opts[:cookie]
-        if WIN32 && opts[:password] == ''
-            # Attempt login by SSPI if no password specified on Windows
-            negotiation = Win32::SSPI::NegotiateAuth.new opts[:user], ENV['USERDOMAIN'].dup
-            begin
-              vim.serviceContent.sessionManager.LoginBySSPI :base64Token => negotiation.get_initial_token
-            rescue RbVmomi::Fault => fault
-              if !fault.fault.is_a?(RbVmomi::VIM::SSPIChallenge)
-                raise
-              else
-                vim.serviceContent.sessionManager.LoginBySSPI :base64Token => negotiation.complete_authentication(fault.base64Token)
-              end
-            end
-        elsif opts[:sso]
-          vim.serviceContent.sessionManager.LoginByToken
-        else
-          vim.serviceContent.sessionManager.Login :userName => opts[:user], :password => opts[:password]
+  class VIM < Connection
+    # Connect to a vSphere SDK endpoint
+    #
+    # @param [Hash] opts The options hash.
+    # @option opts [String]  :host Host to connect to.
+    # @option opts [Numeric] :port (443) Port to connect to.
+    # @option opts [Boolean] :ssl (true) Whether to use SSL.
+    # @option opts [Boolean] :insecure (false) If true, ignore SSL certificate errors.
+    # @option opts [String]  :cookie If set, use cookie to connect instead of user/password
+    # @option opts [String]  :user (root) Username.
+    # @option opts [String]  :password Password.
+    # @option opts [String]  :path (/sdk) SDK endpoint path.
+    # @option opts [Boolean] :debug (false) If true, print SOAP traffic to stderr.
+    # @option opts [String]  :operation_id If set, use for operationID
+    # @option opts [Boolean] :close_on_exit (true) If true, will close connection with at_exit
+    # @option opts [RbVmomi::SSO] :sso (nil) Use SSO token to login if set
+    def self.connect opts
+      raise unless opts.is_a? Hash
+      raise 'host option required' unless opts[:host]
+
+      opts[:cookie] ||= nil
+      opts[:user] ||= 'root'
+      opts[:password] ||= ''
+      opts[:ssl] = true unless opts.member? :ssl or opts[:"no-ssl"]
+      opts[:insecure] ||= false
+      opts[:port] ||= (opts[:ssl] ? 443 : 80)
+      opts[:path] ||= '/sdk'
+      opts[:ns] ||= 'urn:vim25'
+      opts[:rev] = '6.7' if opts[:rev].nil?
+      opts[:debug] = (!ENV['RBVMOMI_DEBUG'].empty? rescue false) unless opts.member? :debug
+
+      conn = new(opts).tap do |vim|
+        unless opts[:cookie]
+          if opts[:sso]
+            vim.serviceContent.sessionManager.LoginByToken
+          else
+            vim.serviceContent.sessionManager.Login userName: opts[:user], password: opts[:password]
+          end
         end
+        rev = vim.serviceContent.about.apiVersion
+        vim.rev = [rev, opts[:rev]].min { |a, b| Gem::Version.new(a) <=> Gem::Version.new(b) }
       end
-      rev = vim.serviceContent.about.apiVersion
-      vim.rev = [rev, opts[:rev]].min { |a, b| Gem::Version.new(a) <=> Gem::Version.new(b) }
-    end
 
-    at_exit { conn.close } if opts.fetch(:close_on_exit, true)
-    conn
-  end
+      at_exit { conn.close } if opts.fetch(:close_on_exit, true)
+      conn
+    end
 
-  def close
-    serviceContent.sessionManager.Logout
-  rescue RbVmomi::Fault => e
-    $stderr.puts(e.message) if debug
-  ensure
-    self.cookie = nil
-    super
-  end
+    def close
+      serviceContent.sessionManager.Logout
+    rescue RbVmomi::Fault => e
+      $stderr.puts(e.message) if debug
+    ensure
+      self.cookie = nil
+      super
+    end
 
-  def rev= x
-    super
-    @serviceContent = nil
-  end
+    def rev= x
+      super
+      @serviceContent = nil
+    end
 
-  # Return the ServiceInstance
-  #
-  # The ServiceInstance is the root of the vSphere inventory.
-  # @see http://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/vim.ServiceInstance.html
-  def serviceInstance
-    VIM::ServiceInstance self, 'ServiceInstance'
-  end
+    # Return the ServiceInstance
+    #
+    # The ServiceInstance is the root of the vSphere inventory.
+    # @see http://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/vim.ServiceInstance.html
+    def serviceInstance
+      VIM::ServiceInstance self, 'ServiceInstance'
+    end
 
-  # Alias to serviceInstance.RetrieveServiceContent
-  def serviceContent
-    @serviceContent ||= serviceInstance.RetrieveServiceContent
-  end
+    # Alias to serviceInstance.RetrieveServiceContent
+    def serviceContent
+      @serviceContent ||= serviceInstance.RetrieveServiceContent
+    end
 
-  # Alias to serviceContent.rootFolder
-  def rootFolder
-    serviceContent.rootFolder
-  end
+    # Alias to serviceContent.rootFolder
+    def rootFolder
+      serviceContent.rootFolder
+    end
 
-  alias root rootFolder
+    alias root rootFolder
 
-  # Alias to serviceContent.propertyCollector
-  def propertyCollector
-    serviceContent.propertyCollector
-  end
-
-  # Alias to serviceContent.searchIndex
-  def searchIndex
-    serviceContent.searchIndex
-  end
+    # Alias to serviceContent.propertyCollector
+    def propertyCollector
+      serviceContent.propertyCollector
+    end
 
-  # @private
-  def pretty_print pp
-    pp.text "VIM(#{@opts[:host]})"
-  end
+    # Alias to serviceContent.searchIndex
+    def searchIndex
+      serviceContent.searchIndex
+    end
 
-  def instanceUuid
-    serviceContent.about.instanceUuid
-  end
+    # @private
+    def pretty_print pp
+      pp.text "VIM(#{@opts[:host]})"
+    end
 
-  def get_log_lines logKey, lines=5, start=nil, host=nil
-    diagMgr = self.serviceContent.diagnosticManager
-    if !start
-      log = diagMgr.BrowseDiagnosticLog(:host => host, :key => logKey, :start => 999999999)
-      lineEnd = log.lineEnd
-      start = lineEnd - lines
+    def instanceUuid
+      serviceContent.about.instanceUuid
     end
-    start = start < 0 ? 0 : start
-    log = diagMgr.BrowseDiagnosticLog(:host => host, :key => logKey, :start => start)
-    if log.lineText.size > 0
-      [log.lineText.slice(-lines, log.lineText.size), log.lineEnd]
-    else
-      [log.lineText, log.lineEnd]
+
+    def get_log_lines logKey, lines=5, start=nil, host=nil
+      diagMgr = self.serviceContent.diagnosticManager
+      if !start
+        log = diagMgr.BrowseDiagnosticLog(host: host, key: logKey, start: 999999999)
+        lineEnd = log.lineEnd
+        start = lineEnd - lines
+      end
+      start = start < 0 ? 0 : start
+      log = diagMgr.BrowseDiagnosticLog(host: host, key: logKey, start: start)
+      if log.lineText.size > 0
+        [log.lineText.slice(-lines, log.lineText.size), log.lineEnd]
+      else
+        [log.lineText, log.lineEnd]
+      end
     end
-  end
 
-  def get_log_keys host=nil
-    diagMgr = self.serviceContent.diagnosticManager
-    keys = []
-    diagMgr.QueryDescriptions(:host => host).each do |desc|
-      keys << "#{desc.key}"
+    def get_log_keys host=nil
+      diagMgr = self.serviceContent.diagnosticManager
+      keys = []
+      diagMgr.QueryDescriptions(host: host).each do |desc|
+        keys << "#{desc.key}"
+      end
+      keys
     end
-    keys
-  end
 
-  add_extension_dir File.join(File.dirname(__FILE__), "vim")
-  (ENV['RBVMOMI_VIM_EXTENSION_PATH']||'').split(':').each { |dir| add_extension_dir dir }
+    add_extension_dir File.join(File.dirname(__FILE__), 'vim')
+    (ENV['RBVMOMI_VIM_EXTENSION_PATH']||'').split(':').each { |dir| add_extension_dir dir }
 
-  load_vmodl(ENV['VMODL'] || File.join(File.dirname(__FILE__), "../../vmodl.db"))
-end
+    load_vmodl(ENV['VMODL'] || File.join(File.dirname(__FILE__), '../../vmodl.db'))
+  end
 
 end

data/lib/rbvmomi.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (c) 2010-2019 VMware, Inc.  All Rights Reserved.
 # SPDX-License-Identifier: MIT