Add exception handling

Roman.py

@@ -1,12 +1,12 @@
 def roman(numero):
     if numero < 1:
-        return "Can't convert to roman number!"
+        raise Exception("Can't convert to roman number!")
 
     if numero > 3999:
-        return "Not a valid number"
+        raise Exception("Not a valid number")
 
     if not isinstance(numero, int):
-        return "Only int numbers"
+        raise Exception("Only int numbers")
 
     numeros = {
         1000: 'M', 900: 'CM', 500: 'D', 400: 'CD', 100: 'C', 90: 'XC', 50: 'L',

roman_to_decimal.py

@@ -25,7 +25,7 @@ def decimal(roman):
         'Z',
     ]
     if any(x in roman for x in a):
-        result = 'Input is not a roman number'
+        raise Exception('Input is not a roman number')
     else:
         for letra in cadena:
             if len(cadena) != count + 1:

test_roman.py

@@ -33,12 +33,21 @@ def test_roman_decimal_roman_845(self):
         self.assertEqual(resultado, 'DCCCXLV')
 
     def test_roman_decimal_roman_minor(self):
-        resultado = roman(-1)
-        self.assertEqual(resultado, "Can't convert to roman number!")
+        with self.assertRaises(Exception) as e:
+            roman(-1)
+        self.assertEqual(
+            e.exception.message,
+            "Can't convert to roman number!",
+        )
 
     def test_roman_decimal_roman_not_type(self):
-        resultado = roman(10.5)
-        self.assertEqual(resultado, "Only int numbers")
+        with self.assertRaises(Exception) as e:
+            roman(10.5)
+
+        self.assertEqual(
+            e.exception.message,
+            "Only int numbers",
+        )
 
     def test_romandecimal_one(self):
         self.assertEqual(decimal('I'), 1)
@@ -62,15 +71,24 @@ def test_romandecimal_3999(self):
         self.assertEqual(decimal('MMMCMXCIX'), 3999)
 
     def test_wrong_letter(self):
-        self.assertEqual(decimal('CCXASDASHDUASBD'), 'Input is not a roman number')
+        with self.assertRaises(Exception) as e:
+            decimal('CCXASDASHDUASBD'),
+        self.assertEqual(
+            e.exception.message,
+            'Input is not a roman number',
+        )
 
     def test_roman_decimal_3999(self):
         resultado = roman(3999)
         self.assertEqual(resultado, 'MMMCMXCIX')
 
     def test_roman_decimal_4000(self):
-        resultado = roman(4000)
-        self.assertEqual(resultado, 'Not a valid number')
+        with self.assertRaises(Exception) as e:
+            roman(4000)
+        self.assertEqual(
+            e.exception.message,
+            'Not a valid number',
+        )
 
     ## definir primer test
 

test_roman_to_int.py

@@ -29,8 +29,10 @@ def test_romandecimal_3999(self):
         self.assertEqual(decimal('MMMCMXCIX'), 3999)
 
     def test_wrong_letter(self):
-        self.assertEqual(
+        with self.assertRaises(Exception) as e:
             decimal('CCXASDASHDUASBD'),
+        self.assertEqual(
+            e.exception.message,
             'Input is not a roman number'
         )