Uh oh!
There was an error while loading. Please reload this page.
.NET Workflows - Re-enable Declarative Integration Tests - #1080
.NET Workflows - Re-enable Declarative Integration Tests#1080Chris (crickman) merged 28 commits into
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR re-enables the .NET Workflows Declarative Integration Tests by addressing the root cause related to test resource presence. The key changes include removing "Skip" attributes from test methods and improving test resource discovery paths.
- Removes skip conditions from previously disabled integration tests
- Updates configuration and file path handling for better test resource location
- Adds workflow-samples directory to CI build paths
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests.csproj | Adds workflow-samples setup files to project includes |
| Framework/IntegrationTest.cs | Adds development configuration sources |
| Framework/AgentFactory.cs | Improves agent definition file path resolution |
| DeclarativeWorkflowTest.cs | Re-enables tests and updates workflow file paths |
| DeclarativeCodeGenTest.cs | Re-enables tests and standardizes skip message |
| AzureAgentProviderTest.cs | Re-enables Azure agent provider tests |
| .github/workflows/dotnet-build-and-test.yml | Includes workflow-samples in CI build paths |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
…tionTests/Framework/AgentFactory.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
) * Investigate * Next * Update initialization * Should be ok * Agent definition dx * Link agent definitions * Link agent definitions * Path resolution microsoft#2 * Fix path resolution * Another pass * Another * Better * One more * Whoopsie * Update dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests/Framework/AgentFactory.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Namespace * Cleanup * Temp config for pipeline * Another temp workaround * Test config: Bing Grounding Tool * Update template * Next pass * Ok now * Cleanup * Test note --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
) * Investigate * Next * Update initialization * Should be ok * Agent definition dx * Link agent definitions * Link agent definitions * Path resolution #2 * Fix path resolution * Another pass * Another * Better * One more * Whoopsie * Update dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests/Framework/AgentFactory.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Namespace * Cleanup * Temp config for pipeline * Another temp workaround * Test config: Bing Grounding Tool * Update template * Next pass * Ok now * Cleanup * Test note --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Motivation and Context
Re-enable Declarative Workflow integration tests.
Description
Addressed root cause w.r.t. test resource presence and config alignment.
Note: Removal of
System.Linq.Asyncpackage affected code-gen scenario.Passing integration tests: https://github.com/microsoft/agent-framework/actions/runs/18176304289/job/51742871881
Contribution Checklist