/offst

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

Adding tests to Funder handler #122

Merged
merged 40 commits into from Nov 14, 2018

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@realcr
@realcr
Collaborator

realcr commented Nov 10, 2018 •
edited

Added a basic test (Sending credits using Request/Response) to the funder handler.
Fixed some bugs in:

  • Funder handler implementation
  • TokenChannel implementation
  • MutualCredit implementation

Test passed.

realcr added some commits Nov 10, 2018

@realcr
Began writing a basic test for funder handler. Not done yet.
Loading status checks…
96b2b2e
@realcr
Derived Debug for some types.
1ec66cb
@realcr
Added to a funder handler test. Not done yet.
Loading status checks…
c240f40
@realcr
Renamed FunderIncomingComm -> IncomingCommMessage.
f7a2cd2
@realcr
Refactored funder main loop.
6c74e60
@realcr
Using RngContainer instead of Rc to wrap random generator.
Loading status checks…
33e8086
@realcr
Added a first Init message when running funder loop.
122dc23
@realcr
Refactored handler test using apply_funder_incoming() function.
Loading status checks…
acae8aa
@realcr
Continued test + Added TODO comments.
Loading status checks…
219735a
@realcr
Fixed bug in control_set_friend_remote_max_debt().
6944522
@realcr
Added missing async! for some calls.
7d883b3
@realcr
Showing unused warnings. Fixed unused warnings.
cbef074
@realcr
Cleaned tests unused warnings.
41f46bb
@realcr
Removed debug print.
517d6c9
@realcr
Added {Request,Response}SendFunds to test. Currently has stack overfl… 
…ow issue.
Loading status checks…
7092946
@realcr
Updated packages. Didn't solve stack overflow.
d5ab444
@realcr
unwrapping the result of event_sender.send().
fb19d0d
@realcr
Bumped Rust's nightly version to 2018.11.13. Didn't solve stack overf… 
…low.
618876b
@realcr
Removed rand dependency for offst-funder.
39873a2
@realcr
Strange case of stack overflow using await! in a generator.
Loading status checks…
876ddf3
@realcr
Added a test to reproduce overflow using simpler code.
Loading status checks…
e68a4e7
@realcr
Solved stack overflow issue with generators by using Box::pinned().
Loading status checks…
da16f67
@realcr
Added comment about use of Box::pinned().
e1545f5
@realcr
Added initial SafeSignedArithmetic. Not tested yet.
73d8173
@realcr
Debug prints.
e532c97
@realcr
Began implementation of shared_credits() and usable_ratio() for Friend.
7615bfb
@realcr
Updated according to rename: SafeArithmetic -> SafeUnsignedArithmetic.
9f19d98
@realcr
Added safe_abs() to safe_arithmetic. Swapped SafeSignedArithmetic and… 
… SafeUnsignedArithmetic.
6f9511a
@realcr
Added State::get_usable_ratio() and Friend::get_shared_credits(). Use… 
…d in handle_friend.rs.
6091354
@realcr
Removed unused imports.
96562d6

@realcr realcr merged commit e3973df into master Nov 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment