9 Commits (6643b80d1c0134bc8ef5edd65fbe09c8b63b01d3)

Author	SHA1	Message	Date
John Newbery	5448a1471d	[tests] don't override __init__() in individual tests ... Almost all test scripts currently need to override the __init__() method. When they do that they need to call into super().__init__() as the base class does some generic initialization. This commit makes the base class __init__() call into set_test_params() method. Individual test cases can override set_test_params() to setup their test parameters.	7 years ago
John Newbery	36b6268670	[tests] TestNode: separate add_node from start_node ... Separates the act of creating a TestNode object from starting the node. The test_framework now keeps track of its list of TestNodes, and test writers can call start_node() and stop_node() without having to update the self.nodes list.	7 years ago
John Newbery	d8c218f9c2	[tests] Functional tests call self.start_node(s) and self.stop_node(s) ... This commit changes the individual test scripts to call the start_node(s) and stop_node(s) methods in BitcoinTestFramework.	8 years ago
John Newbery	c9cc76dcaa	[tests] Remove is_network_split from funtional test cases	8 years ago
John Newbery	2a52ae63bf	Remove duplicate method definitions in NodeConnCB subclasses ... All Node classes in individual test cases subclass from NodeConnCB. Many have duplicate definitions for methods that are defined in the base class. This commit removes those duplicate definitions. This commit removes ~290 lines of duplicate code.	8 years ago
John Newbery	52e15aa4d0	Adds helper functions to NodeConnCB ... This commit adds some helper functions to NodeConnCB which are useful for many tests: - NodeConnCB now keeps track of the number of each message type that it's received and the most recent message of each type. Many tests assert on the most recent block, tx or reject message. - NodeConnCB now keeps track of its connection state by setting a connected boolean in on_open() and on_close() - NodeConnCB now has wait_for_block, wait_for_getdata, wait_for_getheaders, wait_for_inv and wait_for_verack methods I have updated the individual test cases to make sure that there are no namespace problems that cause them to fail with these new definitions. Future commits will remove the duplicate code.	8 years ago
John Newbery	6a18bb9a36	[tests] sync_with_ping should assert that ping hasn't timed out ... sync_with_ping currently returns false if the timeout expires, and it is the caller's responsibility to fail the test. However, none of the tests currently assert on sync_with_ping()'s return code. This commit adds an assert to sync_with_ping so the test will fail if the timeout expires. This commit also removes all the duplicate implementations of sync_with_ping() from the individual tests.	8 years ago
John Newbery	159fe88abf	Remove SingleNodeConnCB ... This commit merges the NodeConnCB and SingleNodeConnCB into a single class (called NodeConnCB). The original intent for the NodeConnCB was to be able to have a python 'mininode' connect to multiple running bitcoinds. This has never been used and can be achieved more easily by having multiple NodeConns backed by a common datastore if it is ever needed. The changes in mininode.py are just code moves (and merging the two classes into a single class). The code changes in the individual test cases are changing the subclasses to subclass from NodeConnCB instead of SingleNodeConnCB. There is a lot of duplicate code in the subclasses that can be removed in future commits.	8 years ago
John Newbery	c28ee91db0	Rename rpc-tests directory to functional	8 years ago