apply coderabbit suggestions

Author: nnethercott

meilisearch/models/document.py

@@ -5,11 +5,10 @@ class Document:
     def __init__(self, doc: Dict[str, Any]) -> None:
         self.__dict__.update(**doc)
 
-    def __getattr__(self, attr: str) -> str:
-        try:
+    def __getattr__(self, attr: str) -> Any:
+        if attr in self.__dict__:
             return self.__dict__[attr]
-        except Exception as _:
-            raise AttributeError(f"{self.__class__.__name__} object has no attribute {attr}")
+        raise AttributeError(f"{self.__class__.__name__} object has no attribute {attr}")
 
     def __iter__(self) -> Iterator:
         return iter(self.__dict__.items())

pyproject.toml

@@ -114,7 +114,6 @@ disable=[
   "too-few-public-methods",
   "line-too-long",
   "too-many-positional-arguments",
-  "raise-missing-from",
 ]
 enable=[
   "use-symbolic-message-instead",

tests/models/test_document.py

@@ -7,22 +7,22 @@
 
 
 def test_doc_init():
-    d = {"field1": "test 1", "fiels2": "test 2"}
+    d = {"field1": "test 1", "field2": "test 2"}
     document = Document(d)
     assert dict(document) == d
 
 
 def test_getattr():
-    document = Document({"field1": "test 1", "fiels2": "test 2"})
+    document = Document({"field1": "test 1", "field2": "test 2"})
     assert document.__getattr__("field1") == "test 1"
 
 
 def test_getattr_not_found():
-    document = Document({"field1": "test 1", "fiels2": "test 2"})
+    document = Document({"field1": "test 1", "field2": "test 2"})
     with pytest.raises(AttributeError):
         document.__getattr__("bad")
 
 
 def test_iter():
-    document = Document({"field1": "test 1", "fiels2": "test 2"})
-    assert list(iter(document)) == [("field1", "test 1"), ("fiels2", "test 2")]
+    document = Document({"field1": "test 1", "field2": "test 2"})
+    assert list(iter(document)) == [("field1", "test 1"), ("field2", "test 2")]