Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

Test/int/connections #75

Merged
merged 6 commits into from
Jul 1, 2021
Merged

Test/int/connections #75

merged 6 commits into from
Jul 1, 2021

Conversation

cjhowland
Copy link
Contributor

No description provided.

@cjhowland
fix: add condition to send_and_await_reply_async() function
9083a6c 
Signed-off-by: Char <char@indicio.tech>
@cjhowland
fix: add assign_thread_from function to ReceiveInvitationHandler
29bd24e 
Signed-off-by: Char <char@indicio.tech>
@cjhowland
fix: clear connection state between tests
03f8eba 
Signed-off-by: Char <char@indicio.tech>
@cjhowland
chore: fold clear_invitations into clear_invitation_state
cef9709 
Signed-off-by: Char <char@indicio.tech>
@cjhowland
test: add test of new message notification to test_basicmessage.py
9df6c08 
Signed-off-by: Char <char@indicio.tech>
@dbluhm
Copy link
Contributor

dbluhm commented Jul 1, 2021

Looks pretty good! Be sure to run black int/tests!

@cjhowland
chore: reformat with black
5c4a11a 
Signed-off-by: Char <char@indicio.tech>
dbluhm
dbluhm approved these changes Jul 1, 2021
View reviewed changes
Copy link
Contributor

@dbluhm dbluhm left a comment

Choose a reason for hiding this comment

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

Excellent work! Looks good and tests are passing!

@dbluhm dbluhm merged commit ebf072e into hyperledger-archives:test/integration-cont Jul 1, 2021
@cjhowland cjhowland deleted the test/int/connections branch September 29, 2021 05:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dbluhm dbluhm dbluhm approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cjhowland @dbluhm