imdb_party 0.7.1 → 0.8.0

data/README.md

@@ -20,6 +20,12 @@
     movie.rating => 8.1
     movie.certification => "PG-13"
 
+### Get a movie trailer poster
+    
+    movie = imdb.find_movie_by_id("tt1210166")
+
+    movie.trailer_url => "http://ia.media-imdb.com/images/M/MV5BODM1NDMxMTI3M15BMl5BanBnXkFtZTcwMDAzODY1Ng@@._V1_.jpg"
+
 ### Find the top 250 movies of all time
 
     imdb.top_250 => [{:title => "Shawshank Redemption", :year => "1994", :imdb_id => "tt0111161"}, {:title => "The Godfather", ...}]
@@ -27,3 +33,15 @@
 ### Get the currently popular tv shows
 
     imdb.popular_shows => [{:title => "Glee", :year => "2009", :imdb_id => "tt1327801"}, {:title => "Dexter", ...}]
+
+## Running the tests
+
+### Install dependencies
+
+    gem install jeweler
+    gem install shoulda
+    gem install httparty
+
+### Run the tests
+
+    rake test

data/Rakefile

@@ -41,7 +41,7 @@ task :test => :check_dependencies
 
 task :default => :test
 
-require 'rake/rdoctask'
+require 'rdoc/task'
 Rake::RDocTask.new do |rdoc|
   if File.exist?('VERSION')
     version = File.read('VERSION')

data/VERSION

@@ -1 +1 @@
-0.7.1
+0.8.0

data/imdb_party.gemspec

@@ -1,50 +1,42 @@
 # Generated by jeweler
-# DO NOT EDIT THIS FILE
-# Instead, edit Jeweler::Tasks in Rakefile, and run `rake gemspec`
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
 
 Gem::Specification.new do |s|
-  s.name = %q{imdb_party}
-  s.version = "0.6.1"
+  s.name = "imdb_party"
+  s.version = "0.8.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jon Maddox"]
-  s.date = %q{2011-05-04}
-  s.description = %q{IMDB client using the IMDB API that their iPhone app uses}
-  s.email = %q{jon@mustacheinc.com}
+  s.date = "2013-07-17"
+  s.description = "IMDB client using the IMDB API that their iPhone app uses"
+  s.email = "jon@mustacheinc.com"
   s.extra_rdoc_files = [
     "LICENSE",
-     "README.md"
+    "README.md"
   ]
   s.files = [
     ".document",
-     ".gitignore",
-     "LICENSE",
-     "README.md",
-     "Rakefile",
-     "VERSION",
-     "imdb_party.gemspec",
-     "lib/imdb_party.rb",
-     "lib/imdb_party/httparty_icebox.rb",
-     "lib/imdb_party/imdb.rb",
-     "lib/imdb_party/movie.rb",
-     "lib/imdb_party/person.rb",
-     "test/movie_test.rb",
-     "test/person_test.rb",
-     "test/search_test.rb",
-     "test/test_helper.rb"
-  ]
-  s.homepage = %q{http://github.com/maddox/imdb_party}
-  s.rdoc_options = ["--charset=UTF-8"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.4.2}
-  s.summary = %q{IMDB client using the IMDB API that their iPhone app uses}
-  s.test_files = [
+    "LICENSE",
+    "README.md",
+    "Rakefile",
+    "VERSION",
+    "imdb_party.gemspec",
+    "lib/imdb_party.rb",
+    "lib/imdb_party/imdb.rb",
+    "lib/imdb_party/movie.rb",
+    "lib/imdb_party/person.rb",
     "test/movie_test.rb",
-     "test/person_test.rb",
-     "test/search_test.rb",
-     "test/test_helper.rb"
+    "test/person_test.rb",
+    "test/search_test.rb",
+    "test/test_helper.rb",
+    "test/trailer_test.rb"
   ]
+  s.homepage = "http://github.com/maddox/imdb_party"
+  s.require_paths = ["lib"]
+  s.rubygems_version = "1.8.23"
+  s.summary = "IMDB client using the IMDB API that their iPhone app uses"
 
   if s.respond_to? :specification_version then
     s.specification_version = 3
@@ -61,3 +53,4 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<httparty>, [">= 0"])
   end
 end
+

data/lib/imdb_party.rb

@@ -2,7 +2,6 @@ require 'rubygems'
 require 'httparty'
 
 directory = File.expand_path(File.dirname(__FILE__))
-require File.join(directory, 'imdb_party', 'httparty_icebox')
 require File.join(directory, 'imdb_party', 'imdb')
 require File.join(directory, 'imdb_party', 'movie')
 require File.join(directory, 'imdb_party', 'person')

data/lib/imdb_party/imdb.rb

@@ -1,8 +1,8 @@
 module ImdbParty
   class Imdb
     include HTTParty
-    include HTTParty::Icebox
-    cache :store => 'file', :timeout => 120, :location => Dir.tmpdir
+#     include HTTParty::Icebox
+#     cache :store => 'file', :timeout => 120, :location => Dir.tmpdir
 
     base_uri 'app.imdb.com'
     format :json
@@ -36,7 +36,6 @@ module ImdbParty
       movie_results = []
 
       results = self.class.get("http://www.imdb.com/xml/find", :query => default_find_by_title_params).parsed_response
-      puts results.inspect
 
       keys = ["title_popular", "title_exact", "title_approx", "title_substring"]
 

data/lib/imdb_party/movie.rb

@@ -1,6 +1,6 @@
 module ImdbParty
   class Movie
-    attr_accessor :imdb_id, :title, :directors, :writers, :tagline, :company, :plot, :runtime, :rating, :poster_url, :release_date, :certification, :genres, :actors, :trailers
+    attr_accessor :imdb_id, :title, :directors, :writers, :tagline, :company, :plot, :runtime, :rating, :poster_url, :release_date, :certification, :genres, :actors, :trailers, :trailer_url
 
     def initialize(options={})
       @imdb_id = options["tconst"]
@@ -13,6 +13,7 @@ module ImdbParty
       @release_date = options["release_date"]["normal"] if options["release_date"] && options["release_date"]["normal"]
       @certification = options["certificate"]["certificate"] if options["certificate"] && options["certificate"]["certificate"]
       @genres = options["genres"] || []
+	    @trailer_url = options["trailer"]["slates"][0]["url"] if(options["trailer"] && options["trailer"]["slates"] && options["trailer"]["slates"][0])
 
       # parse directors
       @directors = options["directors_summary"] ? options["directors_summary"].map { |d| Person.new(d) } : []

data/test/movie_test.rb

@@ -6,73 +6,72 @@ class MovieTest < Test::Unit::TestCase
       @imdb = ImdbParty::Imdb.new
       @movie = @imdb.find_movie_by_id("tt0382932")
     end
-  
+
     should "have a title" do
       assert_equal "Ratatouille", @movie.title
     end
-  
+
     should "have an imdb_id" do
       assert_equal "tt0382932", @movie.imdb_id
     end
-  
+
     should "have a tagline" do
       assert_equal "Dinner is served... Summer 2007", @movie.tagline
     end
-  
+
     should "have a plot" do
-      assert_equal "Remy is a young rat in the French countryside who arrives in Paris, only to find out that his cooking idol is dead. When he makes an unusual alliance with a restaurant's new garbage boy, the culinary and personal adventures begin despite Remy's family's skepticism and the rat-hating world of humans.", @movie.plot
+      assert_equal "With dreams of becoming a chef, a culinary genius in the form of a rat, makes an unusual alliance with a young kitchen worker at a famed restaurant.", @movie.plot
     end
-  
+
     should "have a runtime" do
       assert_equal "111 min", @movie.runtime
     end
-  
+
     should "have a rating" do
