arreglo de la primera linea

.travis.yml

@@ -1,4 +1,9 @@
 language: python
 python:
   - "2.7"
-script: python test_suite.py
+install:
+  - pip install -r requirements.txt
+script: coverage run test_suite.py
+
+after_success:
+- coveralls

README.md

@@ -1,3 +1,5 @@
 # Roman-numbers
 
 [![Build Status](https://travis-ci.org/evbeda/Roman-numbers.svg?branch=master)](https://travis-ci.org/evbeda/Roman-numbers)
+
+[![Coverage Status](https://coveralls.io/repos/github/evbeda/Roman-numbers/badge.svg?branch=master)](https://coveralls.io/github/evbeda/Roman-numbers?branch=master)

Roman.py

@@ -8,22 +8,11 @@ def roman(numero):
     if not isinstance(numero, int):
         return "Only int numbers"
 
-
     numeros = {
-        1000 :'M',
-        900 :'CM',
-        500 :'D',
-        400 :'CD',
-        100 :'C',
-        90 : 'XC',
-        50 : 'L',
-        40 : 'XL',
-        10: 'X',
-        9 : 'IX',
-        5 : 'V',
-        4 : 'IV',
-        1 : 'I',
+        1000: 'M', 900: 'CM', 500: 'D', 400: 'CD', 100: 'C', 90: 'XC', 50: 'L',
+        40: 'XL', 10: 'X', 9: 'IX', 5: 'V', 4: 'IV', 1: 'I',
     }
+
     roman = ''
 
     for key in sorted(numeros.keys(), reverse=True):
@@ -32,4 +21,4 @@ def roman(numero):
             cant = numero // key
             roman += value * cant
             numero -= cant * key
-    return roman
+    return roman

requirements.txt

@@ -0,0 +1,9 @@
+certifi==2018.1.18
+chardet==3.0.4
+coverage==4.0.3
+idna==2.6
+python-coveralls==2.9.1
+PyYAML==3.12
+requests==2.18.4
+six==1.11.0
+urllib3==1.22

roman_to_decimal.py

@@ -1,17 +1,36 @@
-
 def decimal(roman):
     roma = {'M': 1000, 'D': 500, 'C': 100, 'L': 50, 'X': 10, 'V': 5, 'I': 1}
     cadena = list(roman)
     result = 0
     count = 0
-    a = ['A', 'B', 'E', 'F', 'G', 'H', 'J', 'K', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'W', 'Y', 'Z']
+    a = [
+        'A',
+        'B',
+        'E',
+        'F',
+        'G',
+        'H',
+        'J',
+        'K',
+        'N',
+        'O',
+        'P',
+        'Q',
+        'R',
+        'S',
+        'T',
+        'U',
+        'W',
+        'Y',
+        'Z',
+    ]
     if any(x in roman for x in a):
         result = 'Input is not a roman number'
     else:
         for letra in cadena:
             if len(cadena) != count + 1:
                 if len(cadena) > 0:
-                    if roma[letra] < roma[cadena[count+1]]:
+                    if roma[letra] < roma[cadena[count + 1]]:
                         result -= roma[letra]
                     else:
                         result += roma[letra]
@@ -21,5 +40,3 @@ def decimal(roman):
                 result += roma[letra]
             count += 1
     return result
-
-

romandecimalroman.py

@@ -2,4 +2,7 @@
 from Roman import roman
 
 def roman_to_roman(romano):
-    return roman(decimal(romano))
+    return roman(decimal(romano))
+
+
+

test_roman.py

@@ -19,7 +19,7 @@ def test_roman_decimal_roman_two(self):
     def test_roman_decimal_roman_four(self):
         resultado = roman(4)
         self.assertEqual(resultado, 'IV')
-        
+
     def test_roman_decimal_roman_ten(self):
         resultado = roman(10)
         self.assertEqual(resultado, 'X')
@@ -30,22 +30,22 @@ def test_roman_decimal_roman_five(self):
 
     def test_roman_decimal_roman_845(self):
         resultado = roman(845)
-        self.assertEqual(resultado, 'DCCCXLV') 
+        self.assertEqual(resultado, 'DCCCXLV')
 
     def test_roman_decimal_roman_minor(self):
         resultado = roman(-1)
         self.assertEqual(resultado, "Can't convert to roman number!")
 
     def test_roman_decimal_roman_not_type(self):
         resultado = roman(10.5)
-        self.assertEqual(resultado, "Only int numbers")   
+        self.assertEqual(resultado, "Only int numbers")
 
     def test_romandecimal_one(self):
         self.assertEqual(decimal('I'), 1)
 
     def test_romandecimal_two(self):
         self.assertEqual(decimal('II'), 2)
-    
+
     def test_romandecimal_three(self):
         self.assertEqual(decimal('III'), 3)
 
@@ -70,10 +70,10 @@ def test_roman_decimal_3999(self):
 
     def test_roman_decimal_4000(self):
         resultado = roman(4000)
-        self.assertEqual(resultado, 'Not a valid number')   
+        self.assertEqual(resultado, 'Not a valid number')
 
     ## definir primer test
-    
+
 class Roman_to_decimal_to_roman(unittest.TestCase):
     def test_I(self):
         self.assertEqual(roman_to_roman('I'),'I')

test_roman_to_int.py

@@ -12,7 +12,7 @@ def test_romandecimal_one(self):
 
     def test_romandecimal_two(self):
         self.assertEqual(decimal('II'), 2)
-    
+
     def test_romandecimal_three(self):
         self.assertEqual(decimal('III'), 3)
 
@@ -29,10 +29,13 @@ 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')
+        self.assertEqual(
+            decimal('CCXASDASHDUASBD'),
+            'Input is not a roman number'
+        )
 
     ## definir primer test
-    
+
 class Roman_to_decimal_to_roman(unittest.TestCase):
     def test_all(self):
         for number in xrange(1, 4):

test_suite.py

@@ -5,6 +5,7 @@
 )
 from test_roman import TestRoman
 
+
 def suite():
     """
         Gather all the tests from this module in a test suite.
@@ -15,6 +16,7 @@ def suite():
     test_suite.addTest(unittest.makeSuite(TestRoman))
     return test_suite
 
+
 if __name__ == "__main__":
     #So you can run tests from this module individually.
-    unittest.main()
+    unittest.main()