nationbuilder-rb 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 478eaf49468a14664586855b0105a5c0781b9e82
-  data.tar.gz: be6334a496d0b620c74c357515fb026224935594
+  metadata.gz: 7f736fe98de0df24dd6a2e38b3d7013ec6d49245
+  data.tar.gz: fca4e68a8753f003ba0f6815abcf74e9d16c8e7d
 SHA512:
-  metadata.gz: d3f42b168b4b0ce526e17ef7462c1666e27e57b05822c005e3f97fd5ca4832041de1884a3c6cbd11656bf89d1d02d874fd66e5dd57ceb533d9efbc057b7f6652
-  data.tar.gz: a1b4b4fef2cad1592d178c50cc91f385e69b1b9828c7037bdbb963020afb5469129b4ed1f3e23c4f9f98d383ccab76b564d20ec1bf06c0d3daa85efc3fd03c69
+  metadata.gz: 25aec085a33bab04ce580aa96d6f650893a5d603f87caa46f4b910401559b3d66bc161d71bb56370b9bada6a20be5d11b0f4759a587b9edcc3d6c252eb6f2d8f
+  data.tar.gz: a958baba40eaefda72a00248ef5d3ada11d9b751763de3a97ad17ef068fe6a21220f7eafdf74007ffca41c105f9bad5fa4a463f15ca1a708fa1a67921708e080

data/CHANGELOG.md

