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