Skip to content

Commit 81e609b

Browse files
dcgclaude
authored andcommitted
test(dash-spv): improve error message for SyncManager creation failure
Replace unwrap() with expect() in block_download_test.rs to provide clearer error messages when test setup fails. This makes test failures more actionable by showing what specifically failed during initialization. - Changed SyncManager::new().unwrap() to use .expect() with descriptive message - Test verified: all 9 tests in block_download_test pass 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent d19ed33 commit 81e609b

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

dash-spv/tests/block_download_test.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -356,7 +356,8 @@ async fn test_process_multiple_filter_matches() {
356356
async fn test_sync_manager_integration() {
357357
let config = create_test_config();
358358
let received_heights = Arc::new(Mutex::new(HashSet::new()));
359-
let mut sync_manager = SyncManager::new(&config, received_heights).unwrap();
359+
let mut sync_manager = SyncManager::new(&config, received_heights)
360+
.expect("Failed to create SyncManager for integration test");
360361
let mut network = MockNetworkManager::new();
361362

362363
let block_hash = BlockHash::from_slice(&[1u8; 32]).unwrap();

0 commit comments

Comments
 (0)