@@ -1,2 +1,6 @@
+# 1.2.0
+- Native exception types
+- [#9] Exponential backoff when encountering rate limit
+
 # 1.1.0
 - [#6] Pagination class for iterating through paginated sets

data/Gemfile

@@ -3,9 +3,9 @@ source 'http://rubygems.org'
 gem 'httpclient', '~> 2.4.0'
 
 group :development do
-  gem 'jeweler', '~> 2.0.1'
-  gem 'rspec', '~> 2.8.0'
-  gem 'simplecov', '~> 0.8.2'
-  gem 'vcr', '~> 2.9.2'
-  gem 'webmock', '~> 1.18.0'
+  gem 'jeweler', '~> 2.0'
+  gem 'rspec', '~> 3.2'
+  gem 'simplecov', '~> 0.8'
+  gem 'vcr', '~> 2.9'
+  gem 'webmock', '~> 1.18'
 end

data/Gemfile.lock

@@ -1,26 +1,26 @@
 GEM
   remote: http://rubygems.org/
   specs:
-    addressable (2.3.6)
+    addressable (2.3.7)
     builder (3.2.2)
     crack (0.4.2)
       safe_yaml (~> 1.0.0)
     descendants_tracker (0.0.4)
       thread_safe (~> 0.3, >= 0.3.1)
-    diff-lcs (1.1.3)
+    diff-lcs (1.2.5)
     docile (1.1.5)
-    faraday (0.9.0)
+    faraday (0.9.1)
       multipart-post (>= 1.2, < 3)
-    git (1.2.7)
-    github_api (0.11.3)
+    git (1.2.9.1)
+    github_api (0.12.3)
       addressable (~> 2.3)
-      descendants_tracker (~> 0.0.1)
+      descendants_tracker (~> 0.0.4)
       faraday (~> 0.8, < 0.10)
-      hashie (>= 1.2)
+      hashie (>= 3.3)
       multi_json (>= 1.7.5, < 2.0)
-      nokogiri (~> 1.6.0)
+      nokogiri (~> 1.6.3)
       oauth2
-    hashie (3.1.0)
+    hashie (3.4.0)
     highline (1.6.21)
     httpclient (2.4.0)
     jeweler (2.0.1)
@@ -32,41 +32,46 @@ GEM
       nokogiri (>= 1.5.10)
       rake
       rdoc
-    json (1.8.1)
-    jwt (1.0.0)
-    mini_portile (0.6.0)
+    json (1.8.2)
+    jwt (1.2.1)
+    mini_portile (0.6.2)
     multi_json (1.10.1)
     multi_xml (0.5.5)
     multipart-post (2.0.0)
-    nokogiri (1.6.2.1)
-      mini_portile (= 0.6.0)
-    oauth2 (0.9.4)
+    nokogiri (1.6.6.2)
+      mini_portile (~> 0.6.0)
+    oauth2 (1.0.0)
       faraday (>= 0.8, < 0.10)
       jwt (~> 1.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
       rack (~> 1.2)
-    rack (1.5.2)
-    rake (10.3.2)
-    rdoc (4.1.1)
+    rack (1.6.0)
+    rake (10.4.2)
+    rdoc (4.2.0)
       json (~> 1.4)
-    rspec (2.8.0)
-      rspec-core (~> 2.8.0)
-      rspec-expectations (~> 2.8.0)
-      rspec-mocks (~> 2.8.0)
-    rspec-core (2.8.0)
-    rspec-expectations (2.8.0)
-      diff-lcs (~> 1.1.2)
-    rspec-mocks (2.8.0)
-    safe_yaml (1.0.3)
-    simplecov (0.8.2)
+    rspec (3.2.0)
+      rspec-core (~> 3.2.0)
+      rspec-expectations (~> 3.2.0)
+      rspec-mocks (~> 3.2.0)
+    rspec-core (3.2.0)
+      rspec-support (~> 3.2.0)
+    rspec-expectations (3.2.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.2.0)
+    rspec-mocks (3.2.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.2.0)
+    rspec-support (3.2.1)
+    safe_yaml (1.0.4)
+    simplecov (0.9.1)
       docile (~> 1.1.0)
-      multi_json
+      multi_json (~> 1.0)
       simplecov-html (~> 0.8.0)
     simplecov-html (0.8.0)
     thread_safe (0.3.4)
-    vcr (2.9.2)
-    webmock (1.18.0)
+    vcr (2.9.3)
+    webmock (1.20.4)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
 
@@ -75,8 +80,8 @@ PLATFORMS
 
 DEPENDENCIES
   httpclient (~> 2.4.0)
-  jeweler (~> 2.0.1)
-  rspec (~> 2.8.0)
-  simplecov (~> 0.8.2)
-  vcr (~> 2.9.2)
-  webmock (~> 1.18.0)
+  jeweler (~> 2.0)
+  rspec (~> 3.2)
+  simplecov (~> 0.8)
+  vcr (~> 2.9)
+  webmock (~> 1.18)

data/README.md

@@ -30,9 +30,13 @@ Then, create a client by specifying the name of your nation and
 your API token:
 
 ```ruby
-client = NationBuilder::Client.new('my_nation_name', 'my_api_token')
+client = NationBuilder::Client.new('my_nation_name', 'my_api_token', retries: 8)
 ```
 
+The `retries` parameter specifies the maximum number of retries to attempt
+when the client is rate limited. This uses an exponential backoff strategy.
+The default is `8`.
+
 ## Calling the API
 
 The primary method for calling the NationBuilder API in
@@ -81,7 +85,7 @@ page1 = paginated
 page2 = page1.next
 page3 = page2.next
 ```
-Methods `#next` and `#prev` return the results of the next or previous page of results, nil if none. `#next?` and `#prev?` return the path to the next or prev page, or nil if none. The results of a page can be accessed through `.body` - in the above example, `page1.body` returns the same hash as `response`. 
+Methods `#next` and `#prev` return the results of the next or previous page of results, nil if none. `#next?` and `#prev?` return the path to the next or prev page, or nil if none. The results of a page can be accessed through `.body` - in the above example, `page1.body` returns the same hash as `response`.
 
 ## Documentation
 

data/VERSION

@@ -1 +1 @@
-1.1.0
+1.2.0

data/lib/nationbuilder.rb

@@ -12,3 +12,4 @@ require 'nationbuilder/parameter'
 require 'nationbuilder/url'
 require 'nationbuilder/spec_parser'
 require 'nationbuilder/paginator'
+require 'nationbuilder/errors'

data/lib/nationbuilder/client.rb

@@ -1,10 +1,16 @@
 class NationBuilder::Client
 
-  def initialize(nation_name, api_key, base_url = 'https://:nation_name.nationbuilder.com')
+  def initialize(nation_name, api_key, opts = {})
     @nation_name = nation_name
     @api_key = api_key
-    @base_url = base_url
     @name_to_endpoint = {}
+    @base_url = opts[:base_url] || 'https://:nation_name.nationbuilder.com'
+    @retries = opts[:retries] || 8
+
+    if @retries < 1
+      raise 'A positive number of retries must be specified'
+    end
+
     parsed_endpoints.each do |endpoint|
       @name_to_endpoint[endpoint.name] = endpoint
     end
@@ -31,6 +37,8 @@ class NationBuilder::Client
     @base_url.gsub(':nation_name', @nation_name)
   end
 
+  RETRY_DELAY = 0.1 # seconds
+
   def raw_call(path, method, body = {}, args = {})
     url = NationBuilder::URL.new(base_url).generate_url(path, args)
 
@@ -51,8 +59,7 @@ class NationBuilder::Client
       request_args[:body] = JSON(body)
     end
 
-    set_response(HTTPClient.send(method, url, request_args))
-    return parse_response_body(response)
+    perform_request_with_retries(method, url, request_args)
   end
 
   def call(endpoint_name, method_name, args={})
@@ -64,22 +71,52 @@ class NationBuilder::Client
     return raw_call(method.uri, method.http_method, nonmethod_args, args)
   end
 
+  def perform_request_with_retries(method, url, request_args)
+    raw_response = HTTPClient.send(method, url, request_args)
+    parsed_response = nil
+
+    @retries.times do |i|
+      begin
+        parsed_response = parse_response_body(raw_response)
+      rescue NationBuilder::RateLimitedError
+        Kernel.sleep(RETRY_DELAY * 2**i)
+      rescue => e
+        raise e
+      else
+        break
+      end
+    end
+
+    set_response(raw_response)
+    parsed_response
+  end
+
   def set_response(value)
     Thread.current[:nationbuilder_rb_response] = value
   end
 
+  # This getter is used for fetching the raw response
   def response
     Thread.current[:nationbuilder_rb_response]
   end
 
-  class ServerResponseError < StandardError; end
+  def classify_response_error(response)
+    case
+    when response.code == 429
+      return NationBuilder::RateLimitedError.new(response.body)
+    when response.code.to_s.start_with?('4')
+      return NationBuilder::ClientError.new(response.body)
+    when response.code.to_s.start_with?('5')
+      return NationBuilder::ServerError.new(response.body)
+    end
+  end
 
   def parse_response_body(response)
-    success = response.code.to_s.start_with?('2')
+    error = classify_response_error(response)
+    raise error if error
 
     if response.header['Content-Type'].first != 'application/json'
-      return {} if success
-      raise ServerResponseError.new("Non-JSON content-type for server response: #{response.body}")
+      return nil
     end
 
     body = response.body.strip

data/lib/nationbuilder/errors.rb

@@ -0,0 +1,6 @@
+module NationBuilder
+  class BaseError < StandardError; end
+  class ServerError < BaseError; end
+  class ClientError < BaseError; end
+  class RateLimitedError < ClientError; end
+end

data/nationbuilder-rb.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: nationbuilder-rb 1.1.0 ruby lib
+# stub: nationbuilder-rb 1.2.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "nationbuilder-rb"
-  s.version = "1.1.0"
+  s.version = "1.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["David Huie"]
-  s.date = "2015-02-10"
+  s.date = "2015-02-17"
   s.description = "A Ruby client to the NationBuilder API"
   s.email = "david@nationbuilder.com"
   s.executables = ["nbdoc"]
@@ -37,6 +37,7 @@ Gem::Specification.new do |s|
     "lib/nationbuilder/api_spec.json",
     "lib/nationbuilder/client.rb",
     "lib/nationbuilder/endpoint.rb",
+    "lib/nationbuilder/errors.rb",
     "lib/nationbuilder/method.rb",
     "lib/nationbuilder/paginator.rb",
     "lib/nationbuilder/parameter.rb",
@@ -62,26 +63,26 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<httpclient>, ["~> 2.4.0"])
-      s.add_development_dependency(%q<jeweler>, ["~> 2.0.1"])
-      s.add_development_dependency(%q<rspec>, ["~> 2.8.0"])
-      s.add_development_dependency(%q<simplecov>, ["~> 0.8.2"])
-      s.add_development_dependency(%q<vcr>, ["~> 2.9.2"])
-      s.add_development_dependency(%q<webmock>, ["~> 1.18.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 2.0"])
+      s.add_development_dependency(%q<rspec>, ["~> 3.2"])
+      s.add_development_dependency(%q<simplecov>, ["~> 0.8"])
+      s.add_development_dependency(%q<vcr>, ["~> 2.9"])
+      s.add_development_dependency(%q<webmock>, ["~> 1.18"])
     else
       s.add_dependency(%q<httpclient>, ["~> 2.4.0"])
-      s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
-      s.add_dependency(%q<rspec>, ["~> 2.8.0"])
-      s.add_dependency(%q<simplecov>, ["~> 0.8.2"])
-      s.add_dependency(%q<vcr>, ["~> 2.9.2"])
-      s.add_dependency(%q<webmock>, ["~> 1.18.0"])
+      s.add_dependency(%q<jeweler>, ["~> 2.0"])
+      s.add_dependency(%q<rspec>, ["~> 3.2"])
+      s.add_dependency(%q<simplecov>, ["~> 0.8"])
+      s.add_dependency(%q<vcr>, ["~> 2.9"])
+      s.add_dependency(%q<webmock>, ["~> 1.18"])
     end
   else
     s.add_dependency(%q<httpclient>, ["~> 2.4.0"])
-    s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
-    s.add_dependency(%q<rspec>, ["~> 2.8.0"])
-    s.add_dependency(%q<simplecov>, ["~> 0.8.2"])
-    s.add_dependency(%q<vcr>, ["~> 2.9.2"])
-    s.add_dependency(%q<webmock>, ["~> 1.18.0"])
+    s.add_dependency(%q<jeweler>, ["~> 2.0"])
+    s.add_dependency(%q<rspec>, ["~> 3.2"])
+    s.add_dependency(%q<simplecov>, ["~> 0.8"])
+    s.add_dependency(%q<vcr>, ["~> 2.9"])
+    s.add_dependency(%q<webmock>, ["~> 1.18"])
   end
 end
 

data/spec/fixtures/parametered_post.yml

@@ -17,7 +17,7 @@ http_interactions:
       - application/json
   response:
     status:
-      code: 409
+      code: 200
       message: Conflict
     headers:
       Server:
@@ -76,6 +76,6 @@ http_interactions:
         LOS ANGELES\",\"closed_invoices_amount_in_cents\":null,\"closed_invoices_count\":null,\"contact_status_id\":null,\"contact_status_name\":null,\"could_vote_status\":null,\"demo\":\"\",\"donations_amount_in_cents\":0,\"donations_amount_this_cycle_in_cents\":0,\"donations_count\":0,\"donations_count_this_cycle\":0,\"donations_pledged_amount_in_cents\":0,\"donations_raised_amount_in_cents\":0,\"donations_raised_amount_this_cycle_in_cents\":0,\"donations_raised_count\":0,\"donations_raised_count_this_cycle\":0,\"donations_to_raise_amount_in_cents\":0,\"email1\":\"bob@example.com\",\"email1_is_bad\":false,\"email2\":null,\"email2_is_bad\":false,\"email3\":null,\"email3_is_bad\":false,\"email4\":null,\"email4_is_bad\":false,\"ethnicity\":\"\",\"facebook_address\":null,\"facebook_profile_url\":null,\"facebook_updated_at\":null,\"facebook_username\":null,\"fax_number\":\"\",\"federal_donotcall\":false,\"first_donated_at\":null,\"first_fundraised_at\":null,\"first_invoice_at\":null,\"first_prospect_at\":null,\"first_recruited_at\":null,\"first_supporter_at\":\"2014-04-24T10:27:34-07:00\",\"first_volunteer_at\":\"2014-09-02T13:10:28-07:00\",\"full_name\":\"Bob
         Smith\",\"home_address\":null,\"import_id\":null,\"inferred_party\":\"\",\"inferred_support_level\":null,\"invoice_payments_amount_in_cents\":null,\"invoice_payments_referred_amount_in_cents\":null,\"invoices_amount_in_cents\":null,\"invoices_count\":null,\"is_deceased\":false,\"is_donor\":false,\"is_fundraiser\":false,\"is_ignore_donation_limits\":false,\"is_leaderboardable\":true,\"is_mobile_bad\":false,\"is_possible_duplicate\":false,\"is_profile_private\":false,\"is_profile_searchable\":true,\"is_prospect\":false,\"is_supporter\":true,\"is_survey_question_private\":false,\"language\":\"\",\"last_call_id\":null,\"last_contacted_at\":null,\"last_contacted_by\":null,\"last_donated_at\":null,\"last_fundraised_at\":null,\"last_invoice_at\":null,\"last_rule_violation_at\":null,\"legal_name\":null,\"locale\":\"\",\"mailing_address\":null,\"marital_status\":\"\",\"media_market_name\":null,\"meetup_address\":null,\"membership_expires_at\":null,\"membership_level_name\":null,\"membership_started_at\":null,\"middle_name\":null,\"mobile_normalized\":null,\"nbec_precinct_code\":null,\"note_updated_at\":null,\"outstanding_invoices_amount_in_cents\":null,\"outstanding_invoices_count\":null,\"overdue_invoices_count\":null,\"page_slug\":\"ticket_page\",\"parent\":null,\"parent_id\":null,\"party_member\":false,\"phone_normalized\":\"2135551234\",\"phone_time\":\"\",\"precinct_code\":null,\"precinct_name\":null,\"prefix\":null,\"previous_party\":\"\",\"primary_email_id\":1,\"priority_level\":null,\"priority_level_changed_at\":null,\"profile_content\":null,\"profile_content_html\":null,\"profile_headline\":null,\"received_capital_amount_in_cents\":17700,\"recruiter\":null,\"recruits_count\":0,\"registered_address\":null,\"registered_at\":null,\"religion\":\"\",\"rule_violations_count\":0,\"spent_capital_amount_in_cents\":1000,\"submitted_address\":\"\",\"subnations\":[],\"suffix\":null,\"support_level_changed_at\":null,\"support_probability_score\":null,\"turnout_probability_score\":null,\"twitter_address\":null,\"twitter_description\":null,\"twitter_followers_count\":null,\"twitter_friends_count\":null,\"twitter_location\":null,\"twitter_login\":null,\"twitter_updated_at\":null,\"twitter_website\":null,\"unsubscribed_at\":null,\"user_submitted_address\":{\"address1\":\"488
         S Hill St\",\"address2\":null,\"address3\":null,\"city\":\"Los Angeles\",\"state\":\"CA\",\"country_code\":\"US\",\"zip\":\"90013\",\"lat\":\"34.0490467\",\"lng\":\"-118.2515224\"},\"username\":\"randomeveguy\",\"warnings_count\":0,\"website\":null,\"work_address\":null,\"bag_preference\":null,\"do_you_drive_a_fork_lift\":null,\"have_you_ever_been_contacted_by_a_union_representative_before_\":null,\"have_you_been_involved_in_an_election_campaign_before_\":null,\"sub_branch\":null,\"sams_custom_field\":null,\"mrow\":null,\"received_an_email\":null,\"brett_petition_mc\":null,\"headshot_url\":null,\"university\":null,\"membership_number\":null},\"precinct\":null}"
-    http_version: 
+    http_version:
   recorded_at: Thu, 05 Feb 2015 05:54:33 GMT
 recorded_with: VCR 2.9.2

data/spec/nationbuilder_client_spec.rb

@@ -4,13 +4,14 @@ describe NationBuilder::Client do
 
   let(:client) do
     NationBuilder::Client.new('organizeralexandreschmitt',
-                              '53920a524356034a065515a37650df2bd295971975d5742b9daa50eb8c7404d5')
+                              '53920a524356034a065515a37650df2bd295971975d5742b9daa50eb8c7404d5',
+                              retries: 2)
   end
 
   describe '#endpoints' do
 
     it 'should contain all defined endpoints' do
-      client.endpoints.sort.should eq([
+      expect(client.endpoints.sort).to eq([
         :basic_pages,
         :blog_posts,
         :blogs,
@@ -38,7 +39,7 @@ describe NationBuilder::Client do
   describe '#base_url' do
 
     it 'should contain the nation slug' do
-      client.base_url.should eq('https://organizeralexandreschmitt.nationbuilder.com')
+      expect(client.base_url).to eq('https://organizeralexandreschmitt.nationbuilder.com')
     end
   end
 
@@ -48,7 +49,7 @@ describe NationBuilder::Client do
       VCR.use_cassette('parametered_get') do
         response = client.call(:basic_pages, :index, site_slug: 'organizeralexandreschmitt')
         response['results'].each do |result|
-          result['site_slug'].should eq('organizeralexandreschmitt')
+          expect(result['site_slug']).to eq('organizeralexandreschmitt')
         end
       end
     end
@@ -66,7 +67,7 @@ describe NationBuilder::Client do
         client.call(:people, :create, params)
       end
 
-      response['person']['first_name'].should eq('Bob')
+      expect(response['person']['first_name']).to eq('Bob')
     end
 
     it 'should handle a DELETE' do
@@ -78,7 +79,51 @@ describe NationBuilder::Client do
         client.call(:people, :destroy, params)
       end
 
-      response.should eq({})
+      expect(response).to eq(nil)
+    end
+  end
+
+  describe '#classify_response_error' do
+    it 'should account for rate limits' do
+      response = double(code: 429, body: 'rate limiting')
+      expect(client.classify_response_error(response).class).
+        to eq(NationBuilder::RateLimitedError)
+    end
+    it 'should account for client errors' do
+      response = double(code: 404, body: '404ing')
+      expect(client.classify_response_error(response).class).
+        to eq(NationBuilder::ClientError)
+    end
+    it 'should account for client errors' do
+      response = double(code: 500, body: '500ing')
+      expect(client.classify_response_error(response).class).
+        to eq(NationBuilder::ServerError)
+    end
+  end
+
+  describe '#perform_request_with_retries' do
+    before do
+      expect(HTTPClient).to receive(:send)
+    end
+
+    it 'should raise non-rate limiting execeptions' do
+      expect(client).to receive(:parse_response_body) { raise StandardError.new('boom') }
+      expect do
+        client.perform_request_with_retries(nil, nil, nil)
+      end.to raise_error
+    end
+
+    it 'should return a response if the rate limit is eventually dropped' do
+      expect(Kernel).to receive(:sleep).twice
+      allow(client).to receive(:parse_response_body) do
+        @count ||= 0
+        if @count != 2
+          raise NationBuilder::RateLimitedError.new
+        end
+      end
+      expect do
+        client.perform_request_with_retries(nil, nil, nil)
+      end.to_not raise_error
     end
   end
 end

data/spec/nationbuilder_paginator_spec.rb

@@ -19,26 +19,26 @@ describe Paginator do
     end
 
     it 'should check for next and prev page link' do
-      @page1.next?.should_not be_nil
-      @page1.prev?.should be_nil
+      expect(@page1.next?).to_not be_nil
+      expect(@page1.prev?).to be_nil
     end
 
     it 'should return next page' do
       VCR.use_cassette('paginated_get_page2') do
         page2 = @page1.next
-        page2.body.should_not eq(@page1.body)
+        expect(page2.body).to_not eq(@page1.body)
       end
     end
 
     it 'should return additional pages' do
       VCR.use_cassette('paginated_get_page3') do
         page3 = @page2.next
-        page3.body.should_not eq(@page2.body)
+        expect(page3.body).to_not eq(@page2.body)
       end
     end
 
     it 'should return nil if no prev page' do
-      @page1.prev.should be_nil
+      expect(@page1.prev).to be_nil
     end
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nationbuilder-rb
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - David Huie
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-10 00:00:00.000000000 Z
+date: 2015-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httpclient
@@ -30,70 +30,70 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.8.0
+        version: '3.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.8.0
+        version: '3.2'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.2
+        version: '0.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.2
+        version: '0.8'
 - !ruby/object:Gem::Dependency
   name: vcr
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.9.2
+        version: '2.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.9.2
+        version: '2.9'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.18.0
+        version: '1.18'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.18.0
+        version: '1.18'
 description: A Ruby client to the NationBuilder API
 email: david@nationbuilder.com
 executables:
@@ -120,6 +120,7 @@ files:
 - lib/nationbuilder/api_spec.json
 - lib/nationbuilder/client.rb
 - lib/nationbuilder/endpoint.rb
+- lib/nationbuilder/errors.rb
 - lib/nationbuilder/method.rb
 - lib/nationbuilder/paginator.rb
 - lib/nationbuilder/parameter.rb