Remove (or integrate to CI/pyperformance) Module/_decimal/tests
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 77.2k
- Forks
- 35.9k
- PR merge metrics
- PR metrics pending
Description
This is a follow-up of #115119.
The bench.py script already partially integrated to the pyperformance, see https://github.com/python/pyperformance/issues/473.
The Modules/_decimal/tests/deccheck.py currently is broken. Even with the patch
diff --git a/Modules/_decimal/tests/deccheck.py b/Modules/_decimal/tests/deccheck.py
index bf277dd6879..b276700bc01 100644
--- a/Modules/_decimal/tests/deccheck.py
+++ b/Modules/_decimal/tests/deccheck.py
@@ -53,6 +53,8 @@
from formathelper import rand_format, rand_locale
from _pydecimal import _dec_from_triple
+sys.set_int_max_str_digits(0)
+
C = import_fresh_module('decimal', fresh=['_decimal'])
P = import_fresh_module('decimal', blocked=['_decimal'])
EXIT_STATUS = 0
I still got
$ ./python Modules/_decimal/tests/deccheck.py --short
Random seed: 1778644369
testing __abs__ ...
prec: 3 emin: -3 emax: 3
[...]
testing __hash__ ...
prec: 3 emin: -3 emax: 3
Traceback (most recent call last):
[...]
File "/home/sk/src/cpython/Modules/_decimal/tests/deccheck.py", line 570, in skip_error
return getattr(shandler, t.funcname, shandler.default)(t)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
TypeError: 'NoneType' object is not callable
Probably these tests now are partially overlap with the CI tests. I think we should investigate this and merge tests or just run deccheck.py as part of CPython test suite.
Linked PRs
- gh-155198
- gh-155261
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with Modules/_decimal/tests/deccheck.py and reproduce the shown ./python Modules/_decimal/tests/deccheck.py --short failure. Compare these tests with the overlapping CI tests and the partial pyperformance integration referenced in issue #473. Done means the overlap is resolved and deccheck.py is either integrated into the test workflow or removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- ci-cd, testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100