brightbox-cli 3.3.0 → 4.0.0.rc2

data/lib/brightbox-cli/detailed_server.rb

@@ -6,12 +6,12 @@ module Brightbox
       row_attributes[:compatibility_mode] = row_attributes["compatibility_mode"]
 
       if server_type
-        row_attributes[:type] = server_type['id']
-        row_attributes[:type_handle] = server_type['handle']
-        row_attributes[:type_name] = server_type['name']
-        row_attributes[:ram] = server_type['ram']
-        row_attributes[:cores] = server_type['cores']
-        row_attributes[:disk] = server_type['disk_size'].to_i
+        row_attributes[:type] = server_type["id"]
+        row_attributes[:type_handle] = server_type["handle"]
+        row_attributes[:type_name] = server_type["name"]
+        row_attributes[:ram] = server_type["ram"]
+        row_attributes[:cores] = server_type["cores"]
+        row_attributes[:disk] = server_type["disk_size"].to_i
       end
 
       if image
@@ -19,52 +19,52 @@ module Brightbox
         row_attributes[:arch] = image.arch
       end
 
-      row_attributes[:private_ips] = interfaces.map { |i| i['ipv4_address'] }.join(", ")
-      row_attributes[:ipv6_address] = interfaces.map { |i| i['ipv6_address'] }.join(", ")
+      row_attributes[:private_ips] = interfaces.map { |i| i["ipv4_address"] }.join(", ")
+      row_attributes[:ipv6_address] = interfaces.map { |i| i["ipv6_address"] }.join(", ")
 
-      row_attributes[:cloud_ip_ids] = cloud_ips.map { |i| i['id'] }.join(", ")
-      row_attributes[:cloud_ipv4s] = cloud_ips.map { |i| i['public_ipv4'] }.join(", ")
-      row_attributes[:cloud_ipv6s] = cloud_ips.map { |i| i['public_ipv6'] }.join(", ")
-      row_attributes[:snapshots] = snapshots.map { |i| i['id'] }.join(", ")
+      row_attributes[:cloud_ip_ids] = cloud_ips.map { |i| i["id"] }.join(", ")
+      row_attributes[:cloud_ipv4s] = cloud_ips.map { |i| i["public_ipv4"] }.join(", ")
+      row_attributes[:cloud_ipv6s] = cloud_ips.map { |i| i["public_ipv6"] }.join(", ")
+      row_attributes[:snapshots] = snapshots.map { |i| i["id"] }.join(", ")
 
       if server_groups
-        row_attributes[:server_groups] = server_groups.map { |sg| sg['id'] }.join(", ")
+        row_attributes[:server_groups] = server_groups.map { |sg| sg["id"] }.join(", ")
       end
 
       row_attributes
     end
 
     def self.default_field_order
-      [
-        :id,
-        :status,
-        :locked,
-        :name,
-        :created_at,
-        :deleted_at,
-        :zone,
-        :type,
-        :type_name,
-        :type_handle,
-        :ram,
-        :cores,
-        :disk,
-        :disk_encrypted,
-        :compatibility_mode,
-        :image,
-        :image_name,
-        :arch,
-        :private_ips,
-        :ipv6_address,
-        :hostname,
-        :fqdn,
-        :public_hostname,
-        :ipv6_hostname,
-        :cloud_ip_ids,
-        :cloud_ipv4s,
-        :cloud_ipv6s,
-        :snapshots,
-        :server_groups
+      %i[
+        id
+        status
+        locked
+        name
+        created_at
+        deleted_at
+        zone
+        type
+        type_name
+        type_handle
+        ram
+        cores
+        disk
+        disk_encrypted
+        compatibility_mode
+        image
+        image_name
+        arch
+        private_ips
+        ipv6_address
+        hostname
+        fqdn
+        public_hostname
+        ipv6_hostname
+        cloud_ip_ids
+        cloud_ipv4s
+        cloud_ipv6s
+        snapshots
+        server_groups
       ]
     end
   end

data/lib/brightbox-cli/detailed_server_group.rb

@@ -7,7 +7,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :name, :servers, :firewall_policy, :description]
+      %i[id name servers firewall_policy description]
     end
   end
 end

data/lib/brightbox-cli/error_parser.rb

@@ -16,26 +16,26 @@ module Brightbox
       end
     end
 
-    def parse_http_error(e)
-      if e.respond_to?(:response) && e.response.respond_to?(:body)
+    def parse_http_error(err)
+      if err.respond_to?(:response) && err.response.respond_to?(:body)
         begin
