Add dependency injection support to DurableTaskTestHost #613

nytian · 2026-01-13T05:27:51Z

Adds two approaches for dependency injection in DurableTaskTestHost:

ConfigureServices - Register services directly in test host options
AddInMemoryDurableTask() - Integrate with existing hosts (e.g., WebApplicationFactory)

Copilot

Pull request overview

This pull request adds dependency injection support to DurableTaskTestHost, enabling orchestrations and activities to resolve services from a DI container. The implementation provides two approaches: ConfigureServices for simple test scenarios and AddInMemoryDurableTask() for integration with existing hosts.

Changes:

Added Services property to DurableTaskTestHost and ConfigureServices option to DurableTaskTestHostOptions
Introduced AddInMemoryDurableTask() extension method for integrating with existing service collections
Created comprehensive test suites demonstrating both DI approaches
Updated README with detailed documentation and examples

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
src/InProcessTestHost/DurableTaskTestHost.cs	Added Services property exposing worker host's service provider and ConfigureServices option
src/InProcessTestHost/DurableTaskTestExtensions.cs	New extension methods for AddInMemoryDurableTask() and GetInMemoryOrchestrationService()
src/InProcessTestHost/README.md	Comprehensive documentation update with examples for both DI approaches
test/InProcessTestHost.Tests/DependencyInjectionTests.cs	New test suite for ConfigureServices approach with multiple DI scenarios
test/InProcessTestHost.Tests/WebApplicationFactoryIntegrationTests.cs	New test suite for AddInMemoryDurableTask() integration approach
test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs	Test helper types for integration tests
Directory.Packages.props	Added package references (currently unused)

src/InProcessTestHost/DurableTaskTestExtensions.cs

test/InProcessTestHost.Tests/DependencyInjectionTests.cs

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

src/InProcessTestHost/DurableTaskTestExtensions.cs

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

src/InProcessTestHost/DurableTaskTestExtensions.cs

test/InProcessTestHost.Tests/DependencyInjectionTests.cs

src/InProcessTestHost/DurableTaskTestHost.cs

src/InProcessTestHost/README.md

…rabletask-dotnet into nytian/test-host-di

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 31 comments.

Copilot · 2026-01-16T05:02:44Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+public class User
+{
+    public int Id { get; set; }
+    public string Name { get; set; } = "";
+    public string Email { get; set; } = "";
+}
+
+public class Order
+{
+    public int Id { get; set; }
+    public int UserId { get; set; }
+    public decimal Amount { get; set; }
+    public string Status { get; set; } = "";
+}
+
+public class UserDto
+{
+    public int Id { get; set; }
+    public string Name { get; set; } = "";
+    public string Email { get; set; } = "";
+}
+
+public class OrderInput
+{
+    public int OrderId { get; set; }
+    public int UserId { get; set; }
+}
+
+public class OrderOutput


According to coding guidelines, all private classes that do not serve as base classes should be sealed. These test helper classes (User, InMemoryUserRepository, InMemoryOrderRepository, UserService, OrderService, PaymentService) do not serve as base classes and should be marked as sealed for better performance and clarity.

Suggested change

public class User

{

public int Id { get; set; }

public string Name { get; set; } = "";

public string Email { get; set; } = "";

}

public class Order

{

public int Id { get; set; }

public int UserId { get; set; }

public decimal Amount { get; set; }

public string Status { get; set; } = "";

}

public class UserDto

{

public int Id { get; set; }

public string Name { get; set; } = "";

public string Email { get; set; } = "";

}

public class OrderInput

{

public int OrderId { get; set; }

public int UserId { get; set; }

}

public class OrderOutput

public sealed class User

{

public int Id { get; set; }

public string Name { get; set; } = "";

public string Email { get; set; } = "";

}

public sealed class Order

{

public int Id { get; set; }

public int UserId { get; set; }

public decimal Amount { get; set; }

public string Status { get; set; } = "";

}

public sealed class UserDto

{

public int Id { get; set; }

public string Name { get; set; } = "";

public string Email { get; set; } = "";

}

public sealed class OrderInput

{

public int OrderId { get; set; }

public int UserId { get; set; }

}

public sealed class OrderOutput

Copilot · 2026-01-16T05:02:45Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    User? GetById(int id);
+}
+
+public class InMemoryUserRepository : IUserRepository


According to coding guidelines, all private classes that do not serve as base classes should be sealed. These test helper classes (User, InMemoryUserRepository, InMemoryOrderRepository, UserService, OrderService, PaymentService) do not serve as base classes and should be marked as sealed for better performance and clarity.

Copilot · 2026-01-16T05:02:45Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    void Update(Order order);
+}
+
+public class InMemoryOrderRepository : IOrderRepository


According to coding guidelines, all private classes that do not serve as base classes should be sealed. These test helper classes (User, InMemoryUserRepository, InMemoryOrderRepository, UserService, OrderService, PaymentService) do not serve as base classes and should be marked as sealed for better performance and clarity.

Copilot · 2026-01-16T05:02:45Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    UserDto? GetUser(int id);
+}
+
+public class UserService : IUserService


According to coding guidelines, all private classes that do not serve as base classes should be sealed. These test helper classes (User, InMemoryUserRepository, InMemoryOrderRepository, UserService, OrderService, PaymentService) do not serve as base classes and should be marked as sealed for better performance and clarity.

Copilot · 2026-01-16T05:02:46Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    void UpdateStatus(int id, string status);
+}
+
+public class OrderService : IOrderService


