skip if OPENAI_API_KEY not defined

[paul-nechifor]

No description provided.

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[greptile-apps]

Greptile Summary

Added @pytest.mark.skipif decorator to skip the E2E test when OPENAI_API_KEY environment variable is not defined.

• Prevents test failures when API key is missing
• Consistent with similar skip patterns used elsewhere in the codebase (e.g., dimos/models/vl/test_base.py:81)
• The test invokes an LLM agent that requires OpenAI API access, so skipping without the key is appropriate

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The change is minimal, well-targeted, and follows established patterns in the codebase. It adds a single line decorator to gracefully skip a test that requires an OpenAI API key when that key is not available, preventing spurious test failures.
• No files require special attention

Important Files Changed

Filename	Overview
dimos/robot/cli/test_dimos_robot_e2e.py	Added skipif decorator to skip test when OPENAI_API_KEY is not defined, consistent with similar patterns in codebase

Sequence Diagram

sequenceDiagram
    participant Pytest as Pytest Runner
    participant Env as Environment
    participant Test as test_dimos_robot_demo_e2e
    participant LCM as LCMSpy
    participant Dimos as DimosRobotCall
    
    Pytest->>Env: Check CI env variable
    alt CI environment detected
        Env-->>Pytest: CI=true
        Pytest->>Test: Skip (LCM spy doesn't work in CI)
    else CI not detected
        Env-->>Pytest: CI not set
        Pytest->>Env: Check OPENAI_API_KEY
        alt OPENAI_API_KEY not set
            Env-->>Pytest: No API key
            Pytest->>Test: Skip (OPENAI_API_KEY not set)
        else OPENAI_API_KEY exists
            Env-->>Pytest: API key available
            Pytest->>Test: Execute test
            Test->>LCM: Start LCMSpy
            Test->>Dimos: Start DimosRobotCall
            Test->>LCM: Wait for topics
            Test->>Dimos: Send human input
            Test->>LCM: Verify AI response
            Test->>Test: Assert success
        end
    end

Loading

[greptile-apps]

Greptile's behavior is changing!

From now on, if a review finishes with no comments, we will not post an additional "statistics" comment to confirm that our review found nothing to comment on. However, you can confirm that we reviewed your changes in the status check section.

_{This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR".}