hungrytable 0.0.1 → 0.0.2

data/.gitignore

@@ -3,3 +3,4 @@
 Gemfile.lock
 pkg/*
 *.sw*
+.open_table

data/Gemfile

@@ -2,3 +2,4 @@ source "http://rubygems.org"
 
 # Specify your gem's dependencies in hungrytable.gemspec
 gemspec
+

data/Guardfile

@@ -0,0 +1,16 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+notification :libnotify
+
+guard 'minitest' do
+  # with Minitest::Unit
+  watch(%r|^test/(.*)_test\.rb|)
+  watch(%r|^lib/(.*)/([^/]+)\.rb|) { |m| "test/unit/#{m[1]}/#{m[2]}_test.rb" }
+  watch(%r|^test/test_helper\.rb|) { "test" }
+
+  # with Minitest::Spec
+  # watch(%r|^spec/(.*)_spec\.rb|)
+  # watch(%r|^lib/(.*)\.rb|) { |m| "spec/#{m[1]}_spec.rb" }
+  # watch(%r|^spec/spec_helper\.rb|) { "spec" }
+end

data/README.md

@@ -28,6 +28,22 @@ Observe:
     export OT_OAUTH_KEY=<YOUR OPENTABLE OAUTH KEY>
     export OT_OAUTH_SECRET=<YOUR OPENTABLE OAUTH SECRET KEY>
 
+## Example Run
+
+    $ restaurant = Hungrytable::Restaurant.new(82591)  
+
+    > #<Hungrytable::Restaurant:0x0000000032e4098 ... >  
+
+    $ search = Hungrytable::RestaurantSearch.new(restaurant, {date_time: 5.days.from_now, party_size: 3})  
+
+    > #<Hungrytable::RestaurantSearch:0x00000003143388 ... >
+
+    $ slotlock = Hungrytable::RestaurantSlotlock.new(search)
+
+    > #<Hungrytable::RestaurantSlotlock:0x00000002973a68 ... >
+
+    $ reservation = Hungrytable::ReservationMake.new(slotlock, {email_address: 'foo@bar.com', firstname: 'Mike', lastname: 'Jones', phone: '2813308004'})
+
 
 ## Contributing
 

data/hungrytable.gemspec

@@ -22,15 +22,16 @@ Gem::Specification.new do |s|
   # s.add_development_dependency "rspec"
   # s.add_runtime_dependency "rest-client"
 
-  s.add_runtime_dependency 'oauth', '~> 0.4.5'
   s.add_runtime_dependency 'json', '~> 1.7.1'
+  s.add_runtime_dependency 'activesupport'
+  s.add_runtime_dependency 'i18n'
+  s.add_runtime_dependency 'curb'
 
-  s.add_development_dependency "minitest"
-  s.add_development_dependency "minitest-reporters"
-  s.add_development_dependency "rake"
-  s.add_development_dependency "simplecov"
-  s.add_development_dependency 'simplecov-rcov'
-  s.add_development_dependency 'rcov', '0.9.11'
+  s.add_development_dependency 'rake'
+  s.add_development_dependency 'minitest'
+  s.add_development_dependency 'minitest-reporters'
+  s.add_development_dependency 'mocha'
   s.add_development_dependency 'pry'
+  s.add_development_dependency 'webmock'
 end
 

data/lib/hungrytable/config.rb

@@ -0,0 +1,26 @@
+module Hungrytable
+  module Config
+    extend self
+
+    def partner_id
+      ENV['OT_PARTNER_ID'] || config_error('OT_PARTNER_ID')
+    end
+
+    def oauth_key
+      ENV['OT_OAUTH_KEY'] || config_error('OT_OAUTH_KEY')
+    end
+
+    def oauth_secret
+      ENV['OT_OAUTH_SECRET'] || config_error('OT_OAUTH_SECRET')
+    end
+
+    def base_url
+      'https://secure.opentable.com/api/otapi_v2.ashx'
+    end
+
+    private
+    def config_error var
+      raise HungrytableError, "ENV variable #{var} must be set."
+    end
+  end
+end

data/lib/hungrytable/get_request.rb

@@ -0,0 +1,15 @@
+module Hungrytable
+  class GetRequest < Request
+    private
+    def auth_header
+      Hungrytable::RequestHeader.new(:get, @uri, {}, {})
+    end
+
+    def make_request
+      Curl::Easy.perform(@uri) do |curl|
+        curl.headers['Authorization'] = auth_header
+      end
+    end
+
+  end
+end

data/lib/hungrytable/post_request.rb

@@ -0,0 +1,15 @@
+module Hungrytable
+  class PostRequest < Request
+    private
+    def auth_header
+      Hungrytable::RequestHeader.new(:post, @uri, {}, {})
+    end
+
+    def make_request
+      Curl::Easy.http_post(@uri, @params.to_query) do |curl|
+        curl.headers['Authorization'] = auth_header
+      end
+    end
+
+  end
+end

data/lib/hungrytable/request.rb

@@ -0,0 +1,17 @@
+module Hungrytable
+  class Request
+    def initialize uri, params={}
+      @uri    = Hungrytable::Config.base_url << uri
+      @params = params
+    end
+
+    def parsed_response
+      JSON.parse(response)
+    end
+
+    private
+    def response
+      @response ||= make_request.body_str
+    end
+  end
+end

data/lib/hungrytable/request_extensions.rb

@@ -0,0 +1,21 @@
+module Hungrytable
+  module RequestExtensions
+    extend ActiveSupport::Concern
+
+    private
+    def request
+      @request ||= @requester.new(request_uri, params)
+    end
+
+    def ensure_required_opts
+      required_opts.each do |key|
+        raise ArgumentError, "options must include a value for #{key}" unless opts.has_key?(key)
+      end
+    end
+
+    # Will be overwritten in objects that send post requests
+    def params
+      {}
+    end
+  end
+end

data/lib/hungrytable/request_header.rb

@@ -0,0 +1,109 @@
+# Modified from simple_oauth (https://github.com/laserlemon/simple_oauth)
+module Hungrytable
+  class RequestHeader
+
+    ATTRIBUTE_KEYS = %w(consumer_key nonce signature_method timestamp token version).map(&:to_sym)
+
+    def self.default_options
+      {
+        :nonce => OpenSSL::Random.random_bytes(16).unpack('H*')[0],
+        :signature_method => 'HMAC-SHA1',
+        :timestamp => Time.now.to_i.to_s,
+        :version => '1.0',
+        :consumer_key => Hungrytable::Config.oauth_key,
+        :consumer_secret => Hungrytable::Config.oauth_secret,
+        :token => ''
+      }
+    end
+
+    def self.encode(value)
+      URI.encode(value.to_s, /[^a-z0-9\-\.\_\~]/i)
+    end
+
+    def self.decode(value)
+      URI.decode(value.to_s)
+    end
+
+    attr_reader :method, :params, :options
+
+    def initialize(method, url, params, oauth = {})
+      @method = method.to_s.upcase
+      @uri = URI.parse(url.to_s)
+      @uri.scheme = @uri.scheme.downcase
+      @uri.normalize!
+      @uri.fragment = nil
+      @params = params
+      @options =  self.class.default_options.merge(oauth) 
+    end
+
+    def url
+      uri = @uri.dup
+      uri.query = nil
+      uri.to_s
+    end
+
+    def to_s
+      %Q(OAuth realm="http://www.opentable.com/", #{normalized_attributes})
+    end
+
+    def valid?(secrets = {})
+      original_options = options.dup
+      options.merge!(secrets)
+      valid = options[:signature] == signature
+      options.replace(original_options)
+      valid
+    end
+
+    def signed_attributes
+      attributes.merge(:oauth_signature => signature)
+    end
+
+    private
+
+    def normalized_attributes
+      signed_attributes.sort_by{|k,v| k.to_s }.map{|k,v| %(#{k}="#{self.class.encode(v)}") }.join(', ')
+    end
+
+    def attributes
+      ATTRIBUTE_KEYS.inject({}){|a,k| options.key?(k) ? a.merge(:"oauth_#{k}" => options[k]) : a }
+    end
+
+    def signature
+      send(options[:signature_method].downcase.tr('-', '_') + '_signature')
+    end
+
+    def hmac_sha1_signature
+      Base64.encode64(OpenSSL::HMAC.digest(OpenSSL::Digest::SHA1.new, secret, signature_base)).chomp.gsub(/\n/, '')
+    end
+
+    def secret
+      options.values_at(:consumer_secret, :token_secret).map{|v| self.class.encode(v) }.join('&')
+    end
+    alias_method :plaintext_signature, :secret
+
+    def signature_base
+      [method, url, normalized_params].map{|v| self.class.encode(v) }.join('&')
+    end
+
+    def normalized_params
+      signature_params.map{|p| p.map{|v| self.class.encode(v) } }.sort.map{|p| p.join('=') }.join('&')
+    end
+
+    def signature_params
+      attributes.to_a + params.to_a + url_params
+    end
+
+    def url_params
+      CGI.parse(@uri.query || '').inject([]){|p,(k,vs)| p + vs.sort.map{|v| [k, v] } }
+    end
+
+    def rsa_sha1_signature
+      Base64.encode64(private_key.sign(OpenSSL::Digest::SHA1.new, signature_base)).chomp.gsub(/\n/, '')
+    end
+
+    def private_key
+      OpenSSL::PKey::RSA.new(options[:consumer_secret])
+    end
+
+  end
+end

data/lib/hungrytable/reservation_cancel.rb

@@ -0,0 +1,30 @@
+module Hungrytable
+  class ReservationCancel
+    include RequestExtensions
+
+    attr_reader :opts
+
+    def initialize opts={}
+      @opts = opts
+      ensure_required_opts
+      @requester = opts[:requester] || GetRequest
+    end
+
+    def successful?
+      details["ns:ErrorID"] == "0"
+    end
+
+    private
+    def request_uri
+      "/reservation/?pid=#{Config.partner_id}&rid=#{opts[:restaurant_id]}&conf=#{opts[:confirmation_number]}&email=#{CGI.escape(opts[:email_address])}"
+    end
+
+    def details
+      request.parsed_response["Results"]
+    end
+
+    def required_opts
+      %w(email_address confirmation_number restaurant_id).map(&:to_sym)
+    end
+  end
+end

data/lib/hungrytable/reservation_make.rb

@@ -0,0 +1,51 @@
+module Hungrytable
+  class ReservationMake
+    include RequestExtensions
+
+    attr_reader :restaurant_slotlock, :opts
+
+    def initialize restaurant_slotlock, opts={}
+      @opts = opts
+      ensure_required_opts
+      @requester           = opts[:requester] || PostRequest
+      @restaurant_slotlock = restaurant_slotlock
+    end
+
+    def successful?
+      details["ns:ErrorID"] == "0"
+    end
+
+    def confirmation_number
+      return nil unless successful?
+      details["ns:ConfirmationNumber"]
+    end
+
+    private
+    def required_opts
+      %w(email_address firstname lastname phone).map(&:to_sym)
+    end
+
+    def default_options
+      {
+        'OTannouncementOption' => '0',
+        'RestaurantEmailOption' => '0',
+        'firsttimediner' => '0',
+        'specialinstructions' => 'Have a great time',
+        'slotlockid' => restaurant_slotlock.slotlock_id
+      }.merge(restaurant_slotlock.params)
+    end
+
+    def params
+      default_options.merge(opts)
+    end
+
+    def request_uri
+      "/reservation/?pid=#{Config.partner_id}&st=0"
+    end
+
+    def details
+      request.parsed_response["MakeResults"]
+    end
+
+  end
+end

data/lib/hungrytable/restaurant.rb

@@ -1,35 +1,58 @@
 module Hungrytable
-  class Restaurant < ::Hungrytable::Base
-    def initialize
-      validate_environment_variables
-
-      @consumer = OAuth::Consumer.new(
-        ENV['OT_OAUTH_KEY'],
-        ENV['OT_OAUTH_SECRET'],
-        realm: 'http://www.opentable.com/',
-        site: 'https://secure.opentable.com/api/otapi_v2.ashx',
-        request_token_path: '',
-        authorize_path: '',
-        access_token_path: '',
-        oauth_token: '',
-        oauth_version: '1.0',
-        http_method: :get
-      )
-      
-      @access_token = OAuth::AccessToken.new(@consumer)
+  class Restaurant
+    include RequestExtensions
+
+    def initialize restaurant_id, opts={}
+      @requester     = opts[:requester] || GetRequest
+      @restaurant_id = restaurant_id
+    end
+
+    def id
+      @restaurant_id
     end
 
-    def get_details(restaurant_id)
-      uri = "/restaurant/?pid=#{ENV['OT_PARTNER_ID']}&rid=#{restaurant_id}"
-      response = @access_token.get(uri)
-      parse_json(response.body)
+    def valid?
+      error_ID == "0"
     end
 
-    def search(options)
-      parse_uri_options(:restaurant_id, :date_time, :party_size, options)
-      uri = "/table/?pid=#{ENV['OT_PARTNER_ID']}&st=0&rid=#{@restaurant_id}&dt=#{@date_time}&ps=#{@party_size}"
-      response = @access_token.get(uri)
-      parse_json(response.body)
+    def method_missing meth, *args, &blk
+      if %w(
+            address
+            city
+            error_ID
+            error_message
+            image_link
+            latitude
+            longitude
+            metro_name
+            neighborhood_name
+            parking
+            parking_details
+            phone
+            postal_code
+            price_range
+            primary_food_type
+            restaurant_description
+            restaurant_ID
+            restaurant_name
+            state
+            url
+          ).map(&:to_sym).include?(meth)
+        return details["ns:#{meth.to_s.camelize}"]
+      end
+      super
     end
+
+    private
+
+    def request_uri
+      "/restaurant/?pid=#{Config.partner_id}&rid=#{id}"
+    end
+
+    # @return [Hash]
+    def details
+      request.parsed_response["RestaurantDetailsResults"]
+    end
+
   end
 end

data/lib/hungrytable/restaurant_search.rb

@@ -0,0 +1,76 @@
+module Hungrytable
+  class RestaurantSearch
+    include RequestExtensions
+
+    attr_reader :restaurant, :opts
+
+    def initialize restaurant, opts={}
+      @opts = opts
+      ensure_required_opts
+      @requester  = opts[:requester] || GetRequest
+      @restaurant = restaurant
+    end
+
+    def valid?
+      error_ID == "0"
+    end
+
+    def method_missing meth, *args, &blk
+      if %w(
+            cuisine_type
+            early_security_ID
+            early_time
+            error_ID
+            error_message
+            exact_security_ID
+            exact_time
+            later_security_ID
+            later_time
+            latitude
+            longitude
+            neighborhood_name
+            restaurant_name
+            results_key
+          ).map(&:to_sym).include?(meth)
+        return details["ns:#{meth.to_s.camelize}"]
+      end
+      super
+    end
+
+    def ideal_security_id
+      return exact_security_ID unless exact_security_ID.nil?
+      return early_security_ID unless early_security_ID.nil?
+      return later_security_ID unless later_security_ID.nil?
+      nil
+    end
+
+    def ideal_time
+      return exact_time unless exact_time.nil?
+      return early_time unless early_time.nil?
+      return later_time unless later_time.nil?
+      nil
+    end
+
+    def party_size
+      opts[:party_size]
+    end
+
+    private
+    def required_opts
+      %w(date_time party_size).map(&:to_sym)
+    end
+
+    def encoded_date_time
+      URI.encode(opts[:date_time].strftime("%m/%d/%Y %I:%M %p"), /[^a-z0-9\-\.\_\~]/i)
+    end
+
+    def request_uri
+      "/table/?pid=#{Config.partner_id}&rid=#{restaurant.id}&dt=#{encoded_date_time}&ps=#{party_size}"
+    end
+
+    def details
+      request.parsed_response["SearchResults"]
+    end
+
+  end
+end

data/lib/hungrytable/restaurant_slotlock.rb

@@ -0,0 +1,45 @@
+module Hungrytable
+  class RestaurantSlotlock
+    include RequestExtensions
+
+    attr_reader :restaurant_search
+
+    def initialize restaurant_search, requester=PostRequest
+      @restaurant_search = restaurant_search
+      @requester         = requester
+    end
+
+    def successful?
+      details["ns:ErrorID"] == "0"
+    end
+
+    def slotlock_id
+      return nil unless successful?
+      details["ns:SlotLockID"]
+    end
+
+    def params
+      {
+        'RID'            => restaurant.id,
+        'datetime'       => restaurant_search.ideal_time,
+        'partysize'      => restaurant_search.party_size,
+        'timesecurityID' => restaurant_search.ideal_security_id,
+        'resultskey'     => restaurant_search.results_key
+      }
+    end
+
+    private
+    def request_uri
+      "/slotlock/?pid=#{Config.partner_id}&st=0"
+    end
+
+    def restaurant
+      restaurant_search.restaurant
+    end
+
+    def details
+      request.parsed_response["SlotLockResults"]
+    end
+
+  end
+end

data/lib/hungrytable/version.rb

@@ -1,3 +1,3 @@
 module Hungrytable
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/hungrytable.rb

@@ -1,9 +1,24 @@
-require "uri"
-require "json"
-require "hungrytable/base"
-require "hungrytable/version"
-require "hungrytable/oauth_patch"
-require "hungrytable/restaurant"
+require 'uri'
+require 'json'
+require 'openssl'
+require 'base64'
+require 'cgi'
+require 'curb'
+require 'active_support/core_ext'
+require 'hungrytable/version'
+require 'hungrytable/config'
+require 'hungrytable/request_extensions'
+require 'hungrytable/request_header'
+require 'hungrytable/request'
+require 'hungrytable/get_request'
+require 'hungrytable/post_request'
+require 'hungrytable/restaurant'
+require 'hungrytable/restaurant_search'
+require 'hungrytable/restaurant_slotlock'
+require 'hungrytable/reservation_make'
+require 'hungrytable/reservation_status'
+require 'hungrytable/reservation_cancel'
 
 module Hungrytable
+  class HungrytableError < StandardError;end
 end

data/test/restaurant_get_details_result.json

@@ -0,0 +1,6 @@
+HTTP/1.1 200 OK
+Status: 200
+Vary: Accept-Encoding
+Content-Type: application/json
+
+{ "RestaurantDetailsResults": {"ns:Address": null, "ns:City": "799 Market Street", "ns:ErrorID": "0", "ns:ErrorMessage": null, "ns:ImageLink": null, "ns:Latitude": null, "ns:Longitude": null, "ns:MenuURL": null, "ns:MetroID": "1", "ns:MetroName": "Demoland", "ns:NeighborhoodName": "Devland", "ns:Parking": "No", "ns:ParkingDetails": null, "ns:Phone": "4157897485", "ns:PostalCode": "94103", "ns:PriceRange": "$30 and under", "ns:PrimaryFoodType": "American", "ns:ReserveUrl": "1759B23C", "ns:RestaurantDescription": "Coming Soon!", "ns:RestaurantID": "82591", "ns:RestaurantName": "OTConnect-Partner-Rest1", "ns:State": "CA", "ns:Url": null, "xmlns:ns": "com.opentable.webservices.v2" }}

data/test/restaurant_search_result.json

@@ -0,0 +1,7 @@
+HTTP/1.1 200 OK
+Status: 200
+Vary: Accept-Encoding
+Content-Type: application/json
+
+{"SearchResults": {"ns:CuisineType": "American", "ns:EarlyPoints": "100", "ns:EarlySecurityID": "1113043844", "ns:EarlyTime": "5/16/2012 1:45:00 PM", "ns:ErrorID": "0", "ns:ErrorMessage": null, "ns:ExactPoints": "100", "ns:ExactSecurityID": "1113043836", "ns:ExactTime": "5/16/2012 2:00:00 PM", "ns:ImageExists": "0", "ns:Latitude": null, "ns:Longitude": null, "ns:NeighborhoodName": "Devland", "ns:Price": "$$", "ns:RestaurantID": "82591", "ns:RestaurantName": "OTConnect-Partner-Rest1", "ns:ResultsKey": "z0OlOTkH2NbY%2bq7bjf7Lxw%3d%3d", "xmlns:ns": "com.opentable.webservices.v2" }}
+

data/test/test_helper.rb

@@ -1,13 +1,11 @@
-# Set up simplecov
-require 'simplecov'
-require 'simplecov-rcov'
-SimpleCov.formatter = SimpleCov::Formatter::RcovFormatter
-
 require 'minitest/autorun'
 require 'minitest/reporters'
 
 require 'ostruct'
 require 'pry'
+require 'webmock'
+include WebMock::API
+
 
 # Load support files
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
@@ -19,5 +17,5 @@ if ENV['JENKINS']
   MiniTest::Unit.runner.reporters << MiniTest::Reporters::JUnitReporter.new
 end
 
-# Load the app
+require 'mocha'
 require File.expand_path("../../lib/hungrytable.rb", __FILE__)

data/test/unit/config_test.rb

@@ -0,0 +1,43 @@
+require 'test_helper'
+
+describe Hungrytable::Config do
+  describe 'when the correct ENV vars are not set' do
+    before do
+      ENV['OT_PARTNER_ID']   = nil
+      ENV['OT_OAUTH_KEY']    = nil
+      ENV['OT_OAUTH_SECRET'] = nil
+    end
+
+    it 'raises a HungrytableError' do
+      [:partner_id, :oauth_key, :oauth_secret].each do |meth|
+        -> {Hungrytable::Config.send meth}.must_raise Hungrytable::HungrytableError
+      end
+    end
+  end
+  
+  describe 'when the ENV vars are set' do
+    before do
+      ENV['OT_PARTNER_ID']   = 'foo'
+      ENV['OT_OAUTH_KEY']    = 'bar'
+      ENV['OT_OAUTH_SECRET'] = 'baz'
+    end
+
+    describe '.partner_id' do
+      it 'returns the value for OT_PARTNER_ID' do
+        Hungrytable::Config.partner_id.must_equal 'foo'
+      end
+    end
+
+    describe '.oauth_key' do
+      it 'returns the value for OT_OAUTH_KEY' do
+        Hungrytable::Config.oauth_key.must_equal 'bar'
+      end
+    end
+
+    describe '.oauth_secret' do
+      it 'returns the value for OT_OAUTH_SECRET' do
+        Hungrytable::Config.oauth_secret.must_equal 'baz'
+      end
+    end
+  end
+end

data/test/unit/hungrytable/restaurant_test.rb

@@ -13,21 +13,29 @@ describe Hungrytable::Restaurant do
     end
   end
 
-  describe 'when getting details' do
+  describe '#get_details' do
     before do
       @restaurant = subject.new
+      stub_request(:get, /.*secure.opentable.com\/api\/otapi_v2.ashx\/restaurant.*/).to_return(File.new('./test/restaurant_get_details_result.json'))
     end
