address fixes from Andrey

Author: Dimitry

CHANGELOG.md

@@ -9,7 +9,7 @@
 - Added: [#5575](https://github.com/ethereum/aleth/pull/5575) Log active peer count and peer list every 30 seconds.
 - Added: [#5580](https://github.com/ethereum/aleth/pull/5580) Enable syncing from ETC nodes for blocks < dao hard fork block.
 - Added: [#5591](https://github.com/ethereum/aleth/pull/5591) Network logging bugfixes and improvements and add p2pcap log channel.
-- Added: [#5588](https://github.com/ethereum/aleth/pull/5588) Testeth print test suggestions
+- Added: [#5588](https://github.com/ethereum/aleth/pull/5588) Testeth prints similar test suite name suggestions, when the name passed in `-t` argument is not found.
 - Added: [#5593](https://github.com/ethereum/aleth/pull/5593) Dynamically updating host ENR.
 - Changed: [#5559](https://github.com/ethereum/aleth/pull/5559) Update peer validation error messages.
 - Changed: [#5568](https://github.com/ethereum/aleth/pull/5568) Improve rlpx handshake log messages and create new rlpx log channel.

test/tools/libtesteth/boostTest.cpp

@@ -18,7 +18,6 @@
 #include <test/tools/jsontests/vm.h>
 #include <test/tools/libtesteth/TestHelper.h>
 #include <boost/test/included/unit_test.hpp>
-#include <boost/tokenizer.hpp>
 #include <clocale>
 #include <cstdlib>
 #include <iostream>
@@ -162,7 +161,7 @@ int main(int argc, const char* argv[])
 
 void printTestSuiteSuggestions(string const& _sMinusTArg)
 {
-    auto testList = test::testSuggestions(c_allTestNames, _sMinusTArg);
+    auto const testList = test::testSuggestions(c_allTestNames, _sMinusTArg);
     std::cerr << "Did you mean: \n";
     for (auto const& element : testList)
         std::cerr << "-t " << element << "\n";

test/unittests/libtesteth/testHelperTest.cpp

@@ -185,7 +185,7 @@ BOOST_AUTO_TEST_CASE(getTestSuggestions)
     vector<string> const testList = {
         "test1", "test2", "BlockSuite", "BlockSuite/TestCase", "GeneralBlockchainTests"};
     auto list = test::testSuggestions(testList, "blocksuit");
-    BOOST_CHECK_EQUAL(test::inArray(list, string("BlockSuite")), true);
+    BOOST_CHECK(test::inArray(list, string("BlockSuite")));
 }
 
 BOOST_AUTO_TEST_CASE(getTestSuggestions2)
@@ -194,8 +194,8 @@ BOOST_AUTO_TEST_CASE(getTestSuggestions2)
         "GeneralBlockchainTests", "GeneralStateTests/stExample", "BCGeneralStateTests/stExample"};
 
     auto list = test::testSuggestions(testList, "GeneralStateTests/stExample2");
-    BOOST_CHECK_EQUAL(test::inArray(list, string("GeneralStateTests/stExample")), true);
-    BOOST_CHECK_EQUAL(test::inArray(list, string("BCGeneralStateTests/stExample")), true);
+    BOOST_CHECK(test::inArray(list, string("GeneralStateTests/stExample")));
+    BOOST_CHECK(test::inArray(list, string("BCGeneralStateTests/stExample")));
 }
 BOOST_AUTO_TEST_SUITE_END()