V3/bugfix/221#230
Merged
Merged
Conversation
Updated .gitignore to exclude the MigrationBackup/ directory and its contents from version control, preventing accidental commits of backup files.
Update .gitignore to exclude any .claude file or directory from version control. This prevents accidental commits of files or folders named .claude.
…tion Adds AddQueryPipelineBehavior<T> for registering closed (concrete) query pipeline behaviors alongside open generics. Updates DI registration logic to handle both open and closed behaviors correctly. Includes comprehensive integration tests for registration, execution, and error cases, improving flexibility and correctness of query pipeline behavior handling.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
MediatorOptions.cs — Added AddQueryPipelineBehavior() method for closed generic query behaviors, matching the pattern that already exists for commands (AddCommandPipelineBehavior()) and
notifications (AddNotificationPipelineBehavior()). Validates that the type implements IQueryPipelineBehavior<,> and rejects open generics with a helpful error message.
ServiceCollectionExtensions.cs — Fixed query behavior DI registration to handle both open and closed generics. Previously it always registered against the open generic interface
IQueryPipelineBehavior<,>, which doesn't work for closed types. Now it mirrors the notification behavior logic: open generics register against the open interface, closed types register against their
specific implemented interfaces.
QueryBehaviorIntegrationTests.cs (new) — 10 tests covering: