tvrage_api 0.0.2 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 113bc0f11f6442f79fc786189fefb52e634c4367
-  data.tar.gz: 440b7e8e641d79ac98f4a6fd850e142e4d3db3d3
+  metadata.gz: 62f7f9d558b9cd8fe834e83c4349bcdf5665d4bf
+  data.tar.gz: 5c079055971ba0d4b341e3099c30c3d09e5d080b
 SHA512:
-  metadata.gz: 741105dc9cb02c567a3dce94f680fcc00d5246002f477bf3e34f0f9d48fad3ee8b5fbb034bc58400c8a9a553f44619bb52851ca6064eeeeddc00723608da2fac
-  data.tar.gz: 59f45cbc2c174ebd517c3a30ba06bbe63c6795e64302913273ae06c6aedbd0712ecda47f97f721ae90463dab8cc8dc46778b390d6b54bfeb2e41e0bba704f088
+  metadata.gz: 54736a76669139e2e0cb92c85d472d5a68978f2a86ab03359a3d6bd957e02c934a91d5d8e7c7c461e59d29c27c411c4575f093ca04af06919a9eb97f9ebeaf6b
+  data.tar.gz: 173775ab0e8a14ae56c73b18744dca37950ff84c632ff249c16b7842134daa90d21a7064a67512fc2ee800b463e1c75e76fee718be02eeff2fff66b36a0d162e

data/README.md

@@ -19,23 +19,36 @@ Run the bundle command to install it.
 
 ## How to use
 
-All request class return instance where result method contain object or collection.
+There is one entry point:
 
