Address Copilot code review feedback

- Add missing importlib.util import in data_migrator.py
- Use type(None) instead of string comparison for type checking
- Remove debug print statements from test files (security concern)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: abrookins

aredis_om/model/migrations/data_migrator.py

@@ -9,6 +9,7 @@
 import abc
 import asyncio
 import importlib
+import importlib.util
 import os
 import time
 from datetime import datetime

aredis_om/model/model.py

@@ -89,7 +89,7 @@ def convert_timestamp_to_datetime(obj, model_fields):
                     non_none_types = [
                         arg
                         for arg in args
-                        if getattr(arg, "__name__", None) != "NoneType"
+                        if arg is not type(None)
                     ]
                     if len(non_none_types) == 1:
                         field_type = non_none_types[0]

tests/test_datetime_date_fix.py

@@ -45,8 +45,6 @@ async def test_hash_model_date_conversion():
         if isinstance(birth_date_value, bytes):
             birth_date_value = birth_date_value.decode("utf-8")
 
-        print(f"Stored value: {birth_date_value} (type: {type(birth_date_value)})")
-
         # Should be able to parse as a float (timestamp)
         try:
             float(birth_date_value)
@@ -84,8 +82,6 @@ async def test_json_model_date_conversion():
         # The birth_date field should be stored as a timestamp (number)
         birth_date_value = raw_data.get("birth_date")
 
-        print(f"Stored value: {birth_date_value} (type: {type(birth_date_value)})")
-
         assert isinstance(
             birth_date_value, (int, float)
         ), f"Expected timestamp, got: {birth_date_value} ({type(birth_date_value)})"