Movie JSON submission

Gemfile

@@ -0,0 +1,4 @@
+source 'http://rubygems.org'
+
+
+gem 'rspec'

Gemfile.lock

@@ -0,0 +1,18 @@
+GEM
+ remote: http://rubygems.org/
+ specs:
+ diff-lcs (1.2.5)
+ rspec (2.14.1)
+ rspec-core (~> 2.14.0)
+ rspec-expectations (~> 2.14.0)
+ rspec-mocks (~> 2.14.0)
+ rspec-core (2.14.7)
+ rspec-expectations (2.14.5)
+ diff-lcs (>= 1.1.3, < 2.0)
+ rspec-mocks (2.14.5)
+
+PLATFORMS
+ ruby
+
+DEPENDENCIES
+ rspec

lib/api.rb

@@ -9,7 +9,7 @@ class Api
  def self.search_by_title(title)
  url = "http://api.rottentomatoes.com/api/public/v1.0/movies.json?apikey=#{APIKEY}&q=#{URI.encode(title)}&page_limit=1"
  struct = OpenStruct.new(get_url_as_json(url).fetch("movies").first)
- Movie.new(id: struct.id.to_i,
+ Movie.build(id: struct.id.to_i,
  title: struct.title,
  year: struct.year,
  score: struct.ratings["critics_score"]

lib/movie.rb

@@ -1,10 +1,30 @@
 class Movie
 
+ @@movies = []
  attr_reader :id, :title, :year, :score
  def initialize(hash={})
  @id = hash.fetch(:id)
  @title = hash.fetch(:title)
  @year = hash.fetch(:year)
  @score = hash.fetch(:score)
  end
+
+ def self.build(args)
+ movie = Movie.new(args)
+ @@movies << movie
+ movie
+ end
+
+ def self.average_rating
+ all_movies.inject(0.0) { |sum, movie| sum + movie.score } / all_movies.size
+ end
+
+ def self.count
+ @@movies.count
+ end
+
+ def self.all_movies
+ @@movies
+ end
+
 end

movie_json.rb

@@ -2,19 +2,25 @@
 require_relative "lib/api"
 
 def find_movie
- puts "OH HAI. Search?"
- movie_title = gets
- movie = Api.search_by_title(movie_title)
- puts "Found: #{movie.title}. Score: #{movie.score}"
+ begin
+ puts "OH HAI. Add a movie you really like?"
+ movie_title = gets
+ movie = Api.search_by_title(movie_title)
+ puts "Added: #{movie.title}. Score: #{movie.score}"
+ rescue NoMethodError
+ puts "Not found."
+ end
 end
 
 find_movie
+puts "The average rating is #{Movie.average_rating}."
 
 while true do
- puts "Search Again (Y/N)" 
+ puts "Search Again (Y/N)"
  answer = gets.upcase[0]
  if answer == "Y"
  find_movie
+ puts "The average rating is #{Movie.average_rating}."
  else
  break
  end

spec/api_spec.rb

@@ -24,4 +24,8 @@
  it "should return the year" do
  movie.year.should eq(1994)
  end
+
+ it "should not raise an error when searching for a non-existent movie" do
+ expect { Api.search_by_title("NOTHINGFOUNDHERE") }.to_not raise_error
+ end
 end

spec/movie_spec.rb

@@ -1,12 +1,50 @@
+require 'rspec'
 require_relative "../lib/movie"
 describe Movie do
 
- it "should store the title, year, and score" do
- movie = Movie.new(id: "the-id", title: "the-title", year: 1998, score: 50)
- movie.id.should eq("the-id")
- movie.title.should eq("the-title")
- movie.year.should eq(1998)
- movie.score.should eq(50)
+ describe "#new" do
+
+ let (:movie){Movie.new(id: "the-id", title: "the-title", year: 1998, score: 50)}
+
+ it "should store the title, year, and score" do
+ movie.id.should eq("the-id")
+ movie.title.should eq("the-title")
+ movie.year.should eq(1998)
+ movie.score.should eq(50)
+ end
+
+ it 'should inherit from vehicle' do
+ movie.is_a?(Movie).should be_true
+ end
+
+ it 'should not change the movie count' do
+ Movie.count.should eq(0)
+ end
+ end
+
+ describe ".build" do
+
+ it "records the movie when I build it" do
+ movie = Movie.build(id: 10020, title: 'Pirates of the Caribbean: The Curse of the Black Pearl', year: 2003, score: 79)
+ expect(Movie.all_movies).to include (movie)
+ end
+
+ it "should increment the movie count" do
+ expect{Movie.build(id: 10036, title: 'Forrest Gump', year: 1994, score: 71)}.to change{Movie.count}.from(1).to(2)
+ end
+ end
+
+ describe ".average_rating" do
+ it "should have no average rating initially" do
+ #how do you mock this scenario
+ end
+
+ it 'should calculate the average rating while ignoring the Movie#build calls above' do
+ #how do you change scope for RSpec object. Or rather how do you reset the Movie object instance variables?
+ end
+
+ it "should calculate the average rating for Forrest Gump and Pirates of the...as 75 " do
+ Movie.average_rating.should eq(75)
+ end
  end
-
 end