oodler 0.1.7 → 0.2.0

data/VERSION

@@ -1 +1 @@
-0.1.7
+0.2.0

data/lib/oodler/client.rb

@@ -15,6 +15,7 @@ module Oodler
     
     def get(path, options={})
       path = "/listings#{path}"
+      delete_empty_keys(options)
       validate_parameters(options)
       response = web_client.get(path, options)
       raise_errors(response)
@@ -36,9 +37,10 @@ module Oodler
     end
 
     #specify the :location key for city search, i.e :location=>"Houston, Tx"
+    #allow job specializaton, i.e. job/tech
     def usa_job_search(options)
       options.merge!({:region => "usa"})
-      options.merge!({:category => "job"})
+      options.merge!({:category => "job"}) unless options && options[:category]
       self.listing(options)
     end
     
@@ -72,12 +74,17 @@ module Oodler
         
       def validate_parameters(options)
 
+
+        if options.empty?
+          raise OodleArgumentError, 'Missing region and category parameter. Visit http://developer.oodle.com/regions-list/ for possible regions.'
+        end
+
         params = options[:query]
    
-        unless params[:region]
+        unless params.nil? || params[:region]
           raise OodleArgumentError, 'Missing region parameter. Visit http://developer.oodle.com/regions-list/ for possible regions.'
         end
-        unless params[:category] || params[:q]
+        unless  params[:category] || params[:q]
           raise OodleArgumentError, 'You must supply a category or query parameter. Visit http://developer.oodle.com/categories-list/ for possible categories.'
         end
       end
@@ -90,6 +97,10 @@ module Oodler
 
       end
 
+      def delete_empty_keys(options)
+        options.delete_if{|key,value| value.empty?}
+      end
+
       
 
   end

data/lib/oodler/web_client.rb

@@ -11,6 +11,7 @@ module Oodler
      end
 
      def get(path,options)
+       options[:query] ||={}
        options[:query].merge!({:key => @key}) if @key
        validate_parameters(options)
        self.class.get(path,options)

data/test/client_test.rb

@@ -16,6 +16,12 @@ class ClientTest < Test::Unit::TestCase
       @oodler.web_client.should == wc
 
     end
+  context "and using the usa_jobs conv method" do
+    should " allow you to override the category param" do
+       stub_get("/api/v2/listings?region=usa&category=job%2Ftech&key=key", "listings.xml")
+       @oodler.usa_job_search(:category => "job/tech")
+    end
+  end
 
    context "and a region is not provided" do
       should "throw an error" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: oodler
 version: !ruby/object:Gem::Version 
-  version: 0.1.7
+  version: 0.2.0
 platform: ruby
 authors: 
 - Mike Hagedorn
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-16 00:00:00 -05:00
+date: 2010-04-22 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency