brightbox-cli 3.2.0 → 4.0.0

data/lib/brightbox-cli/commands/sql/instances_destroy.rb

@@ -1,12 +1,10 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:instances] do |cmd|
-
       cmd.desc I18n.t("sql.instances.destroy.desc")
       cmd.arg_name "[instance-id...]"
       cmd.command [:destroy] do |c|
         c.action do |_global_options, _options, args|
-
           raise "You must specify instance IDs to destroy" if args.empty?
 
           servers = DatabaseServer.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/sql/instances_list.rb

@@ -2,22 +2,20 @@ module Brightbox
   command [:sql] do |product|
     product.default_command :instances
     product.command [:instances] do |cmd|
-
       cmd.default_command :list
 
       cmd.desc I18n.t("sql.instances.list.desc")
 
       cmd.arg_name "[instance-id...]"
       cmd.command [:list] do |c|
-
         c.action do |global_options, _options, args|
-          if args.empty?
-            servers = DatabaseServer.find(:all)
-          else
-            servers = DatabaseServer.find_or_call(args) do |id|
-              warn "Couldn't find server #{id}"
-            end
-          end
+          servers = if args.empty?
+                      DatabaseServer.find(:all)
+                    else
+                      DatabaseServer.find_or_call(args) do |id|
+                        warn "Couldn't find server #{id}"
+                      end
+                    end
 
           render_table(servers, global_options)
         end

data/lib/brightbox-cli/commands/sql/instances_reset_password.rb

@@ -1,7 +1,6 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:instances] do |cmd|
-
       cmd.desc I18n.t("sql.instances.reset_password.desc")
       cmd.arg_name "server-id"
       cmd.command [:"reset-password"] do |c|

data/lib/brightbox-cli/commands/sql/instances_show.rb

@@ -1,13 +1,10 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:instances] do |cmd|
-
       cmd.desc I18n.t("sql.instances.show.desc")
       cmd.arg_name "instance-id..."
       cmd.command [:show] do |c|
-
         c.action do |global_options, _options, args|
-
           raise "You must specify instance IDs to show" if args.empty?
 
           servers = DatabaseServer.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/sql/instances_snapshot.rb

@@ -1,13 +1,10 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:instances] do |cmd|
-
       cmd.desc I18n.t("sql.instances.snapshot.desc")
       cmd.arg_name "[instance-id...]"
       cmd.command [:snapshot] do |c|
-
         c.action do |_global_options, _options, args|
-
           raise "You must specify SQL instance IDs to snapshot" if args.empty?
 
           servers = DatabaseServer.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/sql/instances_update.rb

@@ -1,13 +1,11 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:instances] do |cmd|
-
       cmd.desc I18n.t("sql.instances.update.desc")
       cmd.arg_name "instance-id"
       cmd.command [:update] do |c|
-
         c.desc I18n.t("options.name.desc")
-        c.flag [:n, :name]
+        c.flag %i[n name]
 
         c.desc I18n.t("options.description.desc")
         c.flag [:d, "description"]

data/lib/brightbox-cli/commands/sql/snapshots_destroy.rb

@@ -1,15 +1,12 @@
 module Brightbox
   command [:sql] do |product|
-
     product.desc I18n.t("sql.snapshots.desc")
     product.command [:snapshots] do |cmd|
-
       cmd.desc I18n.t("sql.snapshots.destroy.desc")
       cmd.arg_name "[snapshot-id...]"
 
       cmd.command [:destroy] do |c|
         c.action do |_global_options, _options, args|
-
           raise "You must specify snapshots to destroy" if args.empty?
 
           snapshots = DatabaseSnapshot.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/sql/snapshots_list.rb

@@ -1,21 +1,19 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:snapshots] do |cmd|
-
       cmd.default_command :list
 
       cmd.desc I18n.t("sql.snapshots.list.desc")
       cmd.arg_name "[snapshot-id...]"
       cmd.command [:list] do |c|
-
         c.action do |global_options, _options, args|
-          if args.empty?
-            snapshots = DatabaseSnapshot.find(:all)
-          else
-            snapshots = DatabaseSnapshot.find_or_call(args) do |id|
-              warn "Couldn't find snapshot #{id}"
-            end
-          end
+          snapshots = if args.empty?
+                        DatabaseSnapshot.find(:all)
+                      else
+                        DatabaseSnapshot.find_or_call(args) do |id|
+                          warn "Couldn't find snapshot #{id}"
+                        end
+                      end
 
           render_table(snapshots, global_options)
         end

data/lib/brightbox-cli/commands/sql/snapshots_show.rb

@@ -1,13 +1,10 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:snapshots] do |cmd|
-
       cmd.desc I18n.t("sql.snapshots.show.desc")
       cmd.arg_name "[snapshot-id...]"
       cmd.command [:show] do |c|
-
         c.action do |global_options, _options, args|
-
           raise "You must specify snapshot ids to show" if args.empty?
 
           servers = DatabaseSnapshot.find_or_call(args) do |id|
@@ -16,13 +13,18 @@ module Brightbox
 
           table_opts = global_options.merge(
             :vertical => true,
-            :fields => [
-              :id,
-              :name,
-              :description,
-              :status,
-              :locked,
-              :created_on
+            :fields => %i[
+              id
+              name
+              description
+              status
+              database_engine
+              database_version
+              locked
+              created_on
+              created_at
+              size
+              source
             ]
           )
           render_table(servers, table_opts)

data/lib/brightbox-cli/commands/sql/snapshots_update.rb

@@ -1,13 +1,11 @@
 module Brightbox
   command [:sql] do |product|
     product.command [:snapshots] do |cmd|
-
       cmd.desc I18n.t("sql.snapshots.update.desc")
       cmd.arg_name "snapshot-id"
       cmd.command [:update] do |c|
-
         c.desc I18n.t("options.name.desc")
-        c.flag [:n, :name]
+        c.flag %i[n name]
 
         c.desc I18n.t("options.description.desc")
         c.flag [:d, "description"]

data/lib/brightbox-cli/commands/sql/types.rb

@@ -1,24 +1,20 @@
 module Brightbox
   command [:sql] do |product|
-
     product.desc I18n.t("sql.types.desc")
     product.command [:types] do |cmd|
-
       cmd.default_command :list
 
       cmd.desc I18n.t("sql.types.list.desc")
       cmd.arg_name "[type-id...]"
       cmd.command [:list] do |c|
-
         c.action do |global_options, _options, args|
-
-          if args.empty?
-            types = DatabaseType.find :all
-          else
-            types = DatabaseType.find_or_call(args) do |id|
-              warn "Couldn't find an SQL type with ID #{id}"
-            end
-          end
+          types = if args.empty?
+                    DatabaseType.find :all
+                  else
+                    DatabaseType.find_or_call(args) do |id|
+                      warn "Couldn't find an SQL type with ID #{id}"
+                    end
+                  end
 
           render_table(types.sort, global_options)
         end
@@ -27,9 +23,7 @@ module Brightbox
       cmd.desc I18n.t("sql.types.show.desc")
       cmd.arg_name "[type-id...]"
       cmd.command [:show] do |c|
-
         c.action do |global_options, _options, args|
-
           if args.empty?
             raise "You must specify the types you want to show"
           end
@@ -40,12 +34,11 @@ module Brightbox
 
           display_options = {
             :vertical => true,
-            :fields => [:id, :name, :description, :ram, :disk]
+            :fields => %i[id name description ram disk]
           }
 
           table_opts = global_options.merge(display_options)
           render_table(types, table_opts)
-
         end
       end
     end

data/lib/brightbox-cli/commands/token.rb

@@ -29,6 +29,5 @@ module Brightbox
         $stdout.puts token.format(options[:format])
       end
     end
-
   end
 end

data/lib/brightbox-cli/commands/types.rb

@@ -1,22 +1,19 @@
 module Brightbox
   desc "Lists the type of templates available for servers"
   command [:types] do |cmd|
-
     cmd.default_command :list
 
     cmd.desc "List types"
     cmd.arg_name "[type-id...]"
     cmd.command [:list] do |c|
-
       c.action do |global_options, _options, args|
-
-        if args.empty?
-          types = Type.find :all
-        else
-          types = Type.find_or_call(args) do |id|
-            warn "Couldn't find type #{id}"
-          end
-        end
+        types = if args.empty?
+                  Type.find :all
+                else
+                  Type.find_or_call(args) do |id|
+                    warn "Couldn't find type #{id}"
+                  end
+                end
 
         render_table(types.sort, global_options)
       end
@@ -25,9 +22,7 @@ module Brightbox
     cmd.desc "Show detailed type info"
     cmd.arg_name "type-id..."
     cmd.command [:show] do |c|
-
       c.action do |global_options, _options, args|
-
         if args.empty?
           raise "You must specify the types you want to show"
         end
@@ -38,12 +33,11 @@ module Brightbox
 
         display_options = {
           :vertical => true,
-          :fields => [:id, :handle, :status, :name, :ram, :disk, :cores]
+          :fields => %i[id handle status name ram disk cores]
         }
 
         table_opts = global_options.merge(display_options)
         render_table(types, table_opts)
-
       end
     end
   end

data/lib/brightbox-cli/commands/{user-collaborations.rb → user_collaborations.rb}

@@ -1,20 +1,19 @@
 module Brightbox
   desc "Lists accounts you can access (when using user credentials)"
   command [:"account-access"] do |cmd|
-
     cmd.default_command :list
 
     cmd.desc "List your collaborations"
     cmd.arg_name "[collaboration-id...]"
     cmd.command [:list] do |c|
       c.action do |global_options, _options, args|
-        if args.empty?
-          collaborations = UserCollaboration.find(:all)
-        else
-          collaborations = UserCollaboration.find_or_call(args) do |id|
-            warn "Couldn't find collaboration #{id}"
-          end
-        end
+        collaborations = if args.empty?
+                           UserCollaboration.find(:all)
+                         else
+                           UserCollaboration.find_or_call(args) do |id|
+                             warn "Couldn't find collaboration #{id}"
+                           end
+                         end
         render_table(collaborations, global_options)
       end
     end

data/lib/brightbox-cli/commands/users/list.rb

@@ -1,13 +1,11 @@
 module Brightbox
   desc I18n.t("users.desc")
   command [:users] do |cmd|
-
     cmd.default_command :list
 
     cmd.desc I18n.t("users.list.desc")
     cmd.arg_name "[user-id...]"
     cmd.command [:list] do |c|
-
       c.action do |global_options, _options, args|
         users = User.find_all_or_warn(args)
         render_table(users, global_options)

data/lib/brightbox-cli/commands/users/show.rb

@@ -1,16 +1,14 @@
 module Brightbox
   command [:users] do |cmd|
-
     cmd.desc I18n.t("users.show.desc")
     cmd.arg_name "user-id..."
     cmd.command [:show] do |c|
-
       c.action do |global_options, _options, args|
         users = User.find_all_or_warn(args)
 
         table_opts = global_options.merge(
           :vertical => true,
-          :fields => [:id, :name, :email_address, :accounts, :ssh_key]
+          :fields => %i[id name email_address accounts ssh_key]
         )
 
         render_table(users, table_opts)

data/lib/brightbox-cli/commands/users/update.rb

@@ -1,26 +1,23 @@
 module Brightbox
   command [:users] do |cmd|
-
     cmd.desc I18n.t("users.update.desc")
     cmd.arg_name "user-id..."
     cmd.command [:update] do |c|
-
       c.desc "Path to public ssh key file"
       c.long_desc "This is the path to the public ssh key that you'd like to use
       for new servers. You can specify '-' to read from stdin"
       c.flag [:f, "ssh-key"]
 
       c.desc I18n.t("options.name.desc")
-      c.flag [:n, :name]
+      c.flag %i[n name]
 
       c.action do |global_options, options, args|
-
         raise "You must specify the user id as the first argument" if args.empty?
 
         user = User.find args.first
 
         if options[:f] == "-"
-          user.ssh_key = STDIN.read
+          user.ssh_key = $stdin.read
         elsif options[:f]
           File.open(File.expand_path(options[:f])) { |f| user.ssh_key = f.read }
         end
@@ -31,7 +28,7 @@ module Brightbox
 
         table_opts = global_options.merge(
           :vertical => true,
-          :fields => [:id, :name, :email_address, :ssh_key]
+          :fields => %i[id name email_address ssh_key]
         )
 
         render_table([user], table_opts)

data/lib/brightbox-cli/commands/zones.rb

@@ -1,22 +1,19 @@
 module Brightbox
   desc "Lists the zones available to an account or see more details"
   command [:zones] do |cmd|
-
     cmd.default_command :list
 
     cmd.desc "List zones"
     cmd.arg_name "[zone-id...]"
     cmd.command [:list] do |c|
-
       c.action do |global_options, _options, args|
-
-        if args.empty?
-          zones = Zone.find(:all)
-        else
-          zones = Zone.find_or_call(args) do |id|
-            warn "Couldn't find zone #{id}"
-          end
-        end
+        zones = if args.empty?
+                  Zone.find(:all)
+                else
+                  Zone.find_or_call(args) do |id|
+                    warn "Couldn't find zone #{id}"
+                  end
+                end
 
         render_table(zones, global_options)
       end

data/lib/brightbox-cli/config/accounts.rb

@@ -3,23 +3,15 @@ module Brightbox
     module Accounts
       def save_default_account(account_id)
         dirty! unless account_id == selected_config["default_account"]
-        selected_config['default_account'] = account_id
+
+        selected_config["default_account"] = account_id
       end
 
-      #
-      #
-      #
       def default_account
-        if selected_config
-          configured_default_account = selected_config["default_account"]
-          if configured_default_account && !configured_default_account.empty?
-            configured_default_account
-          else
-            nil
-          end
-        else
-          nil
-        end
+        return unless selected_config
+
+        configured_default_account = selected_config["default_account"]
+        return configured_default_account if configured_default_account && !configured_default_account.empty?
       end
 
       def determine_account(preferred_account)
@@ -44,19 +36,19 @@ module Brightbox
       def find_or_set_default_account
         # FIXME: API clients are scoped to their account so this code should
         #   never need to run for them.
-        unless default_account
-          begin
-            service = Fog::Compute.new(to_fog)
-            accounts = service.accounts
+        return if default_account
+
+        begin
+          service = Fog::Compute.new(to_fog)
+          accounts = service.accounts
 
-            @account = accounts.select { |acc| %w(pending active).include?(acc.status) }.first.id
-            save_default_account(@account) if @account
-          rescue Brightbox::BBConfigError
-            # We can't get a suitable fog connection so we can't select an
-            # account
-          rescue Excon::Errors::Unauthorized
-            # This is a helper, if it fails let the other code warn and prompt
-          end
+          @account = accounts.select { |acc| %w[pending active].include?(acc.status) }.first.id
+          save_default_account(@account) if @account
+        rescue Brightbox::BBConfigError
+          # We can't get a suitable fog connection so we can't select an
+          # account
+        rescue Excon::Errors::Unauthorized
+          # This is a helper, if it fails let the other code warn and prompt
         end
       end
     end

data/lib/brightbox-cli/config/api_client.rb

@@ -1,7 +1,7 @@
 module Brightbox
   module Config
     class ApiClient
-      NON_BLANK_KEYS = %w(api_url client_id secret)
+      NON_BLANK_KEYS = %w[api_url client_id secret].freeze
 
       attr_accessor :selected_config, :client_name
 
@@ -13,11 +13,11 @@ module Brightbox
       def to_fog
         check_required_params
         {
-          :provider => 'Brightbox',
-          :brightbox_api_url => selected_config['api_url'],
-          :brightbox_auth_url => selected_config['auth_url'] || selected_config['api_url'],
-          :brightbox_client_id => selected_config['client_id'],
-          :brightbox_secret => selected_config['secret'],
+          :provider => "Brightbox",
+          :brightbox_api_url => selected_config["api_url"],
+          :brightbox_auth_url => selected_config["auth_url"] || selected_config["api_url"],
+          :brightbox_client_id => selected_config["client_id"],
+          :brightbox_secret => selected_config["secret"],
           :persistent => persistent?
         }
       end
@@ -31,19 +31,15 @@ module Brightbox
       private
 
       def persistent?
-        if selected_config["persistent"] == "false"
-          false
-        else
-          true
-        end
+        selected_config["persistent"] != "false"
       end
 
       def check_required_params
-        unless valid?
-          NON_BLANK_KEYS.each do |key|
-            unless selected_config.key?(key) && !selected_config[key].to_s.empty?
-              raise Brightbox::BBConfigError, "#{key} option missing from config in section #{client_name}"
-            end
+        return if valid?
+
+        NON_BLANK_KEYS.each do |key|
+          unless selected_config.key?(key) && !selected_config[key].to_s.empty?
+            raise Brightbox::BBConfigError, "#{key} option missing from config in section #{client_name}"
           end
         end
       end