8000 GH-90829: Fix empty iterable error message in min/max by Nnarol · Pull Request #31181 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

GH-90829: Fix empty iterabl 8000 e error message in min/max #31181

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.

Sign up for GitHub

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

Merged
merged 7 commits into from
Jan 8, 2023
Merged

GH-90829: Fix empty iterable error message in min/max #31181

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6802cfc
Fix empty iter error message in min/max
Feb 6, 2022
2f9b0ea
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 7, 2022
4b9ab20
Update Misc/NEWS.d/next/Core and Builtins/2022-02-07-17-40-23.bpo-466…
Nnarol Feb 8, 2022
a55644b
Merge branch 'main' into fix-min-max-empty-it-error-msg-46671
JelleZijlstra May 2, 2022
f8dcf8a
newline in NEWS file
JelleZijlstra May 3, 2022
74ea60f
Delete 2022-02-07-17-40-23.bpo-46671.CmpNlQ.rst
rhettinger May 5, 2022
859848a
Merge branch 'main' into fix-min-max-empty-it-error-msg-46671
kumaraditya303 Jan 8, 2023
File filter

Filter by extension

Filter by extension
< 8000 strong class="js-conversation-menu-button">Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Fix empty iter error message in min/max 
Change misleading "empty sequence" to "empty iterable" in builtins.min/max.
Update corresponding test to be more specific and match the error string.
  • Loading branch information
Nnarol committed Feb 7, 2022
commit 6802cfcca9cdcfd9a75b173f9b1b1392fd962c01
12 changes: 10 additions & 2 deletions Lib/test/test_builtin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1031,7 +1031,11 @@ def test_max(self):
max()

self.assertRaises(TypeError, max, 42)
self.assertRaises(ValueError, max, ())
with self.assertRaisesRegex(
ValueError,
r'max\(\) iterable argument is empty'
):
max(())
class BadSeq:
def __getitem__(self, index):
raise ValueError
Expand Down Expand Up @@ -1090,7 +1094,11 @@ def test_min(self):
min()

self.assertRaises(TypeError, min, 42)
self.assertRaises(ValueError, min, ())
with self.assertRaisesRegex(
ValueError,
r'min\(\) iterable argument is empty'
):
min(())
class BadSeq:
def __getitem__(self, index):
raise ValueError
Expand Down
2 changes: 1 addition & 1 deletion Python/bltinmodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1786,7 +1786,7 @@ min_max(PyObject *args, PyObject *kwds, int op)
maxitem = defaultval;
} else {
PyErr_Format(PyExc_ValueError,
"%s() arg is an empty sequence", name);
"%s() iterable argument is empty", name);
}
}
else
Expand Down
0