Skip to content

Commit 4ac93c1

Browse files
committed
Fix test, test workflow
1 parent e46a9b0 commit 4ac93c1

File tree

2 files changed

+19
-5
lines changed

2 files changed

+19
-5
lines changed

.github/workflows/python-tests.yml

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,27 @@ jobs:
3030
test:
3131
needs: lint
3232
runs-on: ubuntu-latest
33+
strategy:
34+
fail-fast: false
35+
matrix:
36+
python-version: [3.12, 3.13]
37+
redis-version: ['6.2.6-v9', 'latest', '8.0-M03']
38+
3339
steps:
3440
- uses: actions/checkout@v3
3541

42+
- name: Set Redis image name
43+
run: |
44+
if [[ "${{ matrix.redis-version }}" == "8.0-M03" ]]; then
45+
echo "REDIS_IMAGE=redis:${{ matrix.redis-version }}" >> $GITHUB_ENV
46+
else
47+
echo "REDIS_IMAGE=redis/redis-stack-server:${{ matrix.redis-version }}" >> $GITHUB_ENV
48+
f
49+
3650
- name: Set up Python
3751
uses: actions/setup-python@v4
3852
with:
39-
python-version: '3.12'
53+
python-version: ${{ matrix.python-version }}
4054
cache: 'pip'
4155

4256
- name: Install dependencies

tests/test_api.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,8 @@ async def test_post_memory_stores_in_long_term_memory(self, client):
127127
mock_add_task = MagicMock()
128128

129129
with (
130-
patch("redis_memory_server.memory.settings", mock_settings),
131-
patch("redis_memory_server.memory.BackgroundTasks.add_task", mock_add_task),
130+
patch("redis_memory_server.api.settings", mock_settings),
131+
patch("redis_memory_server.api.BackgroundTasks.add_task", mock_add_task),
132132
):
133133
response = await client.post("/sessions/test-session/memory", json=payload)
134134

@@ -159,8 +159,8 @@ async def test_post_memory_compacts_long_conversation(self, client):
159159
mock_add_task = MagicMock()
160160

161161
with (
162-
patch("redis_memory_server.memory.settings", mock_settings),
163-
patch("redis_memory_server.memory.BackgroundTasks.add_task", mock_add_task),
162+
patch("redis_memory_server.api.settings", mock_settings),
163+
patch("redis_memory_server.api.BackgroundTasks.add_task", mock_add_task),
164164
):
165165
response = await client.post("/sessions/test-session/memory", json=payload)
166166

0 commit comments

Comments
 (0)