wow-btc-swap

Commit Graph

Author	SHA1	Message	Date
Daniel Karzel	89b3775e05	Rename config to execution_params	3 years ago
Daniel Karzel	802dc61e7e	Configuration for RPC urls and Bitcoin wallet name	3 years ago
Franck Royer	704a8e7b01	Add swap id to tracing context	3 years ago
Franck Royer	33db688e3a	Bob peer id can be retrieved from the DB This remove branches where Alice resumes from the DB but cannot contact Bob.	3 years ago
Franck Royer	d2a1937f51	Use `Message4`	3 years ago
Franck Royer	f2a25ee49b	Move definitions out of lib.rs	3 years ago
Daniel Karzel	33a9057b1f	Move run_until is_target_state comparison functions into testutils	3 years ago
Daniel Karzel	44c4b5dcea	Remove newlines from import statements to avoid problems Rust fmt automatically groups the imports (from top to bottom) as `pub use` `use crate` and `use`. There is no need to introduce sections which cause annoyance when auto importing using the IDE.	3 years ago
bors[bot]	4e858197c9	Merge #149 149: Fix Alice redeem scenario r=da-kami a=da-kami Follow up of #144, partial fix of https://github.com/comit-network/xmr-btc-swap/issues/137 Fix Alice redeem scenario - Properly check the timelocks before trying to redeem - Distinguish different failure scenarios and reactions to it. - if we fail to construct the redeem transaction: wait for cancel. - if we fail to publish the redeem transaction: wait for cancel but let the user know that restarting the application will result in retrying to publish the tx. - if we succeed to publish the tx but then fail when waiting for finality, print error to the user (secreat already leaked, the user has to check manually if the tx was included) Co-authored-by: Daniel Karzel <daniel@comit.network>	3 years ago
Franck Royer	ae8134f04e	Replace amounts messages with swap res/req	3 years ago
Daniel Karzel	6a75c840b5	Fix Alice redeem scenario - Properly check the timelocks before trying to redeem - Distinguish different failure scenarios and reactions to it. - if we fail to construct the redeem transaction: wait for cancel. - if we fail to publish the redeem transaction: wait for cancel but let the user know that restarting the application will result in retrying to publish the tx. - if we succeed to publish the tx but then fail when waiting for finality, print error to the user (secreat already leaked, the user has to check manually if the tx was included)	3 years ago
Daniel Karzel	170e90ffed	Rename do_run_until to _run_until_internal	3 years ago
Daniel Karzel	8bf467b550	Make the factory code usable in production - Introduce Test abstraction instead of tow harnesses, move test specific data into Test - Change the abstraction from actors to swap, because we are creating swaps, not actors - rename actor::swap to run, because we are running a swap	3 years ago
Daniel Karzel	e4795fa4ee	Fix recursive call to swap by using run_until We should call run_until instead of swap.	3 years ago
bors[bot]	a7f68e4aa1	Merge #144 144: Test refactor r=da-kami a=da-kami This PR is pure refactoring, keeping the logic of the tests we had before. No production code is touched besides re-exports in early commits (no logic changes). In the follow ups improvements will be introduced, that touch the production code as well. All remaining tasks actioned since Friday: - [x] `happy_path_bob _restart` (trivial) - [x] add refund assertions to harnesses (trivial) - [x] convert all refund scenarios currently being tested (trivial) - [x] remove dead test init code once all old tests are converted - [ ] ~~(optional) move alice and bob harness code into separate files~~ -> might action this once re-using test code in production. Out of scope, follow up: - [x] https://github.com/comit-network/xmr-btc-swap/pull/145 - We can do exact assertions for Bob's redeem as well, but have to store Bob's `tx_lock` id in the respective final state. Make `tx_lock` available in `BtcRedeemed` and `BtcPunished` to have better assertions / harmonize test behaviour. - [ ] update the production code to use the `Alice` and `Bob` structs to bundle the params - update tests to use the production struct. - [ ] Re-use test swap setup in production (i.e. `Alice-/BobHarness::new`) to setup the swap. - [ ] add additional tests - [ ] re-try moving the tests from `test` to `src` (if the peer_id was the only problem this should be trivial now - but should be done after the refactor is finished) - [ ] creating new wallets upon restart - [ ] aborting the old event loop after restart Co-authored-by: rishflab <rishflab@hotmail.com> Co-authored-by: Daniel Karzel <daniel@comit.network>	3 years ago
rishflab	f5cfe014be	Fix imports	3 years ago
Daniel Karzel	59f01ad680	Remove Todo that was already actioned This ToDo does not add any value, I would not know what to do here.	4 years ago
Daniel Karzel	af45206fde	Remember the block-height before XMR lock for generated monero wallet restore height Speeds up wallet creation, because only the blocks after the recorded height will be scanned.	4 years ago
Franck Royer	17356eaff9	Do not guard waiting for encrypted signature with arbitrary timeout We already select waiting for this message with the cancellation expiry, we do not need add another guard that tries to guess how long it would for the Monero transaction to be finalised.	4 years ago
rishflab	dcea54dbf1	Move protocol parent states into appropriate module	4 years ago
rishflab	c900d12593	Merge xmr_btc crate Created network, storage and protocol modules. Organised files into the modules where the belong. xmr_btc crate moved into isolated modulein swap crate. Remove the xmr_btc module and integrate into swap crate. Consolidate message related code Reorganise imports Remove unused parent Message enum Remove unused parent State enum Remove unused dependencies from Cargo.toml	4 years ago

21 Commits (89b3775e05f796d1c270a7c8a91589f56f1ad563)