CppCheckExecutor: report all addon loading failures at once

Author: firewave

cli/cppcheckexecutor.cpp

@@ -352,16 +352,18 @@ bool CppCheckExecutor::loadLibraries(Settings& settings)
 
 bool CppCheckExecutor::loadAddons(Settings& settings)
 {
+    bool result = true;
     for (const std::string &addon: settings.addons) {
         AddonInfo addonInfo;
         const std::string failedToGetAddonInfo = addonInfo.getAddonInfo(addon, settings.exename);
         if (!failedToGetAddonInfo.empty()) {
             std::cout << failedToGetAddonInfo << std::endl;
-            return false;
+            result = false;
+            continue;
         }
         settings.addonInfos.emplace_back(std::move(addonInfo));
     }
-    return true;
+    return result;
 }
 
 #ifdef _WIN32

test/cli/test-other.py

@@ -74,9 +74,10 @@ def test_invalid_library(tmpdir):
 # TODO: test missing std.cfg
 
 def test_invalid_addon(tmpdir):
-    args = ['--addon=misra2', 'file.c']
+    args = ['--addon=misra3', '--addon=misra', '--addon=misra2', 'file.c']
 
     exitcode, stdout, stderr = cppcheck(args)
     assert exitcode == 1
-    assert stdout == '(information) Did not find addon misra2.py\n'
+    assert (stdout == 'Did not find addon misra2.py\n'
+                      'Did not find addon misra3.py\n')
     assert stderr == ""