Be consistent with how to #include test data headers

It helps the Bazel scripts to not have multiple search paths to a
single header, and it seems to be the right thing to do anyway.

src/test/alert_tests.cpp

@@ -10,7 +10,7 @@
 #include "chain.h"
 #include "chainparams.h"
 #include "clientversion.h"
-#include "data/alertTests.raw.h"
+#include "test/data/alertTests.raw.h"
 
 #include "main.h"
 #include "rpcprotocol.h"

src/test/base58_tests.cpp

@@ -4,9 +4,9 @@
 
 #include "base58.h"
 
-#include "data/base58_encode_decode.json.h"
-#include "data/base58_keys_invalid.json.h"
-#include "data/base58_keys_valid.json.h"
+#include "test/data/base58_encode_decode.json.h"
+#include "test/data/base58_keys_invalid.json.h"
+#include "test/data/base58_keys_valid.json.h"
 
 #include "key.h"
 #include "script/script.h"

src/test/script_tests.cpp

@@ -2,8 +2,8 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include "data/script_invalid.json.h"
-#include "data/script_valid.json.h"
+#include "test/data/script_invalid.json.h"
+#include "test/data/script_valid.json.h"
 
 #include "core_io.h"
 #include "key.h"

src/test/sighash_tests.cpp

@@ -3,7 +3,7 @@
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
 #include "consensus/validation.h"
-#include "data/sighash.json.h"
+#include "test/data/sighash.json.h"
 #include "main.h"
 #include "random.h"
 #include "script/interpreter.h"

src/test/transaction_tests.cpp

@@ -2,8 +2,8 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include "data/tx_invalid.json.h"
-#include "data/tx_valid.json.h"
+#include "test/data/tx_invalid.json.h"
+#include "test/data/tx_valid.json.h"
 #include "test/test_bitcoin.h"
 
 #include "init.h"