style: Enable some clang-tidy readability checks for unit tests

diff --git a/unittest/.clang-tidy b/unittest/.clang-tidy
index 06d78c32924f61bf27c6b1bd6dedec1c7ba32cb5..978237e36ef5c0b433ceac8e7a8435ae97f9501e 100644 (file)
--- a/unittest/.clang-tidy
+++ b/unittest/.clang-tidy
@@ -1,5 +1,12 @@
 ---
-Checks:          '-*,readability-function-size'
+Checks:          '-*,
+                  readability-*,
+                  -readability-implicit-bool-conversion,
+                  -readability-magic-numbers,
+                  -readability-else-after-return,
+                  -readability-named-parameter,
+                  -readability-qualified-auto,
+                  -readability-redundant-declaration'
 WarningsAsErrors: '*'
 # Only include headers directly in unittest.
 HeaderFilterRegex: 'unittest/[^/]*$'

diff --git a/unittest/test_util_Tokenizer.cpp b/unittest/test_util_Tokenizer.cpp
index c5efea34ca784cbfb1a072f265c8c569b2731ac4..ce3a55313cbee4c3cffeb803f6cadd0c72fd202b 100644 (file)
--- a/unittest/test_util_Tokenizer.cpp
+++ b/unittest/test_util_Tokenizer.cpp
@@ -1,4 +1,4 @@
-// Copyright (C) 2021 Joel Rosdahl and other contributors
+// Copyright (C) 2021-2022 Joel Rosdahl and other contributors
 //
 // See doc/AUTHORS.adoc for a complete list of contributors.
 //
@@ -36,13 +36,14 @@ TEST_CASE("util::Tokenizer")
     void
     operator()(const char* input,
                const char* separators,
-               const std::vector<std::string>& expected)
+               const std::vector<std::string>& expected) const
     {
       const auto res =
         Util::split_into_views(input, separators, m_mode, m_includeDelimiter);
       REQUIRE(res.size() == expected.size());
-      for (int i = 0, total = expected.size(); i < total; ++i)
+      for (int i = 0, total = expected.size(); i < total; ++i) {
         CHECK(res[i] == expected[i]);
+      }
     }
 
     Mode m_mode;