Skip to content

Conversation

@Gsantomaggio
Copy link
Member

Add more tests for #42

complete: #42
Signed-off-by: Gabriele Santomaggio <[email protected]>
Signed-off-by: Gabriele Santomaggio <[email protected]>
Signed-off-by: Gabriele Santomaggio <[email protected]>
@Gsantomaggio Gsantomaggio marked this pull request as ready for review September 12, 2025 13:48
@Gsantomaggio Gsantomaggio changed the title properties filters Add more tests for application properties filters Sep 12, 2025
@Gsantomaggio Gsantomaggio self-assigned this Sep 12, 2025
@Gsantomaggio Gsantomaggio added the enhancement New feature or request label Sep 12, 2025
@Gsantomaggio Gsantomaggio added this to the 0.2.0 milestone Sep 12, 2025
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

Adds comprehensive test coverage for application properties filters in RabbitMQ AMQP stream functionality to support issue #42.

  • Added test for filtering messages by application properties
  • Added test for mixing multiple filter types (values, application properties, and message properties)
  • Introduced corresponding message handler classes to validate filtered message consumption

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

Gsantomaggio and others added 2 commits September 12, 2025 15:59
Signed-off-by: Gabriele Santomaggio <[email protected]>
@Gsantomaggio Gsantomaggio merged commit 62b4608 into main Sep 12, 2025
1 of 2 checks passed
@Gsantomaggio Gsantomaggio deleted the feat/application_properties_filters branch September 12, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant