Skip to content

Commit

Permalink
Update TSS tests to use new registration flow (#1026)
Browse files Browse the repository at this point in the history
* Remove signing integration test

The `register_and_sign` integration test covers the same functionality now.

* Update ETH integration test to use new registration flow

* Get program config test working again

* Delete `test_store_share` test

This isn't relevant in the context of registration anymore. The functionality
of this test was mostly moved to `test_jumpstart_network` anyways.

* Fix `test_fail_infinite_program`

* Fix `test_device_key_proxy` test

* Get `test_sign_tx_no_chain` mostly passing

* Move request limit tests to their own test

* Rename failing signing test

* Move a failing connection test out of `test_sign_tx_no_chain`

* Ignore failing test

Not sure if we still want this, or tbh how to fix it

* Update test name to better reflect what it covers

* Rename `generic_msg` to `signature_request`

This better reflects what the variable actually is

* Fix test for rejecting a TSS node who is not part of the signing committe (#1028)

* Update error message for unexpected participant

* Fix test

* Fix used import and variable warnings

* Use `test_client` for registration in all non-registration tests

* Rustfmt

* Move `update_programs` call to relevant location

---------

Co-authored-by: peg <[email protected]>
  • Loading branch information
HCastano and ameba23 authored Aug 26, 2024
1 parent ae420fe commit 084a2b6
Show file tree
Hide file tree
Showing 4 changed files with 487 additions and 598 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,7 @@ async fn handle_initial_incoming_ws_message(
};

{
// Check that the given public key matches the public key we got in the
// UserTransactionRequest or register transaction
// Check that the given public key is of the ones we are expecting for this protocol session
let mut listeners = app_state
.listener_state
.listeners
Expand All @@ -186,8 +185,7 @@ async fn handle_initial_incoming_ws_message(
// Make the signing process fail, since one of the commitee has misbehaved
listeners.remove(&msg.session_id);
return Err(SubscribeErr::Decryption(
"Public key does not match that given in UserTransactionRequest or register \
transaction"
"Public key does not match any of those expected for this protocol session"
.to_string(),
));
}
Expand Down
Loading

0 comments on commit 084a2b6

Please sign in to comment.