You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ab...@apache.org on 2014/08/12 00:15:36 UTC

[16/17] SQOOP-1379: Sqoop2: From/To: Disable tests

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestFormUtils.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestFormUtils.java b/common/src/test/java/org/apache/sqoop/model/TestFormUtils.java
index 08dfa7b..6c76347 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestFormUtils.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestFormUtils.java
@@ -32,218 +32,218 @@ import java.util.Map;
  */
 public class TestFormUtils extends TestCase {
 
-  public void testToForms() {
-    Config config = new Config();
-    config.aForm.a1 = "value";
-
-    List<MForm> formsByInstance = FormUtils.toForms(config);
-    assertEquals(getForms(), formsByInstance);
-    assertEquals("value", formsByInstance.get(0).getInputs().get(0).getValue());
-
-    List<MForm> formsByClass = FormUtils.toForms(Config.class);
-    assertEquals(getForms(), formsByClass);
-
-    List<MForm> formsByBoth = FormUtils.toForms(Config.class, config);
-    assertEquals(getForms(), formsByBoth);
-    assertEquals("value", formsByBoth.get(0).getInputs().get(0).getValue());
-  }
-
-  public void testToFormsMissingAnnotation() {
-    try {
-      FormUtils.toForms(ConfigWithout.class);
-    } catch(SqoopException ex) {
-      assertEquals(ModelError.MODEL_003, ex.getErrorCode());
-      return;
-    }
-
-    fail("Correct exception wasn't thrown");
-  }
-
-  public void testFailureOnPrimitiveType() {
-    PrimitiveConfig config = new PrimitiveConfig();
-
-    try {
-      FormUtils.toForms(config);
-      fail("We were expecting exception for unsupported type.");
-    } catch(SqoopException ex) {
-      assertEquals(ModelError.MODEL_007, ex.getErrorCode());
-    }
-  }
-
-  public void testFillValues() {
-    List<MForm> forms = getForms();
-
-    ((MStringInput)forms.get(0).getInputs().get(0)).setValue("value");
-
-    Config config = new Config();
-
-    FormUtils.fromForms(forms, config);
-    assertEquals("value", config.aForm.a1);
-  }
-
-  public void testFillValuesObjectReuse() {
-    List<MForm> forms = getForms();
-
-    ((MStringInput)forms.get(0).getInputs().get(0)).setValue("value");
-
-    Config config = new Config();
-    config.aForm.a2 = "x";
-    config.bForm.b1 = "y";
-
-    FormUtils.fromForms(forms, config);
-    assertEquals("value", config.aForm.a1);
-    assertNull(config.aForm.a2);
-    assertNull(config.bForm.b2);
-    assertNull(config.bForm.b2);
-  }
-
-  public void testApplyValidation() {
-    Validation validation = getValidation();
-    List<MForm> forms = getForms();
-
-    FormUtils.applyValidation(forms, validation);
-
-    assertEquals(Status.ACCEPTABLE,
-      forms.get(0).getInputs().get(0).getValidationStatus());
-    assertEquals("e1",
-      forms.get(0).getInputs().get(0).getValidationMessage());
-
-    assertEquals(Status.UNACCEPTABLE,
-      forms.get(0).getInputs().get(1).getValidationStatus());
-    assertEquals("e2",
-      forms.get(0).getInputs().get(1).getValidationMessage());
-  }
-
-  public void testJson() {
-    Config config = new Config();
-    config.aForm.a1 = "A";
-    config.bForm.b2 = "B";
-    config.cForm.intValue = 4;
-    config.cForm.map.put("C", "D");
-    config.cForm.enumeration = Enumeration.X;
-
-    String json = FormUtils.toJson(config);
-
-    Config targetConfig = new Config();
-
-    // Old values from should be always removed
-    targetConfig.aForm.a2 = "X";
-    targetConfig.bForm.b1 = "Y";
-    // Nulls in forms shouldn't be an issue either
-    targetConfig.cForm = null;
-
-    FormUtils.fillValues(json, targetConfig);
-
-    assertEquals("A", targetConfig.aForm.a1);
-    assertNull(targetConfig.aForm.a2);
-
-    assertNull(targetConfig.bForm.b1);
-    assertEquals("B", targetConfig.bForm.b2);
-
-    assertEquals((Integer)4, targetConfig.cForm.intValue);
-    assertEquals(1, targetConfig.cForm.map.size());
-    assertTrue(targetConfig.cForm.map.containsKey("C"));
-    assertEquals("D", targetConfig.cForm.map.get("C"));
-    assertEquals(Enumeration.X, targetConfig.cForm.enumeration);
-  }
-
-  protected Validation getValidation() {
-    Map<Validation.FormInput, Validation.Message> messages
-      = new HashMap<Validation.FormInput, Validation.Message>();
-
-    messages.put(
-      new Validation.FormInput("aForm", "a1"),
-      new Validation.Message(Status.ACCEPTABLE, "e1"));
-    messages.put(
-      new Validation.FormInput("aForm", "a2"),
-      new Validation.Message(Status.UNACCEPTABLE, "e2"));
-
-    return new Validation(Status.UNACCEPTABLE, messages);
-  }
-
-  /**
-   * Form structure that corresponds to Config class declared below
-   * @return Form structure
-   */
-  protected List<MForm> getForms() {
-    List<MForm> ret = new LinkedList<MForm>();
-
-    List<MInput<?>> inputs;
-
-    // Form A
-    inputs = new LinkedList<MInput<?>>();
-    inputs.add(new MStringInput("aForm.a1", false, (short)30));
-    inputs.add(new MStringInput("aForm.a2", true, (short)-1));
-    ret.add(new MForm("aForm", inputs));
-
-    // Form B
-    inputs = new LinkedList<MInput<?>>();
-    inputs.add(new MStringInput("bForm.b1", false, (short)2));
-    inputs.add(new MStringInput("bForm.b2", false, (short)3));
-    ret.add(new MForm("bForm", inputs));
-
-    // Form C
-    inputs = new LinkedList<MInput<?>>();
-    inputs.add(new MIntegerInput("cForm.intValue", false));
-    inputs.add(new MMapInput("cForm.map", false));
-    inputs.add(new MEnumInput("cForm.enumeration", false, new String[]{"X", "Y"}));
-    ret.add(new MForm("cForm", inputs));
-
-    return ret;
-  }
-
-  @ConfigurationClass
-  public static class Config {
-
-    public Config() {
-      aForm = new AForm();
-      bForm = new BForm();
-      cForm = new CForm();
-    }
-
-    @Form AForm aForm;
-    @Form BForm bForm;
-    @Form CForm cForm;
-  }
-
-  @ConfigurationClass
-  public static class PrimitiveConfig {
-    @Form DForm dForm;
-  }
-
-  @FormClass
-  public static class AForm {
-    @Input(size = 30)  String a1;
-    @Input(sensitive = true)  String a2;
-  }
-
-  @FormClass
-  public static class BForm {
-    @Input(size = 2) String b1;
-    @Input(size = 3) String b2;
-  }
-
-  @FormClass
-  public static class CForm {
-    @Input Integer intValue;
-    @Input Map<String, String> map;
-    @Input Enumeration enumeration;
-
-    public CForm() {
-      map = new HashMap<String, String>();
-    }
-  }
-
-  @FormClass
-  public static class DForm {
-    @Input int value;
-  }
-
-  public static class ConfigWithout {
-  }
-
-  enum Enumeration {
-    X,
-    Y,
-  }
+//  public void testToForms() {
+//    Config config = new Config();
+//    config.aForm.a1 = "value";
+//
+//    List<MForm> formsByInstance = FormUtils.toForms(config);
+//    assertEquals(getForms(), formsByInstance);
+//    assertEquals("value", formsByInstance.get(0).getInputs().get(0).getValue());
+//
+//    List<MForm> formsByClass = FormUtils.toForms(Config.class);
+//    assertEquals(getForms(), formsByClass);
+//
+//    List<MForm> formsByBoth = FormUtils.toForms(Config.class, config);
+//    assertEquals(getForms(), formsByBoth);
+//    assertEquals("value", formsByBoth.get(0).getInputs().get(0).getValue());
+//  }
+//
+//  public void testToFormsMissingAnnotation() {
+//    try {
+//      FormUtils.toForms(ConfigWithout.class);
+//    } catch(SqoopException ex) {
+//      assertEquals(ModelError.MODEL_003, ex.getErrorCode());
+//      return;
+//    }
+//
+//    fail("Correct exception wasn't thrown");
+//  }
+//
+//  public void testFailureOnPrimitiveType() {
+//    PrimitiveConfig config = new PrimitiveConfig();
+//
+//    try {
+//      FormUtils.toForms(config);
+//      fail("We were expecting exception for unsupported type.");
+//    } catch(SqoopException ex) {
+//      assertEquals(ModelError.MODEL_007, ex.getErrorCode());
+//    }
+//  }
+//
+//  public void testFillValues() {
+//    List<MForm> forms = getForms();
+//
+//    ((MStringInput)forms.get(0).getInputs().get(0)).setValue("value");
+//
+//    Config config = new Config();
+//
+//    FormUtils.fromForms(forms, config);
+//    assertEquals("value", config.aForm.a1);
+//  }
+//
+//  public void testFillValuesObjectReuse() {
+//    List<MForm> forms = getForms();
+//
+//    ((MStringInput)forms.get(0).getInputs().get(0)).setValue("value");
+//
+//    Config config = new Config();
+//    config.aForm.a2 = "x";
+//    config.bForm.b1 = "y";
+//
+//    FormUtils.fromForms(forms, config);
+//    assertEquals("value", config.aForm.a1);
+//    assertNull(config.aForm.a2);
+//    assertNull(config.bForm.b2);
+//    assertNull(config.bForm.b2);
+//  }
+//
+//  public void testApplyValidation() {
+//    Validation validation = getValidation();
+//    List<MForm> forms = getForms();
+//
+//    FormUtils.applyValidation(forms, validation);
+//
+//    assertEquals(Status.ACCEPTABLE,
+//      forms.get(0).getInputs().get(0).getValidationStatus());
+//    assertEquals("e1",
+//      forms.get(0).getInputs().get(0).getValidationMessage());
+//
+//    assertEquals(Status.UNACCEPTABLE,
+//      forms.get(0).getInputs().get(1).getValidationStatus());
+//    assertEquals("e2",
+//      forms.get(0).getInputs().get(1).getValidationMessage());
+//  }
+//
+//  public void testJson() {
+//    Config config = new Config();
+//    config.aForm.a1 = "A";
+//    config.bForm.b2 = "B";
+//    config.cForm.intValue = 4;
+//    config.cForm.map.put("C", "D");
+//    config.cForm.enumeration = Enumeration.X;
+//
+//    String json = FormUtils.toJson(config);
+//
+//    Config targetConfig = new Config();
+//
+//    // Old values from should be always removed
+//    targetConfig.aForm.a2 = "X";
+//    targetConfig.bForm.b1 = "Y";
+//    // Nulls in forms shouldn't be an issue either
+//    targetConfig.cForm = null;
+//
+//    FormUtils.fillValues(json, targetConfig);
+//
+//    assertEquals("A", targetConfig.aForm.a1);
+//    assertNull(targetConfig.aForm.a2);
+//
+//    assertNull(targetConfig.bForm.b1);
+//    assertEquals("B", targetConfig.bForm.b2);
+//
+//    assertEquals((Integer)4, targetConfig.cForm.intValue);
+//    assertEquals(1, targetConfig.cForm.map.size());
+//    assertTrue(targetConfig.cForm.map.containsKey("C"));
+//    assertEquals("D", targetConfig.cForm.map.get("C"));
+//    assertEquals(Enumeration.X, targetConfig.cForm.enumeration);
+//  }
+//
+//  protected Validation getValidation() {
+//    Map<Validation.FormInput, Validation.Message> messages
+//      = new HashMap<Validation.FormInput, Validation.Message>();
+//
+//    messages.put(
+//      new Validation.FormInput("aForm", "a1"),
+//      new Validation.Message(Status.ACCEPTABLE, "e1"));
+//    messages.put(
+//      new Validation.FormInput("aForm", "a2"),
+//      new Validation.Message(Status.UNACCEPTABLE, "e2"));
+//
+//    return new Validation(Status.UNACCEPTABLE, messages);
+//  }
+//
+//  /**
+//   * Form structure that corresponds to Config class declared below
+//   * @return Form structure
+//   */
+//  protected List<MForm> getForms() {
+//    List<MForm> ret = new LinkedList<MForm>();
+//
+//    List<MInput<?>> inputs;
+//
+//    // Form A
+//    inputs = new LinkedList<MInput<?>>();
+//    inputs.add(new MStringInput("aForm.a1", false, (short)30));
+//    inputs.add(new MStringInput("aForm.a2", true, (short)-1));
+//    ret.add(new MForm("aForm", inputs));
+//
+//    // Form B
+//    inputs = new LinkedList<MInput<?>>();
+//    inputs.add(new MStringInput("bForm.b1", false, (short)2));
+//    inputs.add(new MStringInput("bForm.b2", false, (short)3));
+//    ret.add(new MForm("bForm", inputs));
+//
+//    // Form C
+//    inputs = new LinkedList<MInput<?>>();
+//    inputs.add(new MIntegerInput("cForm.intValue", false));
+//    inputs.add(new MMapInput("cForm.map", false));
+//    inputs.add(new MEnumInput("cForm.enumeration", false, new String[]{"X", "Y"}));
+//    ret.add(new MForm("cForm", inputs));
+//
+//    return ret;
+//  }
+//
+//  @ConfigurationClass
+//  public static class Config {
+//
+//    public Config() {
+//      aForm = new AForm();
+//      bForm = new BForm();
+//      cForm = new CForm();
+//    }
+//
+//    @Form AForm aForm;
+//    @Form BForm bForm;
+//    @Form CForm cForm;
+//  }
+//
+//  @ConfigurationClass
+//  public static class PrimitiveConfig {
+//    @Form DForm dForm;
+//  }
+//
+//  @FormClass
+//  public static class AForm {
+//    @Input(size = 30)  String a1;
+//    @Input(sensitive = true)  String a2;
+//  }
+//
+//  @FormClass
+//  public static class BForm {
+//    @Input(size = 2) String b1;
+//    @Input(size = 3) String b2;
+//  }
+//
+//  @FormClass
+//  public static class CForm {
+//    @Input Integer intValue;
+//    @Input Map<String, String> map;
+//    @Input Enumeration enumeration;
+//
+//    public CForm() {
+//      map = new HashMap<String, String>();
+//    }
+//  }
+//
+//  @FormClass
+//  public static class DForm {
+//    @Input int value;
+//  }
+//
+//  public static class ConfigWithout {
+//  }
+//
+//  enum Enumeration {
+//    X,
+//    Y,
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMAccountableEntity.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMAccountableEntity.java b/common/src/test/java/org/apache/sqoop/model/TestMAccountableEntity.java
index f3d4166..942a056 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMAccountableEntity.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMAccountableEntity.java
@@ -30,32 +30,32 @@ import org.junit.Test;
  */
 public class TestMAccountableEntity {
 
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    MForm form = new MForm("FORMNAME", list);
-    forms.add(form);
-    MAccountableEntity connection = new MConnection(123l, new MConnectionForms(
-        forms), new MConnectionForms(forms));
-    // Initially creation date and last update date is same
-    assertEquals(connection.getCreationDate(), connection.getLastUpdateDate());
-    Date testCreationDate = new Date();
-    Date testLastUpdateDate = new Date();
-    connection.setCreationUser("admin");
-    connection.setCreationDate(testCreationDate);
-    connection.setLastUpdateUser("user");
-    connection.setLastUpdateDate(testLastUpdateDate);
-    connection.setEnabled(false);
-    assertEquals(testCreationDate, connection.getCreationDate());
-    assertEquals("admin", connection.getCreationUser());
-    assertEquals(testLastUpdateDate, connection.getLastUpdateDate());
-    assertEquals(false, connection.getEnabled());
-    assertEquals("user", connection.getLastUpdateUser());
-  }
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    MForm form = new MForm("FORMNAME", list);
+//    forms.add(form);
+//    MAccountableEntity connection = new MConnection(123l, new MConnectionForms(
+//        forms), new MConnectionForms(forms));
+//    // Initially creation date and last update date is same
+//    assertEquals(connection.getCreationDate(), connection.getLastUpdateDate());
+//    Date testCreationDate = new Date();
+//    Date testLastUpdateDate = new Date();
+//    connection.setCreationUser("admin");
+//    connection.setCreationDate(testCreationDate);
+//    connection.setLastUpdateUser("user");
+//    connection.setLastUpdateDate(testLastUpdateDate);
+//    connection.setEnabled(false);
+//    assertEquals(testCreationDate, connection.getCreationDate());
+//    assertEquals("admin", connection.getCreationUser());
+//    assertEquals(testLastUpdateDate, connection.getLastUpdateDate());
+//    assertEquals(false, connection.getEnabled());
+//    assertEquals("user", connection.getLastUpdateUser());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMBooleanInput.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMBooleanInput.java b/common/src/test/java/org/apache/sqoop/model/TestMBooleanInput.java
index cf9cf24..b955aa4 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMBooleanInput.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMBooleanInput.java
@@ -30,70 +30,70 @@ import static org.junit.Assert.assertTrue;
  */
 public class TestMBooleanInput {
 
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    MBooleanInput input = new MBooleanInput("sqoopsqoop", true);
-    assertEquals("sqoopsqoop", input.getName());
-    assertEquals(true, input.isSensitive());
-    assertEquals(MInputType.BOOLEAN, input.getType());
-  }
-
-  /**
-   * Test for equals() method
-   */
-  @Test
-  public void testEquals() {
-    // Positive test
-    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
-    MBooleanInput input2 = new MBooleanInput("sqoopsqoop", true);
-    assertTrue(input1.equals(input2));
-
-    // Negative test
-    MBooleanInput input3 = new MBooleanInput("sqoopsqoop", false);
-    MBooleanInput input4 = new MBooleanInput("sqoopsqoop", true);
-    assertFalse(input3.equals(input4));
-
-    MBooleanInput input5 = new MBooleanInput("sqoopsqoop", false);
-    MBooleanInput input6 = new MBooleanInput("sqoop", false);
-    assertFalse(input5.equals(input6));
-  }
-
-  /**
-   * Test for value
-   */
-  @Test
-  public void testValue() {
-    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
-    input1.setValue(true);
-    assertEquals(true, input1.getValue());
-    input1.setEmpty();
-    assertNull(input1.getValue());
-  }
-
-  /**
-   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
-   */
-  @Test
-  public void testUrlSafe() {
-    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
-    input1.setValue(true);
-    // Getting URL safe string
-    String tmp = input1.getUrlSafeValueString();
-    // Restore to actual value
-    input1.restoreFromUrlSafeValueString(tmp);
-    assertEquals(true, input1.getValue());
-  }
-
-  /**
-   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
-   */
-  @Test
-  public void testNamedElement() {
-    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
-    assertEquals("sqoopsqoop.label", input1.getLabelKey());
-    assertEquals("sqoopsqoop.help", input1.getHelpKey());
-  }
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MBooleanInput input = new MBooleanInput("sqoopsqoop", true);
+//    assertEquals("sqoopsqoop", input.getName());
+//    assertEquals(true, input.isSensitive());
+//    assertEquals(MInputType.BOOLEAN, input.getType());
+//  }
+//
+//  /**
+//   * Test for equals() method
+//   */
+//  @Test
+//  public void testEquals() {
+//    // Positive test
+//    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
+//    MBooleanInput input2 = new MBooleanInput("sqoopsqoop", true);
+//    assertTrue(input1.equals(input2));
+//
+//    // Negative test
+//    MBooleanInput input3 = new MBooleanInput("sqoopsqoop", false);
+//    MBooleanInput input4 = new MBooleanInput("sqoopsqoop", true);
+//    assertFalse(input3.equals(input4));
+//
+//    MBooleanInput input5 = new MBooleanInput("sqoopsqoop", false);
+//    MBooleanInput input6 = new MBooleanInput("sqoop", false);
+//    assertFalse(input5.equals(input6));
+//  }
+//
+//  /**
+//   * Test for value
+//   */
+//  @Test
+//  public void testValue() {
+//    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
+//    input1.setValue(true);
+//    assertEquals(true, input1.getValue());
+//    input1.setEmpty();
+//    assertNull(input1.getValue());
+//  }
+//
+//  /**
+//   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
+//   */
+//  @Test
+//  public void testUrlSafe() {
+//    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
+//    input1.setValue(true);
+//    // Getting URL safe string
+//    String tmp = input1.getUrlSafeValueString();
+//    // Restore to actual value
+//    input1.restoreFromUrlSafeValueString(tmp);
+//    assertEquals(true, input1.getValue());
+//  }
+//
+//  /**
+//   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
+//   */
+//  @Test
+//  public void testNamedElement() {
+//    MBooleanInput input1 = new MBooleanInput("sqoopsqoop", true);
+//    assertEquals("sqoopsqoop.label", input1.getLabelKey());
+//    assertEquals("sqoopsqoop.help", input1.getHelpKey());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMConnection.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMConnection.java b/common/src/test/java/org/apache/sqoop/model/TestMConnection.java
index 27959fb..aa58f05 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMConnection.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMConnection.java
@@ -29,94 +29,94 @@ import org.junit.Test;
  */
 public class TestMConnection {
 
-  /**
-   * Test for initialization
-   */
-  @Test
-  public void testInitialization() {
-    // Test default constructor
-    MConnection connection = connection();
-    assertEquals(123l, connection.getConnectorId());
-    assertEquals("Vampire", connection.getName());
-    assertEquals("Buffy", connection.getCreationUser());
-    assertEquals(forms1(), connection.getConnectorPart());
-    assertEquals(forms2(), connection.getFrameworkPart());
-
-    // Test copy constructor
-    MConnection copy = new MConnection(connection);
-    assertEquals(123l, copy.getConnectorId());
-    assertEquals("Vampire", copy.getName());
-    assertEquals("Buffy", copy.getCreationUser());
-    assertEquals(connection.getCreationDate(), copy.getCreationDate());
-    assertEquals(forms1(), copy.getConnectorPart());
-    assertEquals(forms2(), copy.getFrameworkPart());
-
-    // Test constructor for metadata upgrade (the order of forms is different)
-    MConnection upgradeCopy = new MConnection(connection, forms2(), forms1());
-    assertEquals(123l, upgradeCopy.getConnectorId());
-    assertEquals("Vampire", upgradeCopy.getName());
-    assertEquals("Buffy", upgradeCopy.getCreationUser());
-    assertEquals(connection.getCreationDate(), upgradeCopy.getCreationDate());
-    assertEquals(forms2(), upgradeCopy.getConnectorPart());
-    assertEquals(forms1(), upgradeCopy.getFrameworkPart());
-  }
-
-  @Test
-  public void testClone() {
-    MConnection connection = connection();
-
-    // Clone without value
-    MConnection withoutValue = connection.clone(false);
-    assertEquals(connection, withoutValue);
-    assertEquals(MPersistableEntity.PERSISTANCE_ID_DEFAULT, withoutValue.getPersistenceId());
-    assertNull(withoutValue.getName());
-    assertNull(withoutValue.getCreationUser());
-    assertEquals(forms1(), withoutValue.getConnectorPart());
-    assertEquals(forms2(), withoutValue.getFrameworkPart());
-    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
-    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
-
-    // Clone with value
-    MConnection withValue = connection.clone(true);
-    assertEquals(connection, withValue);
-    assertEquals(connection.getPersistenceId(), withValue.getPersistenceId());
-    assertEquals(connection.getName(), withValue.getName());
-    assertEquals(connection.getCreationUser(), withValue.getCreationUser());
-    assertEquals(forms1(), withValue.getConnectorPart());
-    assertEquals(forms2(), withValue.getFrameworkPart());
-    assertEquals(100, withValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
-    assertEquals("TEST-VALUE", withValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
-  }
-
-  private MConnection connection() {
-    MConnection connection = new MConnection(123l, forms1(), forms2());
-    connection.setName("Vampire");
-    connection.setCreationUser("Buffy");
-    return connection;
-  }
-
-  private MConnectionForms forms1() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
-    input.setValue(100);
-    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
-    strInput.setValue("TEST-VALUE");
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    list.add(strInput);
-    MForm form = new MForm("FORMNAME", list);
-    forms.add(form);
-    return new MConnectionForms(forms);
-  }
-
-  private MConnectionForms forms2() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MMapInput input = new MMapInput("MAP-INPUT", false);
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    MForm form = new MForm("form", list);
-    forms.add(form);
-    return new MConnectionForms(forms);
-  }
+//  /**
+//   * Test for initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    // Test default constructor
+//    MConnection connection = connection();
+//    assertEquals(123l, connection.getConnectorId());
+//    assertEquals("Vampire", connection.getName());
+//    assertEquals("Buffy", connection.getCreationUser());
+//    assertEquals(forms1(), connection.getConnectorPart());
+//    assertEquals(forms2(), connection.getFrameworkPart());
+//
+//    // Test copy constructor
+//    MConnection copy = new MConnection(connection);
+//    assertEquals(123l, copy.getConnectorId());
+//    assertEquals("Vampire", copy.getName());
+//    assertEquals("Buffy", copy.getCreationUser());
+//    assertEquals(connection.getCreationDate(), copy.getCreationDate());
+//    assertEquals(forms1(), copy.getConnectorPart());
+//    assertEquals(forms2(), copy.getFrameworkPart());
+//
+//    // Test constructor for metadata upgrade (the order of forms is different)
+//    MConnection upgradeCopy = new MConnection(connection, forms2(), forms1());
+//    assertEquals(123l, upgradeCopy.getConnectorId());
+//    assertEquals("Vampire", upgradeCopy.getName());
+//    assertEquals("Buffy", upgradeCopy.getCreationUser());
+//    assertEquals(connection.getCreationDate(), upgradeCopy.getCreationDate());
+//    assertEquals(forms2(), upgradeCopy.getConnectorPart());
+//    assertEquals(forms1(), upgradeCopy.getFrameworkPart());
+//  }
+//
+//  @Test
+//  public void testClone() {
+//    MConnection connection = connection();
+//
+//    // Clone without value
+//    MConnection withoutValue = connection.clone(false);
+//    assertEquals(connection, withoutValue);
+//    assertEquals(MPersistableEntity.PERSISTANCE_ID_DEFAULT, withoutValue.getPersistenceId());
+//    assertNull(withoutValue.getName());
+//    assertNull(withoutValue.getCreationUser());
+//    assertEquals(forms1(), withoutValue.getConnectorPart());
+//    assertEquals(forms2(), withoutValue.getFrameworkPart());
+//    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
+//    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
+//
+//    // Clone with value
+//    MConnection withValue = connection.clone(true);
+//    assertEquals(connection, withValue);
+//    assertEquals(connection.getPersistenceId(), withValue.getPersistenceId());
+//    assertEquals(connection.getName(), withValue.getName());
+//    assertEquals(connection.getCreationUser(), withValue.getCreationUser());
+//    assertEquals(forms1(), withValue.getConnectorPart());
+//    assertEquals(forms2(), withValue.getFrameworkPart());
+//    assertEquals(100, withValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
+//    assertEquals("TEST-VALUE", withValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
+//  }
+//
+//  private MConnection connection() {
+//    MConnection connection = new MConnection(123l, forms1(), forms2());
+//    connection.setName("Vampire");
+//    connection.setCreationUser("Buffy");
+//    return connection;
+//  }
+//
+//  private MConnectionForms forms1() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
+//    input.setValue(100);
+//    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
+//    strInput.setValue("TEST-VALUE");
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    list.add(strInput);
+//    MForm form = new MForm("FORMNAME", list);
+//    forms.add(form);
+//    return new MConnectionForms(forms);
+//  }
+//
+//  private MConnectionForms forms2() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MMapInput input = new MMapInput("MAP-INPUT", false);
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    MForm form = new MForm("form", list);
+//    forms.add(form);
+//    return new MConnectionForms(forms);
+//  }
 
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMConnectionForms.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMConnectionForms.java b/common/src/test/java/org/apache/sqoop/model/TestMConnectionForms.java
index e2d2717..0899dc3 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMConnectionForms.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMConnectionForms.java
@@ -29,20 +29,20 @@ import org.junit.Test;
  */
 public class TestMConnectionForms {
 
-  /**
-   * Test for class initialization and values
-   */
-  @Test
-  public void testInitialization() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MConnectionForms connectionForms1 = new MConnectionForms(forms);
-    List<MForm> testForms = new ArrayList<MForm>();
-    assertEquals(testForms, connectionForms1.getForms());
-    MConnectionForms connectionForms2 = new MConnectionForms(testForms);
-    assertEquals(connectionForms2, connectionForms1);
-    // Add a form to list for checking not equals
-    MForm m = new MForm("test", null);
-    testForms.add(m);
-    assertFalse(connectionForms1.equals(connectionForms2));
-  }
+//  /**
+//   * Test for class initialization and values
+//   */
+//  @Test
+//  public void testInitialization() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MConnectionForms connectionForms1 = new MConnectionForms(forms);
+//    List<MForm> testForms = new ArrayList<MForm>();
+//    assertEquals(testForms, connectionForms1.getForms());
+//    MConnectionForms connectionForms2 = new MConnectionForms(testForms);
+//    assertEquals(connectionForms2, connectionForms1);
+//    // Add a form to list for checking not equals
+//    MForm m = new MForm("test", null);
+//    testForms.add(m);
+//    assertFalse(connectionForms1.equals(connectionForms2));
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMConnector.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMConnector.java b/common/src/test/java/org/apache/sqoop/model/TestMConnector.java
index f3ca317..b94c7de 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMConnector.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMConnector.java
@@ -29,83 +29,83 @@ import org.junit.Test;
  */
 public class TestMConnector {
 
-  /**
-   * Test for initialization
-   */
-  @Test
-  public void testInitialization() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MConnectionForms connectionForms1 = new MConnectionForms(forms);
-    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
-    List<MJobForms> jobFormList = new ArrayList<MJobForms>();
-    jobFormList.add(jobform1);
-    MConnector connector1 = new MConnector("NAME", "CLASSNAME", "1.0",
-        connectionForms1, jobFormList);
-    assertEquals("NAME", connector1.getUniqueName());
-    assertEquals("CLASSNAME", connector1.getClassName());
-    assertEquals("1.0", connector1.getVersion());
-    MConnector connector2 = new MConnector("NAME", "CLASSNAME", "1.0",
-        connectionForms1, jobFormList);
-    assertEquals(connector2, connector1);
-    MConnector connector3 = new MConnector("NAME1", "CLASSNAME", "2.0",
-        connectionForms1, jobFormList);
-    assertFalse(connector1.equals(connector3));
-
-    try {
-      connector1 = new MConnector(null, "CLASSNAME", "1.0", connectionForms1,
-          jobFormList); // Expecting null pointer exception
-    } catch (NullPointerException e) {
-      assertTrue(true);
-    }
-    try {
-      connector1 = new MConnector("NAME", null, "1.0", connectionForms1,
-          jobFormList); // Expecting null pointer exception
-    } catch (NullPointerException e) {
-      assertTrue(true);
-    }
-  }
-
-  @Test
-  public void testClone() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
-    input.setValue(100);
-    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
-    strInput.setValue("TEST-VALUE");
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    list.add(strInput);
-    MForm form = new MForm("FORMNAME", list);
-    forms.add(form);
-    MConnectionForms connectionForms1 = new MConnectionForms(forms);
-    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
-    List<MJobForms> jobFormList = new ArrayList<MJobForms>();
-    jobFormList.add(jobform1);
-    MConnector connector1 = new MConnector("NAME", "CLASSNAME", "1.0",
-        connectionForms1, jobFormList);
-    assertEquals("NAME", connector1.getUniqueName());
-    assertEquals("CLASSNAME", connector1.getClassName());
-    assertEquals("1.0", connector1.getVersion());
-    //Clone with values. Checking values copying after the cloning. But form values will be null
-    MConnector clone1 = connector1.clone(true);
-    assertEquals("NAME", clone1.getUniqueName());
-    assertEquals("CLASSNAME", clone1.getClassName());
-    assertEquals("1.0", clone1.getVersion());
-    MForm clonedForm1 = clone1.getConnectionForms().getForms().get(0);
-    assertNull(clonedForm1.getInputs().get(0).getValue());
-    assertNull(clonedForm1.getInputs().get(1).getValue());
-
-    MForm clonedForm2 = clone1.getJobForms(MJob.Type.EXPORT).getForms().get(0);
-    assertNull(clonedForm2.getInputs().get(0).getValue());
-    assertNull(clonedForm2.getInputs().get(1).getValue());
-
-    //Clone without values. Inputs value will be null after cloning.
-    MConnector clone2 = connector1.clone(false);
-    clonedForm1 = clone2.getConnectionForms().getForms().get(0);
-    assertNull(clonedForm1.getInputs().get(0).getValue());
-    assertNull(clonedForm1.getInputs().get(1).getValue());
-    clonedForm2 = clone2.getJobForms(MJob.Type.EXPORT).getForms().get(0);
-    assertNull(clonedForm2.getInputs().get(0).getValue());
-    assertNull(clonedForm2.getInputs().get(1).getValue());
-  }
+//  /**
+//   * Test for initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MConnectionForms connectionForms1 = new MConnectionForms(forms);
+//    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
+//    List<MJobForms> jobFormList = new ArrayList<MJobForms>();
+//    jobFormList.add(jobform1);
+//    MConnector connector1 = new MConnector("NAME", "CLASSNAME", "1.0",
+//        connectionForms1, jobFormList);
+//    assertEquals("NAME", connector1.getUniqueName());
+//    assertEquals("CLASSNAME", connector1.getClassName());
+//    assertEquals("1.0", connector1.getVersion());
+//    MConnector connector2 = new MConnector("NAME", "CLASSNAME", "1.0",
+//        connectionForms1, jobFormList);
+//    assertEquals(connector2, connector1);
+//    MConnector connector3 = new MConnector("NAME1", "CLASSNAME", "2.0",
+//        connectionForms1, jobFormList);
+//    assertFalse(connector1.equals(connector3));
+//
+//    try {
+//      connector1 = new MConnector(null, "CLASSNAME", "1.0", connectionForms1,
+//          jobFormList); // Expecting null pointer exception
+//    } catch (NullPointerException e) {
+//      assertTrue(true);
+//    }
+//    try {
+//      connector1 = new MConnector("NAME", null, "1.0", connectionForms1,
+//          jobFormList); // Expecting null pointer exception
+//    } catch (NullPointerException e) {
+//      assertTrue(true);
+//    }
+//  }
+//
+//  @Test
+//  public void testClone() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
+//    input.setValue(100);
+//    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
+//    strInput.setValue("TEST-VALUE");
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    list.add(strInput);
+//    MForm form = new MForm("FORMNAME", list);
+//    forms.add(form);
+//    MConnectionForms connectionForms1 = new MConnectionForms(forms);
+//    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
+//    List<MJobForms> jobFormList = new ArrayList<MJobForms>();
+//    jobFormList.add(jobform1);
+//    MConnector connector1 = new MConnector("NAME", "CLASSNAME", "1.0",
+//        connectionForms1, jobFormList);
+//    assertEquals("NAME", connector1.getUniqueName());
+//    assertEquals("CLASSNAME", connector1.getClassName());
+//    assertEquals("1.0", connector1.getVersion());
+//    //Clone with values. Checking values copying after the cloning. But form values will be null
+//    MConnector clone1 = connector1.clone(true);
+//    assertEquals("NAME", clone1.getUniqueName());
+//    assertEquals("CLASSNAME", clone1.getClassName());
+//    assertEquals("1.0", clone1.getVersion());
+//    MForm clonedForm1 = clone1.getConnectionForms().getForms().get(0);
+//    assertNull(clonedForm1.getInputs().get(0).getValue());
+//    assertNull(clonedForm1.getInputs().get(1).getValue());
+//
+//    MForm clonedForm2 = clone1.getJobForms(MJob.Type.EXPORT).getForms().get(0);
+//    assertNull(clonedForm2.getInputs().get(0).getValue());
+//    assertNull(clonedForm2.getInputs().get(1).getValue());
+//
+//    //Clone without values. Inputs value will be null after cloning.
+//    MConnector clone2 = connector1.clone(false);
+//    clonedForm1 = clone2.getConnectionForms().getForms().get(0);
+//    assertNull(clonedForm1.getInputs().get(0).getValue());
+//    assertNull(clonedForm1.getInputs().get(1).getValue());
+//    clonedForm2 = clone2.getJobForms(MJob.Type.EXPORT).getForms().get(0);
+//    assertNull(clonedForm2.getInputs().get(0).getValue());
+//    assertNull(clonedForm2.getInputs().get(1).getValue());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMEnumInput.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMEnumInput.java b/common/src/test/java/org/apache/sqoop/model/TestMEnumInput.java
index a25016a..97baa32 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMEnumInput.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMEnumInput.java
@@ -26,38 +26,38 @@ import org.junit.Test;
  */
 public class TestMEnumInput {
 
-  public enum Enumeration { value1, value2}
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    String[] values = { "value1", "value2" };
-    MEnumInput input = new MEnumInput("NAME", false, values);
-    assertEquals("NAME", input.getName());
-    assertArrayEquals(values, input.getValues());
-    assertEquals(MInputType.ENUM, input.getType());
-
-    MEnumInput input1 = new MEnumInput("NAME", false, values);
-    assertEquals(input1, input);
-    String[] testVal = { "val", "test" };
-    MEnumInput input2 = new MEnumInput("NAME1", false, testVal);
-    assertFalse(input1.equals(input2));
-
-    MEnumInput input3 = new MEnumInput("NAME", false, values);
-    input3.setValue(Enumeration.value1);
-    assertEquals("value1", input3.getValue());
-  }
-
-  /**
-   * Test for sensitivity
-   */
-  @Test
-  public void testSensitivity() {
-    String[] values = { "value1", "value2" };
-    MEnumInput input1 = new MEnumInput("NAME", false, values);
-    MEnumInput input2 = new MEnumInput("NAME", true, values);
-    assertFalse(input1.isSensitive());
-    assertTrue(input2.isSensitive());
-  }
+//  public enum Enumeration { value1, value2}
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    String[] values = { "value1", "value2" };
+//    MEnumInput input = new MEnumInput("NAME", false, values);
+//    assertEquals("NAME", input.getName());
+//    assertArrayEquals(values, input.getValues());
+//    assertEquals(MInputType.ENUM, input.getType());
+//
+//    MEnumInput input1 = new MEnumInput("NAME", false, values);
+//    assertEquals(input1, input);
+//    String[] testVal = { "val", "test" };
+//    MEnumInput input2 = new MEnumInput("NAME1", false, testVal);
+//    assertFalse(input1.equals(input2));
+//
+//    MEnumInput input3 = new MEnumInput("NAME", false, values);
+//    input3.setValue(Enumeration.value1);
+//    assertEquals("value1", input3.getValue());
+//  }
+//
+//  /**
+//   * Test for sensitivity
+//   */
+//  @Test
+//  public void testSensitivity() {
+//    String[] values = { "value1", "value2" };
+//    MEnumInput input1 = new MEnumInput("NAME", false, values);
+//    MEnumInput input2 = new MEnumInput("NAME", true, values);
+//    assertFalse(input1.isSensitive());
+//    assertTrue(input2.isSensitive());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMForm.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMForm.java b/common/src/test/java/org/apache/sqoop/model/TestMForm.java
index 0bd55d9..109f1f5 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMForm.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMForm.java
@@ -28,61 +28,61 @@ import org.junit.Test;
  */
 public class TestMForm {
 
-  /**
-   * Test for initialization
-   */
-  @Test
-  public void testInitialization() {
-    MInput<String> input1 = new MStringInput("sqoopsqoop1", true, (short) 5);
-    MInput<String> input2 = new MStringInput("sqoopsqoop2", true, (short) 5);
-
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input1);
-    list.add(input2);
-    MForm mform = new MForm("form", list);
-
-    assertEquals("form", mform.getName());
-    assertEquals(2, mform.getInputs().size());
-  }
-
-  /**
-   * Test for equals method
-   */
-  @Test
-  public void testEquals() {
-    MInput<Integer> input1 = new MIntegerInput("sqoopsqoop1", false);
-    MInput<Integer> input2 = new MIntegerInput("sqoopsqoop2", false);
-    List<MInput<?>> list1 = new ArrayList<MInput<?>>();
-    list1.add(input1);
-    list1.add(input2);
-    MForm mform1 = new MForm("form", list1);
-
-    MInput<Integer> input3 = new MIntegerInput("sqoopsqoop1", false);
-    MInput<Integer> input4 = new MIntegerInput("sqoopsqoop2", false);
-    List<MInput<?>> list2 = new ArrayList<MInput<?>>();
-    list2.add(input3);
-    list2.add(input4);
-    MForm mform2 = new MForm("form", list2);
-    assertEquals(mform2, mform1);
-  }
-
-  @Test
-  public void testGetInputs() {
-    MIntegerInput intInput = new MIntegerInput("Form.A", false);
-    MMapInput mapInput = new MMapInput("Form.B", false);
-    MStringInput stringInput = new MStringInput("Form.C", false, (short)3);
-    MEnumInput enumInput = new MEnumInput("Form.D", false, new String[] {"I", "V"});
-
-    List<MInput<?>> inputs = new ArrayList<MInput<?>>();
-    inputs.add(intInput);
-    inputs.add(mapInput);
-    inputs.add(stringInput);
-    inputs.add(enumInput);
-
-    MForm form = new MForm("Form", inputs);
-    assertEquals(intInput, form.getIntegerInput("Form.A"));
-    assertEquals(mapInput, form.getMapInput("Form.B"));
-    assertEquals(stringInput, form.getStringInput("Form.C"));
-    assertEquals(enumInput, form.getEnumInput("Form.D"));
-  }
+//  /**
+//   * Test for initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MInput<String> input1 = new MStringInput("sqoopsqoop1", true, (short) 5);
+//    MInput<String> input2 = new MStringInput("sqoopsqoop2", true, (short) 5);
+//
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input1);
+//    list.add(input2);
+//    MForm mform = new MForm("form", list);
+//
+//    assertEquals("form", mform.getName());
+//    assertEquals(2, mform.getInputs().size());
+//  }
+//
+//  /**
+//   * Test for equals method
+//   */
+//  @Test
+//  public void testEquals() {
+//    MInput<Integer> input1 = new MIntegerInput("sqoopsqoop1", false);
+//    MInput<Integer> input2 = new MIntegerInput("sqoopsqoop2", false);
+//    List<MInput<?>> list1 = new ArrayList<MInput<?>>();
+//    list1.add(input1);
+//    list1.add(input2);
+//    MForm mform1 = new MForm("form", list1);
+//
+//    MInput<Integer> input3 = new MIntegerInput("sqoopsqoop1", false);
+//    MInput<Integer> input4 = new MIntegerInput("sqoopsqoop2", false);
+//    List<MInput<?>> list2 = new ArrayList<MInput<?>>();
+//    list2.add(input3);
+//    list2.add(input4);
+//    MForm mform2 = new MForm("form", list2);
+//    assertEquals(mform2, mform1);
+//  }
+//
+//  @Test
+//  public void testGetInputs() {
+//    MIntegerInput intInput = new MIntegerInput("Form.A", false);
+//    MMapInput mapInput = new MMapInput("Form.B", false);
+//    MStringInput stringInput = new MStringInput("Form.C", false, (short)3);
+//    MEnumInput enumInput = new MEnumInput("Form.D", false, new String[] {"I", "V"});
+//
+//    List<MInput<?>> inputs = new ArrayList<MInput<?>>();
+//    inputs.add(intInput);
+//    inputs.add(mapInput);
+//    inputs.add(stringInput);
+//    inputs.add(enumInput);
+//
+//    MForm form = new MForm("Form", inputs);
+//    assertEquals(intInput, form.getIntegerInput("Form.A"));
+//    assertEquals(mapInput, form.getMapInput("Form.B"));
+//    assertEquals(stringInput, form.getStringInput("Form.C"));
+//    assertEquals(enumInput, form.getEnumInput("Form.D"));
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMFormList.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMFormList.java b/common/src/test/java/org/apache/sqoop/model/TestMFormList.java
index bd21fcb..4894d2e 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMFormList.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMFormList.java
@@ -29,29 +29,30 @@ import static junit.framework.Assert.assertEquals;
  *
  */
 public class TestMFormList {
-  @Test
-  public void testGetInputs() {
-    List<MForm> forms = new LinkedList<MForm>();
-
-    MIntegerInput intInput = new MIntegerInput("Form1.A", false);
-    MMapInput mapInput = new MMapInput("Form1.B", false);
-
-    List<MInput<?>> inputs = new ArrayList<MInput<?>>();
-    inputs.add(intInput);
-    inputs.add(mapInput);
-    forms.add(new MForm("Form1", inputs));
-
-    MStringInput stringInput = new MStringInput("Form2.C", false, (short)3);
-    MEnumInput enumInput = new MEnumInput("Form2.D", false, new String[] {"I", "V"});
-
-    inputs = new ArrayList<MInput<?>>();
-    inputs.add(stringInput);
-    inputs.add(enumInput);
-    forms.add(new MForm("Form2", inputs));
-
-    MFormList form = new MFormList(forms);
-    assertEquals(intInput, form.getIntegerInput("Form1.A"));
-    assertEquals(mapInput, form.getMapInput("Form1.B"));
-    assertEquals(stringInput, form.getStringInput("Form2.C"));
-    assertEquals(enumInput, form.getEnumInput("Form2.D"));  }
+//  @Test
+//  public void testGetInputs() {
+//    List<MForm> forms = new LinkedList<MForm>();
+//
+//    MIntegerInput intInput = new MIntegerInput("Form1.A", false);
+//    MMapInput mapInput = new MMapInput("Form1.B", false);
+//
+//    List<MInput<?>> inputs = new ArrayList<MInput<?>>();
+//    inputs.add(intInput);
+//    inputs.add(mapInput);
+//    forms.add(new MForm("Form1", inputs));
+//
+//    MStringInput stringInput = new MStringInput("Form2.C", false, (short)3);
+//    MEnumInput enumInput = new MEnumInput("Form2.D", false, new String[] {"I", "V"});
+//
+//    inputs = new ArrayList<MInput<?>>();
+//    inputs.add(stringInput);
+//    inputs.add(enumInput);
+//    forms.add(new MForm("Form2", inputs));
+//
+//    MFormList form = new MFormList(forms);
+//    assertEquals(intInput, form.getIntegerInput("Form1.A"));
+//    assertEquals(mapInput, form.getMapInput("Form1.B"));
+//    assertEquals(stringInput, form.getStringInput("Form2.C"));
+//    assertEquals(enumInput, form.getEnumInput("Form2.D"));
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMFramework.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMFramework.java b/common/src/test/java/org/apache/sqoop/model/TestMFramework.java
index 15d9676..d0720f0 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMFramework.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMFramework.java
@@ -29,18 +29,18 @@ import static org.junit.Assert.*;
  */
 public class TestMFramework {
 
-  @Test
-  public void testFailureOnDuplicateJobTypes() {
-    MConnectionForms connectionForms = new MConnectionForms(new ArrayList<MForm>());
-    List<MJobForms> jobForms = new ArrayList<MJobForms>();
-    jobForms.add(new MJobForms(MJob.Type.IMPORT, new ArrayList<MForm>()));
-    jobForms.add(new MJobForms(MJob.Type.IMPORT, new ArrayList<MForm>()));
-
-    try {
-      new MFramework(connectionForms, jobForms, "1");
-      fail("We we're expecting exception for invalid usage");
-    } catch(Exception ex) {
-      // Expected case
-    }
-  }
+//  @Test
+//  public void testFailureOnDuplicateJobTypes() {
+//    MConnectionForms connectionForms = new MConnectionForms(new ArrayList<MForm>());
+//    List<MJobForms> jobForms = new ArrayList<MJobForms>();
+//    jobForms.add(new MJobForms(MJob.Type.IMPORT, new ArrayList<MForm>()));
+//    jobForms.add(new MJobForms(MJob.Type.IMPORT, new ArrayList<MForm>()));
+//
+//    try {
+//      new MFramework(connectionForms, jobForms, "1");
+//      fail("We we're expecting exception for invalid usage");
+//    } catch(Exception ex) {
+//      // Expected case
+//    }
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMIntegerInput.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMIntegerInput.java b/common/src/test/java/org/apache/sqoop/model/TestMIntegerInput.java
index 1f38e6d..14bca67 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMIntegerInput.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMIntegerInput.java
@@ -28,76 +28,76 @@ import org.junit.Test;
  * Test class for org.apache.sqoop.model.MInputInput
  */
 public class TestMIntegerInput {
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    MIntegerInput input = new MIntegerInput("sqoopsqoop", false);
-    assertEquals("sqoopsqoop", input.getName());
-    assertEquals(MInputType.INTEGER, input.getType());
-  }
-
-  /**
-   * Test for equals() method
-   */
-  @Test
-  public void testEquals() {
-    // Positive test
-    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
-    MIntegerInput input2 = new MIntegerInput("sqoopsqoop", false);
-    assertTrue(input1.equals(input2));
-
-    // Negative test
-    MIntegerInput input3 = new MIntegerInput("sqoopsqoop", false);
-    MIntegerInput input4 = new MIntegerInput("sqoopsqoop1", false);
-    assertFalse(input3.equals(input4));
-  }
-
-  /**
-   * Test for value
-   */
-  @Test
-  public void testValue() {
-    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
-    input1.setValue(99);
-    assertEquals(new Integer(99), input1.getValue());
-    input1.setEmpty();
-    assertNull(input1.getValue());
-  }
-
-  /**
-   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
-   */
-  @Test
-  public void testUrlSafe() {
-    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
-    input1.setValue(1001);
-    // Getting URL safe string
-    String tmp = input1.getUrlSafeValueString();
-    // Restore to actual value
-    input1.restoreFromUrlSafeValueString(tmp);
-    assertEquals(new Integer(1001), input1.getValue());
-  }
-
-  /**
-   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
-   */
-  @Test
-  public void testNamedElement() {
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
-    assertEquals("sqoopsqoop.label", input1.getLabelKey());
-    assertEquals("sqoopsqoop.help", input1.getHelpKey());
-  }
-
-  /**
-   * Test for sensitivity
-   */
-  @Test
-  public void testSensitivity() {
-    MIntegerInput input1 = new MIntegerInput("NAME", false);
-    MIntegerInput input2 = new MIntegerInput("NAME", true);
-    assertFalse(input1.isSensitive());
-    assertTrue(input2.isSensitive());
-  }
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MIntegerInput input = new MIntegerInput("sqoopsqoop", false);
+//    assertEquals("sqoopsqoop", input.getName());
+//    assertEquals(MInputType.INTEGER, input.getType());
+//  }
+//
+//  /**
+//   * Test for equals() method
+//   */
+//  @Test
+//  public void testEquals() {
+//    // Positive test
+//    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
+//    MIntegerInput input2 = new MIntegerInput("sqoopsqoop", false);
+//    assertTrue(input1.equals(input2));
+//
+//    // Negative test
+//    MIntegerInput input3 = new MIntegerInput("sqoopsqoop", false);
+//    MIntegerInput input4 = new MIntegerInput("sqoopsqoop1", false);
+//    assertFalse(input3.equals(input4));
+//  }
+//
+//  /**
+//   * Test for value
+//   */
+//  @Test
+//  public void testValue() {
+//    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
+//    input1.setValue(99);
+//    assertEquals(new Integer(99), input1.getValue());
+//    input1.setEmpty();
+//    assertNull(input1.getValue());
+//  }
+//
+//  /**
+//   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
+//   */
+//  @Test
+//  public void testUrlSafe() {
+//    MIntegerInput input1 = new MIntegerInput("sqoopsqoop", false);
+//    input1.setValue(1001);
+//    // Getting URL safe string
+//    String tmp = input1.getUrlSafeValueString();
+//    // Restore to actual value
+//    input1.restoreFromUrlSafeValueString(tmp);
+//    assertEquals(new Integer(1001), input1.getValue());
+//  }
+//
+//  /**
+//   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
+//   */
+//  @Test
+//  public void testNamedElement() {
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
+//    assertEquals("sqoopsqoop.label", input1.getLabelKey());
+//    assertEquals("sqoopsqoop.help", input1.getHelpKey());
+//  }
+//
+//  /**
+//   * Test for sensitivity
+//   */
+//  @Test
+//  public void testSensitivity() {
+//    MIntegerInput input1 = new MIntegerInput("NAME", false);
+//    MIntegerInput input2 = new MIntegerInput("NAME", true);
+//    assertFalse(input1.isSensitive());
+//    assertTrue(input2.isSensitive());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMJob.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMJob.java b/common/src/test/java/org/apache/sqoop/model/TestMJob.java
index 8b6f5dc..355cdb9 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMJob.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMJob.java
@@ -29,107 +29,107 @@ import org.junit.Test;
  * Test class for org.apache.sqoop.model.MJob
  */
 public class TestMJob {
-  /**
-   * Test class for initialization
-   */
-  @Test
-  public void testInitialization() {
-    // Test default constructor
-    MJob job = job(MJob.Type.IMPORT);
-    assertEquals(123l, job.getConnectorId());
-    assertEquals(MJob.Type.IMPORT, job.getType());
-    assertEquals("Buffy", job.getCreationUser());
-    assertEquals("Vampire", job.getName());
-    assertEquals(forms1(MJob.Type.IMPORT), job.getConnectorPart());
-    assertEquals(forms2(MJob.Type.IMPORT), job.getFrameworkPart());
-
-    // Test copy constructor
-    MJob copy = new MJob(job);
-    assertEquals(123l, copy.getConnectorId());
-    assertEquals(MJob.Type.IMPORT, copy.getType());
-    assertEquals("Vampire", copy.getName());
-    assertEquals("Buffy", copy.getCreationUser());
-    assertEquals(job.getCreationDate(), copy.getCreationDate());
-    assertEquals(forms1(MJob.Type.IMPORT), copy.getConnectorPart());
-    assertEquals(forms2(MJob.Type.IMPORT), copy.getFrameworkPart());
-
-    // Test constructor for metadata upgrade (the order of forms is different)
-    MJob upgradeCopy = new MJob(job, forms2(MJob.Type.IMPORT), forms1(MJob.Type.IMPORT));
-    assertEquals(123l, upgradeCopy.getConnectorId());
-    assertEquals(MJob.Type.IMPORT, upgradeCopy.getType());
-    assertEquals("Vampire", upgradeCopy.getName());
-    assertEquals("Buffy", upgradeCopy.getCreationUser());
-    assertEquals(job.getCreationDate(), upgradeCopy.getCreationDate());
-    assertEquals(forms2(MJob.Type.IMPORT), upgradeCopy.getConnectorPart());
-    assertEquals(forms1(MJob.Type.IMPORT), upgradeCopy.getFrameworkPart());
-  }
-
-  @Test(expected = SqoopException.class)
-  public void testIncorrectDefaultConstructor() {
-    new MJob(1l, 1l, MJob.Type.IMPORT, forms1(MJob.Type.IMPORT), forms2(MJob.Type.EXPORT));
-  }
-
-  @Test(expected = SqoopException.class)
-  public void testIncorrectUpgradeConstructor() {
-    new MJob(job(MJob.Type.EXPORT), forms1(MJob.Type.IMPORT), forms2(MJob.Type.IMPORT));
-  }
-
-  @Test
-  public void testClone() {
-    MJob job = job(MJob.Type.IMPORT);
-
-    // Clone without value
-    MJob withoutValue = job.clone(false);
-    assertEquals(job, withoutValue);
-    assertEquals(MPersistableEntity.PERSISTANCE_ID_DEFAULT, withoutValue.getPersistenceId());
-    assertEquals(MJob.Type.IMPORT, withoutValue.getType());
-    assertNull(withoutValue.getName());
-    assertNull(withoutValue.getCreationUser());
-    assertEquals(forms1(MJob.Type.IMPORT), withoutValue.getConnectorPart());
-    assertEquals(forms2(MJob.Type.IMPORT), withoutValue.getFrameworkPart());
-    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
-    assertNull(withoutValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
-
-    // Clone with value
-    MJob withValue = job.clone(true);
-    assertEquals(job, withValue);
-    assertEquals(job.getPersistenceId(), withValue.getPersistenceId());
-    assertEquals(MJob.Type.IMPORT, withValue.getType());
-    assertEquals(job.getName(), withValue.getName());
-    assertEquals(job.getCreationUser(), withValue.getCreationUser());
-    assertEquals(forms1(MJob.Type.IMPORT), withValue.getConnectorPart());
-    assertEquals(forms2(MJob.Type.IMPORT), withValue.getFrameworkPart());
-    assertEquals(100, withValue.getConnectorPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
-    assertEquals("TEST-VALUE", withValue.getConnectorPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());  }
-
-  private MJob job(MJob.Type type) {
-    MJob job = new MJob(123l, 456l, type, forms1(type), forms2(type));
-    job.setName("Vampire");
-    job.setCreationUser("Buffy");
-    return job;
-  }
-
-  private MJobForms forms1(MJob.Type type) {
-    List<MForm> forms = new ArrayList<MForm>();
-    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
-    input.setValue(100);
-    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
-    strInput.setValue("TEST-VALUE");
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    list.add(strInput);
-    MForm form = new MForm("FORMNAME", list);
-    forms.add(form);
-    return new MJobForms(type, forms);
-  }
-
-  private MJobForms forms2(MJob.Type type) {
-    List<MForm> forms = new ArrayList<MForm>();
-    MMapInput input = new MMapInput("MAP-INPUT", false);
-    List<MInput<?>> list = new ArrayList<MInput<?>>();
-    list.add(input);
-    MForm form = new MForm("form", list);
-    forms.add(form);
-    return new MJobForms(type, forms);
-  }
+//  /**
+//   * Test class for initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    // Test default constructor
+//    MJob job = job(MJob.Type.IMPORT);
+//    assertEquals(123l, job.getFromConnectorId());
+//    assertEquals(MJob.Type.IMPORT, job.getType());
+//    assertEquals("Buffy", job.getCreationUser());
+//    assertEquals("Vampire", job.getName());
+//    assertEquals(forms1(MJob.Type.IMPORT), job.getFromPart());
+//    assertEquals(forms2(MJob.Type.IMPORT), job.getFrameworkPart());
+//
+//    // Test copy constructor
+//    MJob copy = new MJob(job);
+//    assertEquals(123l, copy.getFromConnectorId());
+//    assertEquals(MJob.Type.IMPORT, copy.getType());
+//    assertEquals("Vampire", copy.getName());
+//    assertEquals("Buffy", copy.getCreationUser());
+//    assertEquals(job.getCreationDate(), copy.getCreationDate());
+//    assertEquals(forms1(MJob.Type.IMPORT), copy.getFromPart());
+//    assertEquals(forms2(MJob.Type.IMPORT), copy.getFrameworkPart());
+//
+//    // Test constructor for metadata upgrade (the order of forms is different)
+//    MJob upgradeCopy = new MJob(job, forms2(MJob.Type.IMPORT), forms1(MJob.Type.IMPORT));
+//    assertEquals(123l, upgradeCopy.getFromConnectorId());
+//    assertEquals(MJob.Type.IMPORT, upgradeCopy.getType());
+//    assertEquals("Vampire", upgradeCopy.getName());
+//    assertEquals("Buffy", upgradeCopy.getCreationUser());
+//    assertEquals(job.getCreationDate(), upgradeCopy.getCreationDate());
+//    assertEquals(forms2(MJob.Type.IMPORT), upgradeCopy.getFromPart());
+//    assertEquals(forms1(MJob.Type.IMPORT), upgradeCopy.getFrameworkPart());
+//  }
+//
+//  @Test(expected = SqoopException.class)
+//  public void testIncorrectDefaultConstructor() {
+//    new MJob(1l, 1l, MJob.Type.IMPORT, forms1(MJob.Type.IMPORT), forms2(MJob.Type.EXPORT));
+//  }
+//
+//  @Test(expected = SqoopException.class)
+//  public void testIncorrectUpgradeConstructor() {
+//    new MJob(job(MJob.Type.EXPORT), forms1(MJob.Type.IMPORT), forms2(MJob.Type.IMPORT));
+//  }
+//
+//  @Test
+//  public void testClone() {
+//    MJob job = job(MJob.Type.IMPORT);
+//
+//    // Clone without value
+//    MJob withoutValue = job.clone(false);
+//    assertEquals(job, withoutValue);
+//    assertEquals(MPersistableEntity.PERSISTANCE_ID_DEFAULT, withoutValue.getPersistenceId());
+//    assertEquals(MJob.Type.IMPORT, withoutValue.getType());
+//    assertNull(withoutValue.getName());
+//    assertNull(withoutValue.getCreationUser());
+//    assertEquals(forms1(MJob.Type.IMPORT), withoutValue.getFromPart());
+//    assertEquals(forms2(MJob.Type.IMPORT), withoutValue.getFrameworkPart());
+//    assertNull(withoutValue.getFromPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
+//    assertNull(withoutValue.getFromPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());
+//
+//    // Clone with value
+//    MJob withValue = job.clone(true);
+//    assertEquals(job, withValue);
+//    assertEquals(job.getPersistenceId(), withValue.getPersistenceId());
+//    assertEquals(MJob.Type.IMPORT, withValue.getType());
+//    assertEquals(job.getName(), withValue.getName());
+//    assertEquals(job.getCreationUser(), withValue.getCreationUser());
+//    assertEquals(forms1(MJob.Type.IMPORT), withValue.getFromPart());
+//    assertEquals(forms2(MJob.Type.IMPORT), withValue.getFrameworkPart());
+//    assertEquals(100, withValue.getFromPart().getForm("FORMNAME").getInput("INTEGER-INPUT").getValue());
+//    assertEquals("TEST-VALUE", withValue.getFromPart().getForm("FORMNAME").getInput("STRING-INPUT").getValue());  }
+//
+//  private MJob job(MJob.Type type) {
+//    MJob job = new MJob(123l, 456l, type, forms1(type), forms2(type));
+//    job.setName("Vampire");
+//    job.setCreationUser("Buffy");
+//    return job;
+//  }
+//
+//  private MJobForms forms1(MJob.Type type) {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MIntegerInput input = new MIntegerInput("INTEGER-INPUT", false);
+//    input.setValue(100);
+//    MStringInput strInput = new MStringInput("STRING-INPUT",false,(short)20);
+//    strInput.setValue("TEST-VALUE");
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    list.add(strInput);
+//    MForm form = new MForm("FORMNAME", list);
+//    forms.add(form);
+//    return new MJobForms(type, forms);
+//  }
+//
+//  private MJobForms forms2(MJob.Type type) {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MMapInput input = new MMapInput("MAP-INPUT", false);
+//    List<MInput<?>> list = new ArrayList<MInput<?>>();
+//    list.add(input);
+//    MForm form = new MForm("form", list);
+//    forms.add(form);
+//    return new MJobForms(type, forms);
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMJobForms.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMJobForms.java b/common/src/test/java/org/apache/sqoop/model/TestMJobForms.java
index b2bb0a5..5c44c0a 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMJobForms.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMJobForms.java
@@ -28,20 +28,20 @@ import org.junit.Test;
  * Test class for org.apache.sqoop.model.MJobForms
  */
 public class TestMJobForms {
-  /**
-   * Test for class initialization and values
-   */
-  @Test
-  public void testInitialization() {
-    List<MForm> forms = new ArrayList<MForm>();
-    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
-    assertEquals(MJob.Type.EXPORT, jobform1.getType());
-    List<MForm> forms2 = new ArrayList<MForm>();
-    MJobForms jobform2 = new MJobForms(MJob.Type.EXPORT, forms2);
-    assertEquals(jobform2, jobform1);
-    // Add a form to list for checking not equals
-    MForm m = new MForm("test", null);
-    forms2.add(m);
-    assertFalse(jobform1.equals(jobform2));
-  }
+//  /**
+//   * Test for class initialization and values
+//   */
+//  @Test
+//  public void testInitialization() {
+//    List<MForm> forms = new ArrayList<MForm>();
+//    MJobForms jobform1 = new MJobForms(MJob.Type.EXPORT, forms);
+//    assertEquals(MJob.Type.EXPORT, jobform1.getType());
+//    List<MForm> forms2 = new ArrayList<MForm>();
+//    MJobForms jobform2 = new MJobForms(MJob.Type.EXPORT, forms2);
+//    assertEquals(jobform2, jobform1);
+//    // Add a form to list for checking not equals
+//    MForm m = new MForm("test", null);
+//    forms2.add(m);
+//    assertFalse(jobform1.equals(jobform2));
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMMapInput.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMMapInput.java b/common/src/test/java/org/apache/sqoop/model/TestMMapInput.java
index 120fb07..99d147c 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMMapInput.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMMapInput.java
@@ -32,84 +32,84 @@ import org.junit.Test;
  * Test class for org.apache.sqoop.model.MMapInput
  */
 public class TestMMapInput {
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    MMapInput input = new MMapInput("sqoopsqoop", false);
-    assertEquals("sqoopsqoop", input.getName());
-    assertEquals(MInputType.MAP, input.getType());
-  }
-
-  /**
-   * Test for equals() method
-   */
-  @Test
-  public void testEquals() {
-    // Positive test
-    MMapInput input1 = new MMapInput("sqoopsqoop", false);
-    MMapInput input2 = new MMapInput("sqoopsqoop", false);
-    assertTrue(input1.equals(input2));
-
-    // Negative test
-    MMapInput input3 = new MMapInput("sqoopsqoop", false);
-    MMapInput input4 = new MMapInput("sqoopsqoop1", false);
-    assertFalse(input3.equals(input4));
-  }
-
-  /**
-   * Test for value
-   */
-  @Test
-  public void testValue() {
-    MMapInput input1 = new MMapInput("sqoopsqoop", false);
-    Map<String, String> map1 = new HashMap<String, String>();
-    input1.setValue(map1);
-    assertEquals(map1, input1.getValue());
-    input1.setEmpty();
-    assertNull(input1.getValue());
-  }
-
-  /**
-   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
-   */
-  @Test
-  public void testUrlSafe() {
-    MMapInput input1 = new MMapInput("sqoopsqoop", false);
-    Map<String, String> map1 = new HashMap<String, String>();
-    input1.setValue(map1);
-    // Getting URL safe string
-    String tmp = input1.getUrlSafeValueString();
-    // Restore to actual value
-    input1.restoreFromUrlSafeValueString(tmp);
-    assertNotNull(input1.getValue());
-    assertEquals(0, input1.getValue().size());
-
-    input1.setValue(null);
-    tmp = input1.getUrlSafeValueString();
-    input1.restoreFromUrlSafeValueString(tmp);
-    assertNull(input1.getValue());
-  }
-
-  /**
-   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
-   */
-  @Test
-  public void testNamedElement() {
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
-    assertEquals("sqoopsqoop.label", input1.getLabelKey());
-    assertEquals("sqoopsqoop.help", input1.getHelpKey());
-  }
-
-  /**
-   * Test for sensitivity
-   */
-  @Test
-  public void testSensitivity() {
-    MMapInput input1 = new MMapInput("NAME", false);
-    MMapInput input2 = new MMapInput("NAME", true);
-    assertFalse(input1.isSensitive());
-    assertTrue(input2.isSensitive());
-  }
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MMapInput input = new MMapInput("sqoopsqoop", false);
+//    assertEquals("sqoopsqoop", input.getName());
+//    assertEquals(MInputType.MAP, input.getType());
+//  }
+//
+//  /**
+//   * Test for equals() method
+//   */
+//  @Test
+//  public void testEquals() {
+//    // Positive test
+//    MMapInput input1 = new MMapInput("sqoopsqoop", false);
+//    MMapInput input2 = new MMapInput("sqoopsqoop", false);
+//    assertTrue(input1.equals(input2));
+//
+//    // Negative test
+//    MMapInput input3 = new MMapInput("sqoopsqoop", false);
+//    MMapInput input4 = new MMapInput("sqoopsqoop1", false);
+//    assertFalse(input3.equals(input4));
+//  }
+//
+//  /**
+//   * Test for value
+//   */
+//  @Test
+//  public void testValue() {
+//    MMapInput input1 = new MMapInput("sqoopsqoop", false);
+//    Map<String, String> map1 = new HashMap<String, String>();
+//    input1.setValue(map1);
+//    assertEquals(map1, input1.getValue());
+//    input1.setEmpty();
+//    assertNull(input1.getValue());
+//  }
+//
+//  /**
+//   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
+//   */
+//  @Test
+//  public void testUrlSafe() {
+//    MMapInput input1 = new MMapInput("sqoopsqoop", false);
+//    Map<String, String> map1 = new HashMap<String, String>();
+//    input1.setValue(map1);
+//    // Getting URL safe string
+//    String tmp = input1.getUrlSafeValueString();
+//    // Restore to actual value
+//    input1.restoreFromUrlSafeValueString(tmp);
+//    assertNotNull(input1.getValue());
+//    assertEquals(0, input1.getValue().size());
+//
+//    input1.setValue(null);
+//    tmp = input1.getUrlSafeValueString();
+//    input1.restoreFromUrlSafeValueString(tmp);
+//    assertNull(input1.getValue());
+//  }
+//
+//  /**
+//   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
+//   */
+//  @Test
+//  public void testNamedElement() {
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
+//    assertEquals("sqoopsqoop.label", input1.getLabelKey());
+//    assertEquals("sqoopsqoop.help", input1.getHelpKey());
+//  }
+//
+//  /**
+//   * Test for sensitivity
+//   */
+//  @Test
+//  public void testSensitivity() {
+//    MMapInput input1 = new MMapInput("NAME", false);
+//    MMapInput input2 = new MMapInput("NAME", true);
+//    assertFalse(input1.isSensitive());
+//    assertTrue(input2.isSensitive());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMNamedElement.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMNamedElement.java b/common/src/test/java/org/apache/sqoop/model/TestMNamedElement.java
index f336bab..4fcb212 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMNamedElement.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMNamedElement.java
@@ -26,14 +26,14 @@ import org.junit.Test;
  */
 public class TestMNamedElement {
 
-  /**
-   * Test initialization and values
-   */
-  @Test
-  public void testInitialization() {
-    MNamedElement named = new MIntegerInput("SQOOP", false);
-    assertEquals("SQOOP", named.getName());
-    assertEquals("SQOOP.label", named.getLabelKey());
-    assertEquals("SQOOP.help", named.getHelpKey());
-  }
+//  /**
+//   * Test initialization and values
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MNamedElement named = new MIntegerInput("SQOOP", false);
+//    assertEquals("SQOOP", named.getName());
+//    assertEquals("SQOOP.label", named.getLabelKey());
+//    assertEquals("SQOOP.help", named.getHelpKey());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMPersistableEntity.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMPersistableEntity.java b/common/src/test/java/org/apache/sqoop/model/TestMPersistableEntity.java
index 000c6be..d68f06c 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMPersistableEntity.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMPersistableEntity.java
@@ -22,28 +22,28 @@ import org.junit.Test;
 
 public class TestMPersistableEntity {
 
-  @Test
-  public void testPersistableId() {
-    PersistentId id = new PersistentId();
-
-    assertFalse(id.hasPersistenceId());
-
-    id.setPersistenceId(666);
-    assertTrue(id.hasPersistenceId());
-    assertEquals(666, id.getPersistenceId());
-  }
-
-  /**
-   * Testing class extending MPersistableEntity.
-   *
-   * Empty implementation with purpose to just test methods available
-   * directly in the abstract class.
-   */
-  public static class PersistentId extends MPersistableEntity {
-    @Override
-    public String toString() {
-      return null;
-    }
-  }
+//  @Test
+//  public void testPersistableId() {
+//    PersistentId id = new PersistentId();
+//
+//    assertFalse(id.hasPersistenceId());
+//
+//    id.setPersistenceId(666);
+//    assertTrue(id.hasPersistenceId());
+//    assertEquals(666, id.getPersistenceId());
+//  }
+//
+//  /**
+//   * Testing class extending MPersistableEntity.
+//   *
+//   * Empty implementation with purpose to just test methods available
+//   * directly in the abstract class.
+//   */
+//  public static class PersistentId extends MPersistableEntity {
+//    @Override
+//    public String toString() {
+//      return null;
+//    }
+//  }
 
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMStringInput.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMStringInput.java b/common/src/test/java/org/apache/sqoop/model/TestMStringInput.java
index 2fe0335..b0223a7 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMStringInput.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMStringInput.java
@@ -26,70 +26,70 @@ import org.junit.Test;
  */
 public class TestMStringInput {
 
-  /**
-   * Test for class initialization
-   */
-  @Test
-  public void testInitialization() {
-    short len = 6;
-    MStringInput input = new MStringInput("sqoopsqoop", true, len);
-    assertEquals("sqoopsqoop", input.getName());
-    assertEquals(true, input.isSensitive());
-    assertEquals(len, input.getMaxLength());
-    assertEquals(MInputType.STRING, input.getType());
-  }
-
-  /**
-   * Test for equals() method
-   */
-  @Test
-  public void testEquals() {
-    short len = 6;
-    // Positive test
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, len);
-    MStringInput input2 = new MStringInput("sqoopsqoop", true, len);
-    assertTrue(input1.equals(input2));
-
-    // Negative test
-    MStringInput input3 = new MStringInput("sqoopsqoop", false, len);
-    MStringInput input4 = new MStringInput("sqoopsqoop", true, len);
-    assertFalse(input3.equals(input4));
-  }
-
-  /**
-   * Test for value
-   */
-  @Test
-  public void testValue() {
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
-    input1.setValue("sqoop");
-    assertEquals("sqoop", input1.getValue());
-    input1.setEmpty();
-    assertNull(input1.getValue());
-  }
-
-  /**
-   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
-   */
-  @Test
-  public void testUrlSafe() {
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
-    String s = "Sqoop%$!@#&*()Sqoop";
-    input1.setValue(s);
-    // Getting URL safe string
-    String tmp = input1.getUrlSafeValueString();
-    // Restore to actual value
-    input1.restoreFromUrlSafeValueString(tmp);
-    assertEquals(s, input1.getValue());
-  }
-
-  /**
-   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
-   */
-  @Test
-  public void testNamedElement() {
-    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
-    assertEquals("sqoopsqoop.label", input1.getLabelKey());
-    assertEquals("sqoopsqoop.help", input1.getHelpKey());
-  }
+//  /**
+//   * Test for class initialization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    short len = 6;
+//    MStringInput input = new MStringInput("sqoopsqoop", true, len);
+//    assertEquals("sqoopsqoop", input.getName());
+//    assertEquals(true, input.isSensitive());
+//    assertEquals(len, input.getMaxLength());
+//    assertEquals(MInputType.STRING, input.getType());
+//  }
+//
+//  /**
+//   * Test for equals() method
+//   */
+//  @Test
+//  public void testEquals() {
+//    short len = 6;
+//    // Positive test
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, len);
+//    MStringInput input2 = new MStringInput("sqoopsqoop", true, len);
+//    assertTrue(input1.equals(input2));
+//
+//    // Negative test
+//    MStringInput input3 = new MStringInput("sqoopsqoop", false, len);
+//    MStringInput input4 = new MStringInput("sqoopsqoop", true, len);
+//    assertFalse(input3.equals(input4));
+//  }
+//
+//  /**
+//   * Test for value
+//   */
+//  @Test
+//  public void testValue() {
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
+//    input1.setValue("sqoop");
+//    assertEquals("sqoop", input1.getValue());
+//    input1.setEmpty();
+//    assertNull(input1.getValue());
+//  }
+//
+//  /**
+//   * Test for getUrlSafeValueString() and restoreFromUrlSafeValueString()
+//   */
+//  @Test
+//  public void testUrlSafe() {
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
+//    String s = "Sqoop%$!@#&*()Sqoop";
+//    input1.setValue(s);
+//    // Getting URL safe string
+//    String tmp = input1.getUrlSafeValueString();
+//    // Restore to actual value
+//    input1.restoreFromUrlSafeValueString(tmp);
+//    assertEquals(s, input1.getValue());
+//  }
+//
+//  /**
+//   * Test case for MNamedElement.getLabelKey() and MNamedElement.getHelpKey()
+//   */
+//  @Test
+//  public void testNamedElement() {
+//    MStringInput input1 = new MStringInput("sqoopsqoop", true, (short) 5);
+//    assertEquals("sqoopsqoop.label", input1.getLabelKey());
+//    assertEquals("sqoopsqoop.help", input1.getHelpKey());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java b/common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java
index 3fd5a95..7c9a3d9 100644
--- a/common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java
+++ b/common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java
@@ -27,44 +27,44 @@ import org.junit.Test;
  */
 public class TestMValidatedElement {
 
-  /**
-   * Test for initalization
-   */
-  @Test
-  public void testInitialization() {
-    MValidatedElement input = new MIntegerInput("input", false);
-    assertEquals("input", input.getName());
-    assertEquals(Status.FINE, input.getValidationStatus());
-  }
-
-  /**
-   * Test for validation message and status
-   */
-  @Test
-  public void testValidationMessageStatus() {
-    MValidatedElement input = new MIntegerInput("input", false);
-    // Default status
-    assertEquals(Status.FINE, input.getValidationStatus());
-    // Set status and user message
-    input.setValidationMessage(Status.ACCEPTABLE, "MY_MESSAGE");
-    assertEquals(Status.ACCEPTABLE, input.getValidationStatus());
-    assertEquals("MY_MESSAGE", input.getValidationMessage());
-    // Check for null if status does not equal
-    assertNull(input.getValidationMessage(Status.FINE));
-    assertNull(input.getErrorMessage());
-    assertNotNull(input.getWarningMessage());
-    // Set unacceptable status
-    input.setValidationMessage(Status.UNACCEPTABLE, "MY_MESSAGE");
-    assertNotNull(input.getErrorMessage());
-    assertEquals("MY_MESSAGE", input.getErrorMessage());
-    assertNull(input.getWarningMessage());
-    // Set warning
-    input.setWarningMessage("WARN");
-    assertEquals(Status.ACCEPTABLE, input.getValidationStatus());
-    assertEquals("WARN", input.getValidationMessage());
-    // Unacceptable method
-    input.setErrorMessage("ERROR");
-    assertEquals(Status.UNACCEPTABLE, input.getValidationStatus());
-    assertEquals("ERROR", input.getValidationMessage());
-  }
+//  /**
+//   * Test for initalization
+//   */
+//  @Test
+//  public void testInitialization() {
+//    MValidatedElement input = new MIntegerInput("input", false);
+//    assertEquals("input", input.getName());
+//    assertEquals(Status.FINE, input.getValidationStatus());
+//  }
+//
+//  /**
+//   * Test for validation message and status
+//   */
+//  @Test
+//  public void testValidationMessageStatus() {
+//    MValidatedElement input = new MIntegerInput("input", false);
+//    // Default status
+//    assertEquals(Status.FINE, input.getValidationStatus());
+//    // Set status and user message
+//    input.setValidationMessage(Status.ACCEPTABLE, "MY_MESSAGE");
+//    assertEquals(Status.ACCEPTABLE, input.getValidationStatus());
+//    assertEquals("MY_MESSAGE", input.getValidationMessage());
+//    // Check for null if status does not equal
+//    assertNull(input.getValidationMessage(Status.FINE));
+//    assertNull(input.getErrorMessage());
+//    assertNotNull(input.getWarningMessage());
+//    // Set unacceptable status
+//    input.setValidationMessage(Status.UNACCEPTABLE, "MY_MESSAGE");
+//    assertNotNull(input.getErrorMessage());
+//    assertEquals("MY_MESSAGE", input.getErrorMessage());
+//    assertNull(input.getWarningMessage());
+//    // Set warning
+//    input.setWarningMessage("WARN");
+//    assertEquals(Status.ACCEPTABLE, input.getValidationStatus());
+//    assertEquals("WARN", input.getValidationMessage());
+//    // Unacceptable method
+//    input.setErrorMessage("ERROR");
+//    assertEquals(Status.UNACCEPTABLE, input.getValidationStatus());
+//    assertEquals("ERROR", input.getValidationMessage());
+//  }
 }

http://git-wip-us.apache.org/repos/asf/sqoop/blob/c8108266/common/src/test/java/org/apache/sqoop/submission/TestSubmissionStatus.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/sqoop/submission/TestSubmissionStatus.java b/common/src/test/java/org/apache/sqoop/submission/TestSubmissionStatus.java
index 99f4767..5d6692d 100644
--- a/common/src/test/java/org/apache/sqoop/submission/TestSubmissionStatus.java
+++ b/common/src/test/java/org/apache/sqoop/submission/TestSubmissionStatus.java
@@ -27,38 +27,38 @@ import junit.framework.TestCase;
  */
 public class TestSubmissionStatus extends TestCase {
 
-  /**
-   * unfinished() test
-   */
-  public void testUnfinished() {
-    SubmissionStatus subStatus[] = SubmissionStatus.unfinished();
-    SubmissionStatus subStatusTest[] = new SubmissionStatus[] {
-        SubmissionStatus.RUNNING, SubmissionStatus.BOOTING };
-    List<SubmissionStatus> tempSubmissionStatusList = Arrays.asList(subStatus);
-    for (SubmissionStatus stat : subStatusTest) {
-      assertTrue(tempSubmissionStatusList.contains(stat));
-    }
-  }
-
-  /**
-   * isRunning() test
-   */
-  public void testIsRunning() {
-    assertTrue(SubmissionStatus.RUNNING.isRunning());
-    assertTrue(SubmissionStatus.BOOTING.isRunning());
-    assertFalse(SubmissionStatus.FAILED.isRunning());
-    assertFalse(SubmissionStatus.UNKNOWN.isRunning());
-    assertFalse(SubmissionStatus.FAILURE_ON_SUBMIT.isRunning());
-  }
-
-  /**
-   * isFailure() test
-   */
-  public void testIsFailure() {
-    assertTrue(SubmissionStatus.FAILED.isFailure());
-    assertTrue(SubmissionStatus.UNKNOWN.isFailure());
-    assertTrue(SubmissionStatus.FAILURE_ON_SUBMIT.isFailure());
-    assertFalse(SubmissionStatus.RUNNING.isFailure());
-    assertFalse(SubmissionStatus.BOOTING.isFailure());
-  }
+//  /**
+//   * unfinished() test
+//   */
+//  public void testUnfinished() {
+//    SubmissionStatus subStatus[] = SubmissionStatus.unfinished();
+//    SubmissionStatus subStatusTest[] = new SubmissionStatus[] {
+//        SubmissionStatus.RUNNING, SubmissionStatus.BOOTING };
+//    List<SubmissionStatus> tempSubmissionStatusList = Arrays.asList(subStatus);
+//    for (SubmissionStatus stat : subStatusTest) {
+//      assertTrue(tempSubmissionStatusList.contains(stat));
+//    }
+//  }
+//
+//  /**
+//   * isRunning() test
+//   */
+//  public void testIsRunning() {
+//    assertTrue(SubmissionStatus.RUNNING.isRunning());
+//    assertTrue(SubmissionStatus.BOOTING.isRunning());
+//    assertFalse(SubmissionStatus.FAILED.isRunning());
+//    assertFalse(SubmissionStatus.UNKNOWN.isRunning());
+//    assertFalse(SubmissionStatus.FAILURE_ON_SUBMIT.isRunning());
+//  }
+//
+//  /**
+//   * isFailure() test
+//   */
+//  public void testIsFailure() {
+//    assertTrue(SubmissionStatus.FAILED.isFailure());
+//    assertTrue(SubmissionStatus.UNKNOWN.isFailure());
+//    assertTrue(SubmissionStatus.FAILURE_ON_SUBMIT.isFailure());
+//    assertFalse(SubmissionStatus.RUNNING.isFailure());
+//    assertFalse(SubmissionStatus.BOOTING.isFailure());
+//  }
 }