ruby-tmdb3 0.3.0

data/test/fixtures/person_search.txt

@@ -0,0 +1,15 @@
+HTTP/1.1 200 OK
+Server: nginx
+Date: Mon, 23 Aug 2010 16:39:45 GMT
+Content-Type: text/json; charset=utf-8
+Transfer-Encoding: chunked
+Connection: keep-alive
+Keep-Alive: timeout=20
+Status: 200 OK
+Cache-Control: public, max-age=21600
+X-Varnish: 2542104567 2542101419
+Age: 36
+Via: 1.1 varnish
+X-Cache: HIT
+
+{"page":1,"results":[{"adult":false,"id":558629,"name":"Vince","profile_path":null},{"adult":false,"id":1905,"name":"Vincent Price","profile_path":"/w9wcf7hWLD2I2K0PojJaqd9vbHL.jpg"},{"adult":false,"id":1925,"name":"Vincent Cassel","profile_path":"/dai7LAxPyohPlIO5WYrKkNqO8Wu.jpg"},{"adult":false,"id":4937,"name":"Vince Vaughn","profile_path":"/vTIHDiSvIm3HU93JDGCstFwNGxI.jpg"},{"adult":false,"id":38231,"name":"June Vincent","profile_path":"/b0SZNSaOzmKQm4UOss6SAxTpktT.jpg"},{"adult":false,"id":7132,"name":"Vincent D'Onofrio","profile_path":"/kmoj1mZVNTGDZjWKKPp1xKftC9Z.jpg"},{"adult":false,"id":3418,"name":"Vincent Schiavelli","profile_path":"/q2fEosVlli2T8k5db7c7xueNTtc.jpg"},{"adult":false,"id":9831,"name":"Vincent Regan","profile_path":"/tiyjPHwmX8dMMfx0F8HUpXyLHeY.jpg"},{"adult":false,"id":7164,"name":"Frank Vincent","profile_path":null},{"adult":false,"id":52646,"name":"Vincent Kartheiser","profile_path":"/ePnbREiwxNeMLNLVIkJWY6ODRyd.jpg"},{"adult":false,"id":14830,"name":"Vincent Gardenia","profile_path":"/g7iwqHaaIKh5lSTW39pXT0OOXVC.jpg"},{"adult":false,"id":54233,"name":"Vincent Rottiers","profile_path":"/iqwHkmc6ip4cbLS7pz6RACCiVCO.jpg"},{"adult":false,"id":1141,"name":"Hélène Vincent","profile_path":null},{"adult":false,"id":84458,"name":"Vincent Miller","profile_path":null},{"adult":false,"id":24595,"name":"Vincenzo Nicoli","profile_path":null},{"adult":false,"id":39152,"name":"Yves Vincent","profile_path":null},{"adult":false,"id":54268,"name":"Vince Gerardis","profile_path":null},{"adult":false,"id":76825,"name":"Vincent Desagnat","profile_path":"/chkXB8vpO0NuR66Wi0tGfVKnuBX.jpg"},{"adult":false,"id":575079,"name":"Vincent E. Toto","profile_path":null},{"adult":false,"id":1983,"name":"Vincent Laresca","profile_path":"/sFvw7EIWb84vzcRFRY2LknRFxvI.jpg"}],"total_pages":28,"total_results":550}

data/test/setup/setup_api_key.rb

@@ -0,0 +1,10 @@
+begin
+  file_path = File.absolute_path('tmdb_api_key.txt', File.dirname(__FILE__))
+
+  File.open(file_path) do |file|
+    Tmdb.api_key = file.read
+  end
+rescue Errno::ENOENT => e
+  $stderr.puts "You need place your api key in #{file_path}"
+  exit
+end

data/test/setup/test_unit_extensions.rb

@@ -0,0 +1,21 @@
+module Test::Unit
+  # Used to fix a minor minitest/unit incompatibility in flexmock
+  AssertionFailedError = Class.new(StandardError)
+  
+  class TestCase
+   
+    def self.test(name, &block)
+      test_name = "test_#{name.gsub(/\s+/,'_')}".to_sym
+      defined = instance_method(test_name) rescue false
+      raise "#{test_name} is already defined in #{self}" if defined
+      if block_given?
+        define_method(test_name, &block)
+      else
+        define_method(test_name) do
+          flunk "No implementation provided for #{name}"
+        end
+      end
+    end
+
+  end
+end

data/test/setup/url_mocks.rb

@@ -0,0 +1,46 @@
+def register_api_url_stubs  
+  unless(TEST_LIVE_API)
+  
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "movie_search.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "search/movie" + ".*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "movie_get_info.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "movie/" + ".*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "movie_imdb_lookup.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "movie/tt" + ".*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "person_get_info.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "person/" + ".*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "person_search.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "search/person" + ".*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "incorrect_api_url.txt")) do |file|
+      stub_request(:get, Regexp.new(Tmdb.base_api_url + "Movie.blarg/" + ".*")).to_return(file)
+    end
+
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "example_com.txt")) do |file|
+      stub_request(:get, Regexp.new("http://example.com.*")).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "image.jpg")) do |file|
+      stub_request(:get, Regexp.new('http://i[0-9].themoviedb.org/[backdrops|posters|profiles].*')).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "image.jpg")) do |file|
+      stub_request(:get, Regexp.new('http://hwcdn.themoviedb.org/[backdrops|posters|profiles].*')).to_return(file)
+    end
+    
+    File.open(File.join(File.dirname(__FILE__), "..", "fixtures", "blank_result.txt")) do |file|
+      stub_request(:get, Regexp.new("item_not_found$")).to_return(file)
+    end
+    
+    stub_request(:get, 'http://thisisaurlthatdoesntexist.co.nz').to_return(:body => "", :status => 404, :headers => {'content-length' => 0})
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,21 @@
+TEST_LIVE_API = false
+
+require 'rubygems'
+require 'test/unit'
+require 'mocha'
+
+unless(TEST_LIVE_API)
+  require 'webmock/test_unit'
+  include WebMock::API
+end
+
+require_files = []
+require_files << File.join(File.dirname(__FILE__), '..', 'lib', 'ruby-tmdb3.rb')
+require_files.concat Dir[File.join(File.dirname(__FILE__), 'setup', '*.rb')]
+
+require_files.each do |file|
+  require File.expand_path(file)
+end
+
+#load(File.join('unit', 'test_direct_require.rb'), true)
+system('ruby ' + File.expand_path(File.join(File.dirname(__FILE__), 'unit', 'test_direct_require.rb')))

data/test/unit/test_direct_require.rb

@@ -0,0 +1,25 @@
+require "test/unit"
+
+require_files = []
+require_files << File.join(File.dirname(__FILE__), "..", "..", "lib", "ruby-tmdb3.rb")
+require_files.concat Dir[File.join(File.dirname(__FILE__), '..', 'setup', '*.rb')]
+
+require_files.each do |file|
+  require File.expand_path(file)
+end
+
+class DirectRequireTest < Test::Unit::TestCase
+  
+  test "TmdbMovie should not raise exception when directly required without using rubygems" do
+    assert_nothing_raised do
+      TmdbMovie.find(:id => 187)
+    end
+  end
+  
+  test "TmdbCast should not raise exception when directly required without using rubygems" do
+    assert_nothing_raised do
+      TmdbCast.find(:id => 287)
+    end
+  end
+  
+end

data/test/unit/tmdb_cast_test.rb