-          json_response = MultiJson.load(e.response.body)
-          extract_response_from_json(json_response, e)
-        rescue
+          json_response = MultiJson.load(err.response.body)
+          extract_response_from_json(json_response, err)
+        rescue StandardError
           {}
         end
       else
-        error "ERROR: #{e}"
+        error "ERROR: #{err}"
       end
     end
 
-    def extract_response_from_json(error_json, e)
-      json_error = error_json['errors'] || error_json['error']
+    def extract_response_from_json(error_json, err)
+      json_error = error_json["errors"] || error_json["error"]
       if json_error && !json_error.empty?
         error_string = Array(json_error).join(" ")
         error "ERROR: #{error_string}"
       else
-        error "ERROR: #{e}"
+        error "ERROR: #{err}"
       end
     end
   end

data/lib/brightbox-cli/firewall_policy.rb

@@ -1,6 +1,7 @@
 module Brightbox
   class FirewallPolicy < Api
     def self.require_account?; true; end
+
     def self.create(options)
       new(conn.firewall_policies.create(options))
     end
@@ -26,7 +27,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :server_group, :name]
+      %i[id server_group name]
     end
   end
 end

data/lib/brightbox-cli/firewall_rule.rb

@@ -29,8 +29,8 @@ module Brightbox
 
     def to_row
       attrs = attributes.dup
-      [:protocol, :source, :sport, :destination, :dport, :icmp_type].each do |key|
-        attrs[key] = attributes[key] || '-'
+      %i[protocol source sport destination dport icmp_type].each do |key|
+        attrs[key] = attributes[key] || "-"
       end
       attrs
     end
@@ -40,7 +40,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :protocol, :source, :sport, :destination, :dport, :icmp_type, :description]
+      %i[id protocol source sport destination dport icmp_type description]
     end
   end
 end

data/lib/brightbox-cli/firewall_rules.rb

@@ -3,19 +3,19 @@ module Brightbox
     def self.from_policy(firewall_policy)
       rules = firewall_policy.rules
       rules.map do |rule|
-        fwr = FirewallRule.new('fwr')
+        fwr = FirewallRule.new("fwr")
         fwr.attributes = symbolize_keys(rule)
         fwr
       end
     end
 
-    def self.symbolize_keys(object) #:nodoc:
+    def self.symbolize_keys(object) # :nodoc:
       modify_keys(object) do |key|
         key.is_a?(String) ? key.to_sym : key
       end
     end
 
-    def self.modify_keys(object, &modifier) #:nodoc:
+    def self.modify_keys(object, &modifier) # :nodoc:
       object.reduce({}) do |result, (key, value)|
         new_key = modifier.call(key)
         result.merge! new_key => value

data/lib/brightbox-cli/gli_global_hooks.rb

@@ -11,7 +11,7 @@ module Brightbox
   #
   # Need to locate the source of double loading under Aruba
   #
-  subcommand_files = Dir.glob(File.expand_path("../commands/**/*.rb", __FILE__))
+  subcommand_files = Dir.glob(File.expand_path("commands/**/*.rb", __dir__))
   subcommand_files.sort.each do |cmd_file|
     require cmd_file
   end
@@ -21,34 +21,34 @@ module Brightbox
 
   # Global options
   desc "Simple output (tab separated, don't draw fancy tables)"
-  switch [:s, :simple], :negatable => false
+  switch %i[s simple], :negatable => false
 
   desc "Set the api client to use"
-  flag [:c, :client]
+  flag %i[c client]
 
   desc "Set the account to use"
   flag :account
 
   desc "Disable peer SSL certificate verification"
-  switch [:k, :insecure], :negatable => false
+  switch %i[k insecure], :negatable => false
 
   pre do |global_options, command, _options, _args|
     # Configuration options
     config_opts = {
-      :client_name => ENV["CLIENT"] || global_options[:client],
-      :account => ENV["ACCOUNT"] || global_options[:account]
+      :client_name => ENV.fetch("CLIENT", global_options[:client]),
+      :account => ENV.fetch("ACCOUNT", global_options[:account])
     }
     Brightbox.config = BBConfig.new(config_opts)
 
     # Commands that alter the config files should not error here
-    unless [:config, :login].include?(command.topmost_ancestor.name)
+    unless %i[config login].include?(command.topmost_ancestor.name)
       raise AmbiguousClientError, AMBIGUOUS_CLIENT_ERROR if Brightbox.config.client_name.nil?
 
       if Brightbox.config.has_multiple_clients?
         if Brightbox.config.client_has_alias?
-          info "INFO: client_id: #{Brightbox.config.client_name} (#{Brightbox.config.client_alias})"
+          debug "client_id: #{Brightbox.config.client_name} (#{Brightbox.config.client_alias})"
         else
-          info "INFO: client_id: #{Brightbox.config.client_name}"
+          debug "client_id: #{Brightbox.config.client_name}"
         end
       end
 
@@ -56,11 +56,11 @@ module Brightbox
       Brightbox.config.debug_tokens if Brightbox.config.respond_to?(:debug_tokens)
     end
 
-    Excon.defaults[:headers]['User-Agent'] = "brightbox-cli/#{Brightbox::VERSION} Fog/#{Fog::Core::VERSION}"
+    Excon.defaults[:headers]["User-Agent"] = "brightbox-cli/#{Brightbox::VERSION} Fog/#{Fog::Core::VERSION}"
 
-    Excon.defaults[:headers]['User-Agent'] ||= "brightbox-cli/#{Brightbox::VERSION}"
+    Excon.defaults[:headers]["User-Agent"] ||= "brightbox-cli/#{Brightbox::VERSION}"
 
-    if global_options[:k] || ENV["INSECURE"]
+    if global_options[:k] || ENV.fetch("INSECURE", false)
       Excon.defaults[:ssl_verify_peer] = false
     end
 
@@ -73,20 +73,18 @@ module Brightbox
   end
 
   post do |_global_options, _command, _options, _args|
-    begin
-      # Api.conn is another global which holds the authentication tokens so
-      # we need to shuffle data between globals at a higher level rather than
-      # force one inside the other
-      access_token = Api.conn.access_token
-      refresh_token = Api.conn.refresh_token
-
-      Brightbox.config.update_stored_tokens(access_token, refresh_token)
-      Brightbox.config.save
-    rescue BBConfigError
-    rescue StandardError => e
-      # FIXME: Other StandardErrors are available
-      warn "Error writing auth token #{Brightbox.config.access_token_filename}: #{e.class}: #{e}"
-    end
+    # Api.conn is another global which holds the authentication tokens so
+    # we need to shuffle data between globals at a higher level rather than
+    # force one inside the other
+    access_token = Api.conn.access_token
+    refresh_token = Api.conn.refresh_token
+
+    Brightbox.config.update_stored_tokens(access_token, refresh_token)
+    Brightbox.config.save
+  rescue BBConfigError
+  rescue StandardError => e
+    # FIXME: Other StandardErrors are available
+    warn "Error writing auth token #{Brightbox.config.access_token_filename}: #{e.class}: #{e}"
   end
 
   on_error do |e|
@@ -103,7 +101,7 @@ module Brightbox
         error "Unable to authenticate with supplied details"
         Brightbox.config.debug_tokens if Brightbox.config.respond_to?(:debug_tokens)
         exit(111)
-      rescue Exception => e
+      rescue StandardError => e
         if ENV["DEBUG"]
           debug e
           debug e.class.to_s

data/lib/brightbox-cli/images.rb

@@ -1,6 +1,7 @@
 module Brightbox
   class Image < Api
     def self.require_account?; true; end
+
     def self.all
       conn.images
     end
@@ -11,40 +12,40 @@ module Brightbox
 
     def self.register(options = {})
       image = conn.create_image(options)
-      find image['id']
+      find image["id"]
     end
 
     def self.default_field_order
-      [:id, :owner, :type, :created_on, :status, :size, :name]
+      %i[id owner type created_on status size name]
     end
 
     # Filter out images that are not of the right type, account or status if the option is passed
     def self.filter_images(images, options = {})
       # Remove images that don't match the given type
       if options[:t]
-        images.reject! { |i| i.type != options[:t] }
+        images.select! { |i| i.type == options[:t] }
       end
 
       # Remove statuses that don't match the argument
       if options[:s]
-        images.reject! { |i| i.status != options[:s] }
+        images.select! { |i| i.status == options[:s] }
       end
 
       # Remove images that don't belong to the specified owner id
       if options[:l]
-        if options[:l] == 'brightbox'
-          images.reject! { |i| !i.official }
+        if options[:l] == "brightbox"
+          images.select!(&:official)
         else
-          images.reject! { |i| i.owner_id != options[:l] }
+          images.select! { |i| i.owner_id == options[:l] }
         end
       end
 
       unless options[:a]
         account = Account.conn.get_scoped_account(:nested => false)
