seatgeek 0.1.3 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0417f52b7a1ee59a497f30206bc19bf2aa09df1e
-  data.tar.gz: d08f67022f031051b7d1f875ff1e299d16c433d0
+  metadata.gz: 4fff0f590450a4239ba544957b7015ef1e7a5a5e
+  data.tar.gz: dd74c176439117ac90b7e54d71eff24ff3df1b3c
 SHA512:
-  metadata.gz: 5a7c01abfa900e5ca64a52c363517a15c3be9b2eef708f6201269c1fc1afe3b8f7b64dc0a3bee8d88ee8d2918f73f6ca595e33d91937849c8b4c4b438d82e664
-  data.tar.gz: 1f73bbb1e4f7dacd26c2b35d47358b1a8129223b897706cbc31ce33f1d91201ccfefa8ccc2b078511b37e9bd52ee8d04bc7d88eaea1b799907a8fd6e466c21b9
+  metadata.gz: ee80b3beca64bd2804568275f0664083157429923832700b0328682ac0473f864e3da469fe1f521591162c9fb69ab18172500c4f9c0a35951f4a899dd5b8a751
+  data.tar.gz: 4314cf7470e706942425395ab5996269555a248f27e08860fcec58db6c306a383afe952fd0d4b4b8980ec205f1e42eabf20921c63de674f7fa230054ec3c4eb3

data/.gitignore

@@ -1,5 +1,33 @@
 .DS_Store
-*.swp
-*.swo
-pkg
+
+*.gem
+*.rbc
+/.config
+/coverage/
+/InstalledFiles
+/pkg/
+/spec/reports/
+/spec/examples.txt
+/test/tmp/
+/test/version_tmp/
+/tmp/
+
+## Documentation cache and generated files:
+/.yardoc/
+/_yardoc/
+/doc/
+/rdoc/
+
+## Environment normalization:
+/.bundle/
+/vendor/bundle
+/lib/bundler/man/
+
+# for a library or gem, you might want to ignore these files since the code is
+# intended to run in multiple environments; otherwise, check them in:
+# Gemfile.lock
+# .ruby-version
+# .ruby-gemset
+
+# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
 .rvmrc

data/.travis.yml

@@ -1,8 +1,7 @@
 bundler_args: --without no_travis
 language: ruby
 rvm:
-  - ree
-  - 1.8.7
   - 1.9.2
+  - 2.1.0
 gemfile:
   - Gemfile

data/Gemfile

@@ -1,17 +1,17 @@
 source 'https://rubygems.org'
 
-gem 'faraday'
-gem 'multi_json'
-gem 'rake'
-gem 'yajl-ruby'
+gem 'faraday', '~> 0.8.11'
+gem 'multi_json', '~> 1.3.5'
+gem 'rake', '~> 0.9.2'
+gem 'yajl-ruby', '~> 1.1.0'
 
 group :test do
-  gem 'rspec'
+  gem 'rspec', '~> 3.4.0'
 end
 
 group :no_travis do
-  gem 'awesome_print'
-  RUBY_VERSION =~ /^1\.9/ ? gem('ruby-debug19') : gem('ruby-debug')
+  gem 'awesome_print', '~> 1.6.1'
+  gem 'debugger', '~> 1.6.8'
 end
 
 gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    seatgeek (0.1.2)
+    seatgeek (0.1.3)
       faraday
       multi_json
       rake
