[Tests] Fix chain ordering in budget tests

Fuzzbawls · Fuzzbawls · commit 3a6b15239187 · 2018-05-02T18:05:06.000-07:00
check testnet params first, then check mainnet so as to not interfere
with subsequent unit tests that rely on mainnet params.

Also clean up include ordering.
diff --git a/src/test/budget_tests.cpp b/src/test/budget_tests.cpp
@@ -2,10 +2,11 @@
 // Distributed under the MIT/X11 software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include <boost/test/unit_test.hpp>
-#include <tinyformat.h>
-#include <utilmoneystr.h>
 #include "masternode-budget.h"
+#include "tinyformat.h"
+#include "utilmoneystr.h"
+
+#include <boost/test/unit_test.hpp>
 
 BOOST_AUTO_TEST_SUITE(budget_tests)
 
@@ -19,13 +20,13 @@ void CheckBudgetValue(int nHeight, std::string strNetwork, CAmount nExpectedValu
 
 BOOST_AUTO_TEST_CASE(budget_value)
 {
-    SelectParams(CBaseChainParams::MAIN);
+    SelectParams(CBaseChainParams::TESTNET);
     int nHeightTest = Params().Zerocoin_Block_V2_Start() + 1;
-    CheckBudgetValue(nHeightTest, "mainnet", 43200*COIN);
+    CheckBudgetValue(nHeightTest, "testnet", 7300*COIN);
 
-    SelectParams(CBaseChainParams::TESTNET);
+    SelectParams(CBaseChainParams::MAIN);
     nHeightTest = Params().Zerocoin_Block_V2_Start() + 1;
-    CheckBudgetValue(nHeightTest, "testnet", 7300*COIN);
+    CheckBudgetValue(nHeightTest, "mainnet", 43200*COIN);
 }
 
-BOOST_AUTO_TEST_SUITE_END()
+BOOST_AUTO_TEST_SUITE_END()
