-
Notifications
You must be signed in to change notification settings - Fork 61
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
AEA-684 p2p_stub connection refactoring and tests #1435
Conversation
if not target_file.is_file(): | ||
target_file.touch() | ||
logger.warn("file {} doesn't exist, creating it ...".format(target_file)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file created on open(, "ab")
if it can not be created at this point touch will fail as well, check is_file
quite useless, cause if it's non file object it will fail on open
envelope = make_test_envelope(to_="con2") | ||
await self.connection1.send(envelope) | ||
|
||
await asyncio.wait_for(self.connection2.receive(), timeout=5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add some assertions on envelope content?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, some minor commentd
Codecov Report
@@ Coverage Diff @@
## develop #1435 +/- ##
===========================================
- Coverage 91.37% 88.46% -2.92%
===========================================
Files 217 217
Lines 15251 15254 +3
===========================================
- Hits 13936 13494 -442
- Misses 1315 1760 +445
Continue to review full report at Codecov.
|
d35c515
to
ca8a8d3
Compare
c94f210
to
279aea9
Compare
Proposed changes
small async improvements and tests
Fixes
Types of changes
Checklist
develop
branch (left side). Also you should start your branch off ourdevelop
.aea cli
tool works