@@ -0,0 +1,124 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper.rb'))
+
+class TmdbCastTest < Test::Unit::TestCase
+
+  def setup
+    register_api_url_stubs
+  end
+
+  test "search that returns no results should create empty array" do
+    movie = TmdbCast.find(:name => "item_not_found")
+    assert_equal [], movie
+  end
+  
+  test "cast data should be able to be dumped and re-loaded" do
+    assert_nothing_raised do
+      cast = TmdbCast.find(:id => 287)
+      TmdbCast.new(cast.raw_data)
+    end
+  end
+  
+  test "find by id should return full cast data" do
+    cast = TmdbCast.find(:id => 287)
+    assert_cast_methodized(cast, 287)
+  end
+  
+  test "two cast objects with same data should be equal" do
+    cast1 = TmdbCast.find(:id => 287, :limit => 1)
+    cast2 = TmdbCast.find(:id => 287, :limit => 1)
+    assert_equal cast1, cast2
+  end
+  
+  test "find by name should return full cast data when :expand_results = true" do
+    cast = TmdbCast.find(:name => "Brad Pitt", :expand_results => true)
+    cast = cast.first if cast.class == Array
+    assert_cast_methodized(cast, 287)
+  end
+
+  test "should raise exception if no arguments supplied to find" do
+    assert_raise ArgumentError do
+      TmdbCast.find()
+    end
+  end
+
+  test "find by id should return a single cast member" do
+    assert_kind_of OpenStruct, TmdbCast.find(:id => 287)
+  end
+  
+  test "find by name should return an array of cast members" do
+    cast_members = TmdbCast.find(:name => "vince")
+    assert_kind_of Array, cast_members
+    cast_members.each do |actor|
+      assert_kind_of OpenStruct, actor
+    end
+  end
+  
+  test "should raise error if limit is smaller than 1" do
+    [0, -1, -100].each do |limit|
+      assert_raise ArgumentError do
+        TmdbCast.find(:name => "vince", :limit => limit)
+      end
+    end
+  end
+  
+  test "should raise error if limit is not an integer" do
+    [1.001, "1.2", "hello", [1,2,3], {:test => "1"}].each do |limit|
+      assert_raise ArgumentError do
+        TmdbCast.find(:name => "vince", :limit => limit)
+      end
+    end
+  end
+  
+  test "should only return a single item if limit=1" do
+    actor = TmdbCast.find(:name => "Vince", :limit => 1)
+    assert_kind_of OpenStruct, actor
+  end
+  
+  test "should return X items if limit=X" do
+    (2..5).each do |x|
+      actors = TmdbCast.find(:name => "Vince", :limit => x)
+      assert_kind_of Array, actors
+      assert_equal x, actors.length
+      actors.each do |actor|
+        assert_kind_of OpenStruct, actor
+      end
+    end
+  end
+
+  test "should not pass language to Tmdb.api_call if language is not supplied" do
+    Tmdb.expects(:api_call).with("person", {id: "1"}, nil).returns(nil)
+    Tmdb.expects(:api_call).with("search/person", {query: "1"}, nil).returns(nil)
+    TmdbCast.find(:id => 1, :name => 1)
+  end
+
+  test "should pass through language to Tmdb.api_call when language is supplied" do
+    Tmdb.expects(:api_call).with("person", {id: "1"}, "foo").returns(nil)
+    Tmdb.expects(:api_call).with("search/person", {query: "1"}, "foo").returns(nil)
+    TmdbCast.find(:id => 1, :name => 1, :language => "foo")
+  end
+
+  test "TmdbCast.new should raise error if supplied with raw data for cast member that doesn't exist" do
+    Tmdb.expects(:api_call).with("person", {id: "999999999999"}, nil).returns(nil)
+    assert_raise ArgumentError do
+      TmdbCast.new({"id" => 999999999999}, true)
+    end
+  end
+
+  private
+  
+    def assert_cast_methodized(actor, cast_id)
+      @cast_data = Tmdb.api_call("person", {id: cast_id.to_s})
+      assert_equal @cast_data["adult"], actor.adult
+      assert_equal @cast_data["also_known_as"], actor.also_known_as
+      assert_equal @cast_data["biography"], actor.biography
+      assert_equal @cast_data["birthday"], actor.birthday
+      assert_equal @cast_data["biography"], actor.biography
+      assert_equal @cast_data["deathday"], actor.deathday
+      assert_equal @cast_data["homepage"], actor.homepage
+      assert_equal @cast_data["id"], actor.id
+      assert_equal @cast_data["name"], actor.name
+      assert_equal @cast_data["place_of_birth"], actor.place_of_birth
+      assert_equal @cast_data["profile_path"], actor.profile_path
+    end
+
+end

data/test/unit/tmdb_movie_test.rb

@@ -0,0 +1,158 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper.rb'))
+
+class TmdbMovieTest < Test::Unit::TestCase
+
+  def setup
+    register_api_url_stubs
+  end
+  
+  test "search that returns no results should create empty array" do
+    movie = TmdbMovie.find(:title => "item_not_found")
+    assert_equal [], movie
+  end
+  
+  test "movie should be able to be dumped and re-loaded" do
+    assert_nothing_raised do
+      movie = TmdbMovie.find(:id => 187)
+      TmdbMovie.new(movie.raw_data)
+    end
+  end
+  
+  test "find by id should return the full movie data" do
+    movie = TmdbMovie.find(:id => 187)
+    assert_movie_methodized(movie, 187)
+  end
+  
+  test "movies with same data should be seen as equal" do
+    movie1 = TmdbMovie.find(:id => 187, :limit => 1)
+    movie2 = TmdbMovie.find(:id => 187, :limit => 1)
+    assert_equal movie1, movie2
+  end
+  
+  test "find by imdb should return the full movie data" do
+    movie = TmdbMovie.find(:imdb => "tt0401792")
+    assert_movie_methodized(movie, 187)
+  end
+  
+  test "find by title should return the full movie data when expand_results set to true" do
+    movie = TmdbMovie.find(:title => "Sin City", :limit => 1, :expand_results => true)
+    assert_movie_methodized(movie, 187)
+  end
+  
+  test "should raise exception if no arguments supplied to find" do
+    assert_raise ArgumentError do
+      TmdbMovie.find()
+    end
+  end
+
+  test "find by id should return a single movie" do
+    assert_kind_of OpenStruct, TmdbMovie.find(:id => 187)
+  end
+  
+  test "find by imdb should return a single movie" do
+    assert_kind_of OpenStruct, TmdbMovie.find(:imdb => "tt0401792")
+  end
+  
+  test "find by title should return an array of movies" do
+    movies = TmdbMovie.find(:title => "Iron Man")
+    assert_kind_of Array, movies
+    movies.each do |movie|
+      assert_kind_of OpenStruct, movie
+    end
+  end
+    
+  test "find by title with limit=1 should return a single movie" do
+    assert_kind_of OpenStruct, TmdbMovie.find(:title => "Iron Man", :limit => 1)
+  end
+  
+  test "find by title with limit=X should return an array of X movies" do
+    (2..5).each do |x|
+      movies = TmdbMovie.find(:title => "Iron Man", :limit => x)
+      assert_kind_of Array, movies
+      assert_equal x, movies.length
+      movies.each do |movie|
+        assert_kind_of OpenStruct, movie
+      end
+    end
+  end
+  
+  test "should raise error if limit is smaller than 1" do
+    [0, -1, -100].each do |limit|
+      assert_raise ArgumentError do
+        TmdbMovie.find(:title => "Iron Man", :limit => limit)
+      end
+    end
+  end
+  
+  test "should raise error if limit is not an integer" do
+    [1.001, "1.2", "hello", [1,2,3], {:test => "1"}].each do |limit|
+      assert_raise ArgumentError do
+        TmdbMovie.find(:title => "Iron Man", :limit => limit)
+      end
+    end
+  end
+  
+  test "find should not pass language to Tmdb.api_call if language is not supplied" do
+    Tmdb.expects(:api_call).with("movie", {id: "1"}, nil).twice
+    Tmdb.expects(:api_call).with("search/movie", {query: "1"}, nil)
+    TmdbMovie.find(:id => 1, :imdb => 1, :title => 1)
+  end
+  
+  test "find should pass through language to Tmdb.api_call when language is supplied" do
+    Tmdb.expects(:api_call).with("movie", {id: "1"}, "foo").twice
+    Tmdb.expects(:api_call).with("search/movie", {query: "1"}, "foo")
+    TmdbMovie.find(:id => 1, :imdb => 1, :title => 1, :language => "foo")
+  end
+  
+  test "TmdbMovie.new should raise error if supplied with raw data for movie that doesn't exist" do
+    Tmdb.expects(:api_call).with("movie", {id: "999999999999"}, nil).returns(nil)
+    assert_raise ArgumentError do
+      TmdbMovie.new({"id" => 999999999999}, true)
+    end
+  end
+
+  private
+    
+    def assert_movie_methodized(movie, movie_id)
+      @movie_data = Tmdb.api_call("movie", {id: movie_id.to_s})
+      assert_equal @movie_data["adult"], movie.adult
+      assert_equal @movie_data["budget"], movie.budget
+      assert_equal @movie_data["homepage"], movie.homepage
+      assert_equal @movie_data["id"], movie.id
+      assert_equal @movie_data["imdb_id"], movie.imdb_id
+      assert_equal @movie_data["original_title"], movie.original_title
+      assert_equal @movie_data["overview"], movie.overview
+      assert_equal @movie_data["popularity"], movie.popularity
+      assert_equal @movie_data["poster_path"], movie.poster_path
+      assert_equal @movie_data["release_date"], movie.release_date
+      assert_equal @movie_data["revenue"], movie.revenue
+      assert_equal @movie_data["runtime"], movie.runtime
+      assert_equal @movie_data["tagline"], movie.tagline
+      assert_equal @movie_data["title"], movie.title
+      assert_equal @movie_data["vote_average"], movie.vote_average
+      assert_equal @movie_data["vote_count"], movie.vote_count
+
+      assert_equal @movie_data["belongs_to_collection"]["id"], movie.belongs_to_collection.id
+      assert_equal @movie_data["belongs_to_collection"]["name"], movie.belongs_to_collection.name
+      assert_equal @movie_data["belongs_to_collection"]["poster_path"], movie.belongs_to_collection.poster_path
+      assert_equal @movie_data["belongs_to_collection"]["backdrop_path"], movie.belongs_to_collection.backdrop_path
+
+      @movie_data["genres"].each_index do |x|
+        assert_equal @movie_data["genres"][x]["id"], movie.genres[x].id
+        assert_equal @movie_data["genres"][x]["name"], movie.genres[x].name
+      end
+      @movie_data["production_companies"].each_index do |x|
+        assert_equal @movie_data["production_companies"][x]["name"], movie.production_companies[x].name
+        assert_equal @movie_data["production_companies"][x]["id"], movie.production_companies[x].id
+      end
+      @movie_data["production_countries"].each_index do |x|
+        assert_equal @movie_data["production_countries"][x]["iso_3166_1"], movie.production_countries[x].iso_3166_1
+        assert_equal @movie_data["production_countries"][x]["name"], movie.production_countries[x].name
+      end
+      @movie_data["spoken_languages"].each_index do |x|
+        assert_equal @movie_data["spoken_languages"][x]["iso_639_1"], movie.spoken_languages[x].iso_639_1
+        assert_equal @movie_data["spoken_languages"][x]["name"], movie.spoken_languages[x].name
+      end
+    end
+
+end

data/test/unit/tmdb_test.rb

@@ -0,0 +1,180 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper.rb'))
+
+class TmdbTest < Test::Unit::TestCase
+
+  def setup
+    register_api_url_stubs
+    @@old_default_language = Tmdb.default_language
+  end
+
+  def teardown
+    Tmdb.default_language = @@old_default_language
+  end
+
+  test "should allow setting and getting of api_key" do
+    old_api_key = Tmdb.api_key
+    api_key = "test1234567890"
+    Tmdb.api_key = api_key
+    assert_equal Tmdb.api_key, api_key
+    Tmdb.api_key = old_api_key
+  end
+  
+  test "language should default to 'en'" do
+    assert_equal "en", Tmdb.default_language
+  end
+  
+  test "should allow setting and getting of language" do
+    old_language = Tmdb.default_language
+    new_language = "blah"
+    Tmdb.default_language = new_language
+    assert_equal new_language, Tmdb.default_language
+  end
+  
+  test "should return base API url" do
+    assert_equal "http://api.themoviedb.org/3/", Tmdb.base_api_url
+  end
+
+  test "get url returns a response object" do
+    test_response = Tmdb.get_url("http://example.com/")
+    assert_equal 200, test_response.code.to_i
+  end
+  
+  test "getting nonexistent URL returns response object" do
+    test_response = Tmdb.get_url('http://thisisaurlthatdoesntexist.co.nz')
+    assert_equal 404, test_response.code.to_i
+  end
+  
+  test "API call without explicit language setting should use default language" do
+    method = "search/movie"
+    data = "hello"
+    Tmdb.default_language = "es"
+    url = Tmdb.base_api_url + method + '?api_key=' + Tmdb.api_key + '&language=' + Tmdb.default_language + '&query=' + CGI::escape(data.to_s)
+    mock_response = stub(:code => "200", :body => '{"page":1,"results":[],"total_pages":0,"total_results":0}')
+    Tmdb.expects(:get_url).with(url).returns(mock_response)
+    Tmdb.api_call(method, {query: data})
+  end
+  
+  test "API call with explicit language setting should override default language" do
+    method = "movie"
+    data = "hello"
+    language = "blah"
+    url = Tmdb.base_api_url + method + "?api_key=" + Tmdb.api_key + "&language=" + language + '&query=' + CGI::escape(data.to_s)
+    mock_response = stub(:code => "200", :body => '{"page":1,"results":[],"total_pages":0,"total_results":0}')
+    Tmdb.expects(:get_url).with(url).returns(mock_response)
+    Tmdb.api_call(method, {query: data}, language)
+  end
+  
+  test "api_call should raise exception if api_key is not set" do
+    old_api_key = Tmdb.api_key
+    Tmdb.api_key = ""
+    assert_raises ArgumentError do
+      Tmdb.api_call('Movie.search', 'Transformers')
+    end
+    Tmdb.api_key = old_api_key
+  end
+  
+  test "should perform search/movie API call and return a Hash with an array of results" do
+    movies = Tmdb.api_call("search/movie", {query: "Transformers"})
+    assert_kind_of Hash, movies
+    assert_kind_of Array, movies["results"]
+    movies["results"].each do |movie|
+      assert_kind_of Hash, movie
+      %w(original_title id).each do |item|
+        assert movie[item]
+      end
+    end
+  end
+  
+  test "should perform movie API call and return a single result" do
+    result = Tmdb.api_call("movie", {id: "187"})
+    assert_kind_of Hash, result
+    %w(original_title id).each do |item|
+      assert_not_nil result[item]
+    end
+  end
+  
+  test "should perform Movie.imdbLookup API call and return a single result" do
+    result = Tmdb.api_call("movie", {id: "tt0401792"})
+    assert_kind_of Hash, result
+    %w(original_title id).each do |item|
+      assert result[item]
+    end
+  end
+  
+  test "should perform person API call and return a single result" do
+    result = Tmdb.api_call("person", {id: 287})
+    assert_kind_of Hash, result
+    %w(homepage id name).each do |item|
+      assert_not_nil result[item]
+    end
+  end
+  
+  test "should perform search/person API call and return a Hash with an array of results" do
+    people = Tmdb.api_call("search/person", {query: "vince"})
+    assert_kind_of Array, people["results"]
+    people["results"].each do |person|
+      assert_kind_of Hash, person
+      %w(id name).each do |item|
+        assert_not_nil person[item]
+      end
+    end
+  end
+  
+  test "API call that returns 404 should raise exception" do
+    assert_raise ArgumentError do
+      movies = Tmdb.api_call('Movie.blarg', 'Transformers')
+    end
+  end
+  
+  test "API call that finds no results should return nil" do
+    movies = Tmdb.api_call('search/movie', {query: "item_not_found"})
+    assert_nil movies
+  end
+  
+  test "API call cache should not be changed when data altered in the receiving method" do
+    person = Tmdb.api_call("person", {id: 287})
+    assert_not_nil person[person.keys[0]]
+    person[person.keys[0]] = nil
+    person = Tmdb.api_call("person", {id: 287})
+    assert_not_nil person[person.keys[0]]
+  end
+  
+  test "data_to_object should create object from nested data structures" do
+    test_data = {
+      :test1 => [
+        1,2,3,4
+      ],
+      :test2 => 1
+    }
+    test_object = Tmdb.data_to_object(test_data)
+    assert_nothing_raised do
+      assert_equal [1,2,3,4], test_object.test1
+      assert_equal 1, test_object.test2
+    end
+  end
+  
+  test "data_to_object should include raw_data method that returns original data" do
+    test_data = {
+      :test1 => [1,2,3]
+    }
+    test_object = Tmdb.data_to_object(test_data)
+    assert_equal test_object.raw_data, test_data
+  end
+  
+  test "data_to_object should convert arrays containing images to nicer format" do
+    test_data = {
+      "backdrops" => [
+        {
+          "image" => {
+            :test => 1
+          }
+        }
+      ]
+    }
+    test_object = Tmdb.data_to_object(test_data)
+    assert_nothing_raised do
+      assert_equal 1, test_object.backdrops[0].test
+    end
+  end
+
+end