From f2105c0b6f2bdb3c52b95e6d6fd7fd75b7cbe925 Mon Sep 17 00:00:00 2001
From: "J. Nick Koston" <nick@koston.org>
Date: Sun, 6 Oct 2024 18:20:30 -0500
Subject: [PATCH] Add typing to propcache_module in tests (#33)

Add typing to propcache_module
---
 tests/test_cached_property.py       | 17 +++++++++--------
 tests/test_under_cached_property.py | 18 +++++++++++-------
 2 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/tests/test_cached_property.py b/tests/test_cached_property.py
index 26a14bb..d1f5cc9 100644
--- a/tests/test_cached_property.py
+++ b/tests/test_cached_property.py
@@ -1,9 +1,10 @@
 from operator import not_
+from types import ModuleType
 
 import pytest
 
 
-def test_cached_property(propcache_module) -> None:
+def test_cached_property(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             self._cache = {}
@@ -16,7 +17,7 @@ def prop(self):
     assert a.prop == 1
 
 
-def test_cached_property_class(propcache_module) -> None:
+def test_cached_property_class(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             """Init."""
@@ -30,7 +31,7 @@ def prop(self):
     assert A.prop.__doc__ == "Docstring."
 
 
-def test_cached_property_without_cache(propcache_module) -> None:
+def test_cached_property_without_cache(propcache_module: ModuleType) -> None:
     class A:
 
         __slots__ = ()
@@ -48,7 +49,7 @@ def prop(self):
         a.prop = 123
 
 
-def test_cached_property_check_without_cache(propcache_module) -> None:
+def test_cached_property_check_without_cache(propcache_module: ModuleType) -> None:
     class A:
 
         __slots__ = ()
@@ -65,7 +66,7 @@ def prop(self):
         assert a.prop == 1
 
 
-def test_cached_property_caching(propcache_module) -> None:
+def test_cached_property_caching(propcache_module: ModuleType) -> None:
 
     class A:
         def __init__(self):
@@ -80,7 +81,7 @@ def prop(self):
     assert 1 == a.prop
 
 
-def test_cached_property_class_docstring(propcache_module) -> None:
+def test_cached_property_class_docstring(propcache_module: ModuleType) -> None:
 
     class A:
         def __init__(self):
@@ -94,7 +95,7 @@ def prop(self):
     assert "Docstring." == A.prop.__doc__
 
 
-def test_set_name(propcache_module) -> None:
+def test_set_name(propcache_module: ModuleType) -> None:
     """Test that the __set_name__ method is called and checked."""
 
     class A:
@@ -110,7 +111,7 @@ def prop(self):
         A.prop.__set_name__(A, "something_else")
 
 
-def test_get_without_set_name(propcache_module) -> None:
+def test_get_without_set_name(propcache_module: ModuleType) -> None:
     """Test that get without __set_name__ fails."""
     cp = propcache_module.cached_property(not_)
 
diff --git a/tests/test_under_cached_property.py b/tests/test_under_cached_property.py
index 81c79fc..eae819e 100644
--- a/tests/test_under_cached_property.py
+++ b/tests/test_under_cached_property.py
@@ -1,7 +1,9 @@
+from types import ModuleType
+
 import pytest
 
 
-def test_under_cached_property(propcache_module) -> None:
+def test_under_cached_property(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             self._cache = {}
@@ -14,7 +16,7 @@ def prop(self):
     assert a.prop == 1
 
 
-def test_under_cached_property_class(propcache_module) -> None:
+def test_under_cached_property_class(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             """Init."""
@@ -28,7 +30,7 @@ def prop(self):
     assert A.prop.__doc__ == "Docstring."
 
 
-def test_under_cached_property_assignment(propcache_module) -> None:
+def test_under_cached_property_assignment(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             self._cache = {}
@@ -43,7 +45,7 @@ def prop(self):
         a.prop = 123
 
 
-def test_under_cached_property_without_cache(propcache_module) -> None:
+def test_under_cached_property_without_cache(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             pass
@@ -58,7 +60,9 @@ def prop(self):
         a.prop = 123
 
 
-def test_under_cached_property_check_without_cache(propcache_module) -> None:
+def test_under_cached_property_check_without_cache(
+    propcache_module: ModuleType,
+) -> None:
     class A:
         def __init__(self):
             pass
@@ -72,7 +76,7 @@ def prop(self):
         assert a.prop == 1
 
 
-def test_under_cached_property_caching(propcache_module) -> None:
+def test_under_cached_property_caching(propcache_module: ModuleType) -> None:
 
     class A:
         def __init__(self):
@@ -87,7 +91,7 @@ def prop(self):
     assert 1 == a.prop
 
 
-def test_under_cached_property_class_docstring(propcache_module) -> None:
+def test_under_cached_property_class_docstring(propcache_module: ModuleType) -> None:
     class A:
         def __init__(self):
             """Init."""