-      assert_equal 8.1, @movie.rating
+      assert_equal 8, @movie.rating
     end
-  
+
     should "have a poster_url" do
       assert_match /http:\/\/ia.media-imdb.com\/images\/.*/, @movie.poster_url
     end
-  
+
     should "have a release date" do
-      assert_equal DateTime.parse("2007-06-29"), @movie.release_date
+      assert_equal "2007-06-29", @movie.release_date
     end
-  
+
     should "have a certification" do
       assert_equal "G", @movie.certification
     end
-  
+
     should "have trailers" do
       assert_equal Hash, @movie.trailers.class
-      assert_equal 2, @movie.trailers.keys.size
-      assert_equal "http://www.totaleclips.com/Player/Bounce.aspx?eclipid=e27826&bitrateid=461&vendorid=102&type=.mp4", @movie.trailers[@movie.trailers.keys.first]
+      assert_equal 0, @movie.trailers.keys.size
     end
-  
+
     should "have genres" do
       assert_equal Array, @movie.genres.class
       assert_equal 4, @movie.genres.size
     end
-  
+
     should "have actors" do
       assert_equal Array, @movie.actors.class
       assert_equal 4, @movie.actors.size
     end
-  
+
     should "have directors" do
       assert_equal Array, @movie.directors.class
       assert_equal 2, @movie.directors.size
     end
-  
+
     should "have writers" do
       assert_equal Array, @movie.writers.class
       assert_equal 2, @movie.writers.size
     end
-  
+
   end
-  
+
   context "a movie while anonymized" do
-    setup do 
+    setup do
       @imdb = ImdbParty::Imdb.new(:anonymize => true)
       @movie = @imdb.find_movie_by_id("tt0382932")
     end
@@ -81,5 +80,5 @@ class MovieTest < Test::Unit::TestCase
       assert_equal "Ratatouille", @movie.title
     end
   end
-  
+
 end

data/test/trailer_test.rb

@@ -0,0 +1,16 @@
+require 'test_helper'
+
+class TrailerTest < Test::Unit::TestCase
+  context "a trailer" do
+    setup do
+      @imdb = ImdbParty::Imdb.new
+      @movie = @imdb.find_movie_by_id("tt1210166")
+    end
+
+    should "have a trailer_url" do
+      assert_match /http:\/\/ia.media-imdb.com\/images\/.*/, @movie.trailer_url
+    end
+  
+  end
+end
+

metadata

@@ -1,61 +1,56 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: imdb_party
-version: !ruby/object:Gem::Version 
-  hash: 1
-  prerelease: false
-  segments: 
-  - 0
-  - 7
-  - 1
-  version: 0.7.1
+version: !ruby/object:Gem::Version
+  version: 0.8.0
+  prerelease: 
 platform: ruby
-authors: 
+authors:
 - Jon Maddox
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-06-25 00:00:00 -04:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2013-07-17 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: shoulda
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: httparty
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: httparty
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id002
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: IMDB client using the IMDB API that their iPhone app uses
 email: jon@mustacheinc.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - LICENSE
 - README.md
-files: 
+files:
 - .document
 - LICENSE
 - README.md
@@ -63,7 +58,6 @@ files:
 - VERSION
 - imdb_party.gemspec
 - lib/imdb_party.rb
-- lib/imdb_party/httparty_icebox.rb
 - lib/imdb_party/imdb.rb
 - lib/imdb_party/movie.rb
 - lib/imdb_party/person.rb
@@ -71,39 +65,29 @@ files:
 - test/person_test.rb
 - test/search_test.rb
 - test/test_helper.rb
-has_rdoc: true
+- test/trailer_test.rb
 homepage: http://github.com/maddox/imdb_party
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: IMDB client using the IMDB API that their iPhone app uses
 test_files: []
-

data/lib/imdb_party/httparty_icebox.rb

