On 1/23/19 9:46 AM, Max Reitz wrote:
> Strings can have various representations, this helper function checks
> whether a given value is in any of them.
>
> Signed-off-by: Max Reitz <mre...@redhat.com>
> ---
> tests/qemu-iotests/iotests.py | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
> index f51456fa63..29e4e3306f 100644
> --- a/tests/qemu-iotests/iotests.py
> +++ b/tests/qemu-iotests/iotests.py
> @@ -221,6 +221,12 @@ def image_size(img):
> r = qemu_img_pipe('info', '--output=json', '-f', imgfmt, img)
> return json.loads(r)['virtual-size']
>
> +def is_str(val):
> + if sys.version_info.major >= 3:
> + return isinstance(val, str) or isinstance(val, bytes)
> + else:
> + return isinstance(val, str) or isinstance(val, unicode)
> +
conflating bytes with strings under a function named "is_str()" makes me
nervous, but... I have no concrete objections beyond hypothetically
"What if people take this function name literally?"
*prays for 2.x support to get dropped*
Reviewed-by: John Snow <js...@redhat.com>