refactor: address PR reviews

openedx_events/event_bus/avro/serializer.py

@@ -69,7 +69,6 @@ def _event_data_to_avro_record_dict(event_data, serializers=None):
     def value_to_dict(value):
         # Case 1: Value is an instance of an attrs-decorated class
         if hasattr(value, "__attrs_attrs__"):
-            print("\n\n MY VALUE IN VALUE TO DICT", value, "\n\n")
             return attr.asdict(value, value_serializer=_get_non_attrs_serializer(serializers))
         return _get_non_attrs_serializer(serializers)(None, None, value)
 

openedx_events/event_bus/avro/tests/test_schema.py

@@ -1,7 +1,7 @@
 """
 Tests for event_bus.avro.schema module
 """
-from typing import List
+from typing import List, Dict
 from unittest import TestCase
 
 from openedx_events.event_bus.avro.schema import schema_from_signal
@@ -245,8 +245,9 @@ class UnextendedClass:
 
     def test_throw_exception_to_list_or_dict_types_without_annotation(self):
         LIST_SIGNAL = create_simple_signal({"list_input": list})
-        DICT_SIGNAL = create_simple_signal({"list_input": dict})
+        DICT_SIGNAL = create_simple_signal({"dict_input": dict})
         LIST_WITHOUT_ANNOTATION_SIGNAL = create_simple_signal({"list_input": List})
+        DICT_WITHOUT_ANNOTATION_SIGNAL = create_simple_signal({"dict_input": Dict})
         with self.assertRaises(Exception):
             schema_from_signal(LIST_SIGNAL)
 
@@ -256,6 +257,9 @@ def test_throw_exception_to_list_or_dict_types_without_annotation(self):
         with self.assertRaises(TypeError):
             schema_from_signal(LIST_WITHOUT_ANNOTATION_SIGNAL)
 
+        with self.assertRaises(TypeError):
+            schema_from_signal(DICT_WITHOUT_ANNOTATION_SIGNAL)
+
     def test_list_with_annotation_works(self):
         LIST_SIGNAL = create_simple_signal({"list_input": List[int]})
         expected_dict = {