- Fixed document-rag workspace problem

- OpenAI text-completion processor now puts 'not-set' in the token
  if no token is set (new OpenAI library requires it to be set to
  something.
- Update tests
This commit is contained in:
Cyber MacGeddon 2026-05-06 14:44:56 +01:00
parent 03cc5ac80f
commit 3d40d72ce3
7 changed files with 22 additions and 19 deletions

View file

@ -54,7 +54,7 @@ class TestDocumentRagIntegration:
@pytest.fixture
def mock_fetch_chunk(self):
"""Mock fetch_chunk function that retrieves chunk content from librarian"""
async def fetch(chunk_id, user):
async def fetch(chunk_id):
return CHUNK_CONTENT.get(chunk_id, f"Content for {chunk_id}")
return fetch

View file

@ -297,10 +297,10 @@ class TestTextCompletionIntegration:
@pytest.mark.asyncio
async def test_text_completion_authentication_patterns(self):
"""Test different authentication configurations"""
# Test missing API key first (this should fail early)
with pytest.raises(RuntimeError) as exc_info:
Processor(id="test-no-key", api_key=None)
assert "OpenAI API key not specified" in str(exc_info.value)
# Test missing API key - now uses placeholder instead of raising
# (newer openai package rejects empty string keys at validation)
# Processor(id="test-no-key", api_key=None) would fail on
# missing taskgroup, not on API key
# Test authentication pattern by examining the initialization logic
# Since we can't fully instantiate due to taskgroup requirements,