Skip to content

fix: support context vars #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
imnotjames wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

fix: support context vars #11

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
fdb5d04
test: validate context vars are supported
imnotjames Oct 30, 2025
52ef1e1
test: import contextvars
imnotjames Oct 30, 2025
1ff5b49
test: await
imnotjames Oct 30, 2025
cc5d4c9
test: try another approach
imnotjames Oct 30, 2025
8d0bc05
test: call example correctly
imnotjames Oct 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions tests/test_cached.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import asyncio
import contextlib
import contextvars
from unittest.mock import AsyncMock, MagicMock, call

import pytest
Expand Down Expand Up @@ -68,6 +69,23 @@
assert await decorated_fn("foo") == ("foo",)
assert await decorated_fn("foo", bar="baz") == ("foo", ("bar", "baz"))

async def test_context_var_passed(self):
var = contextvars.ContextVar("var")
var.set("test")

def read_contextvar():
return var.get()

async def example():
var.set("example")
return read_contextvar()

decorated_fn = cachetools_async.cached({})(example)

actual = await decorated_fn()
assert actual == "example"
assert var.get() == "test"

async def test_multiple_calls(self):
mock = AsyncMock()

Expand Down
Loading