Updates

Author: MarkHMorrison

book.rb

@@ -30,4 +30,12 @@ def lost
 	def available?
 		@status == :available
 	end
+
+	def hold
+		@status = :on_hold
+	end
+
+	def release
+		@status = :available
+	end
 end

books.yml

@@ -1,31 +1,31 @@
---- 
-- !ruby/object:Book 
+---
+- !ruby/object:Book
+  title: ! 'JavaScript: The Good Parts'
   author: Douglas Crockford
   category: :development
   status: :available
-  title: "JavaScript: The Good Parts"
-- !ruby/object:Book 
+- !ruby/object:Book
+  title: Designing with Web Standards
   author: Jeffrey Zeldman
   category: :design
   status: :available
-  title: Designing with Web Standards
-- !ruby/object:Book 
+- !ruby/object:Book
+  title: Don't Make me Think
   author: Steve Krug
   category: :usability
   status: :available
-  title: Don't Make me Think
-- !ruby/object:Book 
+- !ruby/object:Book
+  title: JavaScript Patterns
   author: Stoyan Stefanov
   category: :development
   status: :available
-  title: JavaScript Patterns
-- !ruby/object:Book 
+- !ruby/object:Book
+  title: Responsive Web Design
   author: Ethan Marcotte
   category: :design
   status: :available
-  title: Responsive Web Design
-- !ruby/object:Book 
+- !ruby/object:Book
+  title: The Clean Coder
   author: Uncle Bob Martin
   category: :development
   status: :available
-  title: The Clean Coder

spec/book_spec.rb

@@ -27,20 +27,16 @@
 		end
 	end
 
-	describe "#checkout" do
+	describe "checking out a book" do
+		before :each do
+			@book.checkout
+		end
+
 		context "when book is available" do
-			it "changes status to loaned" do
-				@book.status.should eql :available
-				@book.checkout
-				@book.status.should eql :on_loan
-			end
+			it { @book.status.should eql :on_loan } 
 		end
 		context "when book is not available" do
-			it "raises an error" do
-				@book.checkout
-				@book.status.should eql :on_loan
-				lambda { @book.checkout }.should raise_error
-			end
+			it { lambda { @book.checkout }.should raise_error }
 		end
 	end
 
@@ -60,6 +56,7 @@
 			end
 		end
 	end
+
 
 	describe "#lost" do
 		it "changes status to lost" do
@@ -71,4 +68,30 @@
 	describe "when a book is available" do
 		it { @book.should be_available }
 	end
+
+	describe "placeing a book on hold" do
+		before :each do
+			@book.hold
+		end
+
+		context "the book is available" do
+			it "should change status to on hold" do
+				@book.status.should == :on_hold
+			end
+		end
+	end
+
+	describe "when removing a book from hold" do
+		before :each do
+			@book.hold
+			@book.release
+		end
+
+		context "the book is on hold" do
+			it "should change status to available" do
+				@book.status.should == :available
+			end
+		end
+	end
+
 end