[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v9 08/14] iotests: touch up log function signature
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v9 08/14] iotests: touch up log function signature |
Date: |
Mon, 30 Mar 2020 18:19:45 +0200 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
Am 25.03.2020 um 00:20 hat John Snow geschrieben:
> Representing nested, recursive data structures in mypy is notoriously
> difficult; the best we can reliably do right now is denote the atom
> types as "Any" while describing the general shape of the data.
>
> Regardless, this fully annotates the log() function.
>
> Typing notes:
>
> TypeVar is a Type variable that can optionally be constrained by a
> sequence of possible types. This variable is bound per-invocation such
> that the signature for filter=() requires that its callables take e.g. a
> str and return a str.
>
> Signed-off-by: John Snow <address@hidden>
I like it. Does your version of mypy accept this? I actually get a
warning that doesn't make sense to me:
iotests.py:392: error: Argument 1 to "info" of "Logger" has incompatible
type "Dict[str, Any]"; expected "str"
The code looks like this:
if isinstance(msg, (dict, list)):
# Don't sort if it's already sorted
do_sort = not isinstance(msg, OrderedDict)
test_logger.info(json.dumps(msg, sort_keys=do_sort, indent=indent))
else:
test_logger.info(msg)
I have no idea why it would think it can still be Dict[str, Any] in the
else branch. Even after adding an 'assert not instanceof(msg, dict), it
still thinks so.
Probably time to update for me...
Kevin
- Re: [PATCH v9 05/14] iotests: add pylintrc file, (continued)
- [PATCH v9 06/14] iotests: alphabetize standard imports, John Snow, 2020/03/24
- [PATCH v9 07/14] iotests: drop pre-Python 3.4 compatibility code, John Snow, 2020/03/24
- [PATCH v9 08/14] iotests: touch up log function signature, John Snow, 2020/03/24
- [PATCH v9 10/14] iotests: add hmp helper with logging, John Snow, 2020/03/24
- [PATCH v9 12/14] iotest 258: use script_main, John Snow, 2020/03/24
- [PATCH v9 09/14] iotests: limit line length to 79 chars, John Snow, 2020/03/24
- [PATCH v9 13/14] iotests: Mark verify functions as private, John Snow, 2020/03/24