According to coding guidelines, all private classes that do not serve as base classes should be sealed. These test helper classes (User, InMemoryUserRepository, InMemoryOrderRepository, UserService, OrderService, PaymentService) do not serve as base classes and should be marked as sealed for better performance and clarity.

Copilot · 2026-01-16T05:02:51Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    public string Status { get; set; } = "";
+}
+
+public class UserDto


According to coding guidelines, all public classes should have XML documentation comments. These DTO and entity classes are missing XML documentation.

Copilot · 2026-01-16T05:02:51Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    public string Email { get; set; } = "";
+}
+
+public class OrderInput


According to coding guidelines, all public classes should have XML documentation comments. These DTO and entity classes are missing XML documentation.

Copilot · 2026-01-16T05:02:51Z

test/InProcessTestHost.Tests/WebApplicationFactoryTestHelpers.cs

+    public int UserId { get; set; }
+}
+
+public class OrderOutput


According to coding guidelines, all public classes should have XML documentation comments. These DTO and entity classes are missing XML documentation.

Copilot · 2026-01-16T05:02:52Z

test/InProcessTestHost.Tests/DependencyInjectionTests.cs

+
+    public class CalculatorInput
+    {
+        public int A { get; set; }
+        public int B { get; set; }
+    }
+
+    public class CalculatorOutput
+    {
+        public int Sum { get; set; }


According to coding guidelines, all public classes should have XML documentation comments. These DTO classes are missing XML documentation.

Suggested change

public class CalculatorInput

{

public int A { get; set; }

public int B { get; set; }

}

public class CalculatorOutput

{

public int Sum { get; set; }

/// <summary>

/// Represents input values for calculator orchestrator and activity tests.

/// </summary>

public class CalculatorInput

{

/// <summary>

/// Gets or sets the first operand value.

/// </summary>

public int A { get; set; }

/// <summary>

/// Gets or sets the second operand value.

/// </summary>

public int B { get; set; }

}

/// <summary>

/// Represents the calculated results for calculator orchestrator and activity tests.

/// </summary>

public class CalculatorOutput

{

/// <summary>

/// Gets or sets the sum of the input operands.

/// </summary>

public int Sum { get; set; }

/// <summary>

/// Gets or sets the product of the input operands.

/// </summary>

Copilot · 2026-01-16T05:02:52Z

test/InProcessTestHost.Tests/DependencyInjectionTests.cs

+
+    public class CalculatorInput
+    {
+        public int A { get; set; }
+        public int B { get; set; }
+    }
+
+    public class CalculatorOutput
+    {
+        public int Sum { get; set; }


According to coding guidelines, all public classes should have XML documentation comments. These DTO classes are missing XML documentation.

Suggested change

public class CalculatorInput

{

public int A { get; set; }

public int B { get; set; }

}

public class CalculatorOutput

{

public int Sum { get; set; }

/// <summary>

/// Represents the input values for calculator operations in tests.

/// </summary>

public class CalculatorInput

{

/// <summary>

/// Gets or sets the first operand.

/// </summary>

public int A { get; set; }

/// <summary>

/// Gets or sets the second operand.

/// </summary>

public int B { get; set; }

}

/// <summary>

/// Represents the results of calculator operations in tests.

/// </summary>

public class CalculatorOutput

{

/// <summary>

/// Gets or sets the sum of the input operands.

/// </summary>

public int Sum { get; set; }

/// <summary>

/// Gets or sets the product of the input operands.

/// </summary>

initial commit

ac3ef45

Copilot AI review requested due to automatic review settings January 13, 2026 05:27

Merge branch 'main' into nytian/test-host-di

60be02f

Copilot started reviewing on behalf of nytian January 13, 2026 05:28 View session

Copilot AI reviewed Jan 13, 2026

View reviewed changes

Merge branch 'main' into nytian/test-host-di

8edf37f

halspang reviewed Jan 15, 2026

View reviewed changes

src/InProcessTestHost/DurableTaskTestHost.cs Outdated Show resolved Hide resolved

src/InProcessTestHost/README.md Show resolved Hide resolved

nytian added 2 commits January 15, 2026 20:56

update

c806020

Merge branch 'nytian/test-host-di' of https://github.com/microsoft/du…

26a70ff

…rabletask-dotnet into nytian/test-host-di

Copilot AI review requested due to automatic review settings January 16, 2026 04:56

Copilot started reviewing on behalf of nytian January 16, 2026 04:57 View session

Copilot AI reviewed Jan 16, 2026

View reviewed changes

-    public class CalculatorInput
-    {
-        public int A { get; set; }
-        public int B { get; set; }
-    }
-    public class CalculatorOutput
-    {
-        public int Sum { get; set; }
+    /// <summary>
+    /// Represents input values for calculator orchestrator and activity tests.
+    /// </summary>
+    public class CalculatorInput
+    {
+        /// <summary>
+        /// Gets or sets the first operand value.
+        /// </summary>
+        public int A { get; set; }
+        /// <summary>
+        /// Gets or sets the second operand value.
+        /// </summary>
+        public int B { get; set; }
+    }
+    /// <summary>
+    /// Represents the calculated results for calculator orchestrator and activity tests.
+    /// </summary>
+    public class CalculatorOutput
+    {
+        /// <summary>
+        /// Gets or sets the sum of the input operands.
+        /// </summary>
+        public int Sum { get; set; }
+        /// <summary>
+        /// Gets or sets the product of the input operands.
+        /// </summary>

Add dependency injection support to DurableTaskTestHost #613

Are you sure you want to change the base?

Add dependency injection support to DurableTaskTestHost #613

Uh oh!

Conversation

nytian commented Jan 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants