[issue18971] Use argparse in the profile/cProfile modules

2018-07-11 Thread Chih-Hsuan Yen 


Change by Chih-Hsuan Yen :


--
nosy: +yan12125

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2017-04-20 Thread Louie Lu 

Louie Lu added the comment:

haypo, murray, thanks for pointing this thing, I will let go about the bad 
message test, move to output file and sort test tomorrow, also add a new issue: 
#30118

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2017-04-20 Thread R. David Murray 

R. David Murray added the comment:

Well, part of the point of the tests is to make sure the module's behavior 
doesn't change when doing the conversion to argparse.  But "behavior changing" 
doesn't extend to the exact format of the output...that can change, as long as 
what the module *does* given specific cli argument combinations stays the same. 
 Ideally, you write the tests so that it is not sensitive to the exact output, 
but realistically some tests may need to be different between the optparse case 
and the argparse case to get good coverage.

Note that even with good tests, you can still miss behavior changes.  We tried 
to do a good job on the tests for compileall, for example, and we still 
introduced a bug or two when we converted to argparse.

--
nosy: +r.david.murray

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2017-04-20 Thread STINNER Victor 

STINNER Victor added the comment:

> bad news, somehow the output of std.err and std.out have different between 
> optparse and argparse

Only test argparse. I don't think that it matters to test that the change 
doesn't change the behaviour.

You can also redirect stderr to stdout.

What do you want to test?

--
nosy: +haypo

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2017-04-20 Thread Louie Lu 

Louie Lu added the comment:

bad news, somehow the output of std.err and std.out have different between 
optparse and argparse, even if the test is done, it still need to convert some 
testcase to argparse compatible.

--
nosy: +louielu

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2016-12-01 Thread Wolfgang Maier 

Wolfgang Maier added the comment:

oops, typing in wrong window. Very sorry.

--
nosy: +wolma
title: calendar -> Use argparse in the profile/cProfile modules

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2013-09-13 Thread Ezio Melotti 

Ezio Melotti added the comment:

Is the command-line interface covered by tests?

--
nosy: +ezio.melotti

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18971
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2013-09-13 Thread Serhiy Storchaka 

Serhiy Storchaka added the comment:

No. And I know your next proposition. :-(

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18971
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18971] Use argparse in the profile/cProfile modules

2013-09-08 Thread Serhiy Storchaka 

New submission from Serhiy Storchaka:

Here is a patch which replaces optparse to argparse in the profile and cProfile 
modules.

--
components: Library (Lib)
files: profile_argparse.patch
keywords: patch
messages: 197262
nosy: bethard, georg.brandl, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Use argparse in the profile/cProfile modules
type: enhancement
versions: Python 3.4
Added file: http://bugs.python.org/file31665/profile_argparse.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18971
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com