From 5401f35cc3c70a8cf2f21e5fe25f42751a46c449 Mon Sep 17 00:00:00 2001
From: pesse <derpesse@gmail.com>
Date: Fri, 10 Nov 2017 16:52:15 +0100
Subject: [PATCH 1/6] Removed Oracle libaries from release

Included check for specific classes.
Needs some refactoring for we only want to check for orai18n when database is 11g
---
 .travis/create_release.sh                     |  4 ++
 .../org/utplsql/cli/OracleLibraryChecker.java | 39 +++++++++++++++++++
 src/main/java/org/utplsql/cli/RunCommand.java | 26 +++++++++++++
 3 files changed, 69 insertions(+)
 create mode 100644 src/main/java/org/utplsql/cli/OracleLibraryChecker.java

diff --git a/.travis/create_release.sh b/.travis/create_release.sh
index 8daebe9..78c9788 100644
--- a/.travis/create_release.sh
+++ b/.travis/create_release.sh
@@ -5,6 +5,10 @@ VERSION=`date +%Y%m%d%H%M`
 
 mkdir dist
 mv target/appassembler utPLSQL-cli
+# Remove Oracle libraries du to licensing problems
+rm utPLSQL-cli/lib/ojdbc8*
+rm utPLSQL-cli/lib/orai18n*
+
 zip -r -q dist/utPLSQL-cli-${TRAVIS_BRANCH}-${VERSION}.zip utPLSQL-cli
 
 cat > bintray.json <<EOF
diff --git a/src/main/java/org/utplsql/cli/OracleLibraryChecker.java b/src/main/java/org/utplsql/cli/OracleLibraryChecker.java
new file mode 100644
index 0000000..24b02f8
--- /dev/null
+++ b/src/main/java/org/utplsql/cli/OracleLibraryChecker.java
@@ -0,0 +1,39 @@
+package org.utplsql.cli;
+
+/** Simple class to check whether needed Oracle libraries are on classpath or not
+ *
+ * @author pesse
+ */
+class OracleLibraryChecker {
+
+    private static boolean classExists( String classFullName ){
+        try
+        {
+            Class.forName(classFullName);
+
+            return true;
+        }
+        catch ( ClassNotFoundException e )
+        {
+            return false;
+        }
+    }
+
+    /** Checks if OJDBC library is on the classpath by searching for oracle.jdbc.OracleDriver class
+     *
+     * @return true or false
+     */
+    public static boolean checkOjdbcExists() {
+        return classExists("oracle.jdbc.OracleDriver");
+    }
+
+    /** Checks if Orai18n library is on the classpath by searching for oracle.i18n.text.OraCharset
+     *
+     * @return true or false
+     */
+    public static boolean checkOrai18nExists() {
+        return classExists("oracle.i18n.text.OraCharset");
+    }
+
+
+}
diff --git a/src/main/java/org/utplsql/cli/RunCommand.java b/src/main/java/org/utplsql/cli/RunCommand.java
index dd77db5..85d2277 100644
--- a/src/main/java/org/utplsql/cli/RunCommand.java
+++ b/src/main/java/org/utplsql/cli/RunCommand.java
@@ -78,6 +78,9 @@ public List<String> getTestPaths() {
     }
 
     public int run() throws Exception {
+
+        checkOracleLibrariesExist();
+
         final ConnectionInfo ci = getConnectionInfo();
 
         final List<ReporterOptions> reporterOptionsList = getReporterOptionsList();
@@ -246,4 +249,27 @@ public FileMapperOptions getMapperOptions(List<String> mappingParams, List<Strin
         return mapperOptions;
     }
 
+
+    /** Checks that necessary oracle libraries exist
+     *
+     */
+    private void checkOracleLibrariesExist()
+    {
+        if ( !OracleLibraryChecker.checkOjdbcExists() )
+        {
+            System.out.println("Could not find Oracle JDBC driver in classpath. Please download the jar from Oracle website" +
+                    " and copy it to the 'lib' folder of your utPLSQL-cli installation.");
+            System.out.println("Download from http://www.oracle.com/technetwork/database/features/jdbc/jdbc-ucp-122-3110062.html");
+
+            throw new RuntimeException("Can't run utPLSQL-cli without Oracle JDBC driver");
+        }
+
+        if ( !OracleLibraryChecker.checkOrai18nExists() )
+        {
+            System.out.println("Warning: Could not find Oracle i18n driver in classpath. Depending on your database " +
+                    "version (11g) and used charset utPLSQL-cli might not run properly. It is recommended you download " +
+                    "the i18n driver from the Oracle website and copy it to the 'lib' folder of your utPLSQL-cli installation.");
+            System.out.println("Download from http://www.oracle.com/technetwork/database/enterprise-edition/jdbc-112010-090769.html");
+        }
+    }
 }

From 22a560311bf02819e78f026dfcb4d46f37a71236 Mon Sep 17 00:00:00 2001
From: pesse <derpesse@gmail.com>
Date: Sun, 12 Nov 2017 14:44:34 +0100
Subject: [PATCH 2/6] Only check for orai18n when we are on a 11g database

the library is not needed for 12c
---
 .../java/org/utplsql/cli/ConnectionInfo.java  | 40 +++++++++++++++++++
 src/main/java/org/utplsql/cli/RunCommand.java | 21 +++++++---
 2 files changed, 55 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/utplsql/cli/ConnectionInfo.java b/src/main/java/org/utplsql/cli/ConnectionInfo.java
index 287f989..4f76f81 100644
--- a/src/main/java/org/utplsql/cli/ConnectionInfo.java
+++ b/src/main/java/org/utplsql/cli/ConnectionInfo.java
@@ -6,10 +6,14 @@
 
 import java.io.File;
 import java.sql.Connection;
+import java.sql.PreparedStatement;
+import java.sql.ResultSet;
 import java.sql.SQLException;
 
 public class ConnectionInfo {
 
+    private String databaseVersion;
+
     static {
         String oracleHome = System.getenv("ORACLE_HOME");
         if (oracleHome != null) {
@@ -42,4 +46,40 @@ public ConnectionInfo convert(String s) {
         }
     }
 
+    public String getOracleDatabaseVersion() throws SQLException
+    {
+        try ( Connection conn = getConnection() ) {
+            return getOracleDatabaseVersion(conn);
+        }
+    }
+
+    public String getOracleDatabaseVersion( Connection conn ) throws SQLException
+    {
+        if ( databaseVersion == null ) {
+            databaseVersion = getOracleDatabaseVersionFromConnection( conn );
+        }
+
+        return databaseVersion;
+    }
+
+    /** TODO: Outsource this to Java-API
+     *
+     * @param conn
+     * @return
+     * @throws SQLException
+     */
+    public static String getOracleDatabaseVersionFromConnection( Connection conn ) throws SQLException {
+        assert conn != null;
+        String result = null;
+        try (PreparedStatement stmt = conn.prepareStatement("select version from product_component_version where product like 'Oracle Database%'"))
+        {
+            ResultSet rs = stmt.executeQuery();
+
+            if ( rs.next() )
+                result = rs.getString(1);
+        }
+
+        return result;
+    }
+
 }
diff --git a/src/main/java/org/utplsql/cli/RunCommand.java b/src/main/java/org/utplsql/cli/RunCommand.java
index 6cc18ee..b81e189 100644
--- a/src/main/java/org/utplsql/cli/RunCommand.java
+++ b/src/main/java/org/utplsql/cli/RunCommand.java
@@ -90,7 +90,7 @@ public List<String> getTestPaths() {
 
     public int run() throws Exception {
 
-        checkOracleLibrariesExist();
+        checkOracleJDBCExists();
 
         final ConnectionInfo ci = getConnectionInfo();
 
@@ -110,6 +110,9 @@ public int run() throws Exception {
         // Do the reporters initialization, so we can use the id to run and gather results.
         try (Connection conn = ci.getConnection()) {
 
+            // Check if orai18n exists if database version is 11g
+            checkOracleI18nExists(ci.getOracleDatabaseVersion(conn));
+
             // First of all do a compatibility check and fail-fast
             checkFrameworkCompatibility(conn);
 
@@ -324,10 +327,10 @@ public FileMapperOptions getMapperOptions(List<String> mappingParams, List<Strin
     }
 
 
-    /** Checks that necessary oracle libraries exist
+    /** Checks that ojdbc library exists
      *
      */
-    private void checkOracleLibrariesExist()
+    private void checkOracleJDBCExists()
     {
         if ( !OracleLibraryChecker.checkOjdbcExists() )
         {
@@ -337,11 +340,17 @@ private void checkOracleLibrariesExist()
 
             throw new RuntimeException("Can't run utPLSQL-cli without Oracle JDBC driver");
         }
+    }
 
-        if ( !OracleLibraryChecker.checkOrai18nExists() )
+    /** Checks that orai18n library exists if database is an oracle 11
+     *
+     */
+    private void checkOracleI18nExists(String oracleDatabaseVersion )
+    {
+        if ( oracleDatabaseVersion.startsWith("11.") && !OracleLibraryChecker.checkOrai18nExists() )
         {
-            System.out.println("Warning: Could not find Oracle i18n driver in classpath. Depending on your database " +
-                    "version (11g) and used charset utPLSQL-cli might not run properly. It is recommended you download " +
+            System.out.println("Warning: Could not find Oracle i18n driver in classpath. Depending on the database charset " +
+                    "utPLSQL-cli might not run properly. It is recommended you download " +
                     "the i18n driver from the Oracle website and copy it to the 'lib' folder of your utPLSQL-cli installation.");
             System.out.println("Download from http://www.oracle.com/technetwork/database/enterprise-edition/jdbc-112010-090769.html");
         }

From f76f9a744b5152f7155dca197c126198e0a0c946 Mon Sep 17 00:00:00 2001
From: Samuel Nitsche <samuel.nitsche@smenso.de>
Date: Sun, 12 Nov 2017 14:48:20 +0100
Subject: [PATCH 3/6] Updated documentation

---
 README.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/README.md b/README.md
index a98c171..973b531 100644
--- a/README.md
+++ b/README.md
@@ -15,6 +15,9 @@ You can download development versions on [Bintray](https://bintray.com/viniciusa
 ## Requirements
 * [Java SE Runtime Environment 8](http://www.oracle.com/technetwork/java/javase/downloads/jre8-downloads-2133155.html)
 * When using reporters for Sonar or Coveralls client needs to be invoked from project's root directory.
+* Due to Oracle license we can't ship the necessary oracle libraries directly with utPLSQL-cli. <b>Please download the libraries directly from oracle website and put the jars into the "lib" folder of your utPLSQL-cli installation</b>
+  * Oracle JDBC driver: http://www.oracle.com/technetwork/database/features/jdbc/jdbc-ucp-122-3110062.html
+  * If you are on a 11g database you might need the orai18n library, too: http://www.oracle.com/technetwork/database/enterprise-edition/jdbc-112010-090769.html
 
 ## Compatibility
 The latest CLI is always compatible with all database frameworks of the same major version.

From b972be1cc7fe56b602fc9fa77f445292956b49f8 Mon Sep 17 00:00:00 2001
From: pesse <derpesse@gmail.com>
Date: Thu, 16 Nov 2017 15:27:57 +0100
Subject: [PATCH 4/6] Using new API Compatibility check

Let CLI exit gracefully when versions not compatible
---
 src/main/java/org/utplsql/cli/Cli.java        |  3 +++
 src/main/java/org/utplsql/cli/RunCommand.java | 11 ++++-------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/utplsql/cli/Cli.java b/src/main/java/org/utplsql/cli/Cli.java
index 4ea8652..c173ae3 100644
--- a/src/main/java/org/utplsql/cli/Cli.java
+++ b/src/main/java/org/utplsql/cli/Cli.java
@@ -3,6 +3,7 @@
 import com.beust.jcommander.JCommander;
 import com.beust.jcommander.Parameter;
 import com.beust.jcommander.ParameterException;
+import org.utplsql.api.exception.DatabaseNotCompatibleException;
 
 public class Cli {
 
@@ -34,6 +35,8 @@ public static void main(String[] args) {
             } else {
                 jc.usage();
             }
+        } catch ( DatabaseNotCompatibleException e ) {
+            System.out.println(e.getMessage());
         } catch (Exception e) {
             e.printStackTrace();
         }
diff --git a/src/main/java/org/utplsql/cli/RunCommand.java b/src/main/java/org/utplsql/cli/RunCommand.java
index b81e189..3d7dd50 100644
--- a/src/main/java/org/utplsql/cli/RunCommand.java
+++ b/src/main/java/org/utplsql/cli/RunCommand.java
@@ -3,6 +3,7 @@
 import com.beust.jcommander.Parameter;
 import com.beust.jcommander.Parameters;
 import org.utplsql.api.*;
+import org.utplsql.api.compatibility.CompatibilityProxy;
 import org.utplsql.api.exception.DatabaseNotCompatibleException;
 import org.utplsql.api.exception.SomeTestsFailedException;
 import org.utplsql.api.reporter.Reporter;
@@ -264,14 +265,10 @@ public List<ReporterOptions> getReporterOptionsList() {
      */
     private void checkFrameworkCompatibility(Connection conn) throws SQLException {
 
-        if ( !skipCompatibilityCheck ) {
-            try {
-                DBHelper.failOnVersionCompatibilityCheckFailed(conn);
-            } catch (DatabaseNotCompatibleException e) {
-                System.out.println(e.getMessage());
+        CompatibilityProxy proxy = new CompatibilityProxy(conn, skipCompatibilityCheck);
 
-                throw e;
-            }
+        if ( !skipCompatibilityCheck ) {
+            proxy.failOnNotCompatible();
         }
         else {
             System.out.println("Skipping Compatibility check with framework version, expecting the latest version " +

From c140c5e58e02b90689a91734d926df16586f0dc6 Mon Sep 17 00:00:00 2001
From: pesse <derpesse@gmail.com>
Date: Fri, 17 Nov 2017 10:32:32 +0100
Subject: [PATCH 5/6] Shutdown cli gracefully when known exceptions occur

Fixes #44
---
 src/main/java/org/utplsql/cli/Cli.java                 |  4 +++-
 src/main/java/org/utplsql/cli/RunCommand.java          |  9 +++++++--
 .../cli/exception/DatabaseConnectionFailed.java        | 10 ++++++++++
 3 files changed, 20 insertions(+), 3 deletions(-)
 create mode 100644 src/main/java/org/utplsql/cli/exception/DatabaseConnectionFailed.java

diff --git a/src/main/java/org/utplsql/cli/Cli.java b/src/main/java/org/utplsql/cli/Cli.java
index c173ae3..bdfbac3 100644
--- a/src/main/java/org/utplsql/cli/Cli.java
+++ b/src/main/java/org/utplsql/cli/Cli.java
@@ -4,6 +4,8 @@
 import com.beust.jcommander.Parameter;
 import com.beust.jcommander.ParameterException;
 import org.utplsql.api.exception.DatabaseNotCompatibleException;
+import org.utplsql.api.exception.UtPLSQLNotInstalledException;
+import org.utplsql.cli.exception.DatabaseConnectionFailed;
 
 public class Cli {
 
@@ -35,7 +37,7 @@ public static void main(String[] args) {
             } else {
                 jc.usage();
             }
-        } catch ( DatabaseNotCompatibleException e ) {
+        } catch ( DatabaseNotCompatibleException | UtPLSQLNotInstalledException | DatabaseConnectionFailed e ) {
             System.out.println(e.getMessage());
         } catch (Exception e) {
             e.printStackTrace();
diff --git a/src/main/java/org/utplsql/cli/RunCommand.java b/src/main/java/org/utplsql/cli/RunCommand.java
index 3d7dd50..a03045c 100644
--- a/src/main/java/org/utplsql/cli/RunCommand.java
+++ b/src/main/java/org/utplsql/cli/RunCommand.java
@@ -8,6 +8,7 @@
 import org.utplsql.api.exception.SomeTestsFailedException;
 import org.utplsql.api.reporter.Reporter;
 import org.utplsql.api.reporter.ReporterFactory;
+import org.utplsql.cli.exception.DatabaseConnectionFailed;
 
 import java.io.File;
 import java.io.FileNotFoundException;
@@ -120,8 +121,12 @@ public int run() throws Exception {
             reporterList = initReporters(conn, reporterOptionsList);
 
         } catch (SQLException e) {
-            System.out.println(e.getMessage());
-            return Cli.DEFAULT_ERROR_CODE;
+            if ( e.getErrorCode() == 1017 || e.getErrorCode() == 12514 ) {
+                throw new DatabaseConnectionFailed(e);
+            }
+            else {
+                throw e;
+            }
         }
 
         ExecutorService executorService = Executors.newFixedThreadPool(1 + reporterList.size());
diff --git a/src/main/java/org/utplsql/cli/exception/DatabaseConnectionFailed.java b/src/main/java/org/utplsql/cli/exception/DatabaseConnectionFailed.java
new file mode 100644
index 0000000..01b03db
--- /dev/null
+++ b/src/main/java/org/utplsql/cli/exception/DatabaseConnectionFailed.java
@@ -0,0 +1,10 @@
+package org.utplsql.cli.exception;
+
+import java.sql.SQLException;
+
+public class DatabaseConnectionFailed extends SQLException {
+
+    public DatabaseConnectionFailed(SQLException cause ) {
+        super( "Could not establish connection to database. Reason: " + cause.getMessage(), cause);
+    }
+}

From dd0dcdf2d67d6e61d920ab3e21d9e58632ba609b Mon Sep 17 00:00:00 2001
From: pesse <derpesse@gmail.com>
Date: Fri, 17 Nov 2017 11:00:22 +0100
Subject: [PATCH 6/6] Output warning when --failureExitCode is set and
 framework is not compatible with that option

also refactored some things and made run_Default-test work with framework versions < 3.0.3, too
---
 src/main/java/org/utplsql/cli/RunCommand.java | 50 ++++++++---------
 .../org/utplsql/cli/RunCommandChecker.java    | 54 +++++++++++++++++++
 .../java/org/utplsql/cli/RunCommandTest.java  |  8 ++-
 .../utplsql/cli/TestRunCommandChecker.java    | 19 +++++++
 4 files changed, 101 insertions(+), 30 deletions(-)
 create mode 100644 src/main/java/org/utplsql/cli/RunCommandChecker.java
 create mode 100644 src/test/java/org/utplsql/cli/TestRunCommandChecker.java

diff --git a/src/main/java/org/utplsql/cli/RunCommand.java b/src/main/java/org/utplsql/cli/RunCommand.java
index a03045c..690a453 100644
--- a/src/main/java/org/utplsql/cli/RunCommand.java
+++ b/src/main/java/org/utplsql/cli/RunCommand.java
@@ -4,6 +4,7 @@
 import com.beust.jcommander.Parameters;
 import org.utplsql.api.*;
 import org.utplsql.api.compatibility.CompatibilityProxy;
+import org.utplsql.api.compatibility.OptionalFeatures;
 import org.utplsql.api.exception.DatabaseNotCompatibleException;
 import org.utplsql.api.exception.SomeTestsFailedException;
 import org.utplsql.api.reporter.Reporter;
@@ -82,6 +83,8 @@ public class RunCommand {
                     "most actual. Use this if you use CLI with a development version of utPLSQL-framework")
     private boolean skipCompatibilityCheck = false;
 
+    private CompatibilityProxy compatibilityProxy;
+
     public ConnectionInfo getConnectionInfo() {
         return connectionInfoList.get(0);
     }
@@ -92,7 +95,7 @@ public List<String> getTestPaths() {
 
     public int run() throws Exception {
 
-        checkOracleJDBCExists();
+        RunCommandChecker.checkOracleJDBCExists();
 
         final ConnectionInfo ci = getConnectionInfo();
 
@@ -113,10 +116,10 @@ public int run() throws Exception {
         try (Connection conn = ci.getConnection()) {
 
             // Check if orai18n exists if database version is 11g
-            checkOracleI18nExists(ci.getOracleDatabaseVersion(conn));
+            RunCommandChecker.checkOracleI18nExists(ci.getOracleDatabaseVersion(conn));
 
             // First of all do a compatibility check and fail-fast
-            checkFrameworkCompatibility(conn);
+            compatibilityProxy = checkFrameworkCompatibility(conn);
 
             reporterList = initReporters(conn, reporterOptionsList);
 
@@ -129,6 +132,12 @@ public int run() throws Exception {
             }
         }
 
+        // Output a message if --failureExitCode is set but database framework is not capable of
+        String msg = RunCommandChecker.getCheckFailOnErrorMessage(failureExitCode, compatibilityProxy.getDatabaseVersion());
+        if ( msg != null ) {
+            System.out.println(msg);
+        }
+
         ExecutorService executorService = Executors.newFixedThreadPool(1 + reporterList.size());
 
         // Run tests.
@@ -268,7 +277,7 @@ public List<ReporterOptions> getReporterOptionsList() {
      * @param conn Active Connection
      * @throws SQLException
      */
-    private void checkFrameworkCompatibility(Connection conn) throws SQLException {
+    private CompatibilityProxy checkFrameworkCompatibility(Connection conn) throws SQLException {
 
         CompatibilityProxy proxy = new CompatibilityProxy(conn, skipCompatibilityCheck);
 
@@ -279,6 +288,8 @@ private void checkFrameworkCompatibility(Connection conn) throws SQLException {
             System.out.println("Skipping Compatibility check with framework version, expecting the latest version " +
                     "to be installed in database");
         }
+
+        return proxy;
     }
 
     public FileMapperOptions getMapperOptions(List<String> mappingParams, List<String> filePaths) {
@@ -328,33 +339,14 @@ public FileMapperOptions getMapperOptions(List<String> mappingParams, List<Strin
         return mapperOptions;
     }
 
-
-    /** Checks that ojdbc library exists
+    /** Returns the version of the database framework if available
      *
+     * @return
      */
-    private void checkOracleJDBCExists()
-    {
-        if ( !OracleLibraryChecker.checkOjdbcExists() )
-        {
-            System.out.println("Could not find Oracle JDBC driver in classpath. Please download the jar from Oracle website" +
-                    " and copy it to the 'lib' folder of your utPLSQL-cli installation.");
-            System.out.println("Download from http://www.oracle.com/technetwork/database/features/jdbc/jdbc-ucp-122-3110062.html");
-
-            throw new RuntimeException("Can't run utPLSQL-cli without Oracle JDBC driver");
-        }
-    }
+    public Version getDatabaseVersion() {
+        if ( compatibilityProxy != null )
+            return compatibilityProxy.getDatabaseVersion();
 
-    /** Checks that orai18n library exists if database is an oracle 11
-     *
-     */
-    private void checkOracleI18nExists(String oracleDatabaseVersion )
-    {
-        if ( oracleDatabaseVersion.startsWith("11.") && !OracleLibraryChecker.checkOrai18nExists() )
-        {
-            System.out.println("Warning: Could not find Oracle i18n driver in classpath. Depending on the database charset " +
-                    "utPLSQL-cli might not run properly. It is recommended you download " +
-                    "the i18n driver from the Oracle website and copy it to the 'lib' folder of your utPLSQL-cli installation.");
-            System.out.println("Download from http://www.oracle.com/technetwork/database/enterprise-edition/jdbc-112010-090769.html");
-        }
+        return null;
     }
 }
diff --git a/src/main/java/org/utplsql/cli/RunCommandChecker.java b/src/main/java/org/utplsql/cli/RunCommandChecker.java
new file mode 100644
index 0000000..e0c1bb2
--- /dev/null
+++ b/src/main/java/org/utplsql/cli/RunCommandChecker.java
@@ -0,0 +1,54 @@
+package org.utplsql.cli;
+
+import org.utplsql.api.Version;
+import org.utplsql.api.compatibility.OptionalFeatures;
+
+/** Helper class to check several circumstances with RunCommand. Might need refactoring.
+ *
+ * @author pesse
+ */
+class RunCommandChecker {
+
+    /** Checks that ojdbc library exists
+     *
+     */
+    static void checkOracleJDBCExists()
+    {
+        if ( !OracleLibraryChecker.checkOjdbcExists() )
+        {
+            System.out.println("Could not find Oracle JDBC driver in classpath. Please download the jar from Oracle website" +
+                    " and copy it to the 'lib' folder of your utPLSQL-cli installation.");
+            System.out.println("Download from http://www.oracle.com/technetwork/database/features/jdbc/jdbc-ucp-122-3110062.html");
+
+            throw new RuntimeException("Can't run utPLSQL-cli without Oracle JDBC driver");
+        }
+    }
+
+    /** Checks that orai18n library exists if database is an oracle 11
+     *
+     */
+    static void checkOracleI18nExists(String oracleDatabaseVersion )
+    {
+        if ( oracleDatabaseVersion.startsWith("11.") && !OracleLibraryChecker.checkOrai18nExists() )
+        {
+            System.out.println("Warning: Could not find Oracle i18n driver in classpath. Depending on the database charset " +
+                    "utPLSQL-cli might not run properly. It is recommended you download " +
+                    "the i18n driver from the Oracle website and copy it to the 'lib' folder of your utPLSQL-cli installation.");
+            System.out.println("Download from http://www.oracle.com/technetwork/database/enterprise-edition/jdbc-112010-090769.html");
+        }
+    }
+
+    /** Returns a warning message if failureExitCode is specified but database version is too low
+     *
+     * @param failureExitCode
+     * @param databaseVersion
+     */
+    static String getCheckFailOnErrorMessage(int failureExitCode, Version databaseVersion) {
+        if ( failureExitCode != 1 && !OptionalFeatures.FAIL_ON_ERROR.isAvailableFor(databaseVersion)) {
+            return "Your database framework version (" + databaseVersion.getNormalizedString() + ") is not able to " +
+                    "redirect failureCodes. Please upgrade to a newer version if you want to use that feature.";
+        }
+
+        return null;
+    }
+}
diff --git a/src/test/java/org/utplsql/cli/RunCommandTest.java b/src/test/java/org/utplsql/cli/RunCommandTest.java
index 7f976b0..ff72b13 100644
--- a/src/test/java/org/utplsql/cli/RunCommandTest.java
+++ b/src/test/java/org/utplsql/cli/RunCommandTest.java
@@ -4,6 +4,7 @@
 import org.junit.Assert;
 import org.junit.Test;
 import org.utplsql.api.CustomTypes;
+import org.utplsql.api.compatibility.OptionalFeatures;
 
 import java.util.List;
 
@@ -119,7 +120,12 @@ public void run_Default() {
 
         try {
             int result = runCmd.run();
-            Assert.assertEquals(2, result);
+
+            // Only expect failure-exit-code to work on several framework versions
+            if (OptionalFeatures.FAIL_ON_ERROR.isAvailableFor(runCmd.getDatabaseVersion()) )
+                Assert.assertEquals(2, result);
+            else
+                Assert.assertEquals(0, result);
         }
         catch ( Exception e ) {
             Assert.fail(e.getMessage());
diff --git a/src/test/java/org/utplsql/cli/TestRunCommandChecker.java b/src/test/java/org/utplsql/cli/TestRunCommandChecker.java
new file mode 100644
index 0000000..fe807de
--- /dev/null
+++ b/src/test/java/org/utplsql/cli/TestRunCommandChecker.java
@@ -0,0 +1,19 @@
+package org.utplsql.cli;
+
+import org.junit.Assert;
+import org.junit.Test;
+import org.utplsql.api.Version;
+
+public class TestRunCommandChecker {
+
+    @Test
+    public void getCheckFailOnErrorMessage()
+    {
+        // FailOnError option should work since 3.0.3+ framework
+        Assert.assertNotNull(RunCommandChecker.getCheckFailOnErrorMessage(2, new Version("3.0.0")));
+        Assert.assertNotNull(RunCommandChecker.getCheckFailOnErrorMessage(2, new Version("3.0.1")));
+        Assert.assertNotNull(RunCommandChecker.getCheckFailOnErrorMessage(2, new Version("3.0.2")));
+        Assert.assertNull(RunCommandChecker.getCheckFailOnErrorMessage(2, new Version("3.0.3")));
+        Assert.assertNull(RunCommandChecker.getCheckFailOnErrorMessage(2, new Version("3.0.4")));
+    }
+}