[EA] Add support of multi-lines error messages

src/main/java/edu/hm/hafner/analysis/parser/EmbeddedEngineerParser.java

@@ -1,7 +1,9 @@
 package edu.hm.hafner.analysis.parser;
 
 import java.io.UncheckedIOException;
+import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import java.util.stream.Stream;
@@ -22,13 +24,16 @@
  */
 public class EmbeddedEngineerParser extends IssueParser {
     private static final long serialVersionUID = -1251248150731418714L;
+
+    private static final Pattern LOG_BEGINNING_PATTERN = Pattern.compile("\\[.*?\\].*");
     private static final Pattern HEADER_PATTERN = Pattern.compile(
             "^([^\\(Start)]*)(Starting code generation for)\\s(?<file>.*\\})");
     private static final Pattern SPECIAL_WARNING_PATTERN = Pattern.compile(
             "^\\[([^\\]]*)\\]\\s(?<severity>Warn)\\s-\\s(?<description>[^']*)'(?<module>[^']*)"
                     + "'\\s(?<details>\\(?[^{]*)(?<serial>[^)]*\\})");
+
     private static final Pattern WARNING_PATTERN = Pattern.compile(
-            "^\\[([^\\]]*)\\]\\s(?<severity>Error|Warn)\\s-\\s(?<category>.+):\\s(?<description>.+)");
+            "^\\[([^\\]]*)\\]\\s(?<severity>Error|Warn)\\s-\\s(?<category>[^:]*)" + "(:\\s|\\s\\()(?<description>.+)");
 
     @Override
     public Report parse(final ReaderFactory reader) throws ParsingException {
@@ -40,28 +45,47 @@ public Report parse(final ReaderFactory reader) throws ParsingException {
         }
     }
 
