rotten 0.2.0 → 0.3.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rotten (0.1.0)
+    rotten (0.2.0)
 
 GEM
   remote: http://rubygems.org/

data/README.md

@@ -4,12 +4,23 @@
 ### Usage
     require "rotten"
     Rotten.api_key = 'your_key'
-    movies = Rotten::Movies.search "There will be blood"
+
+    # Info about a specific film
+    movies = Rotten::Movies.search("There will be blood").pop
+    movie.reviews
+    movie.cast
+    
+    # list upcoming movies
+    Rotten::Movie.upcoming
+
+    # list movies opening this week
+    Rotten::Movie.opening
 
 ### Features
 - Movie search
 - Movies opening this week
 - Movies upcoming
+- Movie reviews
 
 #### TODO
 - Implement all APIs

data/lib/rotten.rb

@@ -2,10 +2,12 @@ lib = File.expand_path('../', __FILE__)
 $:.unshift lib unless $:.include?(lib)
 
 module Rotten
-  autoload :Api,   "rotten/api"
-  autoload :Actor, "rotten/actor"
-  autoload :Cast,  "rotten/cast"
-  autoload :Movie, "rotten/movie"
+  autoload :Entity, "rotten/entity"
+  autoload :Api,    "rotten/api"
+  autoload :Actor,  "rotten/actor"
+  autoload :Cast,   "rotten/cast"
+  autoload :Review, "rotten/review"
+  autoload :Movie,  "rotten/movie"
 
   def api_key=(val)
     @api_key = val

data/lib/rotten/actor.rb

@@ -1,8 +1,4 @@
 module Rotten
-  class Actor
-    attr_reader :name
-    def initialize name
-      @name = name
-    end
+  class Actor < Entity
   end
 end

data/lib/rotten/api.rb

@@ -25,7 +25,7 @@ module Rotten
 
       def get path, options={}
         if Rotten.api_key.nil?
-          raise UndefinedApiKeyError, "Please define your API key with #{self}.api_key=(your_key)"
+          raise UndefinedApiKeyError, "Please define your API key with Rotten.api_key=(your_key)"
         end
 
         url = url_for(path, options)

data/lib/rotten/cast.rb

@@ -6,7 +6,7 @@ module Rotten
     def initialize array=[]
       @actors     = Set.new
       @characters = []
-      process array
+      process (array || [])
     end
 
     def inspect
@@ -17,11 +17,11 @@ module Rotten
       @characters[index]
     end
 
-    def first; [0]; end
+    def first; at[0]; end
 
     def process array
       array.each do |hash|
-        actor      =  @actors.detect{|a| a.name == hash["name"] } || Actor.new(hash["name"])
+        actor      =  @actors.detect{|a| a.name == hash["name"] } || a=Actor.new; a.attributes=hash
         @actors     << actor
         @characters << [ actor, hash["characters"] ]
       end
@@ -44,6 +44,5 @@ module Rotten
         @characters.include?(other)
       end
     end
-
   end
 end

data/lib/rotten/movie.rb

@@ -1,5 +1,5 @@
 module Rotten
-  class Movie
+  class Movie < Entity
     class InvalidSearchQueryError < StandardError; end
     include Api
 
@@ -37,25 +37,34 @@ module Rotten
 
     attr_reader :actors, :cast
     def initialize movie_hash={}
+      super
       @actors = []
       process movie_hash
     end
 
     def inspect 
-      "<Rotten::Movie title='#{@title}' id='#{@id}'>"
+      "<Rotten::Movie title='#{title}' id='#{id}'>"
     end
 
     def to_s
       title
     end
 
+    # Moview reviews
+    # @return [Array]
+    def reviews options={}
+      Movie.get "movies/#{id}/reviews", options do |json|
+        json["reviews"].map{|review| Review.from_json(review) }
+      end
+    end
+
     # Show cast 
     #
     # @param [Symbol #kind]  Defaults to :abridged, but can accept :full to retrieve full cast info.
     # @return [Rotten::Cast]
     def cast( kind = :abridged )
       if kind == :full
-        Movie.get "movies/#{@id}/cast" do |json|
+        Movie.get "movies/#{id}/cast" do |json|
           @cast = Cast.new json["cast"]
         end
         self.cast
@@ -65,16 +74,16 @@ module Rotten
     end
 
     def process hash
-      hash.each_pair{|k,v| instance_variable_set("@#{k}", v); self.class.send(:attr_reader, k.to_sym) }
-      @cast   = Cast.new( @abridged_cast )
-      @actors = @cast.actors
+      attributes= hash
+      @cast       = Cast.new( abridged_cast )
+      @actors     = @cast.actors
     end
 
     # Fetch updated, potentially additional movie information
     # @return [Rotten::Movie]
     def reload
-      return false unless @id
-      Movie.get "movies/#{@id}" do |json|
+      return false unless id
+      Movie.get "movies/#{id}" do |json|
         process json 
       end
       self

data/lib/rotten/version.rb

@@ -1,3 +1,3 @@
 module Rotten
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/spec/movie_spec.rb

@@ -15,6 +15,19 @@ describe Rotten::Movie do
       end 
     end
   end
+
+  context "#reviews" do
+    before :each do
+      simulate_movie_search
+      @movie = Rotten::Movie.search("There Will Be Blood").pop
+      simulate_movie_reviews(@movie)
+    end
+
+    it "should return an array of Review" do
+      @movie.reviews.should be_an_instance_of(Array)
+      @movie.reviews.shift.should be_an_instance_of(Rotten::Review)
+    end
+  end
   
   context ".opening" do
     before :each do

data/spec/spec_helper.rb

@@ -19,6 +19,11 @@ RSpec.configure do |config|
                          :body => File.read( File.join(fixture_path, "search.json") ))
   end
 
+  def simulate_movie_reviews(movie)
+    FakeWeb.register_uri(:get, Rotten::Movie.url_for("movies/#{movie.id}/reviews"), 
+                         :body => File.read( File.join(fixture_path, "reviews.json") ))
+  end
+
   def simulate_full_cast(movie)
     FakeWeb.register_uri(:get, Rotten::Movie.url_for("movies/#{movie.id}/cast"),
                          :body => File.read( File.join(fixture_path, "cast.json") ))

metadata

@@ -4,9 +4,9 @@ version: !ruby/object:Gem::Version
   prerelease: false
   segments: 
   - 0
-  - 2
+  - 3
   - 0
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors: 
 - James Cook
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-15 00:00:00 -05:00
+date: 2011-04-16 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency