improve test coverage

Author: alex-oleshkevich

starsessions/middleware.py

@@ -20,10 +20,10 @@ def __getattribute__(self, item: str) -> typing.Any:
 
         self._raise()
 
-    def __setitem__(self, key: str, value: typing.Any) -> typing.NoReturn:
+    def __setitem__(self, key: str, value: typing.Any) -> typing.NoReturn:  # pragma: nocover
         self._raise()
 
-    def __getitem__(self, key: str) -> typing.NoReturn:
+    def __getitem__(self, key: str) -> typing.NoReturn:  # pragma: nocover
         self._raise()
 
     def _raise(self) -> typing.NoReturn:

starsessions/stores/redis.py

@@ -11,7 +11,7 @@ def prefix_factory(prefix: str, key: str) -> str:
     return prefix + key
 
 
-if typing.TYPE_CHECKING:
+if typing.TYPE_CHECKING:  # pragma: nocover
     BaseRedis = Redis[bytes]
 else:
     BaseRedis = Redis

tests/backends/test_cookie.py

@@ -28,3 +28,9 @@ async def test_cookie_remove(cookie_store: SessionStore) -> None:
 @pytest.mark.asyncio
 async def test_cookie_exists(cookie_store: SessionStore) -> None:
     assert await cookie_store.exists("session_id") is False
+
+
+@pytest.mark.asyncio
+async def test_returns_empty_string_for_bad_signature(cookie_store: SessionStore) -> None:
+    # the session_id value is a signed session cookie value
+    assert await cookie_store.read("session_id", lifetime=10) == b""

tests/backends/test_memory.py

@@ -21,6 +21,9 @@ async def test_in_memory_remove(in_memory_store: SessionStore) -> None:
     await in_memory_store.remove("session_id")
     assert await in_memory_store.exists("session_id") is False
 
+    # should not fail on missing key
+    await in_memory_store.remove("missing")
+
 
 @pytest.mark.asyncio
 async def test_in_memory_exists(in_memory_store: SessionStore) -> None:

tests/backends/test_redis.py

@@ -1,7 +1,9 @@
 import os
 import pytest
+import redis.asyncio
 from pytest_asyncio.plugin import SubRequest
 
+from starsessions import ImproperlyConfigured
 from starsessions.stores.base import SessionStore
 from starsessions.stores.redis import RedisStore
 
@@ -46,3 +48,22 @@ async def test_redis_exists(redis_store: SessionStore) -> None:
 @pytest.mark.asyncio
 async def test_redis_empty_session(redis_store: SessionStore) -> None:
     assert await redis_store.read("unknown_session_id", lifetime=60) == b""
+
+
+@pytest.mark.asyncio
+async def test_redis_requires_url_or_connection() -> None:
+    with pytest.raises(ImproperlyConfigured):
+        RedisStore()
+
+
+@pytest.mark.asyncio
+async def test_redis_uses_url() -> None:
+    store = RedisStore(url="redis://")
+    assert isinstance(store._connection, redis.asyncio.Redis)
+
+
+@pytest.mark.asyncio
+async def test_redis_uses_connection() -> None:
+    connection = redis.asyncio.Redis.from_url("redis://")
+    store = RedisStore(connection=connection)
+    assert store._connection == connection

tests/test_metadata.py

@@ -11,7 +11,7 @@
 
 
 def test_requires_loaded_session(store: SessionStore) -> None:
-    async def app(scope: Scope, receive: Receive, send: Send) -> None:
+    async def app(scope: Scope, receive: Receive, send: Send) -> None:  # pragma: nocover
         connection = HTTPConnection(scope, receive)
         response = JSONResponse(get_session_metadata(connection))
         await response(scope, receive, send)

tests/test_middleware.py

@@ -240,7 +240,7 @@ async def app(scope: Scope, receive: Receive, send: Send) -> None:
 def test_session_timedelta_lifetime(store: SessionStore) -> None:
     """It should accept datetime.timedelta as lifetime value."""
 
-    async def app(scope: Scope, receive: Receive, send: Send) -> None:
+    async def app(scope: Scope, receive: Receive, send: Send) -> None:  # pragma: nocover
         pass
 
     app = SessionMiddleware(app, store=store, lifetime=datetime.timedelta(seconds=60))