Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unit test to ensure retransmitted message reaches the peer exchange #7142

Merged

Conversation

pan-apple
Copy link
Contributor

Problem

Need an end to end test where

  1. A peer registers a unsolicited message handler
  2. Message sent by the sender gets dropped
  3. The message is retransmitted via CRMP after ack timeout
  4. The peer receives the message via handler
  5. The sender get the ack for the message
  6. All the state is cleaned up correctly

Change overview

Added the test mentioned in the problem statement.

Testing

CheckResendApplicationMessageWithPeerExchange is the new test.
Only unit test was added, and no functional code was modified.

@pan-apple pan-apple force-pushed the crmp-test-peer-exchange branch from 0d5f4fc to ad366d2 Compare May 27, 2021 18:32
@pan-apple
Copy link
Contributor Author

rebased

@bzbarsky-apple bzbarsky-apple merged commit e9d0ac8 into project-chip:master May 27, 2021
@pan-apple pan-apple deleted the crmp-test-peer-exchange branch May 28, 2021 15:16
nikita-s-wrk pushed a commit to nikita-s-wrk/connectedhomeip that referenced this pull request Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants