Make it hopefully run on travis

Author: pesse

pom.xml

@@ -31,18 +31,6 @@
             </exclusion>
         </exclusions>
     </dependency>
-      <dependency>
-          <groupId>com.oracle.jdbc</groupId>
-          <artifactId>ojdbc8</artifactId>
-          <version>12.2.0.1</version>
-          <scope>compile</scope>
-      </dependency>
-      <dependency>
-          <groupId>com.oracle.jdbc</groupId>
-          <artifactId>orai18n</artifactId>
-          <version>12.2.0.1</version>
-          <scope>compile</scope>
-      </dependency>
     <dependency>
       <groupId>com.beust</groupId>
       <artifactId>jcommander</artifactId>

src/main/java/org/utplsql/cli/RunCommand.java

@@ -226,13 +226,14 @@ private List<Reporter> initReporters( Connection conn, List<ReporterOptions> rep
         for (ReporterOptions ro : reporterOptionsList) {
             Reporter reporter = reporterFactory.createReporter(ro.getReporterName());
 
+
+            reporter.init(conn, compatibilityProxy, reporterFactory);
+
             // Quick-hack for CoverageHTML Reporter
             if ( reporter instanceof CoverageHTMLReporter && ro.outputToFile() ) {
                 ((CoverageHTMLReporter)reporter).setAssetsPath(ro.getOutputFileName()+"_assets/");
                 CoverageHTMLReporter.writeReportAssetsTo(Paths.get(ro.getOutputFileName()+"_assets/"));
             }
-
-            reporter.init(conn, compatibilityProxy, reporterFactory);
             ro.setReporterObj(reporter);
             reporterList.add(reporter);
         }

src/test/java/org/utplsql/cli/RunCommandTest.java

@@ -20,7 +20,7 @@ public void reporterOptions_Default() {
         List<ReporterOptions> reporterOptionsList = runCmd.getReporterOptionsList();
 
         ReporterOptions reporterOptions1 = reporterOptionsList.get(0);
-        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER, reporterOptions1.getReporterName());
+        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertNull(reporterOptions1.getOutputFileName());
         assertFalse(reporterOptions1.outputToFile());
         assertTrue(reporterOptions1.outputToScreen());
@@ -33,7 +33,7 @@ public void reporterOptions_OneReporter() {
         List<ReporterOptions> reporterOptionsList = runCmd.getReporterOptionsList();
 
         ReporterOptions reporterOptions1 = reporterOptionsList.get(0);
-        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER, reporterOptions1.getReporterName());
+        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertEquals(reporterOptions1.getOutputFileName(), "output.txt");
         assertTrue(reporterOptions1.outputToFile());
         assertFalse(reporterOptions1.outputToScreen());
@@ -46,7 +46,7 @@ public void reporterOptions_OneReporterForceScreen() {
         List<ReporterOptions> reporterOptionsList = runCmd.getReporterOptionsList();
 
         ReporterOptions reporterOptions1 = reporterOptionsList.get(0);
-        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER, reporterOptions1.getReporterName());
+        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertEquals(reporterOptions1.getOutputFileName(), "output.txt");
         assertTrue(reporterOptions1.outputToFile());
         assertTrue(reporterOptions1.outputToScreen());
@@ -59,7 +59,7 @@ public void reporterOptions_OneReporterForceScreenInverse() {
         List<ReporterOptions> reporterOptionsList = runCmd.getReporterOptionsList();
 
         ReporterOptions reporterOptions1 = reporterOptionsList.get(0);
-        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER, reporterOptions1.getReporterName());
+        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertEquals(reporterOptions1.getOutputFileName(), "output.txt");
         assertTrue(reporterOptions1.outputToFile());
         assertTrue(reporterOptions1.outputToScreen());
@@ -74,13 +74,13 @@ public void reporterOptions_TwoReporters() {
         List<ReporterOptions> reporterOptionsList = runCmd.getReporterOptionsList();
 
         ReporterOptions reporterOptions1 = reporterOptionsList.get(0);
-        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER, reporterOptions1.getReporterName());
+        assertEquals(CoreReporters.UT_DOCUMENTATION_REPORTER.name(), reporterOptions1.getReporterName());
         assertNull(reporterOptions1.getOutputFileName());
         assertFalse(reporterOptions1.outputToFile());
         assertTrue(reporterOptions1.outputToScreen());
 
         ReporterOptions reporterOptions2 = reporterOptionsList.get(1);
-        assertEquals(CoreReporters.UT_COVERAGE_HTML_REPORTER, reporterOptions2.getReporterName());
+        assertEquals(CoreReporters.UT_COVERAGE_HTML_REPORTER.name(), reporterOptions2.getReporterName());
         assertEquals(reporterOptions2.getOutputFileName(), "coverage.html");
         assertTrue(reporterOptions2.outputToFile());
         assertTrue(reporterOptions2.outputToScreen());