Use travis_wait for "wine test_dash.exe" call to fix timeouts (#1812)

* Revert "Adjust tests to avoid Travis timeouts (#1745)"

This reverts commit 32f2169.

* Revert "Show test progress for tests running in wine to avoid Travis timeout (#1740)"

This reverts commit 31bc9d4.

* Use travis_wait when calling "wine test_dash.exe"

Author: codablock

.travis.yml

@@ -83,7 +83,7 @@ script:
     - make $MAKEJOBS $GOAL || ( echo "Build failure. Verbose build follows." && make $GOAL V=1 ; false )
     - export LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/depends/$HOST/lib
     - if [ "$RUN_TESTS" = "true" -a "$WINE" != "true" ]; then travis_wait 30 make $MAKEJOBS check VERBOSE=1; fi
-    - if [ "$RUN_TESTS" = "true" -a "$WINE" = "true" ]; then wine  src/test/test_dash.exe --log_level=test_suite; fi
+    - if [ "$RUN_TESTS" = "true" -a "$WINE" = "true" ]; then travis_wait 30 wine src/test/test_dash.exe; fi
     - if [ "$RUN_TESTS" = "true" ]; then qa/pull-tester/rpc-tests.py --coverage; fi
 after_script:
     - echo $TRAVIS_COMMIT_RANGE

src/test/coins_tests.cpp

@@ -190,7 +190,6 @@ BOOST_AUTO_TEST_CASE(coins_cache_simulation_test)
 
         // Once every 1000 iterations and at the end, verify the full cache.
         if (insecure_rand() % 1000 == 1 || i == NUM_SIMULATION_ITERATIONS - 1) {
-            BOOST_TEST_MESSAGE("coins_cache_simulation_test - verifying full cache");
             for (auto it = result.begin(); it != result.end(); it++) {
                 bool have = stack.back()->HaveCoin(it->first);
                 const Coin& coin = stack.back()->AccessCoin(it->first);
@@ -422,7 +421,6 @@ BOOST_AUTO_TEST_CASE(updatecoins_simulation_test)
 
         // Once every 1000 iterations and at the end, verify the full cache.
         if (insecure_rand() % 1000 == 1 || i == NUM_SIMULATION_ITERATIONS - 1) {
-            BOOST_TEST_MESSAGE("updatecoins_simulation_test - verifying full cache");
             for (auto it = result.begin(); it != result.end(); it++) {
                 bool have = stack.back()->HaveCoin(it->first);
                 const Coin& coin = stack.back()->AccessCoin(it->first);

src/test/prevector_tests.cpp

@@ -186,7 +186,6 @@ class prevector_tester {
 BOOST_AUTO_TEST_CASE(PrevectorTestInt)
 {
     for (int j = 0; j < 64; j++) {
-        BOOST_TEST_MESSAGE("PrevectorTestInt " << j);
         prevector_tester<8, int> test;
         for (int i = 0; i < 2048; i++) {
             int r = insecure_rand();

src/test/versionbits_tests.cpp

@@ -136,8 +136,6 @@ BOOST_FIXTURE_TEST_SUITE(versionbits_tests, TestingSetup)
 BOOST_AUTO_TEST_CASE(versionbits_test)
 {
     for (int i = 0; i < 64; i++) {
-        BOOST_TEST_MESSAGE("versionbits_test " << i);
-
         // DEFINED -> FAILED
         VersionBitsTester().TestDefined()
                            .Mine(1, TestTime(1), 0x100).TestDefined()