Skip to content

.NET Workflows - Add unit tests for DefaultActionExecutor (Declarative Workflows) #3836

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
Copilot wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

.NET Workflows - Add unit tests for DefaultActionExecutor (Declarative Workflows) #3836

Copilot wants to merge 5 commits into from

Conversation

Copy link
Contributor

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

Motivation and Context

The DefaultActionExecutor class in the declarative workflows implementation lacked unit test coverage. This PR adds comprehensive tests following established patterns in the ObjectModel test suite.

Description

Added DefaultActionExecutorTest.cs with 4 test cases achieving 100% line and branch coverage:

  • Constructor validation: Verifies proper initialization of Id and Model properties
  • Basic execution: Tests the no-op ExecuteAsync implementation that returns default
  • Event verification: Confirms DeclarativeActionInvokedEvent and DeclarativeActionCompletedEvent are raised correctly
  • Idempotency: Validates multiple executions behave consistently

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.

Copilot AI and others added 2 commits February 11, 2026 00:06
@crickman
Add comprehensive unit tests for DefaultActionExecutor with 100% code…
f0722f3 
… coverage

Co-authored-by: crickman <66376200+crickman@users.noreply.github.com>
@crickman
Address code review feedback: Add explicit Xunit using and improve va…
203c791 
…riable naming

Co-authored-by: crickman <66376200+crickman@users.noreply.github.com>
Copilot AI changed the title [WIP] Add unit test class for DefaultActionExecutor Add unit tests for DefaultActionExecutor with 100% coverage Feb 11, 2026
Copilot AI requested a review from crickman February 11, 2026 00:14
@crickman crickman added .NET workflows Related to Workflows in agent-framework v1.0 Features being tracked for the version 1.0 GA declarative-workflow labels Feb 11, 2026
@github-actions github-actions bot changed the title Add unit tests for DefaultActionExecutor with 100% coverage .NET: Add unit tests for DefaultActionExecutor with 100% coverage Feb 11, 2026
@crickman crickman changed the title .NET: Add unit tests for DefaultActionExecutor with 100% coverage .NET Workflows - Add unit tests for DefaultActionExecutor Feb 11, 2026
@crickman crickman changed the title .NET Workflows - Add unit tests for DefaultActionExecutor .NET Workflows - Add unit tests for DefaultActionExecutor (Declarative Workflows) Feb 11, 2026
@crickman crickman moved this to In Progress in Agent Framework Feb 11, 2026
@crickman crickman deployed to integration February 11, 2026 00:36 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crickman crickman Awaiting requested review from crickman

At least 2 approving reviews are required to merge this pull request.

Assignees

@crickman crickman

Copilot code review Copilot

Labels

declarative-workflow .NET v1.0 Features being tracked for the version 1.0 GA workflows Related to Workflows in agent-framework

Projects

Agent Framework
Status: In Progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@crickman