bing 0.1.1 → 0.2.0

data/History.txt

@@ -1,3 +1,16 @@
+=== 0.2.0 / 2011-08-05
+* 4 minor enhancement(s):
+
+  * Added ability for user to specify api version.
+  * Namespacing better, no more mixins, and added rest resource superclass.
+  * Only add core extensions if they are not already defined.
+  * Refined the rest resource and added more coverage.
+
+* 2 unknown:
+
+  * Added tests for bing rest resource and various comments.
+  * Namespaced request object in favor of mixin.
+
 === 0.1.1 / 2011-07-25
 * 5 unknown:
 

data/Manifest.txt

@@ -7,15 +7,16 @@ Rakefile
 lib/bing.rb
 lib/bing/core_ext.rb
 lib/bing/errors.rb
-lib/bing/formatting_helper.rb
 lib/bing/location.rb
 lib/bing/request.rb
+lib/bing/rest_resource.rb
 lib/bing/route.rb
 lib/bing/route/itinerary.rb
 test/helper.rb
 test/test_bing.rb
 test/test_bing_location.rb
+test/test_bing_request.rb
+test/test_bing_rest_resource.rb
 test/test_bing_route.rb
 test/test_bing_route_itinerary.rb
 test/test_core_ext.rb
-test/test_request.rb

data/lib/bing/core_ext.rb

@@ -1,46 +1,59 @@
-# core extensions to ruby
-class Object
+# core extensions to ruby.
 
-  def blank?
-    respond_to?(:empty?) ? empty? : !self
+unless respond_to? :blank?
+  class Object
+    def blank?
+      respond_to?(:empty?) ? empty? : !self
+    end
   end
+end
+
 
-  def to_param
-    to_s
+unless respond_to? :to_param
+  class Object
+    def to_param
+      to_s
+    end
   end
+end
 
-  def to_query key
-    require 'cgi' unless defined?(CGI) && defined?(CGI::escape)
-    _key = CGI.escape(key.to_s.camelize(:lower)).gsub(/%(5B|5D)/n) {
-             [$1].pack('H*')
-           }
-    value = CGI.escape(to_param.to_s)
 
-    "#{_key}=#{value}"
-  end
+unless respond_to? :to_lower_camelized_query
+  class Object
+    def to_lower_camelized_query key
+      require 'cgi' unless defined?(CGI) && defined?(CGI::escape)
+      _key = CGI.escape(key.to_s.camelize(:lower)).gsub(/%(5B|5D)/n) {
+               [$1].pack('H*')
+             }
+      value = CGI.escape(to_param.to_s)
 
+      "#{_key}=#{value}"
+    end
+  end
 end
 
-class Hash
 
-  def to_param namespace = nil
-    collect do |key, value|
-      value.to_query(namespace ? "#{namespace}[#{key}]" : key)
-    end.sort * '&'
+unless {}.respond_to? :to_lower_camelized_param
+  class Hash
+    def to_lower_camelized_param namespace = nil
+      collect do |key, value|
+        value.to_lower_camelized_query(namespace ? "#{namespace}[#{key}]" : key)
+      end.sort * '&'
+    end
   end
-
 end
 
-class String
 
-  def camelize first_letter = :upper
-    case first_letter
-      when :upper then
-        self.gsub(/\/(.?)/) {"::#{$1.upcase}"}.gsub(/(?:^|_)(.)/) {$1.upcase}
-      when :lower then
-        self[0].chr.downcase + camelize()[1..-1]
+unless ''.respond_to? :camelize
+  class String
+    def camelize first_letter = :upper
+      case first_letter
+        when :upper then
+          self.gsub(/\/(.?)/) {"::#{$1.upcase}"}.gsub(/(?:^|_)(.)/) {$1.upcase}
+        when :lower then
+          self[0].chr.downcase + camelize()[1..-1]
+      end
     end
   end
-
 end
 

data/lib/bing/errors.rb

@@ -1,55 +1,58 @@
-##
-# DO NOT use these classes. They are too generic.
+module Bing
 
-class ServiceError < StandardError
-  def status() 500 end
-end
+  ##
+  # DO NOT use these classes. They are too generic.
 
-class ResourceMissing < StandardError
-  def status() 502 end
+  class ServiceError < StandardError
+    def status() 500 end
+  end
 
-  def message() 'Resource is empty or nil.' end
-end
+  class ResourceMissing < StandardError
+    def status() 502 end
 
-##
-# Raised when we get an invalid response from an underlying server
+    def message() 'Resource is empty or nil.' end
+  end
 
-class BadGateway < ServiceError
+  ##
+  # Raised when we get an invalid response from an underlying server
 
-  def self.parse_error text, uri = nil
-    error = new "parse error#{uri ? " from #{uri}" : nil} - #{text.inspect}"
-    error.text = text
-    error.uri = uri
-    error
-  end
+  class BadGateway < ServiceError
 
-  def self.bad_response code, uri, server_message=nil
-    server_message = " with message #{server_message}" if server_message
-    error = new "#{uri} returned #{code}#{server_message}"
-    error.code = code
-    error.uri = uri
-    error
-  end
+    def self.parse_error text, uri = nil
+      error = new "parse error#{uri ? " from #{uri}" : nil} - #{text.inspect}"
+      error.text = text
+      error.uri = uri
+      error
+    end
 
-  ##
-  # Status code from underlying server
+    def self.bad_response code, uri, server_message=nil
+      server_message = " with message #{server_message}" if server_message
+      error = new "#{uri} returned #{code}#{server_message}"
+      error.code = code
+      error.uri = uri
+      error
+    end
 
-  attr_accessor :code
+    ##
+    # Status code from underlying server
 
-  ##
-  # Text we were unable to parse
+    attr_accessor :code
 
-  attr_accessor :text
+    ##
+    # Text we were unable to parse
 
-  ##
-  # URI for this request
+    attr_accessor :text
 
-  attr_accessor :uri
+    ##
+    # URI for this request
 
-  def status() 502 end
-end
+    attr_accessor :uri
 
-class ItineraryResourceMissing < ResourceMissing; end
-class LocationResourceMissing < ResourceMissing; end
-class RouteResourceMissing < ResourceMissing; end
+    def status() 502 end
+  end
+
+  class ItineraryResourceMissing < ResourceMissing; end
+  class LocationResourceMissing < ResourceMissing; end
+  class RouteResourceMissing < ResourceMissing; end
 
+end

data/lib/bing/location.rb

@@ -1,36 +1,30 @@
-require 'bing'
+require 'bing/rest_resource'
 
 ##
 # Responsible for obtaining a location based off of data passed in.
-# Most common method is to do:
-#
-# Bing::Location.find :query => '123 Address City State'
 
-class Bing::Location
-
-  ##
-  # Path to location resource.
-
-  PATH = '/REST/v1/Locations'
+class Bing::Location < Bing::RestResource
 
   ##
   # Will find locations based off of +query+ and return an array of
-  # Bing::Location objects.
+  # Bing::Location objects. Will support any param that bing supports.
+  #
+  # === Example
+  #
+  # Bing::Location.find :query => '123 Address City State'
 
   def self.find opts
-    uri = Bing.config[:map_uri].merge(
-            "#{PATH}?key=#{Bing.config[:api_key]}&#{opts.to_param}"
-          )
+    map_find opts.to_lower_camelized_param
+  end
 
-    body = JSON.parse get(uri).body
+  ##
+  # Path to route resource.
 
-    body['resourceSets'].first['resources'].map do |resource|
-      new resource
-    end.compact
+  def self.path
+    super '/Locations'
   end
 
   attr_reader :address
-  attr_reader :bounding_box
   attr_reader :canonical_description
   attr_reader :coordinates
   attr_reader :city
@@ -42,7 +36,7 @@ class Bing::Location
   attr_reader :zip
 
   def initialize resource
-    raise LocationResourceMissing if resource.blank?
+    raise Bing::LocationResourceMissing if resource.blank?
 
     @confidence  = resource['confidence']
     @entity_type = resource['entityType']

