[tests] Rename wallet_* functional tests.

Author: ajtowns

test/functional/test_runner.py

@@ -55,46 +55,46 @@
 BASE_SCRIPTS= [
     # Scripts that are run by the travis build process.
     # Longest test should go first, to favor running tests in parallel
-    'wallet-hd.py',
-    'walletbackup.py',
+    'wallet_hd.py',
+    'wallet_backup.py',
     # vv Tests less than 5m vv
     'feature_block.py',
     'fundrawtransaction.py',
     'p2p-compactblocks.py',
     'feature_segwit.py',
     # vv Tests less than 2m vv
-    'wallet.py',
-    'wallet-accounts.py',
+    'wallet_basic.py',
+    'wallet_accounts.py',
     'p2p-segwit.py',
-    'wallet-dump.py',
+    'wallet_dump.py',
     'listtransactions.py',
     # vv Tests less than 60s vv
     'sendheaders.py',
-    'zapwallettxes.py',
-    'importmulti.py',
+    'wallet_zapwallettxes.py',
+    'wallet_importmulti.py',
     'mempool_limit.py',
     'merkle_blocks.py',
-    'receivedby.py',
-    'abandonconflict.py',
+    'wallet_listreceivedby.py',
+    'wallet_abandonconflict.py',
     'feature_csv_activation.py',
     'rawtransactions.py',
-    'address_types.py',
+    'wallet_address_types.py',
     'feature_reindex.py',
     # vv Tests less than 30s vv
-    'keypool-topup.py',
+    'wallet_keypool_topup.py',
     'zmq_test.py',
     'bitcoin_cli.py',
     'mempool_resurrect_test.py',
-    'txn_doublespend.py --mineblock',
-    'txn_clone.py',
-    'txn_clone.py --segwit',
+    'wallet_txn_doublespend.py --mineblock',
+    'wallet_txn_clone.py',
+    'wallet_txn_clone.py --segwit',
     'getchaintips.py',
     'rest.py',
     'mempool_spendcoinbase.py',
     'mempool_reorg.py',
     'mempool_persist.py',
-    'multiwallet.py',
-    'multiwallet.py --usecli',
+    'wallet_multiwallet.py',
+    'wallet_multiwallet.py --usecli',
     'httpbasics.py',
     'multi_rpc.py',
     'feature_proxy.py',
@@ -103,29 +103,29 @@
     'decodescript.py',
     'blockchain.py',
     'deprecated_rpc.py',
-    'disablewallet.py',
+    'wallet_disable.py',
     'net.py',
-    'keypool.py',
+    'wallet_keypool.py',
     'p2p-mempool.py',
     'prioritise_transaction.py',
     'invalidblockrequest.py',
     'invalidtxrequest.py',
     'feature_versionbits_warning.py',
     'preciousblock.py',
-    'importprunedfunds.py',
+    'wallet_importprunedfunds.py',
     'signmessages.py',
     'feature_nulldummy.py',
-    'import-rescan.py',
+    'wallet_import_rescan.py',
     'mining.py',
-    'bumpfee.py',
+    'wallet_bumpfee.py',
     'rpcnamedargs.py',
-    'listsinceblock.py',
+    'wallet_listsinceblock.py',
     'p2p-leaktests.py',
-    'wallet-encryption.py',
+    'wallet_encryption.py',
     'feature_dersig.py',
     'feature_cltv.py',
     'uptime.py',
-    'resendwallettransactions.py',
+    'wallet_resendwallettransactions.py',
     'feature_minchainwork.py',
     'p2p-fingerprint.py',
     'feature_uacomment.py',
@@ -158,8 +158,8 @@
     # vv Tests less than 30s vv
     'feature_assumevalid.py',
     'example_test.py',
-    'txn_doublespend.py',
-    'txn_clone.py --mineblock',
+    'wallet_txn_doublespend.py',
+    'wallet_txn_clone.py --mineblock',
     'feature_notifications.py',
     'invalidateblock.py',
     'feature_rbf.py',
@@ -474,7 +474,7 @@ def was_successful(self):
 def check_script_prefixes():
     """Check that no more than `EXPECTED_VIOLATION_COUNT` of the
        test scripts don't start with one of the allowed name prefixes."""
-    EXPECTED_VIOLATION_COUNT = 60
+    EXPECTED_VIOLATION_COUNT = 37
 
     # LEEWAY is provided as a transition measure, so that pull-requests
     # that introduce new tests that don't conform with the naming