----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42343/#review116135 -----------------------------------------------------------
why are we doing this? don't the integration tests and unit tests serve two different purposes? the unit tests test that the shell is sending the right message to the server and the integration tests ensure that the server is performing the correct behavior based on the message that the shell sends - Abraham Fine On Jan. 15, 2016, 7:31 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42343/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2016, 7:31 a.m.) > > > 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 > >
