Skip to content

.NET: Disable OpenAIAssistant structured output integration tests#4451

Merged
SergeyMenshykh merged 2 commits intomicrosoft:mainfrom
SergeyMenshykh:disable-so-integration-tests
Mar 4, 2026
Merged

.NET: Disable OpenAIAssistant structured output integration tests#4451
SergeyMenshykh merged 2 commits intomicrosoft:mainfrom
SergeyMenshykh:disable-so-integration-tests

Conversation

@SergeyMenshykh
Copy link
Member

Skip all three structured output run tests in OpenAIAssistantStructuredOutputRunTests as they fail intermittently on the build agent/CI, matching the pattern already used in AzureAIAgentsPersistentStructuredOutputRunTests.

Skip all three structured output run tests in
OpenAIAssistantStructuredOutputRunTests as they fail intermittently
on the build agent/CI, matching the pattern already used in
AzureAIAgentsPersistentStructuredOutputRunTests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 4, 2026 10:27
@github-actions github-actions bot changed the title Disable OpenAIAssistant structured output integration tests .NET: Disable OpenAIAssistant structured output integration tests Mar 4, 2026
@SergeyMenshykh SergeyMenshykh self-assigned this Mar 4, 2026
@SergeyMenshykh SergeyMenshykh moved this to Sprint: In Review in Semantic Kernel Mar 4, 2026
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

Disables the flaky structured-output conformance integration tests for the OpenAI Assistant agent by explicitly skipping the three inherited StructuredOutputRunTests cases, mirroring the existing pattern used by other intermittent integrations.

Changes:

  • Introduce a shared SkipReason constant for CI/build-agent flakiness.
  • Override the three base structured-output run tests and mark them [Fact(Skip = ...)] so they are not executed.
  • Add System.Threading.Tasks import to support the overridden Task-returning methods.

@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Mar 4, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 4, 2026
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Mar 4, 2026
Merged via the queue into microsoft:main with commit f788fdc Mar 4, 2026
19 checks passed
@github-project-automation github-project-automation bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Sprint: Done

Development

Successfully merging this pull request may close these issues.

5 participants