CppCheckExecutor: report all addon loading failures at once

Author: firewave

cli/cppcheckexecutor.cpp

@@ -411,16 +411,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

@@ -532,9 +532,14 @@ class A {
 
 
 def test_missing_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 == 'Did not find addon misra2.py\n'
+    lines = stdout.splitlines()
+    lines.sort()
+    assert lines == [
+        'Did not find addon misra2.py',
+        'Did not find addon misra3.py'
+    ]
     assert stderr == ""