Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unittest lowercase methods #66446

Closed
simonzack mannequin opened this issue Aug 22, 2014 · 7 comments
Closed

unittest lowercase methods #66446

simonzack mannequin opened this issue Aug 22, 2014 · 7 comments
Labels
stdlib Python modules in the Lib dir

Comments

@simonzack
Copy link
Mannequin

simonzack mannequin commented Aug 22, 2014

BPO 22250
Nosy @ezio-melotti, @bitdancer, @cool-RR, @serhiy-storchaka, @remilapeyre

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 2014-08-22.20:07:36.453>
created_at = <Date 2014-08-22.14:57:28.002>
labels = ['library']
title = 'unittest lowercase methods'
updated_at = <Date 2020-05-28.08:21:11.967>
user = 'https://bugs.python.org/simonzack'

bugs.python.org fields:

activity = <Date 2020-05-28.08:21:11.967>
actor = 'serhiy.storchaka'
assignee = 'none'
closed = True
closed_date = <Date 2014-08-22.20:07:36.453>
closer = 'ezio.melotti'
components = ['Library (Lib)']
creation = <Date 2014-08-22.14:57:28.002>
creator = 'simonzack'
dependencies = []
files = []
hgrepos = []
issue_num = 22250
keywords = []
message_count = 7.0
messages = ['225673', '225698', '225702', '225703', '370170', '370172', '370175']
nosy_count = 6.0
nosy_names = ['ezio.melotti', 'r.david.murray', 'cool-RR', 'serhiy.storchaka', 'simonzack', 'remi.lapeyre']
pr_nums = []
priority = 'normal'
resolution = 'rejected'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue22250'
versions = ['Python 3.4']

@simonzack
Copy link
Mannequin Author

simonzack mannequin commented Aug 22, 2014

The python unittest module currently uses camel case. This feels rather inconsistent with the rest of the python library, which is in lower case and follows PEP-8.

Would it be a good idea to add lower-case aliases, and possibly deprecate the camel case methods in the future?

@simonzack simonzack mannequin added the stdlib Python modules in the Lib dir label Aug 22, 2014
@bitdancer
Copy link
Member

We tried going down this path with the Threading module in the python3 transiontion, and it was a mess. I don't think we are going to do it for any other module.

@serhiy-storchaka
Copy link
Member

assertlessequal looks non-readable and assert_less_equal is two characters longer.

@ezio-melotti
Copy link
Member

This has been already proposed and rejected in the past.
It's not something that is going to happen (at least until Python 4 or 5).

@cool-RR
Copy link
Mannequin

cool-RR mannequin commented May 28, 2020

I see it's been 6 years since the last comment here. I think it's important to revisit these kind of decisions once in a while. Maybe now the time is ripe for a change? We could do it backward compatibility with a long deprecation schedule.

Ezio: You said this was proposed and rejected. Can you link to the discussion? I couldn't find it.

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir
Projects
None yet
Development

No branches or pull requests

3 participants