-
+    
     it 'should return relevant details about an individual restaurant' do
       response = @restaurant.get_details(82591)
       response.must_be_kind_of Hash
     end
+  end
 
+  describe '#search' do
+    before do
+      @restaurant = subject.new
+      stub_request(:get, /.*secure.opentable.com\/api\/otapi_v2.ashx\/table.*/).to_return(File.new('./test/restaurant_search_result.json'), :status => 200)
+    end
+    
     it 'should look for availability at an inidividual restaurant' do
       date_time = Time.now.strftime "%-m/#{Time.now.day + 1}/%Y %-l:%M %p"
       response = @restaurant.search({ restaurant_id: 82591, date_time: date_time, party_size: 2 })
       response.must_be_kind_of Hash
     end
-
   end
+
 end

data/test/unit/hungrytable/user_test.rb

@@ -0,0 +1,28 @@
+=begin
+require 'test_helper'
+
+describe Hungrytable::User do
+  subject { Hungrytable::User }
+
+  it 'should be a class' do
+    subject.class.must_be_kind_of Class
+  end
+
+  %w{ login }.each do |meth|
+    it "should respond to #{meth}" do
+      assert subject.new.respond_to?(meth.to_sym), true
+    end
+  end
+
+  describe 'users' do
+    before do
+      stub_request(:get, /.*secure.opentable.com\/api\/otapi_v2.ashx\/user.*/).to_return(File.new('./test/user_login_result.json'), :status => 200)
+      @user = subject.new
+    end
+    it 'should be able to login' do
+      response = @user.login({ email: 'bob@webservices.com', password: 'password' })
+      response.must_be_kind_of Hash
+    end
+  end
+end
+=end

