Skip to content

Conversation

@nikhilNava
Copy link
Contributor

@nikhilNava nikhilNava commented Oct 30, 2025

Task

  1. Add integration test to test with observability
  2. Remove reference of kairo

Copilot AI review requested due to automatic review settings October 30, 2025 13:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds integration tests for the OpenAI trace processor to validate observability functionality with real Azure OpenAI services. The tests verify that telemetry data is correctly captured and processed when using the OpenAI Agents SDK.

Key changes:

  • New integration test suite for OpenAI trace processor with both basic and tool-based scenarios
  • Test infrastructure setup with pytest fixtures for Azure OpenAI and Agent365 configuration
  • CI workflow updates to separate unit and integration tests, with integration tests running conditionally based on available secrets

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/integration/test_openai_trace_processor.py Adds comprehensive integration tests for OpenAI trace processor with mock span exporter and validation logic
tests/integration/conftest.py Provides pytest fixtures for Azure OpenAI and Agent365 configuration with environment variable support
tests/integration/init.py Initializes the integration tests package with proper copyright header
tests/.env.example Documents the new AZURE_OPENAI_API_KEY environment variable requirement
pyproject.toml Adds development dependencies and pytest configuration for unit and integration test markers
.github/workflows/ci.yml Separates unit and integration test execution with conditional integration test runs

@nikhilNava nikhilNava merged commit e06a40a into main Oct 30, 2025
8 checks passed
@nikhilNava nikhilNava deleted the nikhilc/AddIntegrationTests branch October 30, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants