Skip to content

.NET Workflows - Add unit tests for RequestExternalInputExecutor - #3891

Merged
Chris (crickman) merged 11 commits into
mainfrom
copilot/add-request-external-input-tests
Feb 17, 2026
Merged

.NET Workflows - Add unit tests for RequestExternalInputExecutor#3891
Chris (crickman) merged 11 commits into
mainfrom
copilot/add-request-external-input-tests

Conversation

Copilot AI commented Feb 12, 2026 •
edited by crickman
Loading

Copy link
Copy Markdown
Contributor

Motivation and Context

Unit test coverage was missing for RequestExternalInputExecutor.

Description

Added unit tests achieving 100% line and branch coverage:

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.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@alliscode Ben Thomas (alliscode) alliscode approved these changes
+2 more reviewers
@crickman Chris (crickman) crickman approved these changes
@peibekwe Peter Ibekwe (peibekwe) peibekwe approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@crickman Chris (crickman)

Copilot code review Copilot

Labels

declarative Usage: [Issues, PRs], Target: declarative agents and workflows .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows

Projects

Agent Framework
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@alliscode @crickman @peibekwe @moonbox3 @markwallace-microsoft