openai: make instrument fixture explicit

[nehachinnasani23]

Summary

• Remove autouse from the OpenAI instrument test fixture and require tests to opt in explicitly.
• Add a capture-message-content instrumentation fixture for tests that need OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT=true.
• Remove repeated uninstrument/reinstrument blocks from capture-message-content tests.

Closes #55

Testing

• . .venv/bin/activate && cd instrumentation/elastic-opentelemetry-instrumentation-openai && pytest tests/test_instrumentor.py tests/test_embeddings.py tests/test_chat_completions.py
• . .venv/bin/activate && cd instrumentation/elastic-opentelemetry-instrumentation-openai && pytest
• . .venv/bin/activate && python -m compileall -q instrumentation/elastic-opentelemetry-instrumentation-openai/tests/conftest.py instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_instrumentor.py instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_embeddings.py instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_chat_completions.py instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_beta_chat_completions.py
• git diff --check

Note

• python -m ruff check ... was attempted, but ruff is not installed in the dev requirements environment.

[Copilot]

Pull request overview

This PR updates the OpenAI instrumentation test suite to make instrumentation opt-in per test, and introduces a dedicated fixture for tests that must run with OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT=true.

Changes:

• Remove autouse=True from the instrument fixture and require tests to request instrumentation explicitly.
• Add a capture_message_content_instrument fixture that sets OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT=true and instruments accordingly.
• Simplify capture-message-content tests by removing repeated manual uninstrument/reinstrument blocks.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
instrumentation/elastic-opentelemetry-instrumentation-openai/tests/conftest.py	Makes instrumentation fixtures explicit and adds a capture-message-content variant.
instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_instrumentor.py	Adjusts instrumentor tests to no longer rely on autouse instrumentation.
instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_embeddings.py	Updates tests to opt in to instrumentation via the instrument fixture.
instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_chat_completions.py	Updates tests to opt in to instrumentation; uses capture-message-content fixture where required.
instrumentation/elastic-opentelemetry-instrumentation-openai/tests/test_beta_chat_completions.py	Same as above for beta chat completions tests.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.