Skip to content

Conversation

@jithu7432
Copy link
Contributor

@jithu7432 jithu7432 commented Sep 6, 2025

As discussed on #1134, this is an attempt to add the remaining providers.

@jithu7432 jithu7432 marked this pull request as draft September 6, 2025 10:04
@jithu7432 jithu7432 marked this pull request as ready for review September 27, 2025 19:47
@jithu7432 jithu7432 changed the title Add DI tests for LightInject Add DI tests for LightInject, StashBox, Lamar Sep 27, 2025
@jbogard jbogard requested a review from Copilot October 24, 2025 15:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds dependency injection test coverage for three additional IoC containers: LightInject, StashBox, and Lamar. The changes extend the existing test infrastructure to verify that MediatR works correctly with these DI providers.

Key changes:

  • Added test fixtures for LightInject, StashBox, and Lamar DI containers
  • Created corresponding test classes that inherit from BaseAssemblyResolutionTests
  • Updated project dependencies to include the necessary adapter packages

Reviewed Changes

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

Show a summary per file
File Description
StashBoxDependencyInjectionTests.cs Test class for StashBox container integration
StashBoxServiceProviderFixture.cs Service provider fixture implementing StashBox container setup
LightInjectServiceProviderFixture.cs Service provider fixture implementing LightInject container setup
LamarServiceProviderFixture.cs Service provider fixture implementing Lamar container setup
MediatR.DependencyInjectionTests.csproj Added package references for the three new DI containers
LightInjectDependencyInjectionTests.cs Test class for LightInject container integration
LamarDependencyInjectionTests.cs Test class for Lamar container integration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants