.NET: Add Executor RouteBuilder Unit Tests by Copilot · Pull Request #5824 · microsoft/agent-framework

Copilot · 2026-05-13T19:26:30Z

Motivation and Context

This PR adds comprehensive unit tests for the RouteBuilder class in the Workflows library and includes several related fixes and cleanups discovered during the process.

Description

New RouteBuilder Unit Tests (RouteBuilderTests.cs)

Tests all 8 handler overloads (sync/async × void/result × with/without CancellationToken) via HandlerOverload enum
Tests all 8 catch-all fallback routing variants using PortableValue
Tests port handler registration, request posting, response routing, and unknown-port failure handling
Tests registration semantics: duplicate rejection, overwrite behavior, and PortableValue type guard

Contribution Checklist

The code builds clean without any errors or warnings
The PR follows the Contribution Guidelines
All unit tests pass, and I have added new tests where possible
Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/012f3b3b-acb9-4869-9084-b767cbe1885b Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/19397f58-a88a-41cf-bd85-588f520e0d0f Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/a8437809-0898-43a6-a950-09eb3417f58a Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/8573214e-ec42-4969-ba94-76bdc8ad3e59 Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Copilot AI and others added 6 commits May 13, 2026 18:59

Add RouteBuilder unit tests

9ca8ef5

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/012f3b3b-acb9-4869-9084-b767cbe1885b Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Address RouteBuilder test review feedback

224d9ce

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/012f3b3b-acb9-4869-9084-b767cbe1885b Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Fix RouteBuilder test nullability warning

2b66bad

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/012f3b3b-acb9-4869-9084-b767cbe1885b Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Refine RouteBuilder test helpers

e59b932

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/012f3b3b-acb9-4869-9084-b767cbe1885b Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Refactor overload int constants to HandlerOverload enum

1a6e443

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/19397f58-a88a-41cf-bd85-588f520e0d0f Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Fix ValueTask compatibility with .NET Framework 4.7.2

53787a3

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/a8437809-0898-43a6-a950-09eb3417f58a Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Copilot AI assigned Copilot and lokitoth May 13, 2026

Copilot AI review requested due to automatic review settings May 13, 2026 19:26

Copilot created this pull request from a session on behalf of lokitoth May 13, 2026 19:26 View session

Copilot AI removed the request for review from Copilot May 13, 2026 19:26

Merge branch 'main' into copilot/add-unit-tests-workflows-routebuilder

8d72a1c

lokitoth changed the title ~~Add RouteBuilder unit tests with readable HandlerOverload enum~~ Add Executor RouteBuilder Unit Tests May 13, 2026

lokitoth temporarily deployed to integration May 13, 2026 19:27 — with GitHub Actions Inactive

moonbox3 added the .NET Usage: [Issues, PRs], Target: .Net label May 13, 2026

github-actions Bot changed the title ~~Add Executor RouteBuilder Unit Tests~~ .NET: Add Executor RouteBuilder Unit Tests May 13, 2026

Copilot started work on behalf of lokitoth May 13, 2026 19:29 View session

Fix IDE0001 format errors - simplify generic type names

beea1cf

Agent-Logs-Url: https://github.com/microsoft/agent-framework/sessions/8573214e-ec42-4969-ba94-76bdc8ad3e59 Co-authored-by: lokitoth <6936551+lokitoth@users.noreply.github.com>

Copilot AI requested review from Copilot and removed request for Copilot May 13, 2026 19:49

Copilot finished work on behalf of lokitoth May 13, 2026 19:50

Copilot AI requested a review from lokitoth May 13, 2026 19:50

Copilot started work on behalf of lokitoth May 13, 2026 20:04 View session

Copilot finished work on behalf of lokitoth May 13, 2026 20:06

Merge branch 'main' into copilot/add-unit-tests-workflows-routebuilder

0ecd13c

Copilot AI review requested due to automatic review settings May 13, 2026 20:38

lokitoth added the workflows Usage: [Issues, PRs], Target: Workflows label May 13, 2026

lokitoth temporarily deployed to integration May 13, 2026 20:39 — with GitHub Actions Inactive

dependabot Bot mentioned this pull request Jul 26, 2026

Bump the nuget-all group with 6 updates dotnet-presentations/ai-workshop#547

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

.NET: Add Executor RouteBuilder Unit Tests - #5824

.NET: Add Executor RouteBuilder Unit Tests#5824
lokitoth merged 9 commits into
mainfrom
copilot/add-unit-tests-workflows-routebuilder

Copilot AI commented May 13, 2026 •

edited by lokitoth

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

Copilot AI commented May 13, 2026 • edited by lokitoth Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation and Context

Description

Contribution Checklist

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Copilot AI commented May 13, 2026 •

edited by lokitoth

Loading