From 3dd1bdda50ed01596960b443ddd561cfb980439f Mon Sep 17 00:00:00 2001 From: Nipunn Koorapati Date: Wed, 24 Aug 2016 04:20:15 -0700 Subject: [PATCH] Improve eth_getWork timeout test rpc_get_work_should_timeout (#1992) --- rpc/src/v1/tests/mocked/eth.rs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/rpc/src/v1/tests/mocked/eth.rs b/rpc/src/v1/tests/mocked/eth.rs index 4b880419a..d7d93137f 100644 --- a/rpc/src/v1/tests/mocked/eth.rs +++ b/rpc/src/v1/tests/mocked/eth.rs @@ -861,7 +861,7 @@ fn rpc_get_work_should_timeout() { eth_tester.client.set_latest_block_timestamp(get_time().sec as u64 - 1000); // Set latest block to 1000 seconds ago let hash = eth_tester.miner.map_sealing_work(&*eth_tester.client, |b| b.hash()).unwrap(); - // Request with timeout of 0 seconds. This should work since we're disabling timeout. + // Request without providing timeout. This should work since we're disabling timeout. let request = r#"{"jsonrpc": "2.0", "method": "eth_getWork", "params": [], "id": 1}"#; let work_response = format!( r#"{{"jsonrpc":"2.0","result":["0x{:?}","0x0000000000000000000000000000000000000000000000000000000000000000","0x0000800000000000000000000000000000000000000000000000000000000000","0x01"],"id":1}}"#, @@ -869,6 +869,14 @@ fn rpc_get_work_should_timeout() { ); assert_eq!(eth_tester.io.handle_request(request), Some(work_response.to_owned())); + // Request with timeout of 0 seconds. This should work since we're disabling timeout. + let request = r#"{"jsonrpc": "2.0", "method": "eth_getWork", "params": ["0"], "id": 1}"#; + let work_response = format!( + r#"{{"jsonrpc":"2.0","result":["0x{:?}","0x0000000000000000000000000000000000000000000000000000000000000000","0x0000800000000000000000000000000000000000000000000000000000000000","0x01"],"id":1}}"#, + hash, + ); + assert_eq!(eth_tester.io.handle_request(request), Some(work_response.to_owned())); + // Request with timeout of 10K seconds. This should work. let request = r#"{"jsonrpc": "2.0", "method": "eth_getWork", "params": ["10000"], "id": 1}"#; assert_eq!(eth_tester.io.handle_request(request), Some(work_response.to_owned()));