From f19e2a523579db8c28a96febfd3cf35a5d58adc6 Mon Sep 17 00:00:00 2001
From: Venkat Ranganathan <venkat@hortonworks.com>
Date: Wed, 16 Dec 2015 20:49:31 -0800
Subject: [PATCH] SQOOP-2747: Allow customizing test username and password for
 Oracle tests   (Jarek Jarcec Cecho via Venkat Ranganathan)

---
 build.xml                                            | 11 +++++++++--
 src/test/com/cloudera/sqoop/manager/OracleUtils.java |  8 +++-----
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/build.xml b/build.xml
index 148cc6d69..97e550283 100644
--- a/build.xml
+++ b/build.xml
@@ -350,8 +350,9 @@
       value="${name}/[conf]/[artifact]-[revision](-[classifier]).[ext]"/>
 
   <!--test related properties -->
-  <property name="sqoop.test.oracle.connectstring"
-            value="jdbc:oracle:thin:@//localhost/xe"/>
+  <property name="sqoop.test.oracle.connectstring" value="jdbc:oracle:thin:@//localhost/xe"/>
+  <property name="sqoop.test.oracle.username" value="SQOOPTEST"/>
+  <property name="sqoop.test.oracle.password" value="12345"/>
 
   <property name="sqoop.test.mysql.connectstring.host_url"
             value="jdbc:mysql://localhost/"/>
@@ -954,6 +955,12 @@
       <sysproperty key="sqoop.test.oracle.connectstring"
                    value="${sqoop.test.oracle.connectstring}"/>
 
+      <sysproperty key="sqoop.test.oracle.username"
+                   value="${sqoop.test.oracle.username}"/>
+
+      <sysproperty key="sqoop.test.oracle.password"
+                   value="${sqoop.test.oracle.password}"/>
+
       <sysproperty key="sqoop.test.mysql.connectstring.host_url"
                    value="${sqoop.test.mysql.connectstring.host_url}"/>
 
diff --git a/src/test/com/cloudera/sqoop/manager/OracleUtils.java b/src/test/com/cloudera/sqoop/manager/OracleUtils.java
index d9e31b8bb..655aa61f4 100644
--- a/src/test/com/cloudera/sqoop/manager/OracleUtils.java
+++ b/src/test/com/cloudera/sqoop/manager/OracleUtils.java
@@ -37,11 +37,9 @@ public final class OracleUtils {
   // Express edition hardcoded name.
   public static final String ORACLE_DATABASE_NAME = "xe";
 
-  public static final String CONNECT_STRING =
-      System.getProperty("sqoop.test.oracle.connectstring",
-      "jdbc:oracle:thin:@//localhost/" + ORACLE_DATABASE_NAME);
-  public static final String ORACLE_USER_NAME = "SQOOPTEST";
-  public static final String ORACLE_USER_PASS = "12345";
+  public static final String CONNECT_STRING = System.getProperty("sqoop.test.oracle.connectstring", "jdbc:oracle:thin:@//localhost/" + ORACLE_DATABASE_NAME);
+  public static final String ORACLE_USER_NAME = System.getProperty("sqoop.test.oracle.username", "SQOOPTEST");
+  public static final String ORACLE_USER_PASS = System.getProperty("sqoop.test.oracle.password", "12345");
 
   public static final String ORACLE_SECONDARY_USER_NAME = "SQOOPTEST2";
   public static final String ORACLE_SECONDARY_USER_PASS = "ABCDEF";