diff --git a/src/test/prevector_tests.cpp b/src/test/prevector_tests.cpp
--- a/src/test/prevector_tests.cpp
+++ b/src/test/prevector_tests.cpp
@@ -199,9 +199,9 @@
                 test.erase(InsecureRandRange(test.size()));
             }
             if (InsecureRandBits(3) == 2) {
-                int new_size = std::max<int>(
-                    0, std::min<int>(30,
-                                     test.size() + (InsecureRandRange(5)) - 2));
+                int new_size = std::max(
+                    0, std::min(30, int(test.size()) +
+                                        int(InsecureRandRange(5)) - 2));
                 test.resize(new_size);
             }
             if (InsecureRandBits(3) == 3) {
diff --git a/src/test/scheduler_tests.cpp b/src/test/scheduler_tests.cpp
--- a/src/test/scheduler_tests.cpp
+++ b/src/test/scheduler_tests.cpp
@@ -59,11 +59,11 @@
     };
     // [-11, 1000]
     auto randomMsec = [](FastRandomContext &rc) -> int {
-        return -11 + rc.randrange(1012);
+        return -11 + int(rc.randrange(1012));
     };
     // [-1000, 1000]
     auto randomDelta = [](FastRandomContext &rc) -> int {
-        return -1000 + rc.randrange(2001);
+        return -1000 + int(rc.randrange(2001));
     };
 
     boost::chrono::system_clock::time_point start =
diff --git a/src/wallet/test/accounting_tests.cpp b/src/wallet/test/accounting_tests.cpp
--- a/src/wallet/test/accounting_tests.cpp
+++ b/src/wallet/test/accounting_tests.cpp
@@ -78,7 +78,7 @@
     {
         CMutableTransaction tx(*wtx.tx);
         // Just to change the hash :)
-        --tx.nLockTime;
+        ++tx.nLockTime;
         wtx.SetTx(MakeTransactionRef(std::move(tx)));
     }
     m_wallet.AddToWallet(wtx);
@@ -89,7 +89,7 @@
     {
         CMutableTransaction tx(*wtx.tx);
         // Just to change the hash :)
-        --tx.nLockTime;
+        ++tx.nLockTime;
         wtx.SetTx(MakeTransactionRef(std::move(tx)));
     }
     m_wallet.AddToWallet(wtx);