[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Yeah, my first patch applied :) thanks, David -- ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue17987] test.support.captured_stderr, captured_stdin not documented

R. David Murray added the comment: Thanks, Dmi. -- nosy: +r.david.murray resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker

[issue17987] test.support.captured_stderr, captured_stdin not documented

Roundup Robot added the comment: New changeset af2416c2e27c by R David Murray in branch '3.3': #17987: properly document support.captured_xxx. http://hg.python.org/cpython/rev/af2416c2e27c New changeset d0f7f1996001 by R David Murray in branch 'default': Merge #17987: properly document support.c

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Amended -- Added file: http://bugs.python.org/file30430/issue17987_6.patch ___ Python tracker ___ ___ P

[issue17987] test.support.captured_stderr, captured_stdin not documented

Serhiy Storchaka added the comment: LGTM with Fred's suggestion. -- assignee: docs@python -> fdrake ___ Python tracker ___ ___ Python-

[issue17987] test.support.captured_stderr, captured_stdin not documented

Fred L. Drake, Jr. added the comment: >From v5 of the patch: + A context managers that temporarily replaces the :data:`sys.stdin` / + :data:`sys.stdout` / :data:`sys.stderr` stream with :class:`io.StringIO` + object. I'd go with singular nouns instead of trying to map across them with plur

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Fred or Serhiy - any news here? I'm signed Contributor Agreement few days ago, just waiting a change in my profile here, please don't worry about copyright :-) -- ___ Python tracker

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Thanks for review, Serhiy - updated. -- Added file: http://bugs.python.org/file30400/issue17987_5.patch ___ Python tracker ___

[issue17987] test.support.captured_stderr, captured_stdin not documented

Serhiy Storchaka added the comment: > I'd rather see captured_stdin handled separately, perhaps with some > additional comments in the example, to emphasize the intended usage > pattern: However all three functions share a description. sys.stdin also has totally different usage pattern than sys

[issue17987] test.support.captured_stderr, captured_stdin not documented

Fred L. Drake, Jr. added the comment: +1 for issue17987_4.patch Thanks, Dmi! -- ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue17987] test.support.captured_stderr, captured_stdin not documented

Changes by Eli Bendersky : -- nosy: -eli.bendersky ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Amended, as discussed. Also changed documentation from "how to use a context managers with that" to "how it works". -- Added file: http://bugs.python.org/file30398/issue17987_4.patch ___ Python tracker

[issue17987] test.support.captured_stderr, captured_stdin not documented

Fred L. Drake, Jr. added the comment: Joining the documentation for captured_stderr and captured_stdout makes sense, as they can really use a single example, and the usage is completely parallel. I'd rather see captured_stdin handled separately, perhaps with some additional comments in the examp

[issue17987] test.support.captured_stderr, captured_stdin not documented

Serhiy Storchaka added the comment: I think it will be better to describe all three functions together (as sys.stdin/stdout/stderr). .. function:: captured_stdin() captured_stdout() captured_stderr() ... description ... Example use:: ... example for captu

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: `captured_stdin` test changed, docs updated. -- Added file: http://bugs.python.org/file30396/issue17987_3.patch ___ Python tracker ___

[issue17987] test.support.captured_stderr, captured_stdin not documented

Serhiy Storchaka added the comment: I believe it is a mechanical mistake in test_captured_stdin(). -- nosy: +eli.bendersky versions: -Python 3.5 ___ Python tracker ___ _

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Yes, I agree that this is not an obvious sample, but: $ ./python -m test.test_support | grep test_captured_stdin test_captured_stdin (__main__.TestSupport) ... ok What's about that? with captured_stdin() as s: s.write('hello\n') s.seek(0) captured = i

[issue17987] test.support.captured_stderr, captured_stdin not documented

Serhiy Storchaka added the comment: Print to stdin? -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Sorry for noise, missed changes in Doc/library/test.rst. Updated patch added -- hgrepos: +192 Added file: http://bugs.python.org/file30389/issue17987_2.patch ___ Python tracker __

[issue17987] test.support.captured_stderr, captured_stdin not documented

Dmi Baranov added the comment: Added a patch -- keywords: +patch nosy: +dmi.baranov Added file: http://bugs.python.org/file30387/issue17987.patch ___ Python tracker ___ _

[issue17987] test.support.captured_stderr, captured_stdin not documented

New submission from Fred L. Drake, Jr.: The captured_stderr and captured_stdin context managers aren't documented, and should be. -- assignee: docs@python components: Documentation keywords: easy messages: 189311 nosy: docs@python, fdrake priority: normal severity: normal stage: needs p