add block option to min and max

Author: monkstone

lib/jruby_art/helper_methods.rb

@@ -77,15 +77,17 @@ def thread(&block)
     # explicitly provide 'processing.org' min instance method
     # to return a float:- a, b and c need to be floats
 
-    def min(*args)
-      args.min # { |a,b| a <=> b } optional block not reqd
+    def min(*args, &block)
+      args.min unless block_given?
+      args.min(&block) # { |a, b| a.value <=> b.value }
     end
 
     # explicitly provide 'processing.org' max instance method
     # to return a float:- a, b and c need to be floats
 
-    def max(*args)
-      args.max # { |a, b| a <=> b } optional block not reqd
+    def max(*args, &block)
+      args.max unless block_given?
+      args.max(&block) # { |a, b| a.value <=> b.value }
     end
 
     # explicitly provide 'processing.org' dist instance method

test/helper_methods_test.rb

@@ -13,6 +13,8 @@
 Dir.chdir(File.dirname(__FILE__))
 
 class HelperMethodsTest < Minitest::Test
+
+ ARRAY = %w(albatross dog horse)
  def test_hex_color
     col_double = 0.5
     hexcolor = 0xFFCC6600
@@ -47,4 +49,14 @@ def test_dist
     ax, ay, bx, by, cx, cy = 0, 0, 1.0, 0.0, 0, 0
     assert_in_epsilon(dist(ax, ay, bx, by, cx, cy), 1.0, epsilon = 0.0001, msg = 'when x and z dimension are zero')
   end
+
+  def test_min
+    assert_equal(min(*ARRAY), 'albatross')
+    assert_equal(min(*ARRAY) { |a, b| a.length <=> b.length }, 'dog')
+  end
+
+  def test_max
+    assert_equal(max(*ARRAY), 'horse')
+    assert_equal(max(*ARRAY) { |a, b| a.length <=> b.length }, 'albatross')
+  end
 end