data/test/unit/restaurant_test.rb

@@ -0,0 +1,39 @@
+require 'test_helper'
+require 'pry'
+
+describe Hungrytable::Restaurant do
+  before do
+    @mock_requester = mock('GetRequest')
+    @mock_request   = mock('get')
+    @mock_requester.expects(:new).returns(@mock_request)
+    @mock_request.stubs(:parsed_response).returns({"RestaurantDetailsResults" => {}})
+    ENV['OT_PARTNER_ID']   = 'foo'
+    @restaurant = Hungrytable::Restaurant.new(1, requester: @mock_requester)
+  end
+  %w(
+      address
+      city
+      error_ID
+      error_message
+      image_link
+      latitude
+      longitude
+      metro_name
+      neighborhood_name
+      parking
+      parking_details
+      phone
+      postal_code
+      price_range
+      primary_food_type
+      restaurant_description
+      restaurant_ID
+      restaurant_name
+      state
+      url
+    ).map(&:to_sym).each do |meth|
+      it "responds to #{meth.to_s} via method_missing" do
+        @restaurant.send(meth).must_equal nil
+      end
+    end
+end

data/test/user_login_result.json

@@ -0,0 +1,6 @@
+HTTP/1.1 200 OK
+Status: 200
+Vary: Accept-Encoding
+Content-Type: application/json
+
+{ "LoginResults": {"ns:ErrorID": "303", "ns:ErrorMessage": "Invalid User Email. Please re-enter your UserEmail.", "ns:IsAdmin": "0", "xmlns:ns": "com.opentable.webservices.v2" }}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hungrytable
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,55 +9,75 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-11 00:00:00.000000000Z
+date: 2012-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: oauth
-  requirement: &6926620 !ruby/object:Gem::Requirement
+  name: json
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.4.5
+        version: 1.7.1
   type: :runtime
   prerelease: false
