We don't need to enable DBMS output I think

pesse · pesse · commit 92d027ec756a · 2019-02-05T22:00:41.000+01:00
diff --git a/src/main/java/org/utplsql/api/TestRunner.java b/src/main/java/org/utplsql/api/TestRunner.java
@@ -154,7 +154,6 @@ public void run(Connection conn) throws SQLException {
             options.reporterList.add(new DocumentationReporter().init(conn));
         }
 
-        DBHelper.enableDBMSOutput(conn);
         try(TestRunnerStatement testRunnerStatement = compatibilityProxy.getTestRunnerStatement(options, conn)) {
             logger.info("Running tests");
             testRunnerStatement.execute();
@@ -169,8 +168,6 @@ else if (e.getErrorCode() == UtPLSQLNotInstalledException.ERROR_CODE) {
             else {
                 throw e;
             }
-        } finally {
-            DBHelper.disableDBMSOutput(conn);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,6 @@ public void run(Connection conn) throws SQLException {`
`154`	`154`	`options.reporterList.add(new DocumentationReporter().init(conn));`
`155`	`155`	`}`
`156`	`156`
`157`		`- DBHelper.enableDBMSOutput(conn);`
`158`	`157`	`try(TestRunnerStatement testRunnerStatement = compatibilityProxy.getTestRunnerStatement(options, conn)) {`
`159`	`158`	`logger.info("Running tests");`
`160`	`159`	`testRunnerStatement.execute();`
`@@ -169,8 +168,6 @@ else if (e.getErrorCode() == UtPLSQLNotInstalledException.ERROR_CODE) {`
`169`	`168`	`else {`
`170`	`169`	`throw e;`
`171`	`170`	`}`
`172`		`- } finally {`
`173`		`- DBHelper.disableDBMSOutput(conn);`
`174`	`171`	`}`
`175`	`172`	`}`
`176`	`173`