Skip to content

Commit

Permalink
Add test to improve coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
Marlon Rodriguez Garcia committed Nov 19, 2024
1 parent 8c712e9 commit 1be867d
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 4 deletions.
5 changes: 3 additions & 2 deletions num2words/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@ def __init__(self):
self.errmsg_floatord = "Cannot treat float %s as ordinal."
self.errmsg_negord = "Cannot treat negative num %s as ordinal."
self.errmsg_toobig = "abs(%s) must be less than %s."

self.cards = OrderedDict()
self.MAXVAL = None
self.setup()

# uses cards
Expand Down Expand Up @@ -111,7 +112,7 @@ def to_cardinal(self, value):
value = abs(value)
out = "%s " % self.negword.strip()

if value >= self.MAXVAL:
if self.MAXVAL and value >= self.MAXVAL:
raise OverflowError(self.errmsg_toobig % (value, self.MAXVAL))

val = self.splitnum(value)
Expand Down
2 changes: 1 addition & 1 deletion num2words/lang_AR.py
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,7 @@ def convert_to_arabic(self):
return formatted_number

def validate_number(self, number):
if number >= self.MAXVAL:
if self.MAXVAL and number >= self.MAXVAL:
raise OverflowError(self.errmsg_toobig % (number, self.MAXVAL))
return number

Expand Down
2 changes: 1 addition & 1 deletion num2words/lang_ID.py
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ def join(self, word_blocks, float_part):
return ' '.join(word_list) + float_part

def to_cardinal(self, number):
if number >= self.MAXVAL:
if self.MAXVAL and number >= self.MAXVAL:
raise OverflowError(self.errmsg_toobig % (number, self.MAXVAL))
minus = ''
if number < 0:
Expand Down
13 changes: 13 additions & 0 deletions tests/test_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,3 +62,16 @@ def test_is_title(self):
self.base.title("one"),
"one"
)

def test_set_high_numwords_not_implemented(self):
with self.assertRaises(NotImplementedError):
self.base.set_high_numwords()

def test_to_ordinal(self):
from num2words.base import Num2Word_Base
self.base = Num2Word_Base()
self.assertEqual(self.base.to_ordinal(1), "1st")
self.assertEqual(self.base.to_ordinal(11), "11th")
self.assertEqual(self.base.to_ordinal(21), "21st")
self.assertEqual(self.base.to_ordinal(100), "100th")
self.assertEqual(self.base.to_ordinal(123), "123rd")

0 comments on commit 1be867d

Please sign in to comment.