Rendering the report into a basic, non-styled HTML

Author: battila7

src/main/java/cryptid/parsegrind/Application.java

@@ -1,8 +1,15 @@
 package cryptid.parsegrind;
 
+import cryptid.parsegrind.source.SourceReader;
 import cryptid.parsegrind.valgrind.ValgrindPublisher;
+import cryptid.parsegrind.valgrind.model.ValgrindReport;
+import cryptid.parsegrind.valgrind.render.ValgrindReportRenderer;
+import j2html.tags.Tag;
 import picocli.CommandLine;
 
+import java.io.BufferedWriter;
+import java.io.FileWriter;
+import java.io.IOException;
 import java.nio.file.Paths;
 import java.util.List;
 
@@ -41,18 +48,39 @@ public static void main(String[] args) throws Exception {
             defaultValue = "5")
     public int linesAfter;
 
+    @CommandLine.Option(names = {"-o", "--output"},
+            description = "Output file into which the rendered report is written.",
+            required = true)
+    public String outputFile;
+
     @Override
     public void run() {
         final Configuration configuration = configurationFromApplication();
 
         final ValgrindPublisher valgrindPublisher = new ValgrindPublisher();
         try {
-            valgrindPublisher.perform(configuration);
+            final ValgrindReport report = valgrindPublisher.perform(configuration);
+
+            final SourceReader sourceReader = new SourceReader(configuration);
+
+            final ValgrindReportRenderer renderer = new ValgrindReportRenderer(configuration, report, sourceReader);
+
+            final Tag document = renderer.render();
+
+            final String renderedDocument = document.render();
+
+            writeOutput(renderedDocument, configuration.outputFile);
         } catch (final Exception e) {
             e.printStackTrace(System.err);
         }
     }
 
+    private void writeOutput(final String output, final String path) throws IOException  {
+        try (final var writer = new BufferedWriter(new FileWriter(path))) {
+            writer.write(output);
+        }
+    }
+
     private Configuration configurationFromApplication() {
         final Configuration configuration = new Configuration();
 
@@ -61,6 +89,7 @@ private Configuration configurationFromApplication() {
         configuration.sourceGlobs = sourceGlobs;
         configuration.linesBefore = linesBefore;
         configuration.linesAfter = linesAfter;
+        configuration.outputFile = outputFile;
 
         return configuration;
     }

src/main/java/cryptid/parsegrind/Configuration.java

@@ -12,4 +12,6 @@ public final class Configuration {
     public int linesBefore;
 
     public int linesAfter;
+
+    public String outputFile;
 }

src/main/java/cryptid/parsegrind/source/SourceFragment.java

@@ -0,0 +1,29 @@
+package cryptid.parsegrind.source;
+
+import static java.util.Objects.requireNonNull;
+
+public final class SourceFragment {
+    private final int sourceFileStartLine;
+
+    private final int sourceFileErrorLine;
+
+    private final String lines;
+
+    public SourceFragment(final int sourceFileStartLine, final int sourceFileErrorLine, final String lines) {
+        this.sourceFileStartLine = sourceFileStartLine;
+        this.sourceFileErrorLine = sourceFileErrorLine;
+        this.lines = requireNonNull(lines);
+    }
+
+    public int getSourceFileStartLine() {
+        return sourceFileStartLine;
+    }
+
+    public int getSourceFileErrorLine() {
+        return sourceFileErrorLine;
+    }
+
+    public String getLines() {
+        return lines;
+    }
+}

src/main/java/cryptid/parsegrind/source/SourceReader.java

@@ -0,0 +1,38 @@
+package cryptid.parsegrind.source;
+
+import cryptid.parsegrind.Configuration;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+
+import static java.util.Objects.requireNonNull;
+
+public class SourceReader {
+    private static final String NEW_LINE = "\n";
+
+    private final Configuration configuration;
+
+    private final int lineCount;
+
+    public SourceReader(final Configuration configuration) {
+        this.configuration = requireNonNull(configuration);
+
+        this.lineCount = configuration.linesBefore + 1 + configuration.linesAfter;
+    }
+
+    public SourceFragment read(final String path, final int line) throws IOException {
+        final int startLine = Math.max(0, line - configuration.linesBefore);
+
+        try (final Stream<String> lines = Files.lines(Paths.get(path))) {
+            final String contents = lines
+                    .skip(startLine)
+                    .limit(lineCount)
+                    .collect(Collectors.joining(NEW_LINE));
+
+            return new SourceFragment(startLine, line, contents);
+        }
+    }
+}

src/main/java/cryptid/parsegrind/valgrind/ValgrindPublisher.java

@@ -6,15 +6,13 @@
 import cryptid.parsegrind.valgrind.parser.ValgrindParser;
 
 import java.io.IOException;
-import java.util.Collections;
 
 import static java.util.Objects.nonNull;
 
 public final class ValgrindPublisher {
-    public void perform(final Configuration configuration) throws IOException {
+    public ValgrindReport perform(final Configuration configuration) throws IOException {
         final ValgrindParser parser = new ValgrindParser();
 
-        final ValgrindResult valgrindResult = new ValgrindResult();
         final ValgrindReport valgrindReport = parser.parse(configuration);
 
         // Remove workspace path from executable name.
@@ -34,6 +32,6 @@ public void perform(final Configuration configuration) throws IOException {
             }
         }
 
-        valgrindResult.setSourceFiles(Collections.emptyMap());
+        return valgrindReport;
     }
 }