-        images.reject! { |i| !i.official && i.owner_id != account["id"]  }
+        images.reject! { |i| !i.official && i.owner_id != account["id"] }
       end
 
-      snapshots = images.select { |i| i.source_type == 'snapshot' }
+      snapshots = images.select { |i| i.source_type == "snapshot" }
       images -= snapshots
 
       images.sort! { |a, b| a.default_sort_fields <=> b.default_sort_fields }
@@ -68,7 +69,7 @@ module Brightbox
 
     def status
       if fog_model.attributes[:status] == "available"
-        public? ? 'public' : 'private'
+        public? ? "public" : "private"
       else
         fog_model.attributes[:status]
       end
@@ -90,6 +91,7 @@ module Brightbox
       o[:description] = description if description
       o[:licence_name] = licence_name
       o[:size] = virtual_size
+      o[:min_ram] = min_ram
       o
     end
 
@@ -99,9 +101,9 @@ module Brightbox
 
     def status_sort_code
       case fog_model.attributes[:status]
-      when 'available'
+      when "available"
         (public? ? 1 : 2)
-      when 'deprecated'
+      when "deprecated"
         3
       else
         4

data/lib/brightbox-cli/load_balancers.rb

@@ -1,6 +1,7 @@
 module Brightbox
   class LoadBalancer < Api
     def self.require_account?; true; end
+
     def self.create(options)
       new(conn.load_balancers.create(options))
     end
@@ -38,11 +39,7 @@ module Brightbox
     end
 
     def listeners
-      if attributes[:listeners]
-        attributes[:listeners].map { |l| [l["in"], l["out"], l["protocol"], l['timeout']].join(":") }
-      else
-        nil
-      end
+      attributes[:listeners]&.map { |l| [l["in"], l["out"], l["protocol"], l["timeout"]].join(":") }
     end
 
     def buffer_size
@@ -80,7 +77,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :status, :created_on, :cloud_ips, :nodes, :name]
+      %i[id status created_on cloud_ips nodes name]
     end
   end
 end

data/lib/brightbox-cli/logging.rb

@@ -6,33 +6,33 @@ module Brightbox
     end
 
     module MixinMethods
-      def error(s = "")
-        $stderr.write s
+      def error(log_msg = "")
+        $stderr.write log_msg
         $stderr.write "\n"
         $stderr.flush
       end
 
-      def info(s = "")
-        $stderr.write s
+      def info(log_msg = "")
+        $stderr.write log_msg
         $stderr.write "\n"
         $stderr.flush
       end
 
-      def warn(s = "")
-        info "WARN: #{s}"
+      def warn(log_msg = "")
+        info "WARN: #{log_msg}"
       end
 
-      def data(s)
-        $stdout.write s
+      def data(log_msg)
+        $stdout.write log_msg
         $stdout.write "\n"
       end
 
-      def debug(s)
-        if ENV['DEBUG']
-          $stderr.write "DEBUG: "
-          $stderr.write s
-          $stderr.write "\n"
-        end
+      def debug(log_msg)
+        return unless ENV["DEBUG"]
+
+        $stderr.write "DEBUG: "
+        $stderr.write log_msg
+        $stderr.write "\n"
       end
     end
   end

data/lib/brightbox-cli/nilable_hash.rb

@@ -1,7 +1,7 @@
 class NilableHash < Hash
   def nilify_blanks
     keys.each do |k|
-      self[k] = nil if self[k] == ''
+      self[k] = nil if self[k] == ""
     end
   end
 end

data/lib/brightbox-cli/ruby_core_ext.rb

@@ -7,11 +7,11 @@ class Time
   #
   # @return [String] A clipped form of date, ISO 8601 without seconds
   #
-  def clipped_iso_8601
-    strftime(self.utc? ? "%Y-%m-%dT%H:%MZ" : "%Y-%m-%dT%H:%M")
+  def clipped_iso8601
+    strftime(utc? ? "%Y-%m-%dT%H:%MZ" : "%Y-%m-%dT%H:%M")
   end
 
   def to_s
-    clipped_iso_8601
+    clipped_iso8601
   end
 end

data/lib/brightbox-cli/server_groups.rb

@@ -15,7 +15,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :server_count, :name]
+      %i[id server_count name]
     end
 
     def firewall_policy
@@ -46,7 +46,7 @@ module Brightbox
     end
 
     def server_ids
-      attributes[:server_ids].map { |id| id.is_a?(Hash) ? id['id'] : id }
+      attributes[:server_ids].map { |id| id.is_a?(Hash) ? id["id"] : id }
     end
 
     def server_count

data/lib/brightbox-cli/servers.rb

@@ -19,7 +19,7 @@ module Brightbox
     end
 
     def self.default_field_order
-      [:id, :status, :type, :zone, :created_on, :image_id, :cloud_ip_ids, :name]
+      %i[id status type zone created_on image_id cloud_ip_ids name]
     end
 
     def update(options)
@@ -41,20 +41,20 @@ module Brightbox
       a[:image] = image_id
       a[:created_at] = created_at
       a[:created_on] = created_on
-      a[:type] = server_type['handle']
+      a[:type] = server_type["handle"]
       a[:status] = fog_model.state
       a[:locked] = locked?
-      a[:zone] = zone && zone['handle']
+      a[:zone] = zone && zone["handle"]
       a[:hostname] = id
       a[:public_hostname] = "public.#{fqdn}" unless cloud_ips.empty?
-      a[:ipv6_hostname] = ipv6_fqdn if interfaces.any? { |i| i['ipv6_address'] }
+      a[:ipv6_hostname] = ipv6_fqdn if interfaces.any? { |i| i["ipv6_address"] }
       a
     end
 
     def to_row
       o = attributes
-      o[:cloud_ip_ids] = cloud_ips.map { |i| i['id'] }
-      o[:ips] = interfaces.map { |i| i['ipv4_address'] }.join(', ')
+      o[:cloud_ip_ids] = cloud_ips.map { |i| i["id"] }
+      o[:ips] = interfaces.map { |i| i["ipv4_address"] }.join(", ")
       o
     end
 

data/lib/brightbox-cli/tables.rb

@@ -1,4 +1,4 @@
-require 'hirb'
+require "hirb"
 
 module Brightbox
   # Hack to set ascii art table cell width due to limitations in Hirb
@@ -10,9 +10,9 @@ module Brightbox
   # Remove most of the ascii art table output
   class SimpleTable < Hirb::Helpers::Table
     def render_table_header
-      title_row = ' ' + @fields.map do |f|
+      title_row = " " + @fields.map do |f|
         format_cell(@headers[f], @field_lengths[f])
-      end.join('  ')
+      end.join("  ")
       ["", title_row, render_border]
     end
 
@@ -21,14 +21,14 @@ module Brightbox
     end
 
     def render_border
-      '-' + @fields.map { |f| '-' * @field_lengths[f] }.join('--') + '-'
+      "-" + @fields.map { |f| "-" * @field_lengths[f] }.join("--") + "-"
     end
 
     def render_rows
       @rows.map do |row|
-        row = ' ' + @fields.map do |f|
+        row = " " + @fields.map do |f|
           format_cell(row[f], @field_lengths[f])
-        end.join('  ')
+        end.join("  ")
       end
     end
 
@@ -56,7 +56,7 @@ module Brightbox
     def render_footer; []; end
 
     def render_rows
-      longest_header = Hirb::String.size @headers.values.sort_by { |e| Hirb::String.size(e) }.last
+      longest_header = Hirb::String.size(@headers.values.max_by { |e| Hirb::String.size(e) })
       @rows.map do |row|
         fields = @fields.map do |f|
           "#{Hirb::String.rjust(@headers[f], longest_header)}: #{row[f]}"
@@ -87,7 +87,7 @@ module Brightbox
     rows.each do |row|
       # FIXME: default Api subclasses do not respond to #keys so specialising
       #   #to_row is required to not break the following
-      row.keys.each do |k|
+      row.each_key do |k|
         row[k] = row[k].render_cell if row[k].respond_to? :render_cell
       end
     end
@@ -97,16 +97,14 @@ module Brightbox
         if options[:vertical]
           data options[:fields].map { |k| [k, row[k]].join("\t") }.join("\n")
         else
-          data options[:fields].map { |k| row[k].is_a?(Array) ? row[k].join(',') : row[k] }.join("\t")
+          data options[:fields].map { |k| row[k].is_a?(Array) ? row[k].join(",") : row[k] }.join("\t")
         end
       end
-    else
+    elsif options[:vertical]
       # "graphical" table
-      if options[:vertical]
-        data ShowTable.render(rows, options)
-      else
-        data SimpleTable.render(rows, options)
-      end
+      data ShowTable.render(rows, options)
+    else
+      data SimpleTable.render(rows, options)
     end
   end