Minor: Docstring and type hints

Author: Sylvain MARIE

pytest_cases/common_pytest_lazy_values.py

@@ -132,20 +132,6 @@ def __init__(self,
                  id=None,      # type: str  # noqa
                  marks=()      # type: Union[Any, Sequence[Any]]
                  ):
-        """
-        Creates a reference to a value getter, to be used in `parametrize_plus`.
-
-        A `lazy_value` is the same thing than a function-scoped fixture, except that the value getter function is not a
-        fixture and therefore can neither be parametrized nor depend on fixtures. It should have no mandatory argument.
-
-        Note that a `lazy_value` can be included in a `pytest.param` without problem. In that case the id defined by
-        `pytest.param` will take precedence over the one defined in `lazy_value` if any. The marks, however,
-        will all be kept wherever they are defined.
-
-        :param valuegetter: a callable without mandatory arguments
-        :param id: an optional id. Otherwise `valuegetter.__name__` will be used by default
-        :param marks: optional marks. `valuegetter` marks will also be preserved.
-        """
         self.valuegetter = valuegetter
         self._id = id
         if isinstance(marks, (tuple, list, set)):
@@ -289,6 +275,20 @@ def lazy_value(valuegetter,  # type: Callable[[], Any]
                id=None,      # type: str  # noqa
                marks=()      # type: Union[Any, Sequence[Any]]
                ):
+    """
+    Creates a reference to a value getter, to be used in `parametrize_plus`.
+
+    A `lazy_value` is the same thing than a function-scoped fixture, except that the value getter function is not a
+    fixture and therefore can neither be parametrized nor depend on fixtures. It should have no mandatory argument.
+
+    Note that a `lazy_value` can be included in a `pytest.param` without problem. In that case the id defined by
+    `pytest.param` will take precedence over the one defined in `lazy_value` if any. The marks, however,
+    will all be kept wherever they are defined.
+
+    :param valuegetter: a callable without mandatory arguments
+    :param id: an optional id. Otherwise `valuegetter.__name__` will be used by default
+    :param marks: optional marks. `valuegetter` marks will also be preserved.
+    """
     return LazyValue(valuegetter, id=id, marks=marks)
 
 

pytest_cases/fixture_core2.py

@@ -29,7 +29,7 @@
 
 
 def param_fixture(argname,           # type: str
-                  argvalues,         # type: Sequence[Any]
+                  argvalues,         # type: Iterable[Any]
                   autouse=False,     # type: bool
                   ids=None,          # type: Union[Callable, List[str]]
                   scope="function",  # type: str
@@ -126,8 +126,8 @@ def __param_fixture(request):
     return fix
 
 
-def param_fixtures(argnames,
-                   argvalues,
+def param_fixtures(argnames,          # type: str
+                   argvalues,         # type: Iterable[Any]
                    autouse=False,     # type: bool
                    ids=None,          # type: Union[Callable, List[str]]
                    scope="function",  # type: str
@@ -284,6 +284,8 @@ def fixture_plus(scope="function",  # type: str
      - it supports a new argument `unpack_into` where you can provide names for fixtures where to unpack this fixture
        into.
 
+    As a consequence it does not support the `params` and `ids` arguments anymore.
+
     :param scope: the scope for which this fixture is shared, one of "function" (default), "class", "module" or
         "session".
     :param autouse: if True, the fixture func is activated for all tests that can see it.  If False (the default) then

pytest_cases/fixture_parametrize_plus.py

@@ -119,7 +119,7 @@ def _new_fixture(**all_fixtures):
 
 class fixture_ref(object):  # noqa
     """
-    A reference to a fixture, to be used in `parametrize_plus`.
+    A reference to a fixture, to be used in `@parametrize_plus`.
     You can create it from a fixture name or a fixture object (function).
     """
     __slots__ = 'fixture',
@@ -273,8 +273,8 @@ def get(cls, style  # type: str
 _IDGEN = object()
 
 
-def parametrize_plus(argnames=None,
-                     argvalues=None,
+def parametrize_plus(argnames=None,       # type: str
+                     argvalues=None,      # type: Iterable[Any]
                      indirect=False,      # type: bool
                      ids=None,            # type: Union[Callable, List[str]]
                      idstyle='explicit',  # type: str