data/lib/bing/request.rb

@@ -1,7 +1,7 @@
 ##
 # Responsible for making requests to Bing. Uses persistent HTTP connections.
 
-module Bing::Request
+class Bing::Request
 
   HTTP = Net::HTTP::Persistent.new
   USER_AGENT = "Bing Client Version: #{Bing::VERSION}"
@@ -10,10 +10,10 @@ module Bing::Request
   ##
   # Perform a get request and ensure that the response.code == 20\d,
   # otherwise raise a BadGateway.
-  def get uri
+  def self.get uri
     response = HTTP.request uri
 
-    raise BadGateway.bad_response(response.code, uri) unless
+    raise Bing::BadGateway.bad_response(response.code, uri) unless
       response.code =~ /20\d/
 
     response

data/lib/bing/rest_resource.rb

@@ -0,0 +1,51 @@
+require 'bing'
+
+class Bing::RestResource
+
+  ##
+  # Base Bing Rest route.
+
+  BASE_PATH = "/REST/#{Bing.config[:api_version]}"
+
+  def self._find uri
+    body = JSON.parse Bing::Request.get(uri).body
+
+    body['resourceSets'].first['resources'].map do |resource|
+      new resource
+    end.compact
+  end
+
+  def self.map_uri params
+    Bing.config[:map_uri].merge(
+      "#{path}?key=#{Bing.config[:api_key]}&#{params}"
+    )
+  end
+
+  def self.map_find params
+    _find map_uri params
+  end
+
+  def self.path subclass_path = nil
+    "#{BASE_PATH}#{subclass_path}"
+  end
+
+  ##
+  # The map's bounding box.
+
+  attr_reader :bounding_box
+
+  ##
+  # Decipher bounding box from bbox in Bing response.
+
+  def bbox box
+    south, west, north, east = *box
+    {
+      :north => north,
+      :east  => east,
+      :south => south,
+      :west  => west,
+    }
+  end
+
+end
+

data/lib/bing/route/itinerary.rb

@@ -8,7 +8,7 @@ class Bing::Route::Itinerary
   attr_reader :travel_mode
 
   def initialize resource
-    raise ItineraryResourceMissing if resource.blank?
+    raise Bing::ItineraryResourceMissing if resource.blank?
 
     @distance    = resource['travelDistance']
     @duration    = resource['travelDuration']

data/lib/bing/route.rb

@@ -1,11 +1,6 @@
-require 'bing'
+require 'bing/rest_resource'
 
-class Bing::Route
-
-  ##
-  # Path to route resource.
-
-  PATH = '/REST/v1/Routes'
+class Bing::Route < Bing::RestResource
 
   ##
   # === Description
@@ -35,19 +30,18 @@ class Bing::Route
 
   def self.find opts
     waypoints = format_waypoints opts.delete :waypoints
+    params    = [opts.to_lower_camelized_param, waypoints].join '&'
 
-    uri = Bing.config[:map_uri].merge(
-            "#{PATH}?key=#{Bing.config[:api_key]}&#{opts.to_param}&#{waypoints}"
-          )
+    map_find params
+  end
 
-    body = JSON.parse get(uri).body
+  ##
+  # Path to route resource.
 
-    body['resourceSets'].first['resources'].map do |resource|
-      new resource
-    end.compact
+  def self.path
+    super '/Routes'
   end
 
-  attr_reader :bounding_box
   attr_reader :distance_unit
   attr_reader :duration_unit
   attr_reader :ending_coordinates
@@ -58,7 +52,7 @@ class Bing::Route
   attr_reader :type
 
   def initialize resource
-    raise RouteResourceMissing if resource.blank?
+    raise Bing::RouteResourceMissing if resource.blank?
 
     @distance_unit  = resource['distanceUnit']
     @duration_unit  = resource['durationUnit']

data/lib/bing.rb

@@ -5,12 +5,13 @@ require 'json'
 
 module Bing
 
-  VERSION = '0.1.1'
+  VERSION = '0.2.0'
 
   DEFAULTS = {
-    :api_key => 'AtsQ7PXwSqL266EUdxMYj3b4-H5A6ubkf8DwH-B4k3rVmmPycUrhmH-lZKHeWXm-',
-    :api_uri => URI.parse('http://api.bing.net'),
-    :map_uri => URI.parse('http://dev.virtualearth.net'),
+    :api_key     => 'AtsQ7PXwSqL266EUdxMYj3b4-H5A6ubkf8DwH-B4k3rVmmPycUrhmH-lZKHeWXm-',
+    :api_uri     => URI.parse('http://api.bing.net'),
+    :api_version => 'v1',
+    :map_uri     => URI.parse('http://dev.virtualearth.net'),
   }
 
   class << self
@@ -30,9 +31,5 @@ end
 
 require 'bing/core_ext'
 require 'bing/errors'
-require 'bing/formatting_helper'
 require 'bing/request'
 
-include Bing::FormattingHelper
-# TODO figure out how to allow user defined request object.
-include Bing::Request

data/test/test_bing_location.rb

@@ -2,6 +2,8 @@ require 'helper'
 
 class TestBingLocation < MiniTest::Unit::TestCase
 
+  BL = Bing::Location
+
   def test_cls_find_successful
     body = {
       'resourceSets' => [
@@ -11,21 +13,25 @@ class TestBingLocation < MiniTest::Unit::TestCase
       ]
     }.to_json
 
-    mock_map_request 200, '/REST/v1/Locations', body
+    mock_map_request 200, BL.path, body
 
-    locs = Bing::Location.find :query => '123'
+    locs = BL.find :query => '123'
 
     assert_equal 'full name', locs.first.full_name
   end
 
   def test_cls_find_failure
-    mock_map_request 400, '/REST/v1/Locations', '{}'
+    mock_map_request 400, BL.path, '{}'
 
-    assert_raises BadGateway do
-      Bing::Location.find :query => '123'
+    assert_raises Bing::BadGateway do
+      BL.find :query => '123'
     end
   end
 
+  def test_cls_path
+    assert_match %r[Locations], BL.path
+  end
+
   def test_initialize_with_coordinates
     resource = {
       'point' => {
@@ -33,7 +39,7 @@ class TestBingLocation < MiniTest::Unit::TestCase
       }
     }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal [123, 456], bl.coordinates
   end
@@ -49,7 +55,7 @@ class TestBingLocation < MiniTest::Unit::TestCase
       }
     }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal 'address', bl.address
     assert_equal 'city', bl.city
@@ -61,7 +67,7 @@ class TestBingLocation < MiniTest::Unit::TestCase
   def test_initialize_with_confidence
     resource = { 'confidence' => 'High' }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal 'High', bl.confidence
   end
@@ -69,7 +75,7 @@ class TestBingLocation < MiniTest::Unit::TestCase
   def test_initialize_with_bbox
     resource = { 'bbox' => %w[south west north east] }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal 'north', bl.bounding_box[:north]
     assert_equal 'east',  bl.bounding_box[:east]
@@ -80,7 +86,7 @@ class TestBingLocation < MiniTest::Unit::TestCase
   def test_initialize_with_entity_type
     resource = { 'entityType' => 'Postal' }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal 'Postal', bl.entity_type
   end
@@ -88,14 +94,14 @@ class TestBingLocation < MiniTest::Unit::TestCase
   def test_initialize_with_full_name
     resource = { 'name' => '123 street, ca' }
 
-    bl = Bing::Location.new resource
+    bl = BL.new resource
 
     assert_equal '123 street, ca', bl.full_name
   end
 
   def test_initialize_without_resource_raises
-    assert_raises LocationResourceMissing do
-      Bing::Location.new nil
+    assert_raises Bing::LocationResourceMissing do
+      BL.new nil
     end
   end
 

data/test/{test_request.rb → test_bing_request.rb}

@@ -9,15 +9,15 @@ class TestRequest < MiniTest::Unit::TestCase
   def test_get_failure
     stub_request(:any, "http://example.com").to_return(:status => 500)
 
-    assert_raises BadGateway do
-      get @uri
+    assert_raises Bing::BadGateway do
+      Bing::Request.get @uri
     end
   end
 
   def test_get_success
     stub_request(:any, "http://example.com").to_return(:status => 200)
 
-    response = get @uri
+    response = Bing::Request.get @uri
 
     assert_equal '200', response.code
   end

data/test/test_bing_rest_resource.rb

@@ -0,0 +1,66 @@
+require 'helper'
+
+class Bing::RestResource
+  def initialize *args
+  end
+end
+
+class TestBingRestResource < MiniTest::Unit::TestCase
+
+  BRR = Bing::RestResource # so cold
+
+  def test_base_path
+    assert_equal '/REST/v1', BRR::BASE_PATH
+  end
+
+  def test_cls__find
+    stub_request(:any, /.*/).to_return(:status => 200, :body => body)
+
+    assert_resource BRR._find URI.parse 'http://example.com'
+  end
+
+  def test_map_uri
+    assert_match %r[.*virtualearth.*REST.*key.*dunkey],
+      BRR.map_uri('dunkey').to_s
+  end
+
+  def test_map_find
+    mock_map_request 200, BRR.path, body
+
+    assert_resource BRR.map_find 'dunkey'
+  end
+
+  def test_path
+    assert_equal "#{BRR::BASE_PATH}", BRR.path
+    assert_equal "#{BRR::BASE_PATH}/chunky", BRR.path('/chunky')
+  end
+
+  def test_bbox
+    box  = %w[south west north east]
+    bbox = BRR.new.bbox box
+
+    assert_equal 'north', bbox[:north]
+    assert_equal 'east',  bbox[:east]
+    assert_equal 'south', bbox[:south]
+    assert_equal 'west',  bbox[:west]
+  end
+
+  private
+
+  def assert_resource found
+    assert_kind_of Array, found
+    assert_equal BRR.new.class, found.first.class
+  end
+
+  def body
+    {
+      'resourceSets' => [
+        'resources' => [
+          {'name' => 'full name'}
+        ]
+      ]
+    }.to_json
+  end
+
+end
+

data/test/test_bing_route.rb

@@ -8,6 +8,8 @@ end
 
 class TestBingRoute < MiniTest::Unit::TestCase
 
+  BR = Bing::Route
+
   def test_cls_find
     body = {
       'resourceSets' => [
@@ -17,29 +19,33 @@ class TestBingRoute < MiniTest::Unit::TestCase
       ]
     }.to_json
 
-    mock_map_request 200, '/REST/v1/Routes', body
+    mock_map_request 200, BR.path, body
 
-    route = Bing::Route.find :waypoints => ['start', 'end']
+    route = BR.find :waypoints => ['start', 'end']
   end
 
   def test_cls_format_waypoints
     waypoints = ['start', 'end']
 
     assert_equal 'waypoint.0=start&waypoint.1=end',
-      Bing::Route.format_waypoints(waypoints)
+      BR.format_waypoints(waypoints)
 
     waypoints = ["4.9, -1.2", "1.2, 2.2"]
 
     assert_equal 'waypoint.0=4.9%2C+-1.2&waypoint.1=1.2%2C+2.2',
-      Bing::Route.format_waypoints(waypoints)
+      BR.format_waypoints(waypoints)
+
+    assert_equal nil, BR.format_waypoints(nil)
+  end
 
-    assert_equal nil, Bing::Route.format_waypoints(nil)
+  def test_cls_path
+    assert_match %r[Route], BR.path
   end
 
   def test_initialize_with_bbox
     resource = { 'bbox' => %w[south west north east] }
 
-    bl = Bing::Route.new resource
+    bl = BR.new resource
 
     assert_equal 'north', bl.bounding_box[:north]
     assert_equal 'east',  bl.bounding_box[:east]
@@ -50,7 +56,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
   def test_initialize_with_distance_unit
     resource = { 'distanceUnit' => 'miles' }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 'miles', br.distance_unit
   end
@@ -58,7 +64,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
   def test_initialize_with_duration_unit
     resource = { 'durationUnit' => 'sec' }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 'sec', br.duration_unit
   end
@@ -70,7 +76,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
       }]
     }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal [1,2], br.ending_coordinates
   end
@@ -85,7 +91,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
       }]
     }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 1, br.itinerary.first.distance
     assert_equal 2, br.itinerary.last.distance
@@ -98,7 +104,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
       }]
     }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal [1,2], br.starting_coordinates
   end
@@ -106,7 +112,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
   def test_initialize_with_total_distance
     resource = { 'travelDistance' => 117.406223 }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 117.406223, br.total_distance
   end
@@ -114,7 +120,7 @@ class TestBingRoute < MiniTest::Unit::TestCase
   def test_initialize_with_total_duration
     resource = { 'travelDuration' => 11723 }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 11723, br.total_duration
   end
@@ -126,14 +132,14 @@ class TestBingRoute < MiniTest::Unit::TestCase
       }]
     }
 
-    br = Bing::Route.new resource
+    br = BR.new resource
 
     assert_equal 'type', br.type
   end
 
   def test_initialize_without_resource_raises
-    assert_raises RouteResourceMissing do
-      Bing::Route.new nil
+    assert_raises Bing::RouteResourceMissing do
+      BR.new nil
     end
   end
 

data/test/test_bing_route_itinerary.rb

@@ -51,7 +51,7 @@ class TestBingRouteItinerary < MiniTest::Unit::TestCase
   end
 
   def test_initialize_without_resource_raises
-    assert_raises ItineraryResourceMissing do
+    assert_raises Bing::ItineraryResourceMissing do
       Bing::Route::Itinerary.new nil
     end
   end

data/test/test_core_ext.rb

@@ -10,10 +10,10 @@ class TestCoreExt < MiniTest::Unit::TestCase
 
   def test_to_param
     query = {:hi_mom => true}
-    assert_equal 'hiMom=true', query.to_param
+    assert_equal 'hiMom=true', query.to_lower_camelized_param
 
     query[:yep] = false
-    assert_equal 'hiMom=true&yep=false', query.to_param
+    assert_equal 'hiMom=true&yep=false', query.to_lower_camelized_param
   end
 
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: bing
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: 
   segments: 
   - 0
-  - 1
-  - 1
-  version: 0.1.1
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
 authors: 
 - Adam Avilla
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-03 00:00:00 Z
+date: 2011-08-05 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: net-http-persistent
@@ -140,18 +140,19 @@ files:
 - lib/bing.rb
 - lib/bing/core_ext.rb
 - lib/bing/errors.rb
-- lib/bing/formatting_helper.rb
 - lib/bing/location.rb
 - lib/bing/request.rb
+- lib/bing/rest_resource.rb
 - lib/bing/route.rb
 - lib/bing/route/itinerary.rb
 - test/helper.rb
 - test/test_bing.rb
 - test/test_bing_location.rb
+- test/test_bing_request.rb
+- test/test_bing_rest_resource.rb
 - test/test_bing_route.rb
 - test/test_bing_route_itinerary.rb
 - test/test_core_ext.rb
-- test/test_request.rb
 - .gemtest
 homepage: https://github.com/hekaldama/bing
 licenses: []
@@ -190,7 +191,8 @@ summary: Bing api client library that exposes all of Bing's api features.
 test_files: 
 - test/test_bing.rb
 - test/test_bing_location.rb
+- test/test_bing_rest_resource.rb
 - test/test_core_ext.rb
 - test/test_bing_route.rb
 - test/test_bing_route_itinerary.rb
-- test/test_request.rb
+- test/test_bing_request.rb

data/lib/bing/formatting_helper.rb

@@ -1,21 +0,0 @@
-##
-# Responsible for consolidating response formatting logic shared 
-# amongst resources.
-
-module Bing::FormattingHelper
-
-  ##
-  # Decipher bounding box from bbox in Bing response.
-
-  def bbox box
-    south, west, north, east = *box
-    {
-      :north => north,
-      :east  => east,
-      :south => south,
-      :west  => west,
-    }
-  end
-
-end
-