Change exception handling in ReportWriter

Author: simasch

src/main/java/org/utplsql/maven/plugin/UtPlsqlMojo.java

@@ -28,6 +28,7 @@
 import org.utplsql.maven.plugin.model.ReporterParameter;
 
 import java.io.File;
+import java.io.IOException;
 import java.sql.Connection;
 import java.sql.SQLException;
 import java.util.ArrayList;
@@ -193,7 +194,7 @@ public void execute() throws MojoExecutionException {
                         DBHelper.disableDBMSOutput(connection);
                         connection.close();
                     }
-                } catch (SQLException e) {
+                } catch (SQLException | IOException e) {
                     getLog().error(e.getMessage(), e);
                 }
             }

src/main/java/org/utplsql/maven/plugin/io/ReportWriter.java

@@ -65,15 +65,16 @@ public void addReporter(ReporterParameter parameter, Reporter reporter) {
      * Writes the reports to the output.
      *
      * @param connection The database {@link Connection}
-     * @throws MojoExecutionException if any exception happens
+     * @throws SQLException if database access fails
+     * @throws IOException  if files can't be written
      */
-    public void writeReports(Connection connection) throws MojoExecutionException {
+    public void writeReports(Connection connection) throws SQLException, IOException {
         for (Pair<Reporter, ReporterParameter> pair : reporters) {
             writeReports(connection, pair.getLeft(), pair.getRight());
         }
     }
 
-    private void writeReports(Connection connection, Reporter reporter, ReporterParameter reporterParameter) throws MojoExecutionException {
+    private void writeReports(Connection connection, Reporter reporter, ReporterParameter reporterParameter) throws IOException, SQLException {
         FileOutputStream fileOutputStream = null;
         try {
             OutputBuffer buffer = OutputBufferProvider.getCompatibleOutputBuffer(databaseVersion, reporter, connection);
@@ -103,15 +104,9 @@ private void writeReports(Connection connection, Reporter reporter, ReporterPara
 
             buffer.printAvailable(connection, printStreams);
 
-        } catch (FileNotFoundException | SQLException e) {
-            throw new MojoExecutionException(e.getMessage(), e);
         } finally {
             if (fileOutputStream != null) {
-                try {
-                    fileOutputStream.close();
-                } catch (IOException e) {
-                    log.error(format("Couldn't close the output stream %s", reporterParameter.getClass()));
-                }
+                fileOutputStream.close();
             }
         }
     }