9 Commits (efcd39eeef92bcaf90f64dca092e9a9bbbe3b358)

Author	SHA1	Message	Date
Daniel Karzel	0c616c7437	Move loading the state into the function ... In the production code it is a weird indirection that we load the state and then pass in the state and the database. In the tests we have one additional load by doing it inside the command, but loading from the db is not expensive.	3 years ago
Thomas Eizinger	e266fb07ef	Don't stutter	3 years ago
Daniel Karzel	c976358c37	Multiple swaps with the same peer ... - Swap-id is exchanged during execution setup. CLI (Bob) sends the swap-id to be used in his first message. - Transfer poof and encryption signature messages include the swap-id so it can be properly associated with the correct swap. - ASB: Encryption signatures are associated with swaps by swap-id, not peer-id. - ASB: Transfer proofs are still associated to peer-ids (because they have to be sent to the respective peer), but the ASB can buffer multiple - CLI: Incoming transfer proofs are checked for matching swap-id. If a transfer proof with a different swap-id than the current executing swap is received it will be ignored. We can change this to saving into the database. Includes concurrent swap tests with the same Bob. - One test that pauses and starts an additional swap after the transfer proof was received. Results in both swaps being redeemed after resuming the first swap. - One test that pauses and starts an additional swap before the transfer proof is sent (just after BTC locked). Results in the second swap redeeming and the first swap being refunded (because the transfer proof on Bob's side is lost). Once we store transfer proofs that we receive during executing a different swap into the database both swaps should redeem. Note that the monero harness was adapted to allow creating wallets with multiple outputs, which is needed for Alice.	3 years ago
Thomas Eizinger	b9d8cbeaa2	Rename testutils to harness ... This allows us to bring in a dependency named `testutils`.	3 years ago
bors[bot]	95acbc6277	Merge #307 ... 307: Reduce load on electrum r=thomaseizinger a=rishflab . Co-authored-by: rishflab <rishflab@hotmail.com> Co-authored-by: Thomas Eizinger <thomas@eizinger.io>	3 years ago
Thomas Eizinger	a0830f099f	Pass relevant execution params into wallet instead of via functions ... The execution params don't change throughout the lifetime of the program. They can be set in the wallet at the very beginning. This simplifies the interface of the wallet functions.	3 years ago
Daniel Karzel	ea05c306e0	Alice spawns swaps outside the event loop ... Instead of spawning the swap inside the event loop we send the swap back to the caller to be spawned. This means we no longer need the remote handle that was only used in the tests. This now properly logs the swap results in production. It also gives us more control over Alice's swap in the tests.	3 years ago
Thomas Eizinger	6d9b21cb47	Change `imports_granularity` to module ... This reduces the overall amount of LoC that imports take up in our codebase by almost 100. It also makes merge-conflicts less likely because there is less grouping together of imports that may lead to layout changes which in turn can cause merge conflicts.	3 years ago
Franck Royer	eb39add5ff	Fix typo	3 years ago