removed lazy-requiring of rest-client and heroku-api

lib/heroku/auth.rb

@@ -13,7 +13,6 @@ class << self
 
     def api
       @api ||= begin
-        require("heroku-api")
         api = Heroku::API.new(default_params.merge(:api_key => password))
 
         def api.request(params, &block)
@@ -75,7 +74,6 @@ def password    # :nodoc:
     end
 
     def api_key(user = get_credentials[0], password = get_credentials[1])
-      require("heroku-api")
       api = Heroku::API.new(default_params)
       api.post_login(user, password).body["api_key"]
     rescue Heroku::API::Errors::Unauthorized => e
@@ -240,22 +238,19 @@ def ask_for_password
     end
 
     def ask_for_and_save_credentials
-      require("heroku-api") # for the errors
-      begin
-        @credentials = ask_for_credentials
-        write_credentials
-        check
-      rescue Heroku::API::Errors::NotFound, Heroku::API::Errors::Unauthorized => e
-        delete_credentials
-        display "Authentication failed."
-        retry if retry_login?
-        exit 1
-      rescue Exception => e
-        delete_credentials
-        raise e
-      end
+      @credentials = ask_for_credentials
+      write_credentials
+      check
       check_for_associated_ssh_key unless Heroku::Command.current_command == "keys:add"
       @credentials
+    rescue Heroku::API::Errors::NotFound, Heroku::API::Errors::Unauthorized => e
+      delete_credentials
+      display "Authentication failed."
+      retry if retry_login?
+      exit 1
+    rescue Exception => e
+      delete_credentials
+      raise e
     end
 
     def check_for_associated_ssh_key

lib/heroku/cli.rb

@@ -4,12 +4,8 @@
 require "heroku"
 require "heroku/command"
 require "heroku/helpers"
-
-# workaround for rescue/reraise to define errors in command.rb failing in 1.8.6
-if RUBY_VERSION =~ /^1.8.6/
-  require('heroku-api')
-  require('rest_client')
-end
+require 'rest_client'
+require 'heroku-api'
 
 begin
   # attempt to load the JSON parser bundled with ruby for multi_json

lib/heroku/client.rb

@@ -32,7 +32,6 @@ def self.gem_version_string
   attr_accessor :host, :user, :password
 
   def initialize(user, password, host=Heroku::Auth.host)
-    require 'rest_client'
     @user = user
     @password = password
     @host = host
@@ -349,7 +348,6 @@ class Service
     attr_accessor :attached
 
     def initialize(client, app)
-      require 'rest_client'
       @client = client
       @app = app
     end
@@ -435,7 +433,6 @@ class AppCrashed < RuntimeError; end
   # support for console sessions
   class ConsoleSession
     def initialize(id, app, client)
-      require 'rest_client'
       @id = id; @app = app; @client = client
     end
     def run(cmd)

lib/heroku/client/heroku_postgresql.rb

@@ -18,7 +18,6 @@ def self.headers
   attr_reader :attachment
   def initialize(attachment)
     @attachment = attachment
-    require 'rest_client'
   end
 
   def heroku_postgresql_host

lib/heroku/client/organizations.rb

@@ -1,4 +1,3 @@
-require 'heroku-api'
 require "heroku/client"
 
 class Heroku::Client::Organizations

lib/heroku/client/pgbackups.rb

@@ -5,7 +5,6 @@ class Heroku::Client::Pgbackups
   include Heroku::Helpers
 
   def initialize(uri)
-    require 'rest_client'
     @uri = URI.parse(uri)
   end
 

lib/heroku/command.rb

@@ -213,15 +213,8 @@ def self.prepare_run(cmd, args=[])
     end
 
     def self.run(cmd, arguments=[])
-      begin
-        object, method = prepare_run(cmd, arguments.dup)
-        object.send(method)
-      rescue Interrupt, StandardError, SystemExit => error
-        # load likely error classes, as they may not be loaded yet due to defered loads
-        require 'heroku-api'
-        require 'rest_client'
-        raise(error)
-      end
+      object, method = prepare_run(cmd, arguments.dup)
+      object.send(method)
     rescue Heroku::API::Errors::Unauthorized, RestClient::Unauthorized => e
       retry_login = handle_auth_error(e)
       retry if retry_login

spec/spec_helper.rb

@@ -7,11 +7,6 @@
 
 require "excon"
 
-# ensure these are around for errors
-# as their require is generally deferred
-require "heroku-api"
-require "rest_client"
-
 require "heroku/cli"
 require "rspec"
 require "rr"