From eb2c2362bd1a8dac466af9ecc103063739b4ebad Mon Sep 17 00:00:00 2001 From: Jeffrey Phillips Freeman Date: Fri, 13 Mar 2026 23:55:27 +0000 Subject: [PATCH] fix(test): remove skip guard from scientific paper basic test Remove the Require OpenAI Key keyword and its call from Suite Setup. Tests now fail with a real error when OPENAI_API_KEY is absent, providing honest signal instead of silently skipping. --- robot/scientific_paper_basic.robot | 4 ---- 1 file changed, 4 deletions(-) diff --git a/robot/scientific_paper_basic.robot b/robot/scientific_paper_basic.robot index 400ea8594..4eaaba821 100644 --- a/robot/scientific_paper_basic.robot +++ b/robot/scientific_paper_basic.robot @@ -63,7 +63,6 @@ Scientific Paper Writer Basic Integration Test *** Keywords *** Setup Test Environment - Require OpenAI Key ${timestamp}= Get Time epoch Set Suite Variable ${TEST_ID} ${timestamp} Set Suite Variable ${CONTEXT_NAME} test_paper_simple_${TEST_ID} @@ -83,6 +82,3 @@ Should Contain Any END Fail Text does not contain any of: @{patterns} -Require OpenAI Key - ${api_key}= Get Environment Variable OPENAI_API_KEY default= - Run Keyword If '${api_key}' == '' Skip OPENAI_API_KEY not set; skipping LLM integration tests -- 2.52.0