-  version_requirements: *6926620
-- !ruby/object:Gem::Dependency
-  name: json
-  requirement: &6925700 !ruby/object:Gem::Requirement
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.7.1
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *6925700
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
-  requirement: &6924760 !ruby/object:Gem::Requirement
+  name: i18n
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
+  type: :runtime
   prerelease: false
-  version_requirements: *6924760
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest-reporters
-  requirement: &6923840 !ruby/object:Gem::Requirement
+  name: curb
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
+  type: :runtime
   prerelease: false
-  version_requirements: *6923840
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &6922980 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +85,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6922980
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
-  requirement: &6922200 !ruby/object:Gem::Requirement
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +101,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6922200
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov-rcov
-  requirement: &6921360 !ruby/object:Gem::Requirement
+  name: minitest-reporters
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,21 +117,31 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6921360
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rcov
-  requirement: &6854660 !ruby/object:Gem::Requirement
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - =
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 0.9.11
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6854660
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: &6852260 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,7 +149,28 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6852260
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Gem to interact with OpenTable's REST API
 email:
 - david@isotope11.com
@@ -120,20 +181,41 @@ files:
 - .gitignore
 - .rvmrc
 - Gemfile
+- Guardfile
 - LICENSE
 - README.md
 - RELEASE_NOTES.md
 - Rakefile
 - hungrytable.gemspec
 - lib/hungrytable.rb
