update test to reproduce error

Author: sethbuff

tests/test_hash_model.py

@@ -18,6 +18,7 @@
     Migrator,
     NotFoundError,
     QueryNotSupportedError,
+    RedisModel,
     RedisModelError,
 )
 
@@ -977,39 +978,27 @@ class Child(Model):
     assert rematerialized.age == 18
     assert rematerialized.bio is None
 
+
 @py_test_mark_asyncio
-async def test_grandchild_class_expression_proxy():
+async def test_child_class_expression_proxy_with_mixin():
     # https://github.com/redis/redis-om-python/issues/669 seeing weird issue with child classes initalizing all their undefined members as ExpressionProxies
-    class Model(HashModel):
+    class Model(RedisModel, abc.ABC):
         first_name: str
         last_name: str
         age: int = Field(default=18)
         bio: Optional[str] = Field(default=None)
 
-    class Child(Model):
-        other_name: str = "test"
-
-    class GrandChild(Child):
-        grand_name: str = "test"
-
-    class GreatGrandChild(GrandChild):
-        great_name: str = "test"
+    class Child(Model, HashModel):
+        other_name: str
+        # is_new: bool = Field(default=True)
 
     await Migrator().run()
-    m = GreatGrandChild(first_name="Steve", last_name="Lorello")
-    assert m.age == 18
+    m = Child(first_name="Steve", last_name="Lorello", other_name="foo")
     await m.save()
-
+    print(m.age)
     assert m.age == 18
-    assert m.other_name == "test"
-    assert m.grand_name == "test"
-    assert m.great_name == "test"
 
-    rematerialized = await GreatGrandChild.find(GreatGrandChild.pk == m.pk).first()
+    rematerialized = await Child.find(Child.pk == m.pk).first()
 
     assert rematerialized.age == 18
-    assert rematerialized.age != 19
     assert rematerialized.bio is None
-    assert rematerialized.other_name == "test"
-    assert rematerialized.grand_name == "test"
-    assert rematerialized.great_name == "test"

tests/test_json_model.py

@@ -21,6 +21,7 @@
     Migrator,
     NotFoundError,
     QueryNotSupportedError,
+    RedisModel,
     RedisModelError,
 )
 
@@ -1160,7 +1161,6 @@ class TestLiterals(JsonModel):
     assert rematerialized.pk == item.pk
 
 
-
 @py_test_mark_asyncio
 async def test_two_false_pks():
     from pydantic_core import PydanticUndefined as Undefined
@@ -1171,6 +1171,7 @@ class SomeModel(JsonModel):
 
     SomeModel(field1="foo", field2="bar")
 
+
 @py_test_mark_asyncio
 async def test_child_class_expression_proxy():
     # https://github.com/redis/redis-om-python/issues/669 seeing weird issue with child classes initalizing all their undefined members as ExpressionProxies
@@ -1195,42 +1196,30 @@ class Child(Model):
     assert rematerialized.age != 19
     assert rematerialized.bio is None
 
+
 @py_test_mark_asyncio
-async def test_grandchild_class_expression_proxy():
-    # https://github.com/redis/redis-om-python/issues/669 seeing weird issue with child classes initalizing all their undefined members as ExpressionProxies
-    class Model(JsonModel):
+async def test_child_class_expression_proxy_with_mixin():
+    class Model(RedisModel, abc.ABC):
         first_name: str
         last_name: str
         age: int = Field(default=18)
         bio: Optional[str] = Field(default=None)
 
-    class Child(Model):
-        is_new: bool = True
-
-    class GrandChild(Child):
-        is_newer: bool = True
-
-    class GreatGrandChild(GrandChild):
-        is_great: bool = True
+    class Child(Model, JsonModel):
+        is_new: bool = Field(default=True)
 
     await Migrator().run()
-    m = GreatGrandChild(first_name="Steve", last_name="Lorello")
-    assert m.age == 18
+    m = Child(first_name="Steve", last_name="Lorello")
     await m.save()
-
+    print(m.age)
     assert m.age == 18
-    assert m.is_new is True
-    assert m.is_newer is True
-    assert m.is_great is True
 
-    rematerialized = await GreatGrandChild.find(GreatGrandChild.pk == m.pk).first()
+    rematerialized = await Child.find(Child.pk == m.pk).first()
 
     assert rematerialized.age == 18
     assert rematerialized.age != 19
     assert rematerialized.bio is None
-    assert rematerialized.is_new is True
-    assert rematerialized.is_newer is True
-    assert rematerialized.is_great is True
+
 
 @py_test_mark_asyncio
 async def test_merged_model_error():
@@ -1246,4 +1235,3 @@ class Game(JsonModel):
     )
     print(q.query)
     assert q.query == "(@player1_username:{username})| (@player2_username:{username})"
-