Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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.
  • Loading branch information
Tejasrahane authored Oct 23, 2025
commit 13902738fec902082dcf699a5943f71e9cc176d4
6 changes: 3 additions & 3 deletions maths/factorial.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,13 +41,13 @@ def factorial_recursive(n: int) -> int:
https://en.wikipedia.org/wiki/Factorial

>>> import math
>>> all(factorial(i) == math.factorial(i) for i in range(20))
>>> all(factorial_recursive(i) == math.factorial(i) for i in range(20))
True
>>> factorial(0.1)
>>> factorial_recursive(0.1)
Traceback (most recent call last):
...
ValueError: factorial() only accepts integral values
>>> factorial(-1)
>>> factorial_recursive(-1)
Traceback (most recent call last):
...
ValueError: factorial() not defined for negative values
Expand Down