-- lib/hungrytable/base.rb
-- lib/hungrytable/oauth_patch.rb
+- lib/hungrytable/config.rb
+- lib/hungrytable/get_request.rb
+- lib/hungrytable/post_request.rb
+- lib/hungrytable/request.rb
+- lib/hungrytable/request_extensions.rb
+- lib/hungrytable/request_header.rb
+- lib/hungrytable/reservation_cancel.rb
+- lib/hungrytable/reservation_make.rb
+- lib/hungrytable/reservation_status.rb
 - lib/hungrytable/restaurant.rb
+- lib/hungrytable/restaurant_search.rb
+- lib/hungrytable/restaurant_slotlock.rb
 - lib/hungrytable/version.rb
-- test/hungrytable_test.rb
-- test/support/minitest_junit.rb
+- test/restaurant_get_details_result.json
+- test/restaurant_search_result.json
 - test/test_helper.rb
+- test/unit/config_test.rb
+- test/unit/get_request_test.rb
 - test/unit/hungrytable/restaurant_test.rb
+- test/unit/hungrytable/user_test.rb
+- test/unit/post_request_test.rb
+- test/unit/request_test.rb
+- test/unit/reservation_cancel_test.rb
+- test/unit/reservation_make_test.rb
+- test/unit/restaurant_search_test.rb
+- test/unit/restaurant_slotlock_test.rb
+- test/unit/restaurant_test.rb
+- test/user_login_result.json
 homepage: http://www.isotope11.com/
 licenses: []
 post_install_message: 