@@ -10,29 +10,47 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    diff-lcs (1.1.3)
-    faraday (0.8.0)
-      multipart-post (~> 1.1)
-    multi_json (1.3.5)
-    multipart-post (1.1.5)
-    rake (0.9.2.2)
-    rspec (2.10.0)
-      rspec-core (~> 2.10.0)
-      rspec-expectations (~> 2.10.0)
-      rspec-mocks (~> 2.10.0)
-    rspec-core (2.10.0)
-    rspec-expectations (2.10.0)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.10.1)
+    awesome_print (1.6.1)
+    columnize (0.9.0)
+    debugger (1.6.8)
+      columnize (>= 0.3.1)
+      debugger-linecache (~> 1.2.0)
+      debugger-ruby_core_source (~> 1.3.5)
+    debugger-linecache (1.2.0)
+    debugger-ruby_core_source (1.3.8)
+    diff-lcs (1.2.5)
+    faraday (0.8.11)
+      multipart-post (~> 1.2.0)
+    multi_json (1.3.7)
+    multipart-post (1.2.0)
+    rake (0.9.6)
+    rspec (3.4.0)
+      rspec-core (~> 3.4.0)
+      rspec-expectations (~> 3.4.0)
+      rspec-mocks (~> 3.4.0)
+    rspec-core (3.4.4)
+      rspec-support (~> 3.4.0)
+    rspec-expectations (3.4.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-mocks (3.4.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-support (3.4.1)
     yajl-ruby (1.1.0)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  faraday
-  multi_json
-  rake
-  rspec
+  awesome_print (~> 1.6.1)
+  debugger (~> 1.6.8)
+  faraday (~> 0.8.11)
+  multi_json (~> 1.3.5)
+  rake (~> 0.9.2)
+  rspec (~> 3.4.0)
   seatgeek!
-  yajl-ruby
+  yajl-ruby (~> 1.1.0)
+
+BUNDLED WITH
+   1.11.2

data/README.md

@@ -9,8 +9,7 @@ For more information, including a detailed description of the functionality avai
 **Rubies supported**
 
 - 1.9.2
-- 1.8.7
-- ree
+- 2.1.0
 
 Installation
 ------------
@@ -32,6 +31,10 @@ NOTE: Unless it is already defined in your application, SG will be defined as a
 
 SeatGeek::Connection is where all the meat is.
 
+You will first need to provide the `client_id` for your application. For assistance registering and locating your `client_id`, please see [the platform documentation](http://platform.seatgeek.com/#authentication).
+
+    SeatGeek::Connection.client_id = 'some_client_id'
+
 Each of the four endpoints can be accessed with a class method reflecting the endpoint's name.
 
     SeatGeek::Connection.events

data/lib/seat_geek/connection.rb

@@ -1,44 +1,95 @@
 require 'faraday'
 
+##
+# Module: SeatGeek
 module SeatGeek
+  ##
+  # Class: Connection
   class Connection
     class << self; self; end.class_eval do
-      def adapter; @@adapter ||= :net_http; end
-      def adapter=(input); @@adapter = input; end
+      def adapter
+        @@adapter ||= :net_http
+      end
+
+      def adapter=(input)
+        @@adapter = input
+      end
+
+      def events(*args)
+        new.events(*args)
+      end
 
-      def events(*args); new.events(*args); end
+      def logger
+        @@logger ||= nil
+      end
 
-      def logger; @@logger ||= nil; end
-      def logger=(input); @@logger = input; end
+      def logger=(input)
+        @@logger = input
+      end
 
       def options
         {
-          :adapter => adapter,
-          :logger => logger,
-          :protocol => protocol,
-          :response_format => response_format,
-          :url => url,
-          :version => version
+          adapter: adapter,
+          logger: logger,
+          protocol: protocol,
+          response_format: response_format,
+          url: url,
+          version: version,
+          client_id: client_id
         }
       end
 
-      def performers(*args); new.performers(*args); end
+      def performers(*args)
+        new.performers(*args)
+      end
+
+      def protocol
+        @@protocol ||= :https
+      end
 
-      def protocol; @@protocol ||= :https; end
-      def protocol=(input); @@protocol = input; end
+      def protocol=(input)
+        @@protocol = input
+      end
 
-      def response_format; @@response_format ||= :ruby; end
-      def response_format=(input); @@response_format = input; end
+      def response_format
+        @@response_format ||= :ruby
+      end
 
-      def taxonomies(*args); new.taxonomies(*args); end
+      def response_format=(input)
+        @@response_format = input
+      end
 
-      def url; @@url ||= "api.seatgeek.com"; end
-      def url=(input); @@url = input; end
+      def taxonomies(*args)
+        new.taxonomies(*args)
+      end
 
-      def venues(*args); new.venues(*args); end
+      def url
+        @@url ||= 'api.seatgeek.com'
+      end
 
-      def version; @@version ||= 2; end
-      def version=(input); @@version = input; end
+      def url=(input)
+        @@url = input
+      end
+
+      def venues(*args)
+        new.venues(*args)
+      end
+
+      def version
+        @@version ||= 2
+      end
+
+      def version=(input)
+        @@version = input
+      end
+
+      def client_id
+        @@client_id ||= nil
+      end
+
+      def client_id=(input)
+        @@client_id = input
+      end
     end
 
     def initialize(options = {})
@@ -54,26 +105,25 @@ module SeatGeek
     end
 
     def handle_response(response)
-      if response_format == :ruby and response.status == 200
+      if response_format == :ruby && response.status == 200
         MultiJson.decode(response.body)
       else
-        {:status => response.status, :body => response.body}
+        { status: response.status, body: response.body }
       end
     end
 
-    # Ruby 1.8.7 / ree compatibility
-    def id
-      @options[:id]
-    end
-
     def performers(params = {})
       request('/performers', params)
     end
 
     def request(url, params)
+      raise 'You must provide a `client_id` for SeatGeek' unless client_id || params[:client_id]
       handle_response(Faraday.new(*builder(url, params.clone)) do |build|
-        build.adapter adapter
+        if client_id
+          build.use Faraday::Request::BasicAuthentication, client_id, nil
+        end
         build.use Faraday::Response::VerboseLogger, logger unless logger.nil?
+        build.adapter adapter
       end.get)
     end
 
@@ -100,15 +150,14 @@ module SeatGeek
     end
 
     def builder(uri_segment, params)
-      return [
+      [
         uri([].tap do |part|
           part << uri_segment
           part << "/#{params.delete(:id)}" unless params[:id].nil?
         end.join),
         {
-          :params => custom_options.merge((
-            [:jsonp, :xml].include?(response_format) ? \
-              params.merge(:format => response_format) : params))
+          params: custom_options.merge((
+            [:jsonp, :xml].include?(response_format) ? params.merge(format: response_format) : params))
         }
       ]
     end

data/lib/seat_geek/version.rb

@@ -1,3 +1,5 @@
+##
+# Module: SeatGeek
 module SeatGeek
-  VERSION = "0.1.3"
+  VERSION = '1.0.0'
 end

data/spec/fixtures/handle_response.json

@@ -0,0 +1 @@
+{"meta":{"per_page":1,"total":63188,"page":1,"took":4,"geolocation":null},"events":[{"stats":{"listing_count":0,"average_price":0,"lowest_price":null,"highest_price":null},"relative_url":"/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/","title":"Palm Beach Cardinals at Jupiter Hammerheads","url":"https://seatgeek.com/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/","datetime_local":"2012-05-14T10:35:00","performers":[{"away_team":true,"name":"Palm Beach Cardinals","url":"https://seatgeek.com/palm-beach-cardinals-tickets/","image":null,"short_name":"Palm Beach Cardinals","slug":"palm-beach-cardinals","score":0,"images":[],"type":"minor_league_baseball","id":9420},{"home_team":true,"name":"Jupiter Hammerheads","url":"https://seatgeek.com/jupiter-hammerheads-tickets/","image":null,"short_name":"Jupiter Hammerheads","primary":true,"slug":"jupiter-hammerheads","score":0,"images":[],"type":"minor_league_baseball","id":9421}],"venue":{"city":"Jupiter","name":"Roger Dean Stadium","url":"https://seatgeek.com/roger-dean-stadium-tickets/","country":"US","state":"FL","score":16592,"postal_code":"33468","location":{"lat":26.8936,"lon":-80.1156},"extended_address":null,"address":"4751 Main St","id":3927},"short_title":"Palm Beach Cardinals at Jupiter Hammerheads","datetime_utc":"2012-05-14T14:35:00","score":0,"taxonomies":[{"parent_id":null,"id":1000000,"name":"sports"},{"parent_id":1000000,"id":1010000,"name":"baseball"},{"parent_id":1010000,"id":1010300,"name":"minor_league_baseball"}],"type":"minor_league_baseball","id":798459}]}

data/spec/lib/seat_geek/connection_spec.rb

@@ -3,281 +3,339 @@ require 'ostruct'
 
 describe SeatGeek::Connection do
   let(:klass) { SeatGeek::Connection }
-  let(:instance) { klass.new }
+  let(:instance) { klass.new(client_id: 'foobar') }
 
-  describe ".adapter" do
+  describe '.adapter' do
     subject { klass.adapter }
     let(:default) { :net_http }
 
     it { should == default }
-    it "should be writable" do
+    it 'should be writable' do
       klass.adapter = :typhoeus
-      klass.adapter.should == :typhoeus
+      expect(klass.adapter).to eq(:typhoeus)
     end
 
     after { klass.adapter = default }
   end
 
-  describe ".events" do
-    it "proxies to #events" do
-      klass.any_instance.should_receive(:events).and_return(nil)
+  describe '.events' do
+    it 'proxies to #events' do
+      expect_any_instance_of(klass).to receive(:events).and_return(nil)
       klass.events
     end
 
-    it "passes any attributes to #events" do
-      klass.any_instance.should_receive(:events).with(1, 2).and_return(nil)
+    it 'passes any attributes to #events' do
+      expect_any_instance_of(klass).to receive(:events).with(1, 2)
+        .and_return(nil)
       klass.events(1, 2)
     end
   end
 
-  describe ".logger" do
+  describe '.logger' do
     subject { klass.logger }
     let(:default) { nil }
     let(:logger) { Logger.new(STDOUT) }
 
     it { should == default }
-    it "should be writable" do
+    it 'should be writable' do
       klass.logger = logger
-      klass.logger.should == logger
+      expect(klass.logger).to eq(logger)
     end
 
     after { klass.logger = default }
   end
 
-  describe ".options" do
-    it "should return a hash of all of the class level settings" do
-      klass.options.should == {
-        :adapter => :net_http,
-        :logger => nil,
-        :protocol => :https,
-        :response_format => :ruby,
-        :url => "api.seatgeek.com",
-        :version => 2
-      }
+  describe '.options' do
+    it 'should return a hash of all of the class level settings' do
+      expect(klass.options).to eq(
+        adapter: :net_http,
+        logger: nil,
+        protocol: :https,
+        response_format: :ruby,
+        url: 'api.seatgeek.com',
+        version: 2,
+        client_id: nil
+      )
     end
   end
 
-  describe ".performers" do
-    it "proxies to #performers" do
-      klass.any_instance.should_receive(:performers).and_return(nil)
+  describe '.performers' do
+    it 'proxies to #performers' do
+      expect_any_instance_of(klass).to receive(:performers).and_return(nil)
       klass.performers
     end
 
-    it "passes any attributes to #performers" do
-      klass.any_instance.should_receive(:performers).with(1, 2).and_return(nil)
+    it 'passes any attributes to #performers' do
+      expect_any_instance_of(klass).to receive(:performers).with(1, 2)
+        .and_return(nil)
       klass.performers(1, 2)
     end
   end
 
-  describe ".protocol" do
+  describe '.protocol' do
     subject { klass.protocol }
     let(:default) { :https }
 
     it { should == default }
-    it "should be writable" do
+    it 'should be writable' do
       klass.protocol = :https
-      klass.protocol.should == :https
+      expect(klass.protocol).to eq(:https)
     end
 
     after { klass.protocol = default }
   end
 
-  describe ".response_format" do
+  describe '.client_id' do
+    subject { klass.client_id }
+    let(:default) { nil }
+
+    it { should == default }
+    it 'should be writable' do
+      klass.client_id = 'some_client_id'
+      expect(klass.client_id).to eq('some_client_id')
+    end
+
+    after { klass.client_id = default }
+  end
+
+  describe '.response_format' do
     subject { klass.response_format }
     let(:default) { :ruby }
 
     it { should == default }
-    it "should be writable" do
+    it 'should be writable' do
       klass.response_format = :jsonp
-      klass.response_format.should == :jsonp
+      expect(klass.response_format).to eq(:jsonp)
     end
 
     after { klass.response_format = default }
   end
 
-  describe ".taxonomies" do
-    it "proxies to #taxonomies" do
-      klass.any_instance.should_receive(:taxonomies).and_return(nil)
+  describe '.taxonomies' do
+    it 'proxies to #taxonomies' do
+      expect_any_instance_of(klass).to receive(:taxonomies).and_return(nil)
       klass.taxonomies
     end
 
-    it "passes any attributes to #taxonomies" do
-      klass.any_instance.should_receive(:taxonomies).with(1, 2).and_return(nil)
+    it 'passes any attributes to #taxonomies' do
+      expect_any_instance_of(klass).to receive(:taxonomies).with(1, 2)
+        .and_return(nil)
       klass.taxonomies(1, 2)
     end
   end
 
-  describe ".url" do
+  describe '.url' do
     subject { klass.url }
-    let(:default) { "api.seatgeek.com" }
+    let(:default) { 'api.seatgeek.com' }
 
     it { should == default }
-    it "should be writable" do
-      klass.url = "ticketevolution.com"
-      klass.url.should == "ticketevolution.com"
+    it 'should be writable' do
+      klass.url = 'ticketevolution.com'
+      expect(klass.url).to eq('ticketevolution.com')
     end
 
     after { klass.url = default }
   end
 
-  describe ".venues" do
-    it "proxies to #venues" do
-      klass.any_instance.should_receive(:venues).and_return(nil)
+  describe '.venues' do
+    it 'proxies to #venues' do
+      expect_any_instance_of(klass).to receive(:venues).and_return(nil)
       klass.venues
     end
 
-    it "passes any attributes to #venues" do
-      klass.any_instance.should_receive(:venues).with(1, 2).and_return(nil)
+    it 'passes any attributes to #venues' do
+      expect_any_instance_of(klass).to receive(:venues).with(1, 2)
+        .and_return(nil)
       klass.venues(1, 2)
     end
   end
 
-  describe ".version" do
+  describe '.version' do
     subject { klass.version }
     let(:default) { 2 }
 
     it { should == 2 }
-    it "should be writable" do
+    it 'should be writable' do
       klass.version = 1
-      klass.version.should == 1
+      expect(klass.version).to eq(1)
     end
 
     after { klass.version = default }
   end
 
-  describe "#initialize" do
+  describe '#initialize' do
     let(:expected) { klass.options.merge(options) }
-    let(:options) { { :testing => 123, :clone => :hello } }
-    let(:str_options) { {}.tap{|o| options.each{|k, v| o[k.to_s] = v}} }
+    let(:options) { { testing: 123, clone: 'hello', client_id: 'some_data' } }
+    let(:str_options) { {}.tap { |o| options.each { |k, v| o[k.to_s] = v } } }
     let(:instance) { klass.new(options) }
 
-    it "should take options passed and store them on top of the class options hash" do
-      instance.instance_eval('@options').should == expected
+    it 'should take options passed and store '\
+      'them on top of the class options hash' do
+      expect(instance.instance_eval('@options')).to eq(expected)
     end
 
-    it "should convert the keys passed to symbols" do
-      klass.new(str_options).instance_eval('@options').should == expected
+    it 'should convert the keys passed to symbols' do
+      expect(klass.new(str_options).instance_eval('@options')).to eq(expected)
     end
 
-    it "should serve options up via read accessor methods unless the methods already exist" do
-      instance.testing.should == options[:testing]
-      instance.clone.should_not == options[:clone]
+    it 'should serve options up via read accessor '\
+      'methods unless the methods already exist' do
+      expect(instance.testing).to eq(options[:testing])
+      expect(instance.clone).not_to eq(options[:clone])
     end
   end
 
-  describe "#events" do
-    let(:url) { '/events'}
-    let(:params) { {:test => 123} }
+  describe '#events' do
+    let(:url) { '/events' }
+    let(:params) { { test: 123 } }
 
-    it "should call #request with the correct url segment and the params passed" do
-      instance.should_receive(:request).with(url, params)
+    it 'should call #request with the correct'\
+      'url segment and the params passed' do
+      expect(instance).to receive(:request).with(url, params)
       instance.events(params)
     end
   end
 
-  describe "#handle_response" do
-    let(:response) { double(Faraday::Response, :status => status, :body => body)}
+  describe '#handle_response' do
+    let(:response) { double(Faraday::Response, status: status, body: body) }
 
-    context "when response_format is set to ruby" do
-      context "and the request was successful" do
+    context 'when response_format is set to ruby' do
+      context 'and the request was successful' do
         let(:status) { 200 }
-        let(:body) { "{\"meta\":{\"per_page\":1,\"total\":63188,\"page\":1,\"took\":4,\"geolocation\":null},\"events\":[{\"stats\":{\"listing_count\":0,\"average_price\":0,\"lowest_price\":null,\"highest_price\":null},\"relative_url\":\"/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/\",\"title\":\"Palm Beach Cardinals at Jupiter Hammerheads\",\"url\":\"https://seatgeek.com/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/\",\"datetime_local\":\"2012-05-14T10:35:00\",\"performers\":[{\"away_team\":true,\"name\":\"Palm Beach Cardinals\",\"url\":\"https://seatgeek.com/palm-beach-cardinals-tickets/\",\"image\":null,\"short_name\":\"Palm Beach Cardinals\",\"slug\":\"palm-beach-cardinals\",\"score\":0,\"images\":[],\"type\":\"minor_league_baseball\",\"id\":9420},{\"home_team\":true,\"name\":\"Jupiter Hammerheads\",\"url\":\"https://seatgeek.com/jupiter-hammerheads-tickets/\",\"image\":null,\"short_name\":\"Jupiter Hammerheads\",\"primary\":true,\"slug\":\"jupiter-hammerheads\",\"score\":0,\"images\":[],\"type\":\"minor_league_baseball\",\"id\":9421}],\"venue\":{\"city\":\"Jupiter\",\"name\":\"Roger Dean Stadium\",\"url\":\"https://seatgeek.com/roger-dean-stadium-tickets/\",\"country\":\"US\",\"state\":\"FL\",\"score\":16592,\"postal_code\":\"33468\",\"location\":{\"lat\":26.8936,\"lon\":-80.1156},\"extended_address\":null,\"address\":\"4751 Main St\",\"id\":3927},\"short_title\":\"Palm Beach Cardinals at Jupiter Hammerheads\",\"datetime_utc\":\"2012-05-14T14:35:00\",\"score\":0,\"taxonomies\":[{\"parent_id\":null,\"id\":1000000,\"name\":\"sports\"},{\"parent_id\":1000000,\"id\":1010000,\"name\":\"baseball\"},{\"parent_id\":1010000,\"id\":1010300,\"name\":\"minor_league_baseball\"}],\"type\":\"minor_league_baseball\",\"id\":798459}]}" }
+        let(:body) { File.read('spec/fixtures/handle_response.json') }
 
-        it "should parse the json returned and respond with a ruby object" do
-          instance.handle_response(response).should == MultiJson.decode(body)
+        it 'should parse the json returned and respond with a ruby object' do
+          expect(instance.handle_response(response))
+            .to eq(MultiJson.decode(body))
         end
       end
 
-      context "and the request was not successful" do
+      context 'and the request was not successful' do
         let(:status) { 500 }
-        let(:body) { "Internal Server Error" }
+        let(:body) { 'Internal Server Error' }
 
-        it "should return the status code and exact result body" do
-          instance.handle_response(response).should == {:status => status, :body => body}
+        it 'should return the status code and exact result body' do
+          expect(instance.handle_response(response))
+            .to eq(status: status, body: body)
         end
       end
     end
 
-    context "when response format is not set to ruby" do
+    context 'when response format is not set to ruby' do
       let(:status) { 200 }
-      let(:body) { "{\"meta\":{\"per_page\":1,\"total\":63188,\"page\":1,\"took\":4,\"geolocation\":null},\"events\":[{\"stats\":{\"listing_count\":0,\"average_price\":0,\"lowest_price\":null,\"highest_price\":null},\"relative_url\":\"/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/\",\"title\":\"Palm Beach Cardinals at Jupiter Hammerheads\",\"url\":\"https://seatgeek.com/palm-beach-cardinals-at-jupiter-hammerheads-tickets/minor-league-baseball/2012-05-14/798459/\",\"datetime_local\":\"2012-05-14T10:35:00\",\"performers\":[{\"away_team\":true,\"name\":\"Palm Beach Cardinals\",\"url\":\"https://seatgeek.com/palm-beach-cardinals-tickets/\",\"image\":null,\"short_name\":\"Palm Beach Cardinals\",\"slug\":\"palm-beach-cardinals\",\"score\":0,\"images\":[],\"type\":\"minor_league_baseball\",\"id\":9420},{\"home_team\":true,\"name\":\"Jupiter Hammerheads\",\"url\":\"https://seatgeek.com/jupiter-hammerheads-tickets/\",\"image\":null,\"short_name\":\"Jupiter Hammerheads\",\"primary\":true,\"slug\":\"jupiter-hammerheads\",\"score\":0,\"images\":[],\"type\":\"minor_league_baseball\",\"id\":9421}],\"venue\":{\"city\":\"Jupiter\",\"name\":\"Roger Dean Stadium\",\"url\":\"https://seatgeek.com/roger-dean-stadium-tickets/\",\"country\":\"US\",\"state\":\"FL\",\"score\":16592,\"postal_code\":\"33468\",\"location\":{\"lat\":26.8936,\"lon\":-80.1156},\"extended_address\":null,\"address\":\"4751 Main St\",\"id\":3927},\"short_title\":\"Palm Beach Cardinals at Jupiter Hammerheads\",\"datetime_utc\":\"2012-05-14T14:35:00\",\"score\":0,\"taxonomies\":[{\"parent_id\":null,\"id\":1000000,\"name\":\"sports\"},{\"parent_id\":1000000,\"id\":1010000,\"name\":\"baseball\"},{\"parent_id\":1010000,\"id\":1010300,\"name\":\"minor_league_baseball\"}],\"type\":\"minor_league_baseball\",\"id\":798459}]}" }
+      let(:body) { File.read('spec/fixtures/handle_response.json') }
 
-      it "should return the status code and exact result body" do
-        klass.new(:response_format => :json).\
-          handle_response(response).should == {:status => status, :body => body}
+      it 'should return the status code and exact result body' do
+        expect(klass.new(response_format: :json).handle_response(response))
+          .to eq(status: status, body: body)
       end
     end
   end
 
-  describe "#performers" do
-    let(:url) { '/performers'}
-    let(:params) { {:test => 123} }
+  describe '#performers' do
+    let(:url) { '/performers' }
+    let(:params) { { test: 123 } }
 
-    it "should call #request with the correct url segment and the params passed" do
-      instance.should_receive(:request).with(url, params)
+    it 'should call #request with the correct '\
+      'url segment and the params passed' do
+      expect(instance).to receive(:request).with(url, params)
       instance.performers(params)
     end
   end
 
-  describe "#request" do
+  describe '#request' do
     let(:uri_segment) { '/venues' }
     let(:url) { "https://api.seatgeek.com/2#{uri_segment}" }
     let(:params) { {} }
-    let(:faraday) { mock(:faraday, :get => OpenStruct.new({:status => 200, :body => "[]"})) }
+    let(:faraday) { double(:faraday, get: OpenStruct.new(status: 200, body: '[]')) }
+
+    context 'when .client_id is not set as class property' do
+      let(:instance) { klass.new }
+      let(:expected_params) { { params: params.merge(format: :jsonp) } }
+
+      it 'should throw an exception' do
+        expect { instance.request(uri_segment, params) }
+          .to raise_error('You must provide a `client_id` for SeatGeek')
+      end
+
+      it 'should not throw an exception if provided in paramters' do
+        expect {
+          params[:client_id] = 'foobar'
+          instance.request(uri_segment, params)
+        }.to_not raise_error
+      end
+    end
+
+    context 'when .response_format is jsonp' do
+      let(:instance) { klass.new(response_format: :jsonp, client_id: 'foobar') }
+      let(:expected_params) { { params: params.merge(format: :jsonp) } }
+
+      it 'should set the format param to jsonp' do
+        expect(Faraday).to receive(:new).with(url, expected_params)
+          .and_return(faraday)
+        instance.request(uri_segment, params)
+      end
+    end
 
-    context "when .response_format is jsonp" do
-      let(:instance) { klass.new({:response_format => :jsonp}) }
-      let(:expected_params) { {:params => params.merge({:format => :jsonp})} }
+    context 'when .response_format is xml' do
+      let(:instance) { klass.new(response_format: :xml, client_id: 'foobar') }
+      let(:expected_params) { { params: params.merge(format: :xml) } }
 
-      it "should set the format param to jsonp" do
-        Faraday.should_receive(:new).with(url, expected_params).and_return(faraday)
+      it 'should set the format param to xml' do
+        expect(Faraday).to receive(:new).with(url, expected_params)
+          .and_return(faraday)
         instance.request(uri_segment, params)
       end
     end
 
-    context "when .response_format is xml" do
-      let(:instance) { klass.new({:response_format => :xml}) }
-      let(:expected_params) { {:params => params.merge({:format => :xml})} }
+    context 'when additional parameters were passed to #initialize' do
+      let(:instance) { klass.new(response_format: :jsonp, testing: 123, client_id: 'foobar') }
+      let(:expected_params) { { params: params.merge(format: :jsonp, testing: 123)} }
 
-      it "should set the format param to xml" do
-        Faraday.should_receive(:new).with(url, expected_params).and_return(faraday)
+      it 'should add those parameters to the request params' do
+        expect(Faraday).to receive(:new).with(url, expected_params)
+          .and_return(faraday)
         instance.request(uri_segment, params)
       end
     end
 
-    context "when additional parameters were passed to #initialize" do
-      let(:instance) { klass.new({:response_format => :jsonp, :testing => 123}) }
-      let(:expected_params) { {:params => params.merge({:format => :jsonp, :testing => 123})} }
+    context 'when client_id was passed as a parameter' do
+      let(:instance) { klass.new }
+      let(:params) { { client_id: 'foobar' } }
+      let(:expected_params) { { params: params.merge(client_id: 'foobar') } }
 
-      it "should add those parameters to the request params" do
-        Faraday.should_receive(:new).with(url, expected_params).and_return(faraday)
+      it 'should make the request with the client_id parameter set' do
+        expect(Faraday).to receive(:new).with(url, expected_params)
+          .and_return(faraday)
         instance.request(uri_segment, params)
       end
     end
   end
 
-  describe "#taxonomies" do
-    let(:url) { '/taxonomies'}
-    let(:params) { {:test => 123} }
+  describe '#taxonomies' do
+    let(:url) { '/taxonomies' }
+    let(:params) { { test: 123 } }
 
-    it "should call #request with the correct url segment and the params passed" do
-      instance.should_receive(:request).with(url, params)
+    it 'should call #request with the correct '\
+      'url segment and the params passed' do
+      expect(instance).to receive(:request).with(url, params)
       instance.taxonomies(params)
     end
   end
 
-  describe "#uri" do
-    it "should take in a path and combine it with protocol, url and version" do
-      instance.uri('/events').should == "https://api.seatgeek.com/2/events"
+  describe '#uri' do
+    it 'should take in a path and combine it with protocol, url and version' do
+      expect(instance.uri('/events')).to eq('https://api.seatgeek.com/2/events')
     end
   end
 
-  describe "#venues" do
-    let(:url) { '/venues'}
-    let(:params) { {:test => 123} }
+  describe '#venues' do
+    let(:url) { '/venues' }
+    let(:params) { { test: 123 } }
 
-    it "should call #request with the correct url segment and the params passed" do
-      instance.should_receive(:request).with(url, params)
+    it 'should call #request with the correct '\
+      'url segment and the params passed' do
+      expect(instance).to receive(:request).with(url, params)
       instance.venues(params)
     end
   end

data/spec/lib/seat_geek/version_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe SeatGeek::VERSION do
-  it "should match a three segment version number format" do
-    subject.should =~ /^\d+\.\d+\.\d+(rc\d+)?$/
+  it 'should match a three segment version number format' do
+    expect(subject).to match(/^\d+\.\d+\.\d+(rc\d+)?$/)
   end
 end

data/spec/lib/sg_spec.rb

@@ -2,6 +2,6 @@ require 'spec_helper'
 
 describe SG do
   it "should be a subclass of SeatGeek::Connection" do
-    SG.ancestors.should include SeatGeek::Connection
+    expect(SG.ancestors).to include(SeatGeek::Connection)
   end
 end

data/spec/spec_helper.rb

@@ -1,7 +1,6 @@
 require 'seatgeek'
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
   config.filter_run :focus
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: seatgeek
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 1.0.0
 platform: ruby
 authors:
 - Dan Matthews
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-19 00:00:00.000000000 Z
+date: 2016-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -103,6 +103,7 @@ files:
 - lib/seat_geek/version.rb
 - lib/seatgeek.rb
 - seatgeek.gemspec
+- spec/fixtures/handle_response.json
 - spec/lib/seat_geek/connection_spec.rb
 - spec/lib/seat_geek/version_spec.rb
 - spec/lib/sg_spec.rb
@@ -131,6 +132,7 @@ signing_key:
 specification_version: 4
 summary: A Ruby wrapper for the SeatGeek Platform API.
 test_files:
+- spec/fixtures/handle_response.json
 - spec/lib/seat_geek/connection_spec.rb
 - spec/lib/seat_geek/version_spec.rb
 - spec/lib/sg_spec.rb