simplifying senses behavior so it acts on a sense directly instead of…

… memorizing each possible argument

lib/ruby_warrior/abilities/direction_of_stairs.rb

@@ -5,10 +5,6 @@ def description
         "Returns the direction (:left, :right, :forward, :backward) the stairs are from your location."
       end
 
-      def possible_arguments
-        [[]]
-      end
-
       def perform
         @unit.position.relative_direction_of_stairs
       end

lib/ruby_warrior/abilities/distance.rb

@@ -5,10 +5,6 @@ def description
         "Returns the number of spaces the stairs are away."
       end
 
-      def possible_arguments
-        [[]]
-      end
-
       def perform
         @unit.position.distance_from_stairs
       end

lib/ruby_warrior/abilities/feel.rb

@@ -5,10 +5,6 @@ def description
         "Returns a Space for the given direction (forward by default)."
       end
 
-      def possible_arguments
-        [[], *DIRECTIONS.keys]
-      end
-
       def perform(direction = :forward)
         space(direction)
       end

lib/ruby_warrior/abilities/health.rb

@@ -5,10 +5,6 @@ def description
         "Returns an integer representing your health."
       end
 
-      def possible_arguments
-        [[]]
-      end
-
       def perform
         @unit.health
       end

lib/ruby_warrior/abilities/listen.rb

@@ -5,10 +5,6 @@ def description
         "Returns an array of all spaces which have units in them."
       end
 
-      def possible_arguments
-        [[]]
-      end
-
       def perform
         @unit.position.floor.units.map do |unit|
           unit.position.space unless unit == @unit

lib/ruby_warrior/abilities/look.rb

@@ -5,10 +5,6 @@ def description
         "Returns an array of Spaces in given direction (forward by default)."
       end
 
-      def possible_arguments
-        [[], :forward, :left, :right]
-      end
-
       def perform(direction = :forward)
         [1, 2, 3].map do |amount|
           space(direction, amount)

lib/ruby_warrior/turn.rb

@@ -29,13 +29,10 @@ def #{action}(*args)
     def add_sense(name, sense)
       instance_eval <<-EOS
         def #{name}(*args)
-          @senses.fetch([:#{name}, args])
+          @senses[:#{name}].perform(*args)
         end
       EOS
-      sense.possible_arguments.each do |args|
-        args = [args].flatten # in case args aren't in array
-        @senses[[name, args]] = sense.perform(*args)
-      end
+      @senses[name] = sense
     end
   end
 end

spec/ruby_warrior/abilities/feel_spec.rb

@@ -10,10 +10,4 @@
     @unit.position.expects(:relative_space).with(1, 0)
     @feel.perform(:forward)
   end
-
-  it "should have all directions as possible arguments" do
-    [[], :forward, :backward, :left, :right].each do |arg|
-      @feel.possible_arguments.should include(arg)
-    end
-  end
 end

spec/ruby_warrior/abilities/health_spec.rb

@@ -10,8 +10,4 @@
     @warrior.health = 10
     @health.perform.should == 10
   end
-
-  it "should have no possible arguments" do
-    @health.possible_arguments.should == [[]]
-  end
 end

spec/ruby_warrior/abilities/look_spec.rb

@@ -12,8 +12,4 @@
     @unit.position.expects(:relative_space).with(3, 0).returns(3)
     @feel.perform(:forward).should == [1, 2, 3]
   end
-
-  it "should have all directions except back as possible arguments" do
-    @feel.possible_arguments.to_set.should == [[], :forward, :left, :right].to_set
-  end
 end

spec/ruby_warrior/turn_spec.rb

@@ -38,21 +38,5 @@
       @turn.feel.should == @feel.perform
       @turn.feel(:backward).should == @feel.perform(:backward)
     end
-
-    it "should have memorized result" do
-      result = @feel.perform(:backward)
-      @feel.stubs(:space).with(:backward).returns(nil)
-      @turn.feel(:backward).should == result
-    end
-
-    it "should have memorized result" do
-      result = @feel.perform(:backward)
-      @feel.stubs(:space).with(:backward).returns(nil)
-      @turn.feel(:backward).should == result
-    end
-
-    it "should raise an exception for unexpected argument" do
-      lambda { @turn.feel(:bad) }.should raise_error
-    end
   end
 end