@@ -1,259 +0,0 @@
-# = Icebox : Caching for HTTParty
-# 
-# Cache responses in HTTParty models [http://github.com/jnunemaker/httparty]
-# 
-# === Usage
-#
-#   class Foo
-#     include HTTParty
-#     include HTTParty::Icebox
-#     cache :store => 'file', :timeout => 600, :location => MY_APP_ROOT.join('tmp', 'cache')
-#   end
-#
-# Modeled after Martyn Loughran's APICache [http://github.com/newbamboo/api_cache]
-# and Ruby On Rails's caching [http://api.rubyonrails.org/classes/ActiveSupport/Cache.html]
-#
-# Author: Karel Minarik [www.karmi.cz]
-# 
-# === Notes
-# 
-# Thanks to Amit Chakradeo to point out objects have to be stored marhalled on FS
-# Thanks to Marlin Forbes to point out query parameters have to be include in the cache key
-# 
-# 
-
-require 'logger'
-require 'fileutils'
-require 'tmpdir'
-require 'pathname'
-require 'digest/md5'
-
-module HTTParty #:nodoc:
-  module Icebox
-
-    module ClassMethods
-
-      # Enable caching and set cache options
-      # Returns memoized cache object
-      #
-      # Following options are available, default values are in []:
-      #
-      # +store+::       Storage mechanism for cached data (memory, filesystem, your own) [memory]
-      # +timeout+::     Cache expiration in seconds [60]
-      # +logger+::      Path to logfile or logger instance [STDOUT]
-      # 
-      # Any additional options are passed to the Cache constructor
-      #
-      # Usage:
-      #
-      #   # Enable caching in HTTParty, in memory, for 1 minute
-      #   cache # Use default values
-      #
-      #   # Enable caching in HTTParty, on filesystem (/tmp), for 10 minutes
-      #   cache :store => 'file', :timeout => 600, :location => '/tmp/'
-      #
-      #   # Use your own cache store (see AbstractStore class below)
-      #   cache :store => 'memcached', :timeout => 600, :server => '192.168.1.1:1001'
-      #
-      def cache(options={})
-        options[:store]   ||= 'memory'
-        options[:timeout] ||= 60
-        logger = options[:logger]
-        @cache ||= Cache.new( options.delete(:store), options )
-      end
-
-    end
-
-    # When included, extend class with +cache+ method
-    # and redefine +get+ method to use cache
-    # 
-    def self.included(receiver) #:nodoc:
-      receiver.extend ClassMethods
-      receiver.class_eval do
-
-        # Get reponse from network
-        # TODO: Why alias :new :old is not working here? Returns NoMethodError
-        # 
-        def self.get_without_caching(path, options={})
-          perform_request Net::HTTP::Get, path, options
-        end
-
-        # Get response from cache, if available
-        # 
-        def self.get_with_caching(path, options={})
-          key = path.clone
-          key << options[:query].to_s if defined? options[:query]
-          
-          if cache.exists?(key) and not cache.stale?(key)
-            Cache.logger.debug "CACHE -- GET #{path}#{options[:query]}"
-            return cache.get(key)
-          else
-            Cache.logger.debug "/!\\ NETWORK -- GET #{path}#{options[:query]}"
-            response = get_without_caching(path, options)
-            cache.set(key, response) if response.code == 200
-            return response
-          end
-        end
-
-        # Redefine original HTTParty +get+ method to use cache
-        # 
-        def self.get(path, options={})
-          self.get_with_caching(path, options)
-        end
-
-      end
-    end
-
-    # === Cache container
-    # 
-    # Pass a store name ('memory', etc) to initializer
-    # 
-    class Cache
-      attr_accessor :store
-
-      def initialize(store, options={})
-        self.class.logger = options[:logger]
-        @store = self.class.lookup_store(store).new(options)
-      end
-
-      def get(key);            @store.get encode(key) unless stale?(key);        end
-      def set(key, value);     @store.set encode(key), value;                    end
-      def exists?(key);        @store.exists? encode(key);                       end
-      def stale?(key);         @store.stale?  encode(key);                       end
-
-      def self.logger; @logger || default_logger; end
-      def self.default_logger; logger = ::Logger.new(STDERR); end
-
-      # Pass a filename (String), IO object, Logger instance or +nil+ to silence the logger
-      def self.logger=(device); @logger = device.kind_of?(::Logger) ? device : ::Logger.new(device); end
-
-      private
-
-      # Return store class based on passed name
-      def self.lookup_store(name)
-        store_name = "#{name.capitalize}Store"
-        return Store::const_get(store_name)
-      rescue NameError => e
-        raise Store::StoreNotFound, "The cache store '#{store_name}' was not found. Did you loaded any such class?"
-      end
-
-      def encode(key); Digest::MD5.hexdigest(key); end
-    end
-
-
-    # === Cache stores
-    # 
-    module Store
-
-      class StoreNotFound < StandardError; end #:nodoc:
-
-      # ==== Abstract Store
-      # Inherit your store from this class
-      # *IMPORTANT*: Do not forget to call +super+ in your +initialize+ method!
-      # 
-      class AbstractStore
-        def initialize(options={})
-          raise ArgumentError, "You need to set the :timeout parameter" unless options[:timeout]
-          @timeout = options[:timeout]
-          message =  "Cache: Using #{self.class.to_s.split('::').last}"
-          message << " in location: #{options[:location]}" if options[:location]
-          message << " with timeout #{options[:timeout]} sec"
-          Cache.logger.info message unless options[:logger].nil?
-          return self
-        end
-        %w{set get exists? stale?}.each do |method_name|
-          define_method(method_name) { raise NoMethodError, "Please implement method set in your store class" }
-        end
-      end
-
-      # ===== Store objects in memory
-      #
-      Struct.new("TvdbResponse", :code, :body, :headers) { def to_s; self.body; end }
-      class MemoryStore < AbstractStore
-        def initialize(options={})
-          super; @store = {}; self
-        end
-        def set(key, value)
-          Cache.logger.info("Cache: set (#{key})")
-          @store[key] = [Time.now, value]; true
-        end
-        def get(key)
-          data = @store[key][1]
-          Cache.logger.info("Cache: #{data.nil? ? "miss" : "hit"} (#{key})")
-          data
-        end
-        def exists?(key)
-          !@store[key].nil?
-        end
-        def stale?(key)
-          return true unless exists?(key)
-          Time.now - created(key) > @timeout
-        end
-        private
-        def created(key)
-          @store[key][0]
-        end
-      end
-
-      # ===== Store objects on the filesystem
-      # 
-      class FileStore < AbstractStore
-        def initialize(options={})
-          super
-          options[:location] ||= Dir::tmpdir
-          @path = Pathname.new( options[:location] )
-          FileUtils.mkdir_p( @path )
-          self
-        end
-        def set(key, value)
-          Cache.logger.info("Cache: set (#{key})")
-          File.open( @path.join(key), 'w' ) { |file| Marshal.dump(value, file) }
-          true
-        end
-        def get(key)
-          data = Marshal.load(File.new(@path.join(key)))
-          Cache.logger.info("Cache: #{data.nil? ? "miss" : "hit"} (#{key})")
-          data
-        end
-        def exists?(key)
-          File.exists?( @path.join(key) )
-        end
-        def stale?(key)
-          return true unless exists?(key)
-          Time.now - created(key) > @timeout
-        end
-        private
-        def created(key)
-          File.mtime( @path.join(key) )
-        end
-      end
-    end
-
-  end
-end
-
-
-# Major parts of this code are based on architecture of ApiCache.
-# Copyright (c) 2008 Martyn Loughran
-# 
-# Other parts are inspired by the ActiveSupport::Cache in Ruby On Rails.
-# Copyright (c) 2005-2009 David Heinemeier Hansson
-# 
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# "Software"), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-# 
-# 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.