add public openInBrowser method

PhilippSalvisberg · PhilippSalvisberg · commit 55b872743d9d · 2020-05-30T17:23:54.000+02:00
diff --git a/sqldev/src/main/java/org/utplsql/sqldev/coverage/CodeCoverageReporter.java b/sqldev/src/main/java/org/utplsql/sqldev/coverage/CodeCoverageReporter.java
@@ -84,11 +84,26 @@ private void run() {
         logger.fine(() -> "Running code coverage reporter for " + pathList + "...");
         try {
             final UtplsqlDao dal = new UtplsqlDao(conn);
-            final String content = dal.htmlCodeCoverage(pathList, toStringList(schemas),
+            final String html = dal.htmlCodeCoverage(pathList, toStringList(schemas),
                     toStringList(includeObjects), toStringList(excludeObjects));
+            openInBrowser(html);
+        } finally {
+            try {
+                DatabaseTools.closeConnection(conn);
+            } catch (GenericDatabaseAccessException e) {
+                // ignore
+            }
+            if (frame != null) {
+                frame.exit();
+            }
+        }
+    }
+    
+    public static void openInBrowser(String html) {
+        try {
             final File file = File.createTempFile("utplsql_", ".html");
             logger.fine(() -> "Writing result to " + file + "...");
-            FileTools.writeFile(file.toPath(), Arrays.asList(content.split(System.lineSeparator())), StandardCharsets.UTF_8);
+            FileTools.writeFile(file.toPath(), Arrays.asList(html.split(System.lineSeparator())), StandardCharsets.UTF_8);
             final URL url = file.toURI().toURL();
             logger.fine(() -> "Opening " + url.toExternalForm() + " in browser...");
             final Desktop desktop = Desktop.isDesktopSupported() ? Desktop.getDesktop() : null;
@@ -97,21 +112,12 @@ private void run() {
                 logger.fine(() -> url.toExternalForm() + " opened in browser.");
             } else {
                 logger.severe(
-                        () -> "Could not launch " + file + "in browser. No default browser defined on this system.");
+                        () -> "Could not launch " + file + " in browser. No default browser defined on this system.");
             }
         } catch (Exception e) {
-            final String msg = "Error while running code coverage for " + pathList + ".";
+            final String msg = "Error while opening code coverage HTML report in browser.";
             logger.severe(() -> msg);
             throw new GenericRuntimeException(msg, e);
-        } finally {
-            try {
-                DatabaseTools.closeConnection(conn);
-            } catch (GenericDatabaseAccessException e) {
-                // ignore
-            }
-            if (frame != null) {
-                frame.exit();
-            }
         }
     }
 
