lang_ES: improved currency gender handling, improved error messages; test_es: updated accordingly

num2words/lang_ES.py

@@ -27,7 +27,8 @@ CURRENCIES_UNA = ('SLL', 'SEK', 'NOK', 'CZK', 'DKK', 'ISK',
                   'SKK', 'GBP', 'CYP', 'EGP', 'FKP', 'GIP',
                   'LBP', 'SDG', 'SHP', 'SSP', 'SYP', 'INR',
                   'IDR', 'LKR', 'MUR', 'NPR', 'PKR', 'SCR',
-                  'ESP')
+                  'ESP', 'TRY', 'ITL')
+CENTS_UNA = ('EGP', 'JOD', 'LBP', 'SDG', 'SSP', 'SYP')
 
 
 class Num2Word_ES(Num2Word_EU):
@@ -216,9 +217,13 @@ class Num2Word_ES(Num2Word_EU):
         self.high_numwords = self.gen_high_numwords([], [], lows)
         self.negword = "menos "
         self.pointword = "punto"
-        self.errmsg_nonnum = "Solo números pueden ser convertidos a palabras."
+        self.errmsg_nonnum = "type(%s) no es [long, int, float]"
+        self.errmsg_floatord = "El float %s no puede ser tratado como un" \
+            " ordinal."
+        self.errmsg_negord = "El número negativo %s no puede ser tratado" \
+            " como un ordinal."
         self.errmsg_toobig = (
-            "Numero muy grande para ser convertido a palabras."
+            "abs(%s) deber ser inferior a %s."
             )
         self.gender_stem = "o"
         self.exclude_title = ["y", "menos", "punto"]
@@ -352,17 +357,52 @@ class Num2Word_ES(Num2Word_EU):
         result = super(Num2Word_ES, self).to_currency(
             val, currency=currency, cents=cents, separator=separator,
             adjective=adjective)
-        # Handle exception, in spanish is "un euro" and not "uno euro"
+        # Handle exception: In Spanish it's "un euro" and not "uno euro",
-        # except in this currencies: leona, corona,
+        # except in these currencies, where it's "una": leona, corona,
-        # libra, rupia, lempira, peseta, is 'una'
+        # libra, lira, rupia, lempira, peseta.
-        # but only when it's first word, otherwise
+        # The same goes for "veintiuna", "treinta y una"...
-        # it's replaced in others words like 'veintiun'
+        # Also, this needs to be handled separately for "dollars" and
+        # "cents".
+        # All "cents" are masculine except for: piastra.
+        # Source: https://www.rae.es/dpd/una (section 2.2)
+
+        # split "dollars" part from "cents" part
+        list_result = result.split(" con ")
+
+        # "DOLLARS" PART (list_result[0])
+
+        # Feminine currencies ("una libra", "trescientas libras"...)
         if currency in CURRENCIES_UNA:
-            list_result = result.split(" ")
+
-            if list_result[0] == "uno":
+            # "una libra", "veintiuna libras", "treinta y una libras"...
-                list_result[0] = list_result[0].replace("uno", "una")
+            list_result[0] = list_result[0].replace("uno", "una")
-                result = " ".join(list_result)
+
-        result = result.replace("uno", "un")
+            # "doscientas libras", "trescientas libras"...
-        # correct orthography for the specific case of "veintiún":
+            list_result[0] = list_result[0].replace("cientos", "cientas")
-        result = result.replace("veintiun", "veintiún")
+
+        # Masc.: Correct orthography for the specific case of "veintiún":
+        list_result[0] = list_result[0].replace("veintiuno", "veintiún")
+
+        # Masculine currencies: general case ("un euro", "treinta y un
+        # euros"...):
+        list_result[0] = list_result[0].replace("uno", "un")
+
+        # "CENTS" PART (list_result[1])
+
+        # Feminine "cents" ("una piastra", "veintiuna piastras"...)
+        if currency in CENTS_UNA:
+
+            # "una piastra", "veintiuna piastras", "treinta y una piastras"...
+            list_result[1] = list_result[1].replace("uno", "una")
+
+        # Masc.: Correct orthography for the specific case of "veintiún":
+        list_result[1] = list_result[1].replace("veintiuno", "veintiún")
+
+        # Masculine "cents": general case ("un centavo", "treinta y un
+        # centavos"...):
+        list_result[1] = list_result[1].replace("uno", "un")
+
+        # join back "dollars" part with "cents" part
+        result = " con ".join(list_result)
+
         return result

tests/test_es.py

@@ -128,9 +128,9 @@ TEST_CASES_TO_CURRENCY_ESP = (
     (2.00, 'dos pesetas con cero céntimos'),
     (8.00, 'ocho pesetas con cero céntimos'),
     (12.00, 'doce pesetas con cero céntimos'),
-    (21.00, 'veintiún pesetas con cero céntimos'),
+    (21.00, 'veintiuna pesetas con cero céntimos'),
-    (81.25, 'ochenta y un pesetas con veinticinco céntimos'),
+    (81.25, 'ochenta y una pesetas con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta pesetas con noventa céntimos'),
+    (350.90, 'trescientas cincuenta pesetas con noventa céntimos'),
     (100.00, 'cien pesetas con cero céntimos'),
 )
 
@@ -175,9 +175,9 @@ TEST_CASES_TO_CURRENCY_GBP = (
     (2.00, 'dos libras con cero peniques'),
     (8.00, 'ocho libras con cero peniques'),
     (12.00, 'doce libras con cero peniques'),
-    (21.00, 'veintiún libras con cero peniques'),
+    (21.00, 'veintiuna libras con cero peniques'),
-    (81.25, 'ochenta y un libras con veinticinco peniques'),
+    (81.25, 'ochenta y una libras con veinticinco peniques'),
-    (350.90, 'trescientos cincuenta libras con noventa peniques'),
+    (350.90, 'trescientas cincuenta libras con noventa peniques'),
     (100.00, 'cien libras con cero peniques'),
     (4150.83,
         'cuatro mil ciento cincuenta libras con ochenta y tres peniques'),
@@ -201,9 +201,9 @@ TEST_CASES_TO_CURRENCY_SEK = (
     (2.00, 'dos coronas con cero öre'),
     (8.00, 'ocho coronas con cero öre'),
     (12.00, 'doce coronas con cero öre'),
-    (21.00, 'veintiún coronas con cero öre'),
+    (21.00, 'veintiuna coronas con cero öre'),
-    (81.25, 'ochenta y un coronas con veinticinco öre'),
+    (81.25, 'ochenta y una coronas con veinticinco öre'),
-    (350.90, 'trescientos cincuenta coronas con noventa öre'),
+    (350.90, 'trescientas cincuenta coronas con noventa öre'),
     (100.00, 'cien coronas con cero öre'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres öre'),
@@ -214,9 +214,9 @@ TEST_CASES_TO_CURRENCY_NOK = (
     (2.00, 'dos coronas con cero øre'),
     (8.00, 'ocho coronas con cero øre'),
     (12.00, 'doce coronas con cero øre'),
-    (21.00, 'veintiún coronas con cero øre'),
+    (21.00, 'veintiuna coronas con cero øre'),
-    (81.25, 'ochenta y un coronas con veinticinco øre'),
+    (81.25, 'ochenta y una coronas con veinticinco øre'),
-    (350.90, 'trescientos cincuenta coronas con noventa øre'),
+    (350.90, 'trescientas cincuenta coronas con noventa øre'),
     (100.00, 'cien coronas con cero øre'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres øre'),
@@ -266,9 +266,9 @@ TEST_CASES_TO_CURRENCY_INR = (
     (2.00, 'dos rupias con cero paisas'),
     (8.00, 'ocho rupias con cero paisas'),
     (12.00, 'doce rupias con cero paisas'),
-    (21.00, 'veintiún rupias con cero paisas'),
+    (21.00, 'veintiuna rupias con cero paisas'),
-    (81.25, 'ochenta y un rupias con veinticinco paisas'),
+    (81.25, 'ochenta y una rupias con veinticinco paisas'),
-    (350.90, 'trescientos cincuenta rupias con noventa paisas'),
+    (350.90, 'trescientas cincuenta rupias con noventa paisas'),
     (100.00, 'cien rupias con cero paisas'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres paisas'),
@@ -318,9 +318,9 @@ TEST_CASES_TO_CURRENCY_CZK = (
     (2.00, 'dos coronas con cero haléř'),
     (8.00, 'ocho coronas con cero haléř'),
     (12.00, 'doce coronas con cero haléř'),
-    (21.00, 'veintiún coronas con cero haléř'),
+    (21.00, 'veintiuna coronas con cero haléř'),
-    (81.25, 'ochenta y un coronas con veinticinco haléř'),
+    (81.25, 'ochenta y una coronas con veinticinco haléř'),
-    (350.90, 'trescientos cincuenta coronas con noventa haléř'),
+    (350.90, 'trescientas cincuenta coronas con noventa haléř'),
     (100.00, 'cien coronas con cero haléř'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres haléř'),
@@ -405,13 +405,13 @@ TEST_CASES_TO_CURRENCY_KPW = (
 )
 
 TEST_CASES_TO_CURRENCY_TRY = (
-    (1.00, 'un lira con cero kuruş'),
+    (1.00, 'una lira con cero kuruş'),
     (2.00, 'dos liras con cero kuruş'),
     (8.00, 'ocho liras con cero kuruş'),
     (12.00, 'doce liras con cero kuruş'),
-    (21.00, 'veintiún liras con cero kuruş'),
+    (21.00, 'veintiuna liras con cero kuruş'),
-    (81.25, 'ochenta y un liras con veinticinco kuruş'),
+    (81.25, 'ochenta y una liras con veinticinco kuruş'),
-    (350.90, 'trescientos cincuenta liras con noventa kuruş'),
+    (350.90, 'trescientas cincuenta liras con noventa kuruş'),
     (100.00, 'cien liras con cero kuruş'),
     (4150.83,
         'cuatro mil ciento cincuenta liras con ochenta y tres kuruş'),
@@ -708,9 +708,9 @@ TEST_CASES_TO_CURRENCY_CYP = (
     (2.00, 'dos libras con cero céntimos'),
     (8.00, 'ocho libras con cero céntimos'),
     (12.00, 'doce libras con cero céntimos'),
-    (21.00, 'veintiún libras con cero céntimos'),
+    (21.00, 'veintiuna libras con cero céntimos'),
-    (81.25, 'ochenta y un libras con veinticinco céntimos'),
+    (81.25, 'ochenta y una libras con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta libras con noventa céntimos'),
+    (350.90, 'trescientas cincuenta libras con noventa céntimos'),
     (100.00, 'cien libras con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta libras con ochenta y tres céntimos'),
@@ -721,9 +721,9 @@ TEST_CASES_TO_CURRENCY_DKK = (
     (2.00, 'dos coronas con cero øre'),
     (8.00, 'ocho coronas con cero øre'),
     (12.00, 'doce coronas con cero øre'),
-    (21.00, 'veintiún coronas con cero øre'),
+    (21.00, 'veintiuna coronas con cero øre'),
-    (81.25, 'ochenta y un coronas con veinticinco øre'),
+    (81.25, 'ochenta y una coronas con veinticinco øre'),
-    (350.90, 'trescientos cincuenta coronas con noventa øre'),
+    (350.90, 'trescientas cincuenta coronas con noventa øre'),
     (100.00, 'cien coronas con cero øre'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres øre'),
@@ -760,9 +760,10 @@ TEST_CASES_TO_CURRENCY_EGP = (
     (2.00, 'dos libras con cero piastras'),
     (8.00, 'ocho libras con cero piastras'),
     (12.00, 'doce libras con cero piastras'),
-    (21.00, 'veintiún libras con cero piastras'),
+    (21.00, 'veintiuna libras con cero piastras'),
-    (81.25, 'ochenta y un libras con veinticinco piastras'),
+    (81.21, 'ochenta y una libras con veintiuna piastras'),
-    (350.90, 'trescientos cincuenta libras con noventa piastras'),
+    (81.25, 'ochenta y una libras con veinticinco piastras'),
+    (350.90, 'trescientas cincuenta libras con noventa piastras'),
     (100.00, 'cien libras con cero piastras'),
     (4150.83,
         'cuatro mil ciento cincuenta libras con ochenta y tres piastras'),
@@ -799,9 +800,9 @@ TEST_CASES_TO_CURRENCY_FKP = (
     (2.00, 'dos libras con cero peniques'),
     (8.00, 'ocho libras con cero peniques'),
     (12.00, 'doce libras con cero peniques'),
-    (21.00, 'veintiún libras con cero peniques'),
+    (21.00, 'veintiuna libras con cero peniques'),
-    (81.25, 'ochenta y un libras con veinticinco peniques'),
+    (81.25, 'ochenta y una libras con veinticinco peniques'),
-    (350.90, 'trescientos cincuenta libras con noventa peniques'),
+    (350.90, 'trescientas cincuenta libras con noventa peniques'),
     (100.00, 'cien libras con cero peniques'),
     (4150.83,
         'cuatro mil ciento cincuenta libras con ochenta y tres peniques'),
@@ -903,9 +904,9 @@ TEST_CASES_TO_CURRENCY_IDR = (
     (2.00, 'dos rupias con cero céntimos'),
     (8.00, 'ocho rupias con cero céntimos'),
     (12.00, 'doce rupias con cero céntimos'),
-    (21.00, 'veintiún rupias con cero céntimos'),
+    (21.00, 'veintiuna rupias con cero céntimos'),
-    (81.25, 'ochenta y un rupias con veinticinco céntimos'),
+    (81.25, 'ochenta y una rupias con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta rupias con noventa céntimos'),
+    (350.90, 'trescientas cincuenta rupias con noventa céntimos'),
     (100.00, 'cien rupias con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres céntimos'),
@@ -955,22 +956,22 @@ TEST_CASES_TO_CURRENCY_ISK = (
     (2.00, 'dos coronas con cero aurar'),
     (8.00, 'ocho coronas con cero aurar'),
     (12.00, 'doce coronas con cero aurar'),
-    (21.00, 'veintiún coronas con cero aurar'),
+    (21.00, 'veintiuna coronas con cero aurar'),
-    (81.25, 'ochenta y un coronas con veinticinco aurar'),
+    (81.25, 'ochenta y una coronas con veinticinco aurar'),
-    (350.90, 'trescientos cincuenta coronas con noventa aurar'),
+    (350.90, 'trescientas cincuenta coronas con noventa aurar'),
     (100.00, 'cien coronas con cero aurar'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres aurar'),
 )
 
 TEST_CASES_TO_CURRENCY_ITL = (
-    (1.00, 'un lira con cero céntimos'),
+    (1.00, 'una lira con cero céntimos'),
     (2.00, 'dos liras con cero céntimos'),
     (8.00, 'ocho liras con cero céntimos'),
     (12.00, 'doce liras con cero céntimos'),
-    (21.00, 'veintiún liras con cero céntimos'),
+    (21.00, 'veintiuna liras con cero céntimos'),
-    (81.25, 'ochenta y un liras con veinticinco céntimos'),
+    (81.25, 'ochenta y una liras con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta liras con noventa céntimos'),
+    (350.90, 'trescientas cincuenta liras con noventa céntimos'),
     (100.00, 'cien liras con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta liras con ochenta y tres céntimos'),
@@ -1059,9 +1060,9 @@ TEST_CASES_TO_CURRENCY_LKR = (
     (2.00, 'dos rupias con cero céntimos'),
     (8.00, 'ocho rupias con cero céntimos'),
     (12.00, 'doce rupias con cero céntimos'),
-    (21.00, 'veintiún rupias con cero céntimos'),
+    (21.00, 'veintiuna rupias con cero céntimos'),
-    (81.25, 'ochenta y un rupias con veinticinco céntimos'),
+    (81.25, 'ochenta y una rupias con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta rupias con noventa céntimos'),
+    (350.90, 'trescientas cincuenta rupias con noventa céntimos'),
     (100.00, 'cien rupias con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres céntimos'),
@@ -1241,9 +1242,9 @@ TEST_CASES_TO_CURRENCY_MUR = (
     (2.00, 'dos rupias con cero céntimos'),
     (8.00, 'ocho rupias con cero céntimos'),
     (12.00, 'doce rupias con cero céntimos'),
-    (21.00, 'veintiún rupias con cero céntimos'),
+    (21.00, 'veintiuna rupias con cero céntimos'),
-    (81.25, 'ochenta y un rupias con veinticinco céntimos'),
+    (81.25, 'ochenta y una rupias con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta rupias con noventa céntimos'),
+    (350.90, 'trescientas cincuenta rupias con noventa céntimos'),
     (100.00, 'cien rupias con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres céntimos'),
@@ -1319,9 +1320,9 @@ TEST_CASES_TO_CURRENCY_NPR = (
     (2.00, 'dos rupias con cero paisas'),
     (8.00, 'ocho rupias con cero paisas'),
     (12.00, 'doce rupias con cero paisas'),
-    (21.00, 'veintiún rupias con cero paisas'),
+    (21.00, 'veintiuna rupias con cero paisas'),
-    (81.25, 'ochenta y un rupias con veinticinco paisas'),
+    (81.25, 'ochenta y una rupias con veinticinco paisas'),
-    (350.90, 'trescientos cincuenta rupias con noventa paisas'),
+    (350.90, 'trescientas cincuenta rupias con noventa paisas'),
     (100.00, 'cien rupias con cero paisas'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres paisas'),
@@ -1371,9 +1372,9 @@ TEST_CASES_TO_CURRENCY_PKR = (
     (2.00, 'dos rupias con cero paisas'),
     (8.00, 'ocho rupias con cero paisas'),
     (12.00, 'doce rupias con cero paisas'),
-    (21.00, 'veintiún rupias con cero paisas'),
+    (21.00, 'veintiuna rupias con cero paisas'),
-    (81.25, 'ochenta y un rupias con veinticinco paisas'),
+    (81.25, 'ochenta y una rupias con veinticinco paisas'),
-    (350.90, 'trescientos cincuenta rupias con noventa paisas'),
+    (350.90, 'trescientas cincuenta rupias con noventa paisas'),
     (100.00, 'cien rupias con cero paisas'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres paisas'),
@@ -1462,9 +1463,9 @@ TEST_CASES_TO_CURRENCY_SCR = (
     (2.00, 'dos rupias con cero céntimos'),
     (8.00, 'ocho rupias con cero céntimos'),
     (12.00, 'doce rupias con cero céntimos'),
-    (21.00, 'veintiún rupias con cero céntimos'),
+    (21.00, 'veintiuna rupias con cero céntimos'),
-    (81.25, 'ochenta y un rupias con veinticinco céntimos'),
+    (81.25, 'ochenta y una rupias con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta rupias con noventa céntimos'),
+    (350.90, 'trescientas cincuenta rupias con noventa céntimos'),
     (100.00, 'cien rupias con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta rupias con ochenta y tres céntimos'),
@@ -1475,9 +1476,9 @@ TEST_CASES_TO_CURRENCY_SHP = (
     (2.00, 'dos libras con cero peniques'),
     (8.00, 'ocho libras con cero peniques'),
     (12.00, 'doce libras con cero peniques'),
-    (21.00, 'veintiún libras con cero peniques'),
+    (21.00, 'veintiuna libras con cero peniques'),
-    (81.25, 'ochenta y un libras con veinticinco peniques'),
+    (81.25, 'ochenta y una libras con veinticinco peniques'),
-    (350.90, 'trescientos cincuenta libras con noventa peniques'),
+    (350.90, 'trescientas cincuenta libras con noventa peniques'),
     (100.00, 'cien libras con cero peniques'),
     (4150.83,
         'cuatro mil ciento cincuenta libras con ochenta y tres peniques'),
@@ -1488,9 +1489,9 @@ TEST_CASES_TO_CURRENCY_SKK = (
     (2.00, 'dos coronas con cero haliers'),
     (8.00, 'ocho coronas con cero haliers'),
     (12.00, 'doce coronas con cero haliers'),
-    (21.00, 'veintiún coronas con cero haliers'),
+    (21.00, 'veintiuna coronas con cero haliers'),
-    (81.25, 'ochenta y un coronas con veinticinco haliers'),
+    (81.25, 'ochenta y una coronas con veinticinco haliers'),
-    (350.90, 'trescientos cincuenta coronas con noventa haliers'),
+    (350.90, 'trescientas cincuenta coronas con noventa haliers'),
     (100.00, 'cien coronas con cero haliers'),
     (4150.83,
         'cuatro mil ciento cincuenta coronas con ochenta y tres haliers'),
@@ -1501,9 +1502,9 @@ TEST_CASES_TO_CURRENCY_SLL = (
     (2.00, 'dos leonas con cero céntimos'),
     (8.00, 'ocho leonas con cero céntimos'),
     (12.00, 'doce leonas con cero céntimos'),
-    (21.00, 'veintiún leonas con cero céntimos'),
+    (21.00, 'veintiuna leonas con cero céntimos'),
-    (81.25, 'ochenta y un leonas con veinticinco céntimos'),
+    (81.25, 'ochenta y una leonas con veinticinco céntimos'),
-    (350.90, 'trescientos cincuenta leonas con noventa céntimos'),
+    (350.90, 'trescientas cincuenta leonas con noventa céntimos'),
     (100.00, 'cien leonas con cero céntimos'),
     (4150.83,
         'cuatro mil ciento cincuenta leonas con ochenta y tres céntimos'),