Merge pull request #1981 from k-tsj/fix-method-name

Fix method name in Array#reverse_each

2 files changed, 11 insertions, 1 deletions

diff --git a/mrbgems/mruby-array-ext/mrblib/array.rb b/mrbgems/mruby-array-ext/mrblib/array.rb
index 18ed5e303..2e66c5fbc 100644
--- a/mrbgems/mruby-array-ext/mrblib/array.rb
+++ b/mrbgems/mruby-array-ext/mrblib/array.rb
@@ -204,7 +204,7 @@ class Array
 
   # for efficiency
   def reverse_each(&block)
-    return to_enum :sort_by unless block_given?
+    return to_enum :reverse_each unless block_given?
 
     i = self.size - 1
     while i>=0
diff --git a/mrbgems/mruby-array-ext/test/array.rb b/mrbgems/mruby-array-ext/test/array.rb
index 7f296f591..ed1edb5c2 100644
--- a/mrbgems/mruby-array-ext/test/array.rb
+++ b/mrbgems/mruby-array-ext/test/array.rb
@@ -131,3 +131,13 @@ assert("Array#fill") do
   assert_equal [1, 2, 3, 27], a.fill(0..1) { |i| i+1 }
   assert_raise(ArgumentError) { a.fill }
 end
+
+assert("Array#reverse_each") do
+  a = [ "a", "b", "c", "d" ]
+  b = []
+  a.reverse_each do |i|
+    b << i
+  end
+  assert_equal [ "d", "c", "b", "a" ], b
+  assert_equal [ "d", "c", "b", "a" ], a.reverse_each.to_a
+end