Skip to content

Conversation

Copy link

Copilot AI commented Dec 1, 2025

Addresses reviewer feedback that the Direct Reply-To test only verified queue address format without testing actual message send/receive functionality.

Changes

  • Added test_direct_reply_to_send_and_receive test that exercises the full message flow:
    • Creates consumer with DirectReplyToConsumerOptions
    • Publishes messages to the dynamic reply-to address
    • Consumes and validates message content
    • Verifies all messages delivered successfully
def test_direct_reply_to_send_and_receive(environment: Environment, connection: Connection) -> None:
    consumer = connection.consumer("", consumer_options=DirectReplyToConsumerOptions())
    addr = consumer.get_queue_address()
    
    publisher = environment.connection().publisher(addr)
    publisher.publish(Message(body=Converter.string_to_bytes("test")))
    
    message = consumer.consume()
    assert Converter.bytes_to_string(message.body) == "test"
  • Added proper cleanup for existing test_consumer_create_reply_name test

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update implementation for direct reply to feature based on feedback Add end-to-end test for Direct Reply-To message flow Dec 1, 2025
Copilot AI requested a review from Gsantomaggio December 1, 2025 09:12
@Gsantomaggio
Copy link
Member

added test manually here

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