Add UT for tooManyConfigsError function

Author: GNL10

lib/cppcheck.cpp

@@ -1686,6 +1686,9 @@ void CppCheck::tooManyConfigsError(const std::string &file, const int numberOfCo
         msg << " of " << numberOfConfigurations << " configurations. Use --force to check all configurations.\n";
     else if (file.empty())
         msg << " configurations. Use --force to check all configurations. For more details, use --enable=information.\n";
+    else
+        msg << " configurations.\n";
+
     msg << "The checking of the file will be interrupted because there are too many "
         "#ifdef configurations. Checking of all #ifdef configurations can be forced "
         "by --force command line option or from GUI preferences. However that may "

test/testcppcheck.cpp

@@ -79,6 +79,7 @@ class TestCppcheck : public TestFixture {
         TEST_CASE(isPremiumCodingStandardId);
         TEST_CASE(getDumpFileContentsRawTokens);
         TEST_CASE(getDumpFileContentsLibrary);
+        TEST_CASE(tooManyConfigsError);
         TEST_CASE(checkPlistOutput);
         TEST_CASE(premiumResultsCache);
         TEST_CASE(toomanyconfigs);
@@ -523,6 +524,81 @@ class TestCppcheck : public TestFixture {
         }
     }
 
+    void tooManyConfigsError() const {
+        Suppressions supprs;
+        ErrorLogger2 errorLogger;
+
+        {
+            const Settings s;
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            cppcheck.tooManyConfigsError("file", 0);
+            ASSERT_EQUALS(0, errorLogger.errmsgs.size());
+        }
+
+        {
+            const auto s = dinit(Settings, $.severity.enable (Severity::information));
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            cppcheck.tooManyConfigsError("", 0);
+            ASSERT_EQUALS(0, errorLogger.errmsgs.size());
+        }
+
+        const std::string forceFlagInstruction {"The checking of the file will be interrupted because there are too many "
+                                                "#ifdef configurations. Checking of all #ifdef configurations can be forced "
+                                                "by --force command line option or from GUI preferences. However that may "
+                                                "increase the checking time."};
+
+        {
+            const auto s = dinit(Settings, $.templateFormat = templateFormat, $.severity.enable (Severity::information));
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+
+            const int numberOfConfigurations = s.maxConfigs+1;
+            cppcheck.tooManyConfigsError("file", numberOfConfigurations);
+
+            ASSERT_EQUALS(1, errorLogger.errmsgs.size());
+            auto it = errorLogger.errmsgs.cbegin();
+            const std::string shortMsg = it->toString(false, templateFormat, "");
+            const std::string debugMsg = it->toString(true, templateFormat, "");
+
+            ASSERT_EQUALS(shortMsg, "file:0:0: information: Too many #ifdef configurations - cppcheck only checks " + std::to_string(s.maxConfigs) + " of " + std::to_string(numberOfConfigurations) + " configurations. Use --force to check all configurations. [toomanyconfigs]");
+            ASSERT_EQUALS(debugMsg, "file:0:0: information: " + forceFlagInstruction + " [toomanyconfigs]");
+            errorLogger.errmsgs.clear();
+        }
+
+        {
+            const auto s = dinit(Settings, $.templateFormat = templateFormat);
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            cppcheck.mTooManyConfigs = true;
+
+            cppcheck.tooManyConfigsError("", 0);
+
+            ASSERT_EQUALS(1, errorLogger.errmsgs.size());
+            auto it = errorLogger.errmsgs.cbegin();
+            const std::string shortMsg = it->toString(false, templateFormat, "");
+            const std::string debugMsg = it->toString(true, templateFormat, "");
+
+            ASSERT_EQUALS(shortMsg, "nofile:0:0: information: Too many #ifdef configurations - cppcheck only checks " + std::to_string(s.maxConfigs) + " configurations. Use --force to check all configurations. For more details, use --enable=information. [toomanyconfigs]");
+            ASSERT_EQUALS(debugMsg, "nofile:0:0: information: " + forceFlagInstruction + " For more details, use --enable=information. [toomanyconfigs]");
+            errorLogger.errmsgs.clear();
+        }
+
+        {
+            const auto s = dinit(Settings, $.templateFormat = templateFormat, $.severity.enable (Severity::information));
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            cppcheck.mTooManyConfigs = true;
+
+            cppcheck.tooManyConfigsError("file", 0);
+
+            ASSERT_EQUALS(1, errorLogger.errmsgs.size());
+            auto it = errorLogger.errmsgs.cbegin();
+            const std::string shortMsg = it->toString(false, templateFormat, "");
+            const std::string debugMsg = it->toString(true, templateFormat, "");
+
+            ASSERT_EQUALS(shortMsg, "file:0:0: information: Too many #ifdef configurations - cppcheck only checks " + std::to_string(s.maxConfigs) + " configurations. [toomanyconfigs]");
+            ASSERT_EQUALS(debugMsg, "file:0:0: information: " + forceFlagInstruction + " [toomanyconfigs]");
+            errorLogger.errmsgs.clear();
+        }
+    }
+
     void checkPlistOutput() const {
         Suppressions supprs;
         ErrorLogger2 errorLogger;