Skip to content

Commit 135c748

Browse files
Fix doctests in factorial_recursive function (#13703)
* Fix doctests in factorial_recursive function The doctests in factorial_recursive were calling factorial() instead of factorial_recursive(). This fix ensures that the tests correctly validate the factorial_recursive function itself. * Update factorial.py --------- Co-authored-by: Maxim Smolskiy <mithridatus@mail.ru>
1 parent af131b7 commit 135c748

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

maths/factorial.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,21 +41,21 @@ def factorial_recursive(n: int) -> int:
4141
https://en.wikipedia.org/wiki/Factorial
4242
4343
>>> import math
44-
>>> all(factorial(i) == math.factorial(i) for i in range(20))
44+
>>> all(factorial_recursive(i) == math.factorial(i) for i in range(20))
4545
True
46-
>>> factorial(0.1)
46+
>>> factorial_recursive(0.1)
4747
Traceback (most recent call last):
4848
...
49-
ValueError: factorial() only accepts integral values
50-
>>> factorial(-1)
49+
ValueError: factorial_recursive() only accepts integral values
50+
>>> factorial_recursive(-1)
5151
Traceback (most recent call last):
5252
...
53-
ValueError: factorial() not defined for negative values
53+
ValueError: factorial_recursive() not defined for negative values
5454
"""
5555
if not isinstance(n, int):
56-
raise ValueError("factorial() only accepts integral values")
56+
raise ValueError("factorial_recursive() only accepts integral values")
5757
if n < 0:
58-
raise ValueError("factorial() not defined for negative values")
58+
raise ValueError("factorial_recursive() not defined for negative values")
5959
return 1 if n in {0, 1} else n * factorial_recursive(n - 1)
6060

6161

0 commit comments

Comments
 (0)