From ca3ec3fab783327fd72427a087745ae10ef6e860 Mon Sep 17 00:00:00 2001 From: HAL9000 Date: Thu, 23 Apr 2026 10:56:16 +0000 Subject: [PATCH] fix(e2e): add Force Tags E2E and remove duplicate Skip If No LLM Keys keyword in workflow tests --- robot/e2e/test_correction_workflow.robot | 19 +++++-------------- robot/e2e/test_project_plan_workflow.robot | 19 +++++-------------- robot/e2e/test_subplan_workflow.robot | 19 +++++-------------- 3 files changed, 15 insertions(+), 42 deletions(-) diff --git a/robot/e2e/test_correction_workflow.robot b/robot/e2e/test_correction_workflow.robot index 782060789..e71f9a443 100644 --- a/robot/e2e/test_correction_workflow.robot +++ b/robot/e2e/test_correction_workflow.robot @@ -11,6 +11,7 @@ Documentation E2E workflow test for plan correction workflows. Resource common_e2e.resource Suite Setup E2E Suite Setup Suite Teardown E2E Suite Teardown +Force Tags E2E *** Test Cases *** Correction Revert Mode Workflow @@ -19,7 +20,7 @@ Correction Revert Mode Workflow ... Creates a project, generates a plan, executes it, ... then corrects it using revert mode to restore ... the plan to a previous state. - [Tags] E2E correction-revert + [Tags] correction-revert Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory correction-revert-test @@ -37,7 +38,7 @@ Correction Append Mode Workflow ... Creates a project, generates a plan, executes it, ... then corrects it using append mode to add corrections ... to the existing plan. - [Tags] E2E correction-append + [Tags] correction-append Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory correction-append-test @@ -54,7 +55,7 @@ Correction State Transition Validation ... ... Verifies that correction workflows properly transition ... plan state through correction phases. - [Tags] E2E correction-state + [Tags] correction-state Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory correction-state-test @@ -70,15 +71,5 @@ Correction State Transition Validation Create Temp Directory [Documentation] Create a temporary directory for testing. [Arguments] ${name} - ${temp_dir}= Evaluate __import__('tempfile').mkdtemp(prefix='${name}-') + ${temp_dir}= Evaluate __import__("tempfile").mkdtemp(prefix="${name}-") RETURN ${temp_dir} - -Skip If No LLM Keys - [Documentation] Skip test if LLM API keys are not configured. - ... - ... Checks for OPENAI_API_KEY or other LLM provider keys - ... in the environment. Skips the test if none are found. - ${openai_key}= Get Environment Variable OPENAI_API_KEY ${EMPTY} - ${anthropic_key}= Get Environment Variable ANTHROPIC_API_KEY ${EMPTY} - ${has_keys}= Evaluate bool('${openai_key}' or '${anthropic_key}') - Skip If not ${has_keys} LLM API keys not configured diff --git a/robot/e2e/test_project_plan_workflow.robot b/robot/e2e/test_project_plan_workflow.robot index ded93c88b..645be1c76 100644 --- a/robot/e2e/test_project_plan_workflow.robot +++ b/robot/e2e/test_project_plan_workflow.robot @@ -12,6 +12,7 @@ Documentation E2E workflow test for project creation, actor setup, and plan e Resource common_e2e.resource Suite Setup E2E Suite Setup Suite Teardown E2E Suite Teardown +Force Tags E2E *** Test Cases *** Project Creation Workflow @@ -19,7 +20,7 @@ Project Creation Workflow ... ... Creates a new project via CLI, verifies project ... is created successfully, and validates output format. - [Tags] E2E project-creation + [Tags] project-creation Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory project-creation-test @@ -37,7 +38,7 @@ Actor Setup Workflow ... ... Creates a project, adds actors to it, and verifies ... actors are properly registered. - [Tags] E2E actor-setup + [Tags] actor-setup Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory actor-setup-test @@ -53,7 +54,7 @@ Plan Execution Workflow ... ... Creates a project, generates a plan, and executes it ... through the complete workflow. - [Tags] E2E plan-execution + [Tags] plan-execution Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory plan-execution-test @@ -69,15 +70,5 @@ Plan Execution Workflow Create Temp Directory [Documentation] Create a temporary directory for testing. [Arguments] ${name} - ${temp_dir}= Evaluate __import__('tempfile').mkdtemp(prefix='${name}-') + ${temp_dir}= Evaluate __import__("tempfile").mkdtemp(prefix="${name}-") RETURN ${temp_dir} - -Skip If No LLM Keys - [Documentation] Skip test if LLM API keys are not configured. - ... - ... Checks for OPENAI_API_KEY or other LLM provider keys - ... in the environment. Skips the test if none are found. - ${openai_key}= Get Environment Variable OPENAI_API_KEY ${EMPTY} - ${anthropic_key}= Get Environment Variable ANTHROPIC_API_KEY ${EMPTY} - ${has_keys}= Evaluate bool('${openai_key}' or '${anthropic_key}') - Skip If not ${has_keys} LLM API keys not configured diff --git a/robot/e2e/test_subplan_workflow.robot b/robot/e2e/test_subplan_workflow.robot index 959175ed1..16345b9ae 100644 --- a/robot/e2e/test_subplan_workflow.robot +++ b/robot/e2e/test_subplan_workflow.robot @@ -11,6 +11,7 @@ Documentation E2E workflow test for subplan spawning and three-way merge. Resource common_e2e.resource Suite Setup E2E Suite Setup Suite Teardown E2E Suite Teardown +Force Tags E2E *** Test Cases *** Subplan Spawning Workflow @@ -18,7 +19,7 @@ Subplan Spawning Workflow ... ... Creates a project, generates a plan, and spawns ... subplans from the parent plan. - [Tags] E2E subplan-spawn + [Tags] subplan-spawn Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory subplan-spawn-test @@ -35,7 +36,7 @@ Three-Way Merge Workflow ... ... Creates a project, generates a plan with subplans, ... executes them, and performs three-way merge. - [Tags] E2E three-way-merge + [Tags] three-way-merge Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory three-way-merge-test @@ -52,7 +53,7 @@ Merge Result Validation ... ... Verifies that merge results are properly validated ... and conform to specification requirements. - [Tags] E2E merge-validation + [Tags] merge-validation Skip If No LLM Keys # Create a temporary directory for the project ${project_dir}= Create Temp Directory merge-validation-test @@ -68,15 +69,5 @@ Merge Result Validation Create Temp Directory [Documentation] Create a temporary directory for testing. [Arguments] ${name} - ${temp_dir}= Evaluate __import__('tempfile').mkdtemp(prefix='${name}-') + ${temp_dir}= Evaluate __import__("tempfile").mkdtemp(prefix="${name}-") RETURN ${temp_dir} - -Skip If No LLM Keys - [Documentation] Skip test if LLM API keys are not configured. - ... - ... Checks for OPENAI_API_KEY or other LLM provider keys - ... in the environment. Skips the test if none are found. - ${openai_key}= Get Environment Variable OPENAI_API_KEY ${EMPTY} - ${anthropic_key}= Get Environment Variable ANTHROPIC_API_KEY ${EMPTY} - ${has_keys}= Evaluate bool('${openai_key}' or '${anthropic_key}') - Skip If not ${has_keys} LLM API keys not configured