wasili conv

Test_Converter_Römische_Zahlen.py

@@ -5,6 +5,7 @@ from abc import ABC, abstractmethod
 #from muhamed_directory.other.alma.test2converter import RomanNumber as Alma
 #from muhamed_directory.other.daniel.converter import Converter as Dani
 #from muhamed_directory.other.hatice.converter import RomanNumber as Hati
+from muhamed_directory.other.wasili.TDD_Converter import RomanNumber as was
 
 class IStringCalculator(ABC):
     @abstractmethod
@@ -27,52 +28,31 @@ class Converter(IStringCalculator):
         return result
     
     
-'''class TestRomanConverter(unittest.TestCase):
+class TestRomanConverter(unittest.TestCase):
     
     def setUp(self):
-        self.calculator = Hati
+        self.calculator = was
 
     def test_1(self):
-        self.assertEqual(self.calculator.convert(1), "I")
+        self.assertEqual(self.calculator.convert_int_to_str(1), "I")
 
     def test_10(self):
-        self.assertEqual(self.calculator.convert(10), "X")
+        self.assertEqual(self.calculator.convert_int_to_str(10), "X")
 
     def test_21(self):
-        self.assertEqual(self.calculator.convert(21), "XXI")
+        self.assertEqual(self.calculator.convert_int_to_str(21), "XXI")
 
     def test_50(self):
-        self.assertEqual(self.calculator.convert(50), "L")
+        self.assertEqual(self.calculator.convert_int_to_str(50), "L")
 
     def test_100(self):
-        self.assertEqual(self.calculator.convert(100), "C")
+        self.assertEqual(self.calculator.convert_int_to_str(100), "C")
 
     def test_1000(self):
-        self.assertEqual(self.calculator.convert(1000), "M")
+        self.assertEqual(self.calculator.convert_int_to_str(1000), "M")
 
     def test_1999(self):
-        self.assertEqual(self.calculator.convert(1999), "MCMXCIX")'''
-class TestRomanConverter(unittest.TestCase):
-    def setUp(self):
-        self.converter = Converter
+        self.assertEqual(self.calculator.convert_int_to_str(1999), "MCMXCIX")
 
-    def test_single_value(self):
-        self.assertEqual(self.converter.convert(1), "I")
-        self.assertEqual(self.converter.convert(2), "II")
-        self.assertEqual(self.converter.convert(3), "III")
-        self.assertEqual(self.converter.convert(4), "IV")
-        self.assertEqual(self.converter.convert(5), "V")
-        self.assertEqual(self.converter.convert(9), "IX")
-        self.assertEqual(self.converter.convert(21), "XXI")
-        self.assertEqual(self.converter.convert(50), "L")
-        self.assertEqual(self.converter.convert(100), "C")
-        self.assertEqual(self.converter.convert(500), "D")
-        self.assertEqual(self.converter.convert(1000), "M")
-    
-    def test_inivalid_numbers(self):
-        self.assertEqual(self.converter.convert(6), "")
-        self.assertEqual(self.converter.convert(99), "")
-        self.assertEqual(self.converter.convert(-1), "")
- 
 if __name__ == "__main__":
     unittest.main()
\ No newline at end of file

report.md

@@ -5,4 +5,6 @@
 | Daniel                  | yes             | all              | Die Tests rufen Converter.convert() als statische Methode auf, aber die convert-Methode aus Skript 1 ist eine Instanzmethode, die eine Objektinstanz (self) benötigt. Dieser falsche Aufruf verursacht bei jedem der 7 Tests einen TypeError, weil das erwartete self-Argument fehlt. |
 | Ich vs Daniel Tests     | no              | none             | 
 | Hatice                    | yes             | all              | TypeError: RomanNumber.convert() missing 1 required positional argument: 'n'                                                  |
-| ich vs Hatice Tests       | no              | 1             | FAIL: test_inivalid_numbers (__main__.TestRomanConverter.test_inivalid_numbers)                                                                                  |
\ No newline at end of file
+| ich vs Hatice Tests       | no              | 1             | FAIL: test_inivalid_numbers (__main__.TestRomanConverter.test_inivalid_numbers)                                                                                  |
+| Lasse                    | yes             | all              | TypeError: RomanNumerals.to_roman() missing 1 required positional argument: 'num'                                                  |
+| Wasili                    | yes             | all              | TypeError: RomanNumber.convert_int_to_str() missing 1 required positional argument: 'n'                                                  |