Incorrect error message in the module asyncio.selector_events.

[Paradisee]

BPO	26125
Nosy	@gvanrossum, @vstinner, @ezio-melotti, @berkerpeksag, @1st1
Files	patch.diff: Patch patch.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2016-02-01.17:57:41.022>
created_at = <Date 2016-01-15.13:52:59.768>
labels = ['type-bug', 'expert-asyncio']
title = 'Incorrect error message in the module asyncio.selector_events.'
updated_at = <Date 2016-02-01.17:57:41.018>
user = 'https://bugs.python.org/Paradisee'

bugs.python.org fields:

activity = <Date 2016-02-01.17:57:41.018>
actor = 'berker.peksag'
assignee = 'none'
closed = True
closed_date = <Date 2016-02-01.17:57:41.022>
closer = 'berker.peksag'
components = ['asyncio']
creation = <Date 2016-01-15.13:52:59.768>
creator = 'Paradisee'
dependencies = []
files = ['41624', '41625']
hgrepos = []
issue_num = 26125
keywords = ['patch']
message_count = 7.0
messages = ['258294', '258296', '258301', '258307', '259203', '259211', '259338']
nosy_count = 6.0
nosy_names = ['gvanrossum', 'vstinner', 'ezio.melotti', 'berker.peksag', 'yselivanov', 'Paradisee']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue26125'
versions = ['Python 3.5', 'Python 3.6']

[Paradisee]

Incorrect error message in the module asyncio.selector_events for the methods:
_SelectorSocketTransport.write
_SelectorSslTransport.write
_SelectorDatagramTransport.sendto.

The previous error was raising a Tuple:
TypeError: ('data argument must be byte-ish (%r)', <class 'str'>)

Patched:
TypeError: data argument must be a bytes-like object, not 'str'

[ezio-melotti]

LGTM unless you think we should also add a test that checks that the name of the incorrect type is included in the error message.

[gvanrossum]

Remember also to apply to asyncio's own GitHub repo.

--Guido (mobile)
On Jan 15, 2016 6:03 AM, "Ezio Melotti" <report@bugs.python.org> wrote:

Ezio Melotti added the comment:

LGTM unless you think we should also add a test that checks that the name
of the incorrect type is included in the error message.

----------
nosy: +ezio.melotti
stage: -> commit review
type: -> behavior
versions: +Python 3.5, Python 3.6 -Python 3.4

---

Python tracker <report@bugs.python.org>
<http://bugs.python.org/issue26125\>

---

[Paradisee]

python/asyncio#313

[berkerpeksag]

python/asyncio#313 has been merged. Do we need to commit this patch to the CPython repo or will it be merged with the next sync? (e.g. f4fe55dd5659)

[gvanrossum]

Usually Yury or Victor takes care of this sooner or later. If you want
to you can keep this open as a release blocker with that as a task.

[berkerpeksag]

Victor committed this in 97c80e317ab8 (3.5) and c7f1acdd8be1 (default). Closing as "fixed".