seatgeek 0.1.1 → 1.0.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: 7ea02bcbb451f64c78d3298bc0d9a580b80f0f4229955ec3b8e1a92248492580
+  data.tar.gz: 164188c7662ebabfb0f3719fd923ed8e6cf31afd0bc4898b9f6bef6c00b393d6
+SHA512:
+  metadata.gz: 4d7670113e4596d936c93ff9e2e619770f5398354cf133f61a187e310131961d154700fd183770674456b0fa48ea6a5e69cbbd0595c6a8ecf34948ba6b332eb4
+  data.tar.gz: b382ea251fc3baa64fd0b66446922a1dcae28651b7355dec2b9c594bc35be5bcb8d097398ab1bbe54625acb8938e07d25b8e2f44a0b83828e44d2d6a20890a77

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,14 @@
-bundler_args: --without no_travis
 language: ruby
+dist: xenial
+os: linux
+
 rvm:
-  - ree
-  - 1.8.7
-  - 1.9.2
-gemfile:
-  - Gemfile
+  - 2.5.3
+
+before_install:
+  - gem install bundler
+
+install: bundle install --retry=3
+
+notifications:
+  email: false

data/Gemfile

@@ -1,17 +1,16 @@
 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', '~> 13.0.1'
+gem 'yajl-ruby', '~> 1.3.1'
 
 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.8.0'
 end
 
 gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    seatgeek (0.1.1)
+    seatgeek (1.0.2)
       faraday
       multi_json
       rake
@@ -10,46 +10,39 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    archive-tar-minitar (0.5.2)
-    awesome_print (1.0.2)
-    columnize (0.3.6)
-    diff-lcs (1.1.3)
-    faraday (0.8.0)
-      multipart-post (~> 1.1)
-    linecache19 (0.5.12)
-      ruby_core_source (>= 0.1.4)
-    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)
-    ruby-debug-base19 (0.11.25)
-      columnize (>= 0.3.1)
-      linecache19 (>= 0.5.11)
-      ruby_core_source (>= 0.1.4)
-    ruby-debug19 (0.11.6)
-      columnize (>= 0.3.1)
-      linecache19 (>= 0.5.11)
-      ruby-debug-base19 (>= 0.11.19)
-    ruby_core_source (0.1.5)
-      archive-tar-minitar (>= 0.5.2)
-    yajl-ruby (1.1.0)
+    awesome_print (1.8.0)
+    diff-lcs (1.4.4)
+    faraday (0.8.11)
+      multipart-post (~> 1.2.0)
+    multi_json (1.3.7)
+    multipart-post (1.2.0)
+    rake (13.0.1)
+    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.3.1)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  awesome_print
-  faraday
-  multi_json
-  rake
-  rspec
-  ruby-debug19
+  awesome_print (~> 1.8.0)
+  faraday (~> 0.8.11)
+  multi_json (~> 1.3.5)
+  rake (~> 13.0.1)
+  rspec (~> 3.4.0)
   seatgeek!
-  yajl-ruby
+  yajl-ruby (~> 1.3.1)
+
+BUNDLED WITH
+   2.1.4

data/{README.markdown → README.md}

@@ -1,4 +1,4 @@
-SeatGeek [![Build Status](https://secure.travis-ci.org/bluefocus/seatgeek.png)](http://travis-ci.org/bluefocus/seatgeek)
+SeatGeek [![Build Status](https://secure.travis-ci.org/seatshare/seatgeek.png)](http://travis-ci.org/seatshare/seatgeek)
 ========
 SeatGeek maintains a canonical directory of live events in the United States. The SeatGeek Platform makes it easy to construct applications that help users find and discover events by exposing the wealth of data and powerful search behind SeatGeek.com in an easy-to-consume format.
 
@@ -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
@@ -57,12 +60,12 @@ SeatGeek::Connection objects can be instantiated before calls. Any parameters pa
 
 There are a number of class level options that can be overridden.
 
- - SeatGeek::Connection.adapter - the Faraday adapter to use (default: :net_http)
- - SeatGeek::Connection.logger - a Logger object for logging requests (default: nil)
- - SeatGeek::Connection.protocol - http or https (default: :http)
- - SeatGeek::Connection.response_format - ruby, json, jsonp or xml (default: ruby)
- - SeatGeek::Connection.url - api.seatgeek::Connection.com [Advanced tuning]
- - SeatGeek::Connection.version - 2 [Forward thinking]
+    SeatGeek::Connection.adapter          # the Faraday adapter to use (default: :net_http)
+    SeatGeek::Connection.logger           # a Logger object for logging requests (default: nil)
+    SeatGeek::Connection.protocol         # http or https (default: :http)
+    SeatGeek::Connection.response_format  # ruby, json, jsonp or xml (default: ruby)
+    SeatGeek::Connection.url              # api.seatgeek.com [Advanced tuning]
+    SeatGeek::Connection.version          # 2 [Forward thinking]
 
 Example:
 
@@ -105,4 +108,3 @@ Permission is hereby granted, free of charge, to any person obtaining a copy of
 The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
 
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-

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 ||= :http; 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 << "/events"
+          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.1"
+  VERSION = '1.0.2'
 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,280 +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 => :http,
-        :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) { :http }
+    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\":\"http://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\":\"http://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\":\"http://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\":\"http://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\":\"http://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\":\"http://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\":\"http://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\":\"http://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
-    let(:url) { 'http://api.seatgeek.com/2/events' }
+  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)
-        instance.request(url, params)
+      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)
-        instance.request(url, params)
+      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)
-        instance.request(url, params)
+      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 == "http://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,71 +1,85 @@
 --- !ruby/object:Gem::Specification
 name: seatgeek
 version: !ruby/object:Gem::Version
-  version: 0.1.1
-  prerelease: 
+  version: 1.0.2
 platform: ruby
 authors:
 - Dan Matthews
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2012-05-15 00:00:00.000000000Z
+date: 2020-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70345881699200 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70345881699200
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &70345881697820 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70345881697820
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70345881696440 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70345881696440
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: yajl-ruby
-  requirement: &70345881695420 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70345881695420
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70345881693420 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70345881693420
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: This gem provides Ruby functionality around the SeatGeek Platform API
   (http://platform.seatgeek.com). It is designed to be framework agnostic and was
   originally developed for use in my day job at Ticket Evolution.
@@ -75,54 +89,48 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rdebugrc
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".rdebugrc"
+- ".rspec"
+- ".travis.yml"
 - Gemfile
 - Gemfile.lock
-- README.markdown
+- README.md
 - Rakefile
 - lib/faraday/response/verbose_logger.rb
 - lib/seat_geek/connection.rb
 - 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
 - spec/spec_helper.rb
 homepage: http://platform.seatgeek.com
 licenses: []
-post_install_message: 
+metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 2766271802599134346
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 2766271802599134346
 requirements: []
-rubyforge_project: seatgeek
-rubygems_version: 1.8.6
-signing_key: 
-specification_version: 3
+rubygems_version: 3.1.2
+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