From eef0b1d562b914ae28433e568b42d9dcf3bf9c04 Mon Sep 17 00:00:00 2001 From: Jelle Zijlstra Date: Mon, 4 Jun 2018 15:34:09 -0700 Subject: [PATCH] fix type for TestCase.assertIn (#2186) * fix type for TestCase.assertIn This does essentially `assert member in container`, so we want a `Container`, not an `Iterable`. This came up in https://github.com/ambv/black/pull/288/files/68e9d426a86edc187a3f58ea889a2002620f0de6..0bbee43d60dfc16d8bbdd0bbdaad754a2c8c7ec0#r192525658. * use any for assertIn --- stdlib/3/unittest/__init__.pyi | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/stdlib/3/unittest/__init__.pyi b/stdlib/3/unittest/__init__.pyi index febc71883..f78a9a4bd 100644 --- a/stdlib/3/unittest/__init__.pyi +++ b/stdlib/3/unittest/__init__.pyi @@ -1,9 +1,9 @@ # Stubs for unittest from typing import ( - Any, Callable, ContextManager, Dict, FrozenSet, Generic, Iterable, Iterator, - List, NoReturn, Optional, overload, Pattern, Sequence, Set, TextIO, Tuple, - Type, TypeVar, Union + Any, Callable, Container, ContextManager, Dict, FrozenSet, Generic, Iterable, + Iterator, List, NoReturn, Optional, overload, Pattern, Sequence, Set, TextIO, + Tuple, Type, TypeVar, Union ) import logging import sys @@ -50,9 +50,9 @@ class TestCase: msg: Any = ...) -> None: ... def assertIsNone(self, expr: Any, msg: Any = ...) -> None: ... def assertIsNotNone(self, expr: Any, msg: Any = ...) -> None: ... - def assertIn(self, first: _T, second: Iterable[_T], + def assertIn(self, member: Any, container: Container[Any], msg: Any = ...) -> None: ... - def assertNotIn(self, first: _T, second: Iterable[_T], + def assertNotIn(self, member: Any, container: Container[Any], msg: Any = ...) -> None: ... def assertIsInstance(self, obj: Any, cls: Union[type, Tuple[type, ...]],