Skip to content

gh-114563: C decimal falls back to pydecimal for unsupported format strings #114879

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
serhiy-storchaka merged 12 commits into from
Feb 12, 2024
+88 −122
Merged

gh-114563: C decimal falls back to pydecimal for unsupported format strings #114879

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d49d679
add decimal tests
belm0 Feb 1, 2024
b526004
revert 'z' format support in C decimal
belm0 Feb 1, 2024
6ac60d7
gh-114563: C decimal falls back to pydecimal for unsupported format s…
belm0 Feb 1, 2024
d37b5b8
Apply suggestions from code review
belm0 Feb 10, 2024
d2b6893
disable failing null fill test
belm0 Feb 10, 2024
feb7469
only override message string on ValueError
belm0 Feb 10, 2024
5803d9c
corrected null fill char test
belm0 Feb 10, 2024
dcee58f
honor round and capitals context in fallback
belm0 Feb 11, 2024
7a2ba4b
format fallback context tests
belm0 Feb 11, 2024
ab4f2d7
pass context directly to __format__()
belm0 Feb 11, 2024
3b1b8f4
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 11, 2024
06e371c
fix NEWS formatting
belm0 Feb 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
format fallback context tests
  • Loading branch information
@belm0
belm0 committed Feb 11, 2024
commit 7a2ba4bf2acccdec766c4f7e2633c5df8921c018
15 changes: 15 additions & 0 deletions Lib/test/test_decimal.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5733,6 +5733,21 @@ def test_c_signaldict_segfault(self):
with self.assertRaisesRegex(ValueError, err_msg):
sd.copy()

def test_format_fallback_capitals(self):
# Fallback to _pydecimal formatting (triggered by `#` format which
# is unsupported by mpdecimal) should honor the current context.
x = C.Decimal('6.09e+23')
self.assertEqual(format(x, '#'), '6.09E+23')
with C.localcontext(capitals=0):
self.assertEqual(format(x, '#'), '6.09e+23')

def test_format_fallback_rounding(self):
y = C.Decimal('6.09')
self.assertEqual(format(y, '#.1f'), '6.1')
with C.localcontext(rounding=C.ROUND_DOWN):
self.assertEqual(format(y, '#.1f'), '6.0')


@requires_docstrings
@requires_cdecimal
class SignatureTest(unittest.TestCase):
Expand Down