Fix test suite

Author: Shehab Abdel-Salam

chapters/chapter02_variables/exercises/exercise_04.py

@@ -8,4 +8,4 @@
 def compute_length_difference(word1, word2):
     # Your code should go here.
 
-    return ...
+    return -1

chapters/chapter02_variables/tests/test_ch02.py

@@ -29,10 +29,10 @@ def test_e03():
 
 
 def test_e04():
-    compute_length_difference("", "") == 0
-    compute_length_difference("Hello", "World") == 0
-    compute_length_difference("Python", "Java") == 2
-    compute_length_difference("Python", "Programming") == 5
+    assert compute_length_difference("", "") == 0
+    assert compute_length_difference("Hello", "World") == 0
+    assert compute_length_difference("Python", "Java") == 2
+    assert compute_length_difference("Python", "Programming") == 5
 
 
 def test_e05():

chapters/chapter08_files/exercises/exercise_63.py

@@ -1,6 +1,18 @@
-# Exercise 63 - File Handler
-# Write a custom context manager using a class FileHandler that opens
-# and closes a file. Use it to read from data.txt.
+# Exercise 63 - How Many emails?
+# Write a function that reads a file and returns the number of emails in the file.
+#
+# Example: emails.txt
+#   test1@abc.com
+#   test2@abc.com
+#   fullname1
+#   fullname2
+#   test3@abc.com
 
+# The function should return 3.
+# A valid email address contains a single @ symbol.
 
-class FileHandler: ...
+
+def count_emails(file_name: str) -> int:
+    # Your code should go here.
+
+    ...

chapters/chapter08_files/tests/test_ch08.py

@@ -7,8 +7,7 @@
 from ..exercises.exercise_60 import course_grades_summary
 from ..exercises.exercise_61 import inventory_management
 from ..exercises.exercise_62 import is_file_sorted
-
-# from ..exercises.exercise_63 import
+from ..exercises.exercise_63 import count_emails
 
 
 @pytest.mark.parametrize(
@@ -287,3 +286,44 @@ def test_e61(source, restock_threshold, expected):
 )
 def test_e62(source, expected):
     assert is_file_sorted(source) is expected
+
+
+@pytest.mark.parametrize(
+    "file_content, expected",
+    [
+        (
+            textwrap.dedent(
+                """
+                username1
+                username2
+            """
+            ),
+            0,
+        ),
+        (
+            textwrap.dedent(
+                """
+                username1
+                test1@abc.com
+                username2
+            """
+            ),
+            1,
+        ),
+        (
+            textwrap.dedent(
+                """
+                username1
+                username2
+                user1@abc.com
+                user2@abc.com
+                username3
+                user3@abc.com
+                """
+            ),
+            3,
+        ),
+    ],
+)
+def test_e63(source, expected):
+    assert count_emails(source) == expected

chapters/chapter09_error_handling/exercises/exercise_65.py

@@ -7,7 +7,7 @@
 class NegativeValueError: ...
 
 
-def check_positive(number):
+def check_positive(number: int):
     # Your code should go here.
 
     return ...

chapters/chapter09_error_handling/exercises/exercise_66.py

@@ -1 +1,18 @@
-# Exercise 66 - Custom Exception
+# Exercise 66 - ProductSalesCalculator
+# Given the following class, fix the code so that it raises a custom exception called `InvalidQuantityError`
+# instead of the print message.
+
+
+class InvalidQuantityError(Exception):
+    pass
+
+
+class ProductSalesCalculator:
+    def __init__(self, price: float, quantity: int):
+        if quantity <= 0:
+            print("Quantity must be greater than zero")
+        self.price = price
+        self.quantity = quantity
+
+    def calculate_total(self) -> float:
+        return self.price * self.quantity

chapters/chapter09_error_handling/exercises/exercise_67.py

