Removed Chef for simplicity at this point

added some tests and refactored mood for the customer
now use instance_variable_set instead of using attr_accessor to
make a setter

Author: charissa

Diff for: .gitignore

@@ -0,0 +1,2 @@
+.DS_Store
+/tmp/

Diff for: Rakefile

@@ -14,10 +14,6 @@ desc 'Run Minitest Tests'
 Rake::TestTask.new(:test) do |t|
 end
 
-#desc 'Storm test'
-#Rake::TestTask.new(:storm) do |t|
-#end
-
 Cucumber::Rake::Task.new(:features) do |t|
   t.cucumber_opts = "features --format pretty"
 end

Diff for: features/step_definitions/restaurant_steps.rb

@@ -2,7 +2,6 @@
 
 Before do
   @w=Waiter.new
-  @chef=Chef.new
   @c=Customer.new
 end
 
@@ -102,7 +101,7 @@
    # but since I don;t know how to do that and don't want to spend hours figuring it out
    # since there are not a lot of great examples out there or useful documentation
    # I added a setter for mood (bad idea) just so I can run this test....
-   @c.mood=arg1
+   @c.instance_variable_set(:@mood, arg1)
 end
 
 When /^I greet him with "(.*)"$/ do |arg1|

Diff for: lib/restaurant.rb

@@ -13,10 +13,11 @@ def initialize
 
 # check item on menu
   def take_order(item)
-     !$menu[item].nil?
+     $menu[item].nil? ? "So sorry we don't have #{item}" : "I will take your order straight to the kitchen"
   end
 
   def place_order(order)
+    "Dear chef can you please prepare this order?"
   end
 
   def serve_order(order)
@@ -36,7 +37,7 @@ def take_money
 
 
 class Customer
-  attr_accessor :mood # don't really want mood up here - need it for testing
+ # attr_accessor :mood # don't really want mood up here - need it for testing - now using instance_variable_set
   attr_reader :order, :mood, :tip
 
   def initialize
@@ -68,23 +69,6 @@ def leave
 end
 
 
-class Chef
-  def initialize
-  end
-  
-  def cook(item)
-  end
-  
-  def prepare (ingrediant)
-  end
-  
-  def manage(orders)
-  end
-  
-  def ready_order(order)
-  end
-end
-
 class Order
 # something funky going on with the names in here. too many 'orders'
 # probably not a good practice- can cause confusion...

Diff for: test/restaurant_test.rb

@@ -3,34 +3,41 @@
   before do
      @w=Waiter.new
    end
+   
   it 'should take orders' do
     assert_respond_to @w, :take_order
   end
 
   it 'should only accept order on menu' do
-    assert @w.take_order(:pizza)
-    refute @w.take_order(1)
-    assert !@w.take_order(:flanksteak)
+    assert_equal  "I will take your order straight to the kitchen", @w.take_order(:pizza) # these tests makes my skin crawl! So hard coded. 
+    assert_equal "So sorry we don't have 1", @w.take_order(1)
+    refute_equal "I will take your order straight to the kitchen", @w.take_order(:flanksteak)
   end
 
   it 'should place order' do
-    assert_respond_to @w, :place_order
-   
+    assert_respond_to @w, :place_order 
+    @mo=MiniTest::Mock.new
+    @mo.expect(:order, [:pizza, :plankton])
+    assert_equal "Dear chef can you please prepare this order?", @w.place_order(@mo.order)
   end
 
   it 'should serve order' do
     assert_respond_to @w, :serve_order
+    
   end
 
   it 'should greet customer' do
     assert_respond_to @w, :greet
     refute_nil @w.greet
+    assert_includes($greetings.keys, @w.greet)
+  # refute_includes($menu.keys, @w.greet) # this is a bad test because theoretically the greeting could also be on the menu. E.g.,the waiter could greet the customer by saying 'plankton'
   end
 
 
   it 'should deliver check' do
     assert_respond_to @w, :deliver_check
   end
+  
   it 'should take money' do
     assert_respond_to @w, :take_money
   end
@@ -55,7 +62,7 @@
 
 
 it 'should pay the bill'do
-assert_respond_to @c, :pay
+  assert_respond_to @c, :pay
 end
 
 
@@ -66,28 +73,6 @@
 it 'should tip based on its mood' do
   # this should be mocked?
   # or it could be a feature?
-  
-end
-end
-
-describe Chef do
-  before do
-    @chef=Chef.new
   end
-    it 'should cook the item ordered' do
-      assert_respond_to @chef, :cook
-    end
-  
-it 'should prepare ingrediants'do
- assert_respond_to @chef, :prepare
- end
-  
-  it 'should manage multiple orders' do
-  assert_respond_to @chef, :manage
-  # not sure about this one...
 end
-it 'should ready an order' do
-  assert_respond_to @chef, :ready_order
-  end
-  
-end
+