Merge pull request #10 from kvsm/Two-bucket-regenerate_tests

Fix two-bucket solution

Added short-cut for case where non-starting bucket size is equal to goal

exercises/two-bucket/.meta/solutions/two_bucket.rb

@@ -15,11 +15,15 @@ def initialize(first_size, second_size, goal, start_bucket)
   def moves
     if @start_bucket == 'one'
       @levels = [@first_size, 0]
-      solve method(:start_from_first)
+      strategy = method(:start_from_first)
     else
       @levels = [0, @second_size]
-      solve method(:start_from_second)
+      strategy = method(:start_from_second)
     end
+    if other_bucket_matches_goal
+      strategy = method(:fill_other_bucket)
+    end
+    solve strategy
   end
 
   private
@@ -68,6 +72,22 @@ def start_from_second
     end
   end
 
+  def fill_other_bucket
+    if first_bucket_empty?
+      fill_first_bucket
+    else
+      fill_second_bucket
+    end
+  end
+
+  def other_bucket_matches_goal
+    if @start_bucket == 'one'
+      return @second_size == @goal
+    else
+      return @first_size == @goal
+    end
+  end
+
   def first_bucket_empty?
     @levels.first == 0
   end