@@ -146,20 +228,22 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -1516024462257141481
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -1516024462257141481
 requirements: []
 rubyforge_project: hungrytable
-rubygems_version: 1.8.10
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Gem to interact with OpenTable's REST API
-test_files:
-- test/hungrytable_test.rb
-- test/support/minitest_junit.rb
-- test/test_helper.rb
-- test/unit/hungrytable/restaurant_test.rb
+test_files: []

data/lib/hungrytable/base.rb

@@ -1,26 +0,0 @@
-module Hungrytable
-  class Base
-    def validate_environment_variables
-      raise "Please set the environment variable OT_OAUTH_KEY to your OpenTable OAuth Table." unless ENV['OT_OAUTH_KEY']
-      raise "Please set the environment variable OT_OAUTH_SECRET to your OpenTable OAuth Secret Key." unless ENV['OT_OAUTH_SECRET']
-      raise "Please set the environment variable OT_PARTNER_ID to your OpentTable Partner ID." unless ENV['OT_PARTNER_ID']
-    end
-
-    def parse_uri_option(option_name, options)
-      val = options[option_name.to_sym]
-      raise "Missing required option: #{option_name}" if val.nil?
-      if val.is_a?(String)
-        val = CGI.escape(val).gsub(/[+]/, '%20')
-      end
-      instance_variable_set("@#{option_name}", val)
-    end
-
-    def parse_uri_options(*option_names, options)
-      option_names.each { |option_name| parse_uri_option(option_name, options) }
-    end
-
-    def parse_json(string)
-      JSON.parse(string)
-    end
-  end
-end