-For example:
-```console
-series = TvrageApi::Request::Show.find('buffy')
-series.result
-=> #<TvrageApi::Show ...>
+```ruby
+client = TvrageApi::Client.new
+```
+
+Search show by name:
+
+```ruby
+client = TvrageApi::Client.new
+client.search.by_name('buffy') # return pure Hash
+client.search.full_by_name('buffy') # return pure Hash
 ```
 
-* TvrageApi::Request::Search.find(show_name) - return collection of show
-* TvrageApi::Request::Search.find_full(show_name) - return collection of show with more details
-* TvrageApi::Request::Show.find(show_id) - return show
-* TvrageApi::Request::Show.find_full(show_id) - return show with more details
-* TvrageApi::Request::Show.episodes(show_id) - return simple show information with all episodes
-* TvrageApi::Request::Show.episode(show_id, season, episode) - return simple show information with specific episode
-* TvrageApi::Request::Update.all - return all shows (simple information: id, name)
+Search show by id:
 
+```ruby
+client = TvrageApi::Client.new
+client.show.find('123') # return pure Hash
+client.show.find_full('123') # return pure Hash
+client.show.episodes('123') # return pure Hash, with all episodes
+client.show.episode('123', season, episode) # return pure Hash, with specific episode
+```
+
+Search all show ids:
+
+```ruby
+client = TvrageApi::Client.new
+client.update.all # return pure Hash
+```
 
 ## Contributing
 

data/lib/tvrage_api/base.rb

@@ -1,7 +1,26 @@
-require 'hashie'
+require 'httparty'
 
-class TvrageApi::Base < Hashie::Mash
-  include Hashie::Extensions::MethodAccess
-  include Hashie::Extensions::KeyReplace
-  include Hashie::Extensions::Coercion
-end
+class TvrageApi::Base
+  include HTTParty
+
+  attr_reader :uri, :options
+
+  def get(uri, options = {})
+    @uri = uri
+    @options = options
+
+    self
+  end
+
+  def response
+    response = self.class.get(uri, request_options(options))
+    response.code == 200 ? response.parsed_response : nil
+  end
+
+  def request_options(options = {})
+    {
+      query: options,
+      base_uri: 'http://services.tvrage.com/feeds/'
+    }
+  end
+end

data/lib/tvrage_api/client.rb

@@ -0,0 +1,13 @@
+class TvrageApi::Client
+  def search
+    @search ||= TvrageApi::Search.new
+  end
+
+  def show
+    @show ||= TvrageApi::Show.new
+  end
+
+  def update
+    @show ||= TvrageApi::Update.new
+  end
+end

data/lib/tvrage_api/search.rb

@@ -0,0 +1,9 @@
+class TvrageApi::Search < TvrageApi::Base
+  def by_name(name)
+    get('search.php', show: name).response
+  end
+
+  def full_by_name(name)
+    get('full_search.php', show: name).response
+  end
+end

data/lib/tvrage_api/show.rb

@@ -1,22 +1,17 @@
 class TvrageApi::Show < TvrageApi::Base
-  replace_key :Episodelist, :episode_list
-  replace_key :latestepisode, :latest_episode
-  replace_key :showid, :show_id
-  replace_key :totalseasons, :total_seasons
+  def find(id)
+    get('showinfo.php', sid: id).response
+  end
 
-  coerce_key :id, Integer
-  coerce_key :akas, AkaArray
-  coerce_key :ended, DateOrInteger
-  coerce_key :episode, TvrageApi::Episode
-  coerce_key :episode_list, EpisodeArray
-  coerce_key :genres, GenreArray
-  coerce_key :latest_episode, TvrageApi::Episode
-  coerce_key :network, TvrageApi::Element
-  coerce_key :runtime, Integer
-  coerce_key :seasons, Integer
-  coerce_key :show_id, Integer
-  coerce_key :startdate, Date
-  coerce_key :started, Integer
-  coerce_key :status, Integer
-  coerce_key :total_seasons, Integer
+  def find_full(id)
+    get('full_show_info.php', sid: id).response
+  end
+
+  def episodes(show_id)
+    get('episode_list.php', sid: show_id).response
+  end
+
+  def episode(series_id, season, episode)
+    get('episodeinfo.php', sid: series_id, ep: "#{season}x#{episode}").response
+  end
 end

data/lib/tvrage_api/update.rb

@@ -0,0 +1,5 @@
+class TvrageApi::Update < TvrageApi::Base
+  def all
+    get('show_list.php').response
+  end
+end

data/lib/tvrage_api/version.rb

@@ -1,3 +1,3 @@
 module TvrageApi
-  VERSION = '0.0.2'
+  VERSION = '0.1.0'
 end

data/lib/tvrage_api.rb

@@ -3,12 +3,8 @@ module TvrageApi
 end
 
 require 'tvrage_api/version'
-require 'tvrage_api/utility'
+require 'tvrage_api/client'
 require 'tvrage_api/base'
-require 'tvrage_api/element'
-require 'tvrage_api/episode'
+require 'tvrage_api/search'
 require 'tvrage_api/show'
-require 'tvrage_api/request/base'
-require 'tvrage_api/request/search'
-require 'tvrage_api/request/show'
-require 'tvrage_api/request/update'
+require 'tvrage_api/update'

data/spec/integrations/client_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe TvrageApi::Client do
+  let(:client) { TvrageApi::Client.new }
+
+  describe 'real request' do
+    describe '.search' do
+      describe '.by_name' do
+        it 'should return Hash class' do
+          client.search.by_name('buffy').class.should == Hash
+        end
+      end
+
+      describe '.full_by_name' do
+        it 'should return show class' do
+          client.search.full_by_name('buffy').class.should == Hash
+        end
+      end
+    end
+
+    describe '.show' do
+      describe '.find' do
+        it 'should return Hash class' do
+          client.show.find('2930').class.should == Hash
+        end
+      end
+
+      describe '.find_full' do
+        it 'should return Hash class' do
+          client.show.find_full('2930').class.should == Hash
+        end
+      end
+
+      describe '.episodes' do
+        it 'should return show class' do
+          client.show.episodes('2930').class.should == Hash
+        end
+      end
+
+      describe '.episode' do
+        it 'should return show class' do
+          client.show.episode('2930', '2',  '4').class.should == Hash
+        end
+      end
+    end
+
+    describe '.update' do
+      describe '.all' do
+        it 'should return Hash class' do
+          client.update.all.class.should == Hash
+        end
+      end
+    end
+  end
+end

data/spec/integrations/search_spec.rb

@@ -1,18 +1,18 @@
 require 'spec_helper'
 
-describe TvrageApi::Request::Search do
-  let(:klass) { TvrageApi::Request::Search }
+describe TvrageApi::Search do
+  let(:model) { TvrageApi::Search.new }
 
   describe 'real request' do
-    describe '#find' do
-      it 'should return show class' do
-        klass.find('buffy').result.class.should == Array
+    describe '.by_name' do
+      it 'should return Hash class' do
+        model.by_name('buffy').class.should == Hash
       end
     end
 
-    describe '#find_full' do
+    describe '.full_by_name' do
       it 'should return show class' do
-        klass.find_full('buffy').result.class.should == Array
+        model.full_by_name('buffy').class.should == Hash
       end
     end
   end

data/spec/integrations/show_spec.rb

@@ -1,30 +1,30 @@
 require 'spec_helper'
 
-describe TvrageApi::Request::Show do
-  let(:klass) { TvrageApi::Request::Show }
+describe TvrageApi::Show do
+  let(:model) { TvrageApi::Show.new }
 
   describe 'real request' do
-    describe '#find' do
-      it 'should return show class' do
-        klass.find('2930').result.class.should == TvrageApi::Show
+    describe '.find' do
+      it 'should return Hash class' do
+        model.find('2930').class.should == Hash
       end
     end
 
-    describe '#find_full' do
-      it 'should return show class' do
-        klass.find_full('2930').result.class.should == TvrageApi::Show
+    describe '.find_full' do
+      it 'should return Hash class' do
+        model.find_full('2930').class.should == Hash
       end
     end
 
-    describe '#episodes' do
+    describe '.episodes' do
       it 'should return show class' do
-        klass.episodes('2930').result.class.should == TvrageApi::Show
+        model.episodes('2930').class.should == Hash
       end
     end
 
-    describe '#episode' do
+    describe '.episode' do
       it 'should return show class' do
-        klass.episode('2930', '2',  '4').result.class.should == TvrageApi::Show
+        model.episode('2930', '2',  '4').class.should == Hash
       end
     end
   end

data/spec/integrations/update_spec.rb

@@ -1,13 +1,12 @@
 require 'spec_helper'
 
-describe TvrageApi::Request::Update do
-  let(:klass) { TvrageApi::Request::Update }
+describe TvrageApi::Update do
+  let(:model) { TvrageApi::Update.new }
 
   describe 'real request' do
-    describe '#find' do
-      it 'should return show class' do
-        puts klass.all.result.inspect
-        # klass.all.result.class.should == Array
+    describe '.all' do
+      it 'should return Hash class' do
+        model.all.class.should == Hash
       end
     end
   end

data/spec/tvrage_api/base_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+class ExampleRequestClass < TvrageApi::Base
+end
+
+describe TvrageApi::Base do
+  let(:klass) { ExampleRequestClass }
+  let(:model) { klass.new }
+
+  describe '.get' do
+    it 'should set uri' do
+      model.get('http://example.com')
+
+      model.uri.should == 'http://example.com'
+    end
+
+    it 'should set options' do
+      model.get('http://example.com', sample_options: true)
+
+      model.options.should == { sample_options: true }
+    end
+
+    it 'should return self' do
+      model.get('http://example.com').should == model
+    end
+  end
+
+  describe '.response' do
+    it 'should call get klass method' do
+      klass.should_receive(:get).and_return(double(code: 200, parsed_response: {}))
+      model.stub(:request_options).and_return({})
+
+      model.response
+    end
+
+    it 'should return Hash' do
+      klass.should_receive(:get).and_return(double(code: 200, parsed_response: {}))
+      model.stub(:request_options).and_return({})
+
+      model.response.class.should == Hash
+    end
+
+    it 'should return nil' do
+      klass.should_receive(:get).and_return(double(code: 400))
+      model.stub(:request_options).and_return({})
+
+      model.response.should be_nil
+    end
+  end
+
+  describe '.request_options' do
+    it 'should return correct keys' do
+      model.request_options.keys.sort.should == [:query, :base_uri].sort
+    end
+  end
+end

data/spec/tvrage_api/client_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe TvrageApi::Client do
+  let(:klass) { TvrageApi::Client }
+
+  describe '.search' do
+    it 'should return search class' do
+      klass.new.search.class.should == TvrageApi::Search
+    end
+  end
+
+  describe '.show' do
+    it 'should return show class' do
+      klass.new.show.class.should == TvrageApi::Show
+    end
+  end
+
+  describe '.update' do
+    it 'should return update class' do
+      klass.new.update.class.should == TvrageApi::Update
+    end
+  end
+end

data/spec/tvrage_api/search_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe TvrageApi::Search do
+  let(:klass) { TvrageApi::Search }
+  let(:model) { klass.new }
+
+  describe '#find' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('search.php', show: 'buffy').and_return(double(response: true))
+
+      model.by_name('buffy')
+    end
+  end
+
+  describe '#find_full' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('full_search.php', show: 'buffy').and_return(double(response: true))
+
+      model.full_by_name('buffy')
+    end
+  end
+end

data/spec/tvrage_api/show_spec.rb

@@ -2,78 +2,37 @@ require 'spec_helper'
 
 describe TvrageApi::Show do
   let(:klass) { TvrageApi::Show }
+  let(:model) { klass.new }
 
-  shared_examples 'mapping' do |field, field_mapped|
-    it "should save as #{field_mapped}" do
-      klass.new(field => '123').send(field_mapped).should_not == nil
-    end
-  end
-
-  shared_examples 'integer mapping' do |field|
-    it 'should save as Integer' do
-      klass.new(field => '123').send(field).should == 123
-    end
-  end
+  describe '#find' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('showinfo.php', sid: 123).and_return(double(response: true))
 
-  shared_examples 'date mapping' do |field|
-    it 'should save as Date' do
-      klass.new(field => '2000-01-01').send(field).should == Date.new(2000, 1, 1)
+      model.find(123)
     end
   end
 
-  describe 'key mapping' do
-    describe 'showid attribute' do
-      include_examples 'mapping', :showid, :show_id
-    end
-  end
+  describe '#find_full' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('full_show_info.php', sid: 123).and_return(double(response: true))
 
-  describe 'key mapping' do
-    describe 'Episodelist attribute' do
-      it 'should mapping' do
-        TvrageApi::Show.new(Episodelist: double(season: [double(episode: [double(to_hash: {})])])).send(:episode_list).should_not == nil
-      end
-    end
-    
-    describe 'showid attribute' do
-      include_examples 'mapping', :showid, :show_id
-    end
-    
-    describe 'totalseasons attribute' do
-      include_examples 'mapping', :totalseasons, :total_seasons
+      model.find_full(123)
     end
   end
 
-  describe 'ceorce' do
-    describe 'id attribute' do
-      include_examples 'integer mapping', :id
-    end
-
-    describe 'runtime attribute' do
-      include_examples 'integer mapping', :runtime
-    end
-
-    describe 'seasons attribute' do
-      include_examples 'integer mapping', :seasons
-    end
-
-    describe 'show_id attribute' do
-      include_examples 'integer mapping', :show_id
-    end
-
-    describe 'startdate attribute' do
-      include_examples 'date mapping', :startdate
-    end
+  describe '#episodes' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('episode_list.php', sid: 123).and_return(double(response: true))
 
-    describe 'started attribute' do
-      include_examples 'integer mapping', :started
+      model.episodes(123)
     end
+  end
 
-    describe 'status attribute' do
-      include_examples 'integer mapping', :status
-    end
+  describe '#episode' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('episodeinfo.php', sid: 123, ep: '1x2').and_return(double(response: true))
 
-    describe 'total_seasons attribute' do
-      include_examples 'integer mapping', :total_seasons
+      model.episode(123, 1, 2)
     end
   end
 end

data/spec/tvrage_api/update_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe TvrageApi::Update do
+  let(:klass) { TvrageApi::Update }
+  let(:model) { klass.new }
+
+  describe '#all' do
+    it 'should call new with specific params' do
+      model.should_receive(:get).with('show_list.php').and_return(double(response: true))
+
+      model.all
+    end
+  end
+end

data/tvrage_api.gemspec

@@ -18,8 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'httparty', '>= 0.12.0'
-  spec.add_runtime_dependency 'hashie', '>= 2.0.5'
+  spec.add_runtime_dependency 'httparty', '>= 0.12.0'
 
   spec.add_development_dependency 'bundler', '~> 1.3'
   spec.add_development_dependency 'rake'