repr(ElementList()) should return repr() of the internal container (#…

…1045)

splinter/element_list.py

@@ -8,17 +8,18 @@
 
 
 class ElementList:
-    """
-    List of elements. Each member of the list is (usually) an instance
+    """Collection of elements.
+
+    Each member of the collection is by default an instance
     of :class:`ElementAPI <splinter.driver.ElementAPI>`.
 
-    Beyond the traditional list methods, the ``ElementList`` provides some
+    Beyond the traditional list methods, ``ElementList`` provides some
     other methods, listed below.
 
     There is a peculiar behavior on ElementList: you never get an
-    ``IndexError``. Instead, you can an :class:`ElementDoesNotExist
+    ``IndexError``. Instead, you get an :class:`ElementDoesNotExist
     <splinter.exceptions.ElementDoesNotExist>` exception when trying to
-    access an inexistent item in the list:
+    access a non-existent item:
 
         >>> element_list = ElementList([])
         >>> element_list[0] # raises ElementDoesNotExist
@@ -64,7 +65,7 @@ def last(self):
         """
         return self[-1]
 
-    def is_empty(self):
+    def is_empty(self) -> bool:
         """Check if the ElementList is empty.
 
         Returns:
@@ -92,3 +93,7 @@ def __iter__(self):
     def __len__(self):
         """__len__ checks the internal container."""
         return len(self._container)
+
+    def __repr__(self):
+        """Return the repr of the internal container."""
+        return repr(self._container)

tests/test_element_list.py

@@ -108,3 +108,12 @@ def test_not_found_exception_with_query_and_method():
 
     expected_message = 'no elements could be found with id "menu"'
     assert expected_message == str(e.value)
+
+
+def test_elementlist_repr():
+    """repr() of ElementList is identical to repr() of the internal container.
+    """
+    the_list = [Person(), Person()]
+    elementlist = ElementList(the_list)
+
+    assert repr(elementlist) == repr(the_list)