diff --git a/lib/errorlogger.cpp b/lib/errorlogger.cpp index b9792f51dff..7554611a686 100644 --- a/lib/errorlogger.cpp +++ b/lib/errorlogger.cpp @@ -47,7 +47,6 @@ const std::set ErrorLogger::mCriticalErrorIds{ "internalAstError", "instantiationError", "internalError", - "misra-config", "premium-internalError", "premium-invalidArgument", "premium-invalidLicense", diff --git a/test/testerrorlogger.cpp b/test/testerrorlogger.cpp index 49da36cfa6b..b166bd0e7e3 100644 --- a/test/testerrorlogger.cpp +++ b/test/testerrorlogger.cpp @@ -65,6 +65,8 @@ class TestErrorLogger : public TestFixture { TEST_CASE(substituteTemplateFormatStatic); TEST_CASE(substituteTemplateLocationStatic); + + TEST_CASE(isCriticalErrorId); } void TestPatternSearchReplace(const std::string& idPlaceholder, const std::string& id) const { @@ -519,6 +521,11 @@ class TestErrorLogger : public TestFixture { ASSERT_EQUALS("{", s); } } + + void isCriticalErrorId() { + // It does not abort all the analysis of the file. Like "missingInclude" there can be false negatives. + ASSERT_EQUALS(false, ErrorLogger::isCriticalErrorId("misra-config")); + } }; REGISTER_TEST(TestErrorLogger)