----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42343/ -----------------------------------------------------------
Review request for Sqoop. Repository: sqoop-sqoop2 Description ------- The test cases for shell will be refactored as following: 1. Because the integration tests are created for shell, remove the duplicated cases in shell module. 2. Move the negative test cases from shell module to integration test. The authorization test cases won't be changed in this ticket, and keep them in shell module for now. Diffs ----- shell/src/test/java/org/apache/sqoop/shell/TestCloneCommand.java 4515b1c shell/src/test/java/org/apache/sqoop/shell/TestCreateCommand.java c1c23db shell/src/test/java/org/apache/sqoop/shell/TestDeleteCommand.java 47a8f87 shell/src/test/java/org/apache/sqoop/shell/TestDisableCommand.java 9e4e532 shell/src/test/java/org/apache/sqoop/shell/TestEnableCommand.java 4c52448 shell/src/test/java/org/apache/sqoop/shell/TestSetCommand.java 93e1e3e shell/src/test/java/org/apache/sqoop/shell/TestShowCommand.java 49affa3 shell/src/test/java/org/apache/sqoop/shell/TestStartCommand.java 7dc407c shell/src/test/java/org/apache/sqoop/shell/TestStatusCommand.java ce01842 shell/src/test/java/org/apache/sqoop/shell/TestStopCommand.java ca9d03b shell/src/test/java/org/apache/sqoop/shell/TestUpdateCommand.java 469ded7 test/src/test/java/org/apache/sqoop/integration/shell/CloneCommandTest.java bcf6334 test/src/test/java/org/apache/sqoop/integration/shell/CreateCommandTest.java 81b7c8a test/src/test/java/org/apache/sqoop/integration/shell/DeleteCommandTest.java 8ed8570 test/src/test/java/org/apache/sqoop/integration/shell/DisableCommandTest.java e524312 test/src/test/java/org/apache/sqoop/integration/shell/EnableCommandTest.java b104348 test/src/test/java/org/apache/sqoop/integration/shell/SetCommandTest.java f673a58 test/src/test/java/org/apache/sqoop/integration/shell/StartCommandTest.java 5504586 test/src/test/java/org/apache/sqoop/integration/shell/StatusCommandTest.java 7294b05 test/src/test/java/org/apache/sqoop/integration/shell/StopCommandTest.java 69110b2 test/src/test/java/org/apache/sqoop/integration/shell/UpdateCommandTest.java f9a35fe Diff: https://reviews.apache.org/r/42343/diff/ Testing ------- Thanks, Colin Ma
