[GR-9749] List.index ends with IndexError instead ValueError.

PullRequest: graalpython-open/35

Author: ppisl

graalpython/com.oracle.graal.python.test/src/tests/list_tests.py

@@ -361,7 +361,7 @@ def __eq__(self, other):
         for x, y in zip(d, e):
             # verify that original order and values are retained.
             self.assertIs(x, y)
-    '''
+
     def test_count(self):
         a = self.type2test([0, 1, 2])*3
         self.assertEqual(a.count(0), 3)
@@ -447,7 +447,7 @@ def test_reverse(self):
         self.assertEqual(u, u2)
 
         self.assertRaises(TypeError, u.reverse, 42)
-    '''
+
     def test_clear(self):
         u = self.type2test([2, 3, 4])
         u.clear()

graalpython/com.oracle.graal.python.test/src/tests/test_list.py

@@ -303,6 +303,7 @@ class MyInt(int):
         a = [1,2,3]
         a.remove(MyInt(2))
         self.assertEqual([1,3], a)
+
     def test_insert_spec(self):
         a = [1,2]
         self.assertRaises(TypeError, a.insert, [1,2,3], 1)

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/list/ListBuiltins.java

@@ -808,8 +808,7 @@ private static int correctIndex(PList list, PInt index) {
         }
 
         private int findIndex(PList list, Object value, int start, int end, BinaryComparisonNode eqNode) {
-            int len = list.len();
-            for (int i = start; i < end && i < len; i++) {
+            for (int i = start; i < end && i < list.len(); i++) {
                 Object object = list.getItem(i);
                 if (eqNode.executeBool(object, value)) {
                     return i;