+    private List<String> readLogInfo(final Iterator<String> lineIterator) {
+        List<String> logInfo = new ArrayList<>();
+
+        while (lineIterator.hasNext()) {
+            String currentLine = lineIterator.next();
+            Matcher logBeginningMatcher = LOG_BEGINNING_PATTERN.matcher(currentLine);
+            if (logBeginningMatcher.matches()) {
+                logInfo.add(currentLine);
+            }
+            else {
+                int lastIdx = logInfo.size() - 1;
+                StringBuilder multiLineInfo = new StringBuilder(logInfo.get(lastIdx));
+                multiLineInfo.append(" ").append(currentLine);
+                logInfo.set(lastIdx, multiLineInfo.toString());
+            }
+        }
+        return logInfo;
+    }
+
     private Report parse(final Stream<String> lines) {
         try (IssueBuilder builder = new IssueBuilder()) {
             Iterator<String> lineIterator = lines.iterator();
             String file = parseFileName(lineIterator);
             Report report = new Report();
-
-            while (lineIterator.hasNext()) {
-                String line = lineIterator.next();
-                Matcher matcher = SPECIAL_WARNING_PATTERN.matcher(line);
-                Matcher warningMatcher = WARNING_PATTERN.matcher(line);
+            List<String> logLines = this.readLogInfo(lineIterator);
+            for (String logLine:logLines) {
+                Matcher matcher = SPECIAL_WARNING_PATTERN.matcher(logLine);
+                Matcher warningMatcher = WARNING_PATTERN.matcher(logLine);
 
                 if (matcher.matches() || warningMatcher.matches()) {
                     String group;
                     String description;
                     if (matcher.matches()) {
                         group = matcher.group("severity");
+
                         description = String.format("%s'%s' %s%s",
                                 matcher.group("description"),
                                 matcher.group("module"),
                                 matcher.group("details"),
                                 matcher.group("serial"));
-                        builder.setCategory(setCategory(line));
+                        builder.setCategory(setCategory(logLine));
                         builder.setModuleName(matcher.group("module"));
                     }
                     else {
@@ -71,7 +95,7 @@ private Report parse(final Stream<String> lines) {
                                 warningMatcher.group("category"),
                                 warningMatcher.group("description"));
                     }
-                    Severity priority = mapPriority(line, group);
+                    Severity priority = mapPriority(logLine, group);
                     builder.setDescription(description);
                     builder.setFileName(file);
                     builder.setSeverity(priority);
@@ -121,4 +145,4 @@ else if (group.contains("Error")) {
         }
         return Severity.WARNING_NORMAL;
     }
-}
+}

src/test/java/edu/hm/hafner/analysis/parser/EmbeddedEngineerParserTest.java

@@ -22,7 +22,7 @@ protected EmbeddedEngineerParser createParser() {
 
     @Override
     protected void assertThatIssuesArePresent(final Report report, final SoftAssertions softly) {
-        softly.assertThat(report).hasSize(8);
+        softly.assertThat(report).hasSize(9);
         softly.assertThat(report.get(0))
                 .hasModuleName("index_module")
                 .hasDescription("Complex type definition without referenced element found 'index_module' (uint8_t); {98CF1FE6-EC9C-43f1-e476-40EFCD63cA8D}")
@@ -45,6 +45,30 @@ protected void assertThatIssuesArePresent(final Report report, final SoftAsserti
                 .hasCategory("Error loading plugins from")
                 .hasDescription("Error loading plugins from C:\\file1\\idc\\sample_ext.x64.dll")
                 .hasSeverity(Severity.ERROR);
+        softly.assertThat(report.get(8))
+                .hasCategory("Error loading plugins from")
+                .hasDescription("Error loading plugins from C:\\file1\\idc\\sample_ext.x64.dll File name: "
+                        + "'file:///C:\\AA\\BB\\System.Text.Json.dll' ---> System.NotSupportedException: "
+                        + "An attempt was made to load an assembly from a network location which would have caused the "
+                        + "assembly to be sandboxed in previous versions of the .NET Framework. "
+                        + "This release of the .NET Framework does not enable CAS policy by default, "
+                        + "so this load may be dangerous. If this load is not intended to sandbox the assembly,"
+                        + " please enable the loadFromRemoteSources switch. "
+                        + "See http://go.microsoft.com/fwlink/?LinkId=155569 for more information.   "
+                        + " at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase,"
+                        + " Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, "
+                        + "IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, "
+                        + "Boolean suppressSecurityChecks)    at System.Reflection.RuntimeAssembly."
+                        + "InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, "
+                        + "RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, "
+                        + "Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)    "
+                        + "at System.Reflection.RuntimeAssembly.InternalLoadFrom(String assemblyFile, "
+                        + "Evidence securityEvidence, Byte[] hashValue, AssemblyHashAlgorithm hashAlgorithm, "
+                        + "Boolean forIntrospection, Boolean suppressSecurityChecks, StackCrawlMark& stackMark)   "
+                        + " at System.Reflection.Assembly.LoadFrom(String assemblyFile)    "
+                        + "at A.A`1.a(String , Predicate`1 , Object[] )    "
+                        + "at A.A`1.A(String , Predicate`1 , Object[] )")
+                .hasSeverity(Severity.ERROR);
     }
 }
 

src/test/resources/edu/hm/hafner/analysis/parser/ea.log

@@ -7,4 +7,12 @@
 [2023-03-10 12:04:25.0777] Warn - Code generation for element 'Module1' failed. Could not start code generation ; {ffeee99-9BD1-12345678}
 [2023-03-10 12:04:25.0777] Warn - Code generation for element 'Module1' passed. Could not start code generation ; {ffeee99-9BD1-12345678}
 [2023-03-10 12:04:25.1377] Warn - SampleValidation: no requirement linked to final node 'Module1_Node'.
-[2023-03-10 12:04:22.1045] Error - Error loading plugins from: C:\file1\idc\sample_ext.x64.dll
+[2023-03-10 12:04:22.1045] Error - Error loading plugins from: C:\file1\idc\sample_ext.x64.dll
+[2023-04-10 12:04:22.1045] Error - Error loading plugins from: C:\file1\idc\sample_ext.x64.dll
+File name: 'file:///C:\AA\BB\System.Text.Json.dll' ---> System.NotSupportedException: An attempt was made to load an assembly from a network location which would have caused the assembly to be sandboxed in previous versions of the .NET Framework. This release of the .NET Framework does not enable CAS policy by default, so this load may be dangerous. If this load is not intended to sandbox the assembly, please enable the loadFromRemoteSources switch. See http://go.microsoft.com/fwlink/?LinkId=155569 for more information.
+   at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
+   at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
+   at System.Reflection.RuntimeAssembly.InternalLoadFrom(String assemblyFile, Evidence securityEvidence, Byte[] hashValue, AssemblyHashAlgorithm hashAlgorithm, Boolean forIntrospection, Boolean suppressSecurityChecks, StackCrawlMark& stackMark)
+   at System.Reflection.Assembly.LoadFrom(String assemblyFile)
+   at A.A`1.a(String , Predicate`1 , Object[] )
+   at A.A`1.A(String , Predicate`1 , Object[] )