data/lib/hungrytable/oauth_patch.rb

@@ -1,19 +0,0 @@
-require 'oauth'
-module OAuth::Client
-  class Helper
-    def oauth_parameters
-      {   
-        'oauth_body_hash'        => options[:body_hash],
-        'oauth_callback'         => options[:oauth_callback],
-        'oauth_consumer_key'     => options[:consumer].key,
-        'oauth_token'            => options[:token] ? options[:token].token : '', 
-        'oauth_signature_method' => options[:signature_method],
-        'oauth_timestamp'        => timestamp,
-        'oauth_nonce'            => nonce,
-        'oauth_verifier'         => options[:oauth_verifier],
-        'oauth_version'          => (options[:oauth_version] || '1.0'),
-        'oauth_session_handle'   => options[:oauth_session_handle]
-      }.reject { |k,v| v.to_s == "" && k.to_s != 'oauth_token' }
-    end 
-  end 
-end

data/test/hungrytable_test.rb

@@ -1,8 +0,0 @@
-require 'test_helper'
-
-describe Hungrytable do
-  subject { Hungrytable }
-  it 'should be a class' do
-    subject.class.must_equal Module
-  end
-end

data/test/support/minitest_junit.rb

@@ -1,81 +0,0 @@
-require 'ansi'
-
-module MiniTest
-  module Reporters
-    # A reporter identical to the standard MiniTest reporter.
-    # 
-    # Based upon Ryan Davis of Seattle.rb's MiniTest (MIT License).
-    # 
-    # @see https://github.com/seattlerb/minitest MiniTest
-    class JUnitReporter
-      include MiniTest::Reporter
-
-      def before_suites(suites, type)
-      end
-
-      def before_test(suite, test)
-      end
-
-      def pass(suite, test, test_runner)
-      end
-
-      def skip(suite, test, test_runner)
-      end
-
-      def failure(suite, test, test_runner)
-      end
-
-      def error(suite, test, test_runner)
-      end
-
-      def after_suites(suites, type)
-        time = Time.now - runner.start_time
-        i = 0
-        runner.report.each do |suite, tests|
-          File.open("#{suite.to_s.gsub(" ", '_').downcase}.xml", 'w') do |out|
-            out.puts '<?xml version="1.0" encoding="UTF-8" ?>'
-            out.puts "<testsuite errors=\"#{runner.errors}\" failures=\"#{runner.failures}\" tests=\"#{runner.test_count}\" time=\"#{time}\" timestamp=\"#{runner.start_time.strftime('%FT%R')}\">"
-            tests.each do |test, test_runner|
-              out.print "<testcase classname=\"#{suite}\" name=\"#{test}\" time=\"0\">"
-              message = message_for(test_runner)
-              out.puts "\n#{message}" if message
-              out.puts '</testcase>'
-            end
-            out.puts '</testsuite>'
-          end
-        end
-      end
-
-      private
-      def location(exception)
-        last_before_assertion = ''
-
-        exception.backtrace.reverse_each do |s|
-          break if s =~ /in .(assert|refute|flunk|pass|fail|raise|must|wont)/
-          last_before_assertion = s
-        end
-
-        last_before_assertion.sub(/:in .*$/, '')
-      end
-
-      def message_for(test_runner)
-        suite = test_runner.suite
-        test = test_runner.test
-        e = test_runner.exception
-
-        case test_runner.result
-        when :pass then nil
-        when :skip then "<skipped message=\"\"/>"
-        when :failure then "<failure message=\"#{e.message}\" type=\"failure\">#{location(e)}:\n#{e.message}\n</failure>"
-        when :error then "<error message=\"#{e.message}\" type=\"error\">#{location(e)}:\n#{e.message}\n</error>"
-        end
-      end
-
-      def status
-        '%d tests, %d assertions, %d failures, %d errors, %d skips' %
-          [runner.test_count, runner.assertion_count, runner.failures, runner.errors, runner.skips]
-      end
-    end
-  end
-end
-