more cleanup

Author: chayim

Makefile

@@ -54,7 +54,7 @@ lint: $(INSTALL_STAMP) dist
 	$(POETRY) run isort --profile=black --lines-after-imports=2 ./tests/ $(NAME) $(SYNC_NAME)
 	$(POETRY) run black ./tests/ $(NAME)
 	$(POETRY) run flake8 --ignore=W503,E501,F401,E731 ./tests/ $(NAME) $(SYNC_NAME)
-	$(POETRY) run mypy ./tests/ $(NAME) $(SYNC_NAME) --ignore-missing-imports
+	$(POETRY) run mypy ./tests/ $(NAME) $(SYNC_NAME) --ignore-missing-imports --exclude migrate.py
 	$(POETRY) run bandit -r $(NAME) $(SYNC_NAME) -s B608
 
 .PHONY: format

aredis_om/model/cli/migrate.py

@@ -5,7 +5,7 @@
 
 @click.command()
 @click.option("--module", default="aredis_om")
-def migrate(module):
+def migrate(module: str):
     migrator = Migrator(module)
     migrator.detect_migrations()
 

pyproject.toml

@@ -43,7 +43,7 @@ pptree = "^3.1"
 types-redis = ">=3.5.9,<5.0.0"
 python-ulid = "^1.0.3"
 typing-extensions = "^4.4.0"
-hiredis = "^2.0.0"
+hiredis = "^2.2.3"
 more-itertools = ">=8.14,<10.0"
 
 [tool.poetry.dev-dependencies]

tests/test_hash_model.py

@@ -165,11 +165,11 @@ async def test_delete_non_exist(members, m):
 async def test_full_text_search_queries(members, m):
     member1, member2, member3 = members
 
-    actual = await (m.Member.find(m.Member.bio % "great").all())
+    actual = await m.Member.find(m.Member.bio % "great").all()
 
     assert actual == [member1]
 
-    actual = await (m.Member.find(~(m.Member.bio % "anxious")).sort_by("age").all())
+    actual = await m.Member.find(~(m.Member.bio % "anxious")).sort_by("age").all()
 
     assert actual == [member1, member3]
 
@@ -245,10 +245,10 @@ async def test_tag_queries_punctuation(m):
     )
     await member2.save()
 
-    result = await (m.Member.find(m.Member.first_name == "Andrew, the Michael").first())
+    result = await m.Member.find(m.Member.first_name == "Andrew, the Michael").first()
     assert result == member1
 
-    result = await (m.Member.find(m.Member.last_name == "St. Brookins-on-Pier").first())
+    result = await m.Member.find(m.Member.last_name == "St. Brookins-on-Pier").first()
     assert result == member1
 
     # Notice that when we index and query multiple values that use the internal