linode 0.6.3 → 0.7.0

data/CHANGELOG

@@ -1,4 +1,11 @@
 
+0.7.0 / 2011-07-07
+==================
+  * Added support for Nodebalancer API calls
+  * Changed HTTP GETs to HTTP POSTs to work around errors with large stackscripts (credit: Dan Hodos)
+  * Updated documentation references in error messages to match API documentation on Linode website
+
+
 0.6.3 / 2010-11-23 
 ==================
   * Can now initialize w/ username/pass; also support for user.getapikey API call

data/README

@@ -140,4 +140,5 @@ Here is an annoyingly exhaustive IRB session where I play around with the API:
  irb> ^D@ Wed Aug 05 01:50:52 rick@Yer-Moms-Computer
 
 CREDITS:
+ - Thanks to Dan Hodos (github: danhodos) for diagnosing and fixing an issue with sending GET requests instead of POST request.
  - Thanks to Aaron Hamid for updates for RSpec 2 and work on user.getapikey + username/password initialization.

data/VERSION

@@ -1 +1 @@
-0.6.3
+0.7.0

data/lib/linode.rb

@@ -10,7 +10,7 @@ class Linode
       define_method(action.to_sym) do |*data|
         data = data.shift if data
         data ||= {}
-        send_request(self.class.name.downcase.sub(/^linode::/, '').gsub(/::/, '.') + ".#{action}", data)
+        send_request(Linode.action_path(self.class.name, action), data)
       end
     end
   end
@@ -27,7 +27,30 @@ class Linode
     end
   end
   
-  has_namespace :test, :avail, :user, :domain, :linode
+  has_namespace :test, :avail, :user, :domain, :linode, :nodebalancer
+  
+  @@documentation_category = {}
+  
+  def self.documentation_category(category)
+    @@documentation_category[class_to_path(name)] = category
+  end
+  
+  def documentation_categories
+    @@documentation_category
+  end
+  
+  def self.action_path(class_name, action)
+    Linode.class_to_path(class_name) + ".#{action}"
+  end
+  
+  def self.class_to_path(class_name)
+    class_name.downcase.sub(/^linode::/, '').gsub(/::/, '.')
+  end
+  
+  def documentation_path(action)
+     hits = action.match(/^(.*)\.[^.]+$/)
+    "http://www.linode.com/api/" + @@documentation_category[hits[1]] + '/' + action
+  end
   
   def initialize(args)
     @api_url = args[:api_url] if args[:api_url]
@@ -52,7 +75,7 @@ class Linode
   
   def send_request(action, data)
     data.delete_if {|k,v| [:api_key, :api_action, :api_responseFormat].include?(k) }
-    response = get({ :api_key => api_key, :api_action => action, :api_responseFormat => 'json' }.merge(data))
+    response = post({ :api_key => api_key, :api_action => action, :api_responseFormat => 'json' }.merge(data))
     raise "Errors completing request [#{action}] @ [#{api_url}] with data [#{data.inspect}]:\n#{error_message(response, action)}" if error?(response)
     reformat_response(response)
   end
@@ -60,13 +83,13 @@ class Linode
   protected
   
   def fetch_api_key
-    response = get(:api_action => 'user.getapikey', :api_responseFormat => 'json', :username => username, :password => password)
+    response = post(:api_action => 'user.getapikey', :api_responseFormat => 'json', :username => username, :password => password)
     raise "Errors completing request [user.getapikey] @ [#{api_url}] for username [#{username}]:\n#{error_message(response, 'user.getapikey')}" if error?(response)
     reformat_response(response).api_key
   end
   
-  def get(data)
-    Crack::JSON.parse(HTTParty.get(api_url, :query => data))
+  def post(data)
+    Crack::JSON.parse(HTTParty.post(api_url, :body => data))
   end
   
   def error?(response)
@@ -76,7 +99,7 @@ class Linode
   def error_message(response, action)
     response["ERRORARRAY"].collect { |err|
       "  - Error \##{err["ERRORCODE"]} - #{err["ERRORMESSAGE"]}.  "+
-      "(Please consult http://www.linode.com/api/autodoc.cfm?method=#{action})"
+      "(Please consult #{documentation_path(action)})"
     }.join("\n")
   end
   

data/lib/linode/avail.rb

@@ -1,3 +1,4 @@
 class Linode::Avail < Linode
+  documentation_category 'utility'
   has_method :datacenters, :kernels, :linodeplans, :distributions, :stackscripts
 end

data/lib/linode/domain.rb

@@ -1,4 +1,5 @@
 class Linode::Domain < Linode
+  documentation_category 'dns'
   has_namespace :resource
   has_method :update, :create, :list, :delete
 end

data/lib/linode/domain/resource.rb

@@ -1,3 +1,4 @@
 class Linode::Domain::Resource < Linode
+  documentation_category 'dns'
   has_method :create, :delete, :update, :list
 end

data/lib/linode/linode.rb

@@ -1,4 +1,5 @@
 class Linode::Linode < Linode
+  documentation_category "linode"
   has_namespace :config, :disk, :ip, :job
   has_method :update, :create, :list, :shutdown, :boot, :delete, :reboot
 end

data/lib/linode/linode/config.rb

@@ -1,3 +1,4 @@
 class Linode::Linode::Config < Linode
+  documentation_category 'linode'
   has_method :create, :delete, :update, :list
 end

data/lib/linode/linode/disk.rb

@@ -1,3 +1,4 @@
 class Linode::Linode::Disk < Linode
+  documentation_category 'linode'
   has_method :update, :create, :list, :createfromdistribution, :createfromstackscript, :duplicate, :delete, :resize
 end

data/lib/linode/linode/ip.rb

@@ -1,3 +1,4 @@
 class Linode::Linode::Ip < Linode
+  documentation_category 'linode'
   has_method :list
 end

data/lib/linode/linode/job.rb

@@ -1,3 +1,4 @@
 class Linode::Linode::Job < Linode
+  documentation_category 'linode'
   has_method :list
 end

data/lib/linode/nodebalancer.rb

@@ -0,0 +1,5 @@
+class Linode::Nodebalancer < Linode
+  documentation_category 'nodebalancer'
+  has_namespace :config, :node
+  has_method :update, :create, :list, :delete
+end

data/lib/linode/nodebalancer/config.rb

@@ -0,0 +1,4 @@
+class Linode::Nodebalancer::Config < Linode
+  documentation_category 'nodebalancer'
+  has_method :create, :delete, :update, :list
+end

data/lib/linode/nodebalancer/node.rb

@@ -0,0 +1,4 @@
+class Linode::Nodebalancer::Node < Linode
+  documentation_category 'nodebalancer'
+  has_method :create, :delete, :update, :list
+end

data/lib/linode/stackscript.rb

@@ -1,3 +1,4 @@
 class Linode::Stackscript < Linode
+  documentation_category 'stackscript'
   has_method :update, :create, :list, :delete
 end

data/lib/linode/test.rb

@@ -1,3 +1,4 @@
 class Linode::Test < Linode
+  documentation_category 'utility'
   has_method :echo
 end

data/lib/linode/user.rb

@@ -1,4 +1,6 @@
 class Linode::User < Linode
+  documentation_category 'utility'
+  
   def getapikey
     api_key
   end

data/linode.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{linode}
-  s.version = "0.6.3"
+  s.version = "0.7.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Rick Bradley"]
-  s.date = %q{2010-11-23}
+  s.date = %q{2011-07-07}
   s.description = %q{This is a wrapper around Linode's automation facilities.}
   s.email = %q{rick@rickbradley.com}
   s.extra_rdoc_files = [
@@ -32,6 +32,9 @@ Gem::Specification.new do |s|
     "lib/linode/linode/disk.rb",
     "lib/linode/linode/ip.rb",
     "lib/linode/linode/job.rb",
+    "lib/linode/nodebalancer.rb",
+    "lib/linode/nodebalancer/config.rb",
+    "lib/linode/nodebalancer/node.rb",
     "lib/linode/stackscript.rb",
     "lib/linode/test.rb",
     "lib/linode/user.rb",
@@ -44,6 +47,9 @@ Gem::Specification.new do |s|
     "spec/linode/linode/ip_spec.rb",
     "spec/linode/linode/job_spec.rb",
     "spec/linode/linode_spec.rb",
+    "spec/linode/nodebalancer/config_spec.rb",
+    "spec/linode/nodebalancer/node_spec.rb",
+    "spec/linode/nodebalancer_spec.rb",
     "spec/linode/stackscript_spec.rb",
     "spec/linode/test_spec.rb",
     "spec/linode/user_spec.rb",
@@ -63,6 +69,9 @@ Gem::Specification.new do |s|
     "spec/linode/linode/ip_spec.rb",
     "spec/linode/linode/job_spec.rb",
     "spec/linode/linode_spec.rb",
+    "spec/linode/nodebalancer/config_spec.rb",
+    "spec/linode/nodebalancer/node_spec.rb",
+    "spec/linode/nodebalancer_spec.rb",
     "spec/linode/stackscript_spec.rb",
     "spec/linode/test_spec.rb",
     "spec/linode/user_spec.rb",

data/spec/linode/avail_spec.rb

@@ -39,7 +39,10 @@ describe Linode::Avail do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/utility/avail.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/utility/avail.#{action}"
+      end
     end
   end
-  
 end

data/spec/linode/domain/resource_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Domain::Resource do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/dns/domain.resource.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/dns/domain.resource.#{action}"
+      end
     end
   end
 end

data/spec/linode/domain_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Domain do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/dns/domain.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/dns/domain.#{action}"
+      end
     end
   end
   

data/spec/linode/linode/config_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Linode::Config do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.config.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/linode/linode.config.#{action}"
+      end
     end
   end
 end

data/spec/linode/linode/disk_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Linode::Disk do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.disk.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/linode/linode.disk.#{action}"
+      end
     end
   end
 end

data/spec/linode/linode/ip_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Linode::Ip do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.ip.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/linode/linode.ip.#{action}"
+      end
     end
   end
 end

data/spec/linode/linode/job_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Linode::Job do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.job.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/linode/linode.job.#{action}"
+      end
     end
   end
 end

data/spec/linode/linode_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Linode do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/linode/linode.#{action}"
+      end
     end
   end
   

data/spec/linode/nodebalancer/config_spec.rb

@@ -0,0 +1,48 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require 'linode'
+
+describe Linode::Nodebalancer::Config do
+  before :each do
+    @api_key = 'foo'
+    @linode = Linode::Nodebalancer::Config.new(:api_key => @api_key)
+  end
+  
+  it 'should be a Linode instance' do
+    @linode.class.should < Linode
+  end
+  
+  %w(create delete update list).each do |action|
+    it "should allow accessing the #{action} API" do
+      @linode.should respond_to(action.to_sym)
+    end
+  
+    describe "when accessing the #{action} API" do
+      it 'should allow a data hash' do
+        lambda { @linode.send(action.to_sym, {}) }.should_not raise_error(ArgumentError)
+      end
+    
+      it 'should not require arguments' do
+        lambda { @linode.send(action.to_sym) }.should_not raise_error(ArgumentError)      
+      end
+    
+      it "should request the nodebalancer.config.#{action} action" do
+        @linode.expects(:send_request).with {|api_action, data| api_action == "nodebalancer.config.#{action}" }
+        @linode.send(action.to_sym)
+      end
+    
+      it 'should provide the data hash when making its request' do
+        @linode.expects(:send_request).with {|api_action, data| data = { :foo => :bar } }
+        @linode.send(action.to_sym, {:foo => :bar})
+      end
+    
+      it 'should return the result of the request' do
+        @linode.expects(:send_request).returns(:bar => :baz)      
+        @linode.send(action.to_sym).should == { :bar => :baz }      
+      end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/nodebalancer/nodebalancer.config.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/nodebalancer/nodebalancer.config.#{action}"
+      end
+    end
+  end
+end

data/spec/linode/nodebalancer/node_spec.rb

@@ -0,0 +1,48 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require 'linode'
+
+describe Linode::Nodebalancer::Node do
+  before :each do
+    @api_key = 'foo'
+    @linode = Linode::Nodebalancer::Node.new(:api_key => @api_key)
+  end
+  
+  it 'should be a Linode instance' do
+    @linode.class.should < Linode
+  end
+  
+  %w(create delete update list).each do |action|
+    it "should allow accessing the #{action} API" do
+      @linode.should respond_to(action.to_sym)
+    end
+  
+    describe "when accessing the #{action} API" do
+      it 'should allow a data hash' do
+        lambda { @linode.send(action.to_sym, {}) }.should_not raise_error(ArgumentError)
+      end
+    
+      it 'should not require arguments' do
+        lambda { @linode.send(action.to_sym) }.should_not raise_error(ArgumentError)      
+      end
+    
+      it "should request the nodebalancer.node.#{action} action" do
+        @linode.expects(:send_request).with {|api_action, data| api_action == "nodebalancer.node.#{action}" }
+        @linode.send(action.to_sym)
+      end
+    
+      it 'should provide the data hash when making its request' do
+        @linode.expects(:send_request).with {|api_action, data| data = { :foo => :bar } }
+        @linode.send(action.to_sym, {:foo => :bar})
+      end
+    
+      it 'should return the result of the request' do
+        @linode.expects(:send_request).returns(:bar => :baz)      
+        @linode.send(action.to_sym).should == { :bar => :baz }      
+      end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/nodebalancer/nodebalancer.node.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/nodebalancer/nodebalancer.node.#{action}"
+      end
+    end
+  end
+end

data/spec/linode/nodebalancer_spec.rb

@@ -0,0 +1,124 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper.rb')
+require 'linode'
+
+describe Linode::Nodebalancer do
+  before :each do
+    @api_key = 'foo'
+    @linode = Linode::Nodebalancer.new(:api_key => @api_key)
+  end
+  
+  it 'should be a Linode instance' do
+    @linode.class.should < Linode
+  end
+  
+  %w(update create list delete).each do |action|
+    it "should allow accessing the #{action} API" do
+      @linode.should respond_to(action.to_sym)
+    end
+  
+    describe "when accessing the #{action} API" do
+      it 'should allow a data hash' do
+        lambda { @linode.send(action.to_sym, {}) }.should_not raise_error(ArgumentError)
+      end
+    
+      it 'should not require arguments' do
+        lambda { @linode.send(action.to_sym) }.should_not raise_error(ArgumentError)      
+      end
+    
+      it "should request the nodebalancer.#{action} action" do
+        @linode.expects(:send_request).with {|api_action, data| api_action == "nodebalancer.#{action}" }
+        @linode.send(action.to_sym)
+      end
+    
+      it 'should provide the data hash when making its request' do
+        @linode.expects(:send_request).with {|api_action, data| data = { :foo => :bar } }
+        @linode.send(action.to_sym, {:foo => :bar})
+      end
+    
+      it 'should return the result of the request' do
+        @linode.expects(:send_request).returns(:bar => :baz)      
+        @linode.send(action.to_sym).should == { :bar => :baz }      
+      end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/nodebalancer/nodebalancer.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/nodebalancer/nodebalancer.#{action}"
+      end
+    end
+  end
+  
+  it 'should be able to provide access to the Linode Nodebalancer Config API' do
+    @linode.should respond_to(:config)
+  end
+  
+  describe 'when providing access to the Linode Nodebalancer Config API' do
+    before :each do
+      @api_key = 'foo'
+      @api_url = 'https://fake.linode.com/'
+      @linode = Linode::Nodebalancer.new(:api_key => @api_key, :api_url => @api_url)
+    end
+
+    it 'should allow no arguments' do
+      lambda { @linode.config }.should_not raise_error(ArgumentError)
+    end
+    
+    it 'should require no arguments' do
+      lambda { @linode.config(:foo) }.should raise_error(ArgumentError)
+    end
+    
+    it 'should return a Linode::Nodebalancer::Config instance' do
+      @linode.config.class.should == Linode::Nodebalancer::Config
+    end
+    
+    it 'should set the API key on the Linode::Nodebalancer::Config instance to be our API key' do
+      @linode.config.api_key.should == @api_key
+    end
+    
+    it 'should set the API url on the Linode::Nodebalancer::Config instance to be our API url' do
+      @linode.config.api_url.should == @api_url
+    end
+    
+    it 'should return the same Linode::Nodebalancer::Config instance when called again' do
+      linode = Linode::Nodebalancer.new(:api_key => @api_key)
+      result = linode.config
+      linode.config.should == result
+    end
+  end
+  
+  it 'should be able to provide access to the Linode Nodebalancer Node API' do
+    @linode.should respond_to(:node)
+  end
+  
+  describe 'when providing access to the Linode Nodebalancer Node API' do
+    before :each do
+      @api_key = 'foo'
+      @api_url = 'https://fake.linode.com/'
+      @linode = Linode::Nodebalancer.new(:api_key => @api_key, :api_url => @api_url)
+    end
+
+    it 'should allow no arguments' do
+      lambda { @linode.node }.should_not raise_error(ArgumentError)
+    end
+    
+    it 'should require no arguments' do
+      lambda { @linode.node(:foo) }.should raise_error(ArgumentError)
+    end
+    
+    it 'should return a Linode::Nodebalancer::Node instance' do
+      @linode.node.class.should == Linode::Nodebalancer::Node
+    end
+    
+    it 'should set the API key on the Linode::Nodebalancer::Node instance to be our API key' do
+      @linode.node.api_key.should == @api_key
+    end
+    
+    it 'should set the API url on the Linode::Nodebalancer::Node instance to be our API url' do
+      @linode.node.api_url.should == @api_url
+    end
+    
+    it 'should return the same Linode::Nodebalancer::Node instance when called again' do
+      linode = Linode::Nodebalancer.new(:api_key => @api_key)
+      result = linode.node
+      linode.node.should == result
+    end
+  end
+end

data/spec/linode/stackscript_spec.rb

@@ -39,6 +39,10 @@ describe Linode::Stackscript do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/stackscript/stackscript.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/stackscript/stackscript.#{action}"
+      end
     end
   end
 end

data/spec/linode/test_spec.rb

@@ -40,6 +40,10 @@ describe Linode::Test do
         @linode.expects(:send_request).returns(:bar => :baz)      
         @linode.send(action.to_sym).should == { :bar => :baz }      
       end
+      
+      it "should consider the documentation to live at http://www.linode.com/api/linode/linode.#{action}" do
+        @linode.documentation_path(Linode.action_path(@linode.class.name, action)).should == "http://www.linode.com/api/utility/test.#{action}"
+      end
     end
   end
 end

data/spec/linode/user_spec.rb

@@ -28,5 +28,9 @@ describe Linode::User do
       @linode.stubs(:api_key).returns('foo')
       @linode.getapikey.should == 'foo'
     end
+      
+    it "should consider the documentation to live at http://www.linode.com/api/utility/user.getapikey" do
+      @linode.documentation_path(Linode.action_path(@linode.class.name, 'getapikey')).should == "http://www.linode.com/api/utility/user.getapikey"
+    end
   end
 end

data/spec/linode_spec.rb

@@ -70,8 +70,8 @@ describe 'Linode' do
 
     it 'should use the user.getapikey remote call to look up the API key associated with the username/password combo specified at creation time' do
       @json = '{"ERRORARRAY":[],"DATA":{"USERNAME":"ogc","API_KEY":"blahblahblah"},"ACTION":"user.getapikey"}'
-      HTTParty.expects(:get).with(@api_url, 
-        :query => { 
+      HTTParty.expects(:post).with(@api_url, 
+        :body => { 
           :api_action => 'user.getapikey', 
           :api_responseFormat => 'json', 
           :username => @username, 
@@ -136,7 +136,7 @@ describe 'Linode' do
       		"ACTION":"test.echo",
       		"DATA":{"FOO":"bar"}
       	}!
-      HTTParty.stubs(:get).returns(@json)
+      HTTParty.stubs(:post).returns(@json)
       @linode.stubs(:api_url).returns('https://fake.linode.com/')
     end
     
@@ -150,63 +150,63 @@ describe 'Linode' do
     
     it 'should make a request to the API url' do
       @linode.stubs(:api_url).returns('https://fake.linode.com/')
-      HTTParty.expects(:get).with { |path, args|
+      HTTParty.expects(:post).with { |path, args|
         path == 'https://fake.linode.com/'
       }.returns(@json)
       @linode.send_request('test.echo', { })
     end
     
     it 'should provide the API key when making its request' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_key] == @api_key
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_key] == @api_key
       }.returns(@json)
       @linode.send_request('test.echo', { })      
     end
     
     it 'should set the designated request method as the HTTP API action' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_action] == 'test.echo'
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_action] == 'test.echo'
       }.returns(@json)
       @linode.send_request('test.echo', { })            
     end
     
     it 'should set the response format to JSON' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_responseFormat] == 'json'
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_responseFormat] == 'json'
       }.returns(@json)
       @linode.send_request('test.echo', { })      
     end
     
     it 'should provide the data hash to the HTTP API request' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query]['foo'] == 'bar'
+      HTTParty.expects(:post).with { |path, args|
+        args[:body]['foo'] == 'bar'
       }.returns(@json)
       @linode.send_request('test.echo', { 'foo' => 'bar' })                  
     end
     
     it 'should not allow overriding the API key via the data hash' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_key] == @api_key
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_key] == @api_key
       }.returns(@json)
       @linode.send_request('test.echo', { :api_key => 'h4x0r' })                        
     end
     
     it 'should not allow overriding the API action via the data hash' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_action] == 'test.echo'
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_action] == 'test.echo'
       }.returns(@json)
       @linode.send_request('test.echo', { :api_action => 'h4x0r' })
     end
     
     it 'should not allow overriding the API response format via the data hash' do
-      HTTParty.expects(:get).with { |path, args|
-        args[:query][:api_responseFormat] == 'json'
+      HTTParty.expects(:post).with { |path, args|
+        args[:body][:api_responseFormat] == 'json'
       }.returns(@json)
       @linode.send_request('test.echo', { :api_responseFormat => 'h4x0r' })
     end
     
     it 'should fail when the request submission fails' do
-      HTTParty.stubs(:get).returns(%Q!{
+      HTTParty.stubs(:post).returns(%Q!{
       		"ERRORARRAY":["failure"],
       		"ACTION":"test.echo",
       		"DATA":{"foo":"bar"}
@@ -216,7 +216,7 @@ describe 'Linode' do
     
     describe 'when the result is a list of hashes' do
       it 'should return a list of objects with lower-cased methods for the data fields' do
-        HTTParty.stubs(:get).returns(%Q!{
+        HTTParty.stubs(:post).returns(%Q!{
         		"ERRORARRAY":[],
         		"ACTION":"test.echo",
         		"DATA":[{"FOO":"bar"},{"BAR":"baz"}]
@@ -226,7 +226,7 @@ describe 'Linode' do
       end
       
       it 'should return a list of objects which do not respond to upper-case URLs for the data fields' do
-        HTTParty.stubs(:get).returns(%Q!{
+        HTTParty.stubs(:post).returns(%Q!{
         		"ERRORARRAY":[],
         		"ACTION":"test.echo",
         		"DATA":[{"FOO":"bar"},{"BAR":"baz"}]
@@ -248,7 +248,7 @@ describe 'Linode' do
     
     describe 'when the result is neither a list nor a hash' do
       it 'should return the result unchanged' do
-        HTTParty.stubs(:get).returns(%Q!{
+        HTTParty.stubs(:post).returns(%Q!{
         		"ERRORARRAY":[],
         		"ACTION":"test.echo",
         		"DATA":"thingie"
@@ -417,5 +417,37 @@ describe 'Linode' do
       linode.linode.should == result
     end
   end
+
+  it 'should be able to provide access to the Linode Nodebalancer API' do
+    @linode.should respond_to(:nodebalancer)
+  end
+  
+  describe 'when providing access to the Linode Nodebalancer API' do
+    it 'should allow no arguments' do
+      lambda { @linode.nodebalancer }.should_not raise_error(ArgumentError)
+    end
+    
+    it 'should require no arguments' do
+      lambda { @linode.nodebalancer(:foo) }.should raise_error(ArgumentError)
+    end
+    
+    it 'should return a Linode::Nodebalancer instance' do
+      @linode.nodebalancer.class.should == Linode::Nodebalancer
+    end
+    
+    it 'should set the API key on the Linode::Nodebalancer instance to be our API key' do
+      @linode.nodebalancer.api_key.should == @api_key
+    end
+    
+    it 'should set the API url on the Linode::Nodebalancer instance to be our API url' do
+      @linode.nodebalancer.api_url.should == @api_url
+    end
+    
+    it 'should return the same Linode::Nodebalancer instance when called again' do
+      linode = Linode.new(:api_key => @api_key)
+      result = linode.nodebalancer
+      linode.nodebalancer.should == result
+    end
+  end
 end
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: linode
 version: !ruby/object:Gem::Version 
-  hash: 1
+  hash: 3
   prerelease: false
   segments: 
   - 0
-  - 6
-  - 3
-  version: 0.6.3
+  - 7
+  - 0
+  version: 0.7.0
 platform: ruby
 authors: 
 - Rick Bradley
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-11-23 00:00:00 -06:00
+date: 2011-07-07 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -59,6 +59,9 @@ files:
 - lib/linode/linode/disk.rb
 - lib/linode/linode/ip.rb
 - lib/linode/linode/job.rb
+- lib/linode/nodebalancer.rb
+- lib/linode/nodebalancer/config.rb
+- lib/linode/nodebalancer/node.rb
 - lib/linode/stackscript.rb
 - lib/linode/test.rb
 - lib/linode/user.rb
@@ -71,6 +74,9 @@ files:
 - spec/linode/linode/ip_spec.rb
 - spec/linode/linode/job_spec.rb
 - spec/linode/linode_spec.rb
+- spec/linode/nodebalancer/config_spec.rb
+- spec/linode/nodebalancer/node_spec.rb
+- spec/linode/nodebalancer_spec.rb
 - spec/linode/stackscript_spec.rb
 - spec/linode/test_spec.rb
 - spec/linode/user_spec.rb
@@ -119,6 +125,9 @@ test_files:
 - spec/linode/linode/ip_spec.rb
 - spec/linode/linode/job_spec.rb
 - spec/linode/linode_spec.rb
+- spec/linode/nodebalancer/config_spec.rb
+- spec/linode/nodebalancer/node_spec.rb
+- spec/linode/nodebalancer_spec.rb
 - spec/linode/stackscript_spec.rb
 - spec/linode/test_spec.rb
 - spec/linode/user_spec.rb