add #slice and make #elements use it.

remove extra #clear test.

Author: Mikhail Varabyou

lib/kredis/types/list.rb

@@ -7,11 +7,15 @@ class Kredis::Types::List < Kredis::Types::Proxying
 
   attr_accessor :typed
 
-  def elements(start = 0, stop = -1)
-    strings_to_types(lrange(start, stop) || [], typed)
+  def elements
+    slice(0, -1)
   end
   alias to_a elements
 
+  def slice(start = 0, stop = -1)
+    strings_to_types(lrange(start, stop) || [], typed)
+  end
+
   def remove(*elements)
     types_to_strings(elements, typed).each { |element| lrem 0, element }
   end
@@ -30,11 +34,11 @@ def clear
   end
 
   def first(n = nil)
-    n ? elements(0, n - 1) : elements(0, 0).first
+    n ? slice(0, n - 1) : slice(0, 0).first
   end
 
   def last(n = nil)
-    n ? elements(-n, -1) : elements(-1, -1).first
+    n ? slice(-n, -1) : slice(-1, -1).first
   end
 
   def size

test/types/list_test.rb

@@ -71,15 +71,11 @@ class ListTest < ActiveSupport::TestCase
 
   test "size" do
     @list.clear
+    assert_equal 0, @list.size
     @list.append(%w[ 1 2 3 ])
     assert_equal 3, @list.size
   end
 
-  test "size when list removed" do
-    @list.clear
-    assert_equal 0, @list.size
-  end
-
   test "exists?" do
     assert_not @list.exists?