@@ -1 +1,24 @@
-# Exercise 67 - Custom Exception
+# Exercise 67 - Exception Group
+# Given the following function, we want to raise an exception called `ExceptionGroup` if multiple errors occur during processing.
+# Fix this function so that it appends the exceptions to a list and raises an `ExceptionGroup` if multiple errors occur.
+
+
+def process_strings(strings: list[str]):
+    exceptions = []
+
+    for string in strings:
+        try:
+            # Operation 1: Convert to Integer
+            print(f"Converting '{string}' to integer: {int(string)}")
+
+            # Operation 2: Access First Character
+            print(f"First character of '{string}' is '{string[0]}'")
+
+        except ValueError as e:
+            print("TODO: Cannot convert to integer")
+
+        except IndexError as e:
+            print("TODO: Cannot access first character")
+
+    if exceptions:
+        raise ExceptionGroup("Multiple errors occurred during processing", exceptions)

chapters/chapter09_error_handling/tests/test_ch09.py

@@ -1,9 +1,8 @@
 import pytest
 from ..exercises.exercise_64 import divide
 from ..exercises.exercise_65 import check_positive, NegativeValueError
-
-# from ..exercises.exercise_66 import
-# from ..exercises.exercise_67 import
+from ..exercises.exercise_66 import ProductSalesCalculator, InvalidQuantityError
+from ..exercises.exercise_67 import process_strings
 
 
 def test_e64():
@@ -21,3 +20,33 @@ def test_e65():
         check_positive(-1)
     with pytest.raises(NegativeValueError):
         check_positive(-10)
+
+
+def test_e66():
+    with pytest.raises(InvalidQuantityError):
+        ProductSalesCalculator(10, 0)
+    with pytest.raises(InvalidQuantityError):
+        ProductSalesCalculator(10, -1)
+
+    psc = ProductSalesCalculator(10, 1)
+    assert psc.calculate_total() == 10
+    psc = ProductSalesCalculator(10, 2)
+    assert psc.calculate_total() == 20
+
+
+def test_e67():
+    test_strings = ["123", "", "abc", "456", "789", ""]
+
+    with pytest.raises(ExceptionGroup) as exc_info:
+        process_strings(test_strings)
+
+    exceptions = exc_info.value.exceptions
+    assert len(exceptions) == 3
+
+    value_error_raised = any(isinstance(e, ValueError) for e in exceptions)
+    assert value_error_raised, "ValueError was not raised as expected"
+
+    index_error_count = sum(isinstance(e, IndexError) for e in exceptions)
+    assert (
+        index_error_count == 2
+    ), "IndexError was not raised the correct number of times"

chapters/chapter10_oop/tests/test_ch10.py

@@ -117,6 +117,10 @@ def test_e74():
 def test_e75():
     customer = Customer("John", "Doe")
     assert isinstance(customer, Customer)
+    assert customer.add_account(Account("1234", "User A", 1000, [])) is None
+    assert customer.add_card(Card("1234", "User A", "2024-12-31", "1234")) is None
+    assert customer.get_total_balance() == 1000
+    assert customer.withdraw(1234, 500, 1234, 1234) is None
 
 
 def test_e76():

chapters/chapter11_standard_library/exercises/exercise_85.py

@@ -1,14 +1,18 @@
 # Exercise 85 - Parse transaction files
-# Given this "RateInterestCalculator" class with two methods,
-# we need to replace the print() messages with log statements instead with
-# the appropriate Log Level.
-
-# Can you fix this?
+# Given the following class and function, implement the `parse_transactions` function that reads a file
+# and returns a list of transactions. Each line in the file contains a timestamp and a description separated by a comma.
+# The timestamp should be converted to a `datetime` object.
+#
+from dataclasses import dataclass
 
 
+@dataclass
 class Transaction:
     timestamp: str  # Replace with `datetime`
     description: str
 
 
-def parse_transactions(filename): ...
+def parse_transactions(filename: str) -> list[Transaction]:
+    # Your code should go here.
+
+    ...