> On April 6, 2015, 9:30 p.m., Himanshu Gahlaut wrote: > > lens-cli/src/test/java/org/apache/lens/cli/TestLensCubeCommands.java, line > > 57 > > <https://reviews.apache.org/r/32871/diff/1/?file=916305#file916305line57> > > > > Please consider adding separate modular test case for new command. > > Rajat Khandelwal wrote: > I wanted to have minimal diff. Cleanup of test cases can be separate > task. We have several issues: modularity, relative paths, coverage etc. > > Himanshu Gahlaut wrote: > For new commands, we shall try to add them as separate test methods. > Adding newer test cases is unrelated to cleaning existing test cases.
Defining separate method requires you to have a `Before` method that creates the cube and an `After` method that destroys. The existing function `testCubeCommands` is doing both these things and executing tests in between. This function needs to be refactored for the new tests to be modular. Which makes it pretty related. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32871/#review78924 ----------------------------------------------------------- On April 6, 2015, 12:54 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32871/ > ----------------------------------------------------------- > > (Updated April 6, 2015, 12:54 p.m.) > > > Review request for lens. > > > Bugs: LENS-474 > https://issues.apache.org/jira/browse/LENS-474 > > > Repository: lens > > > Description > ------- > > added method > > > Diffs > ----- > > lens-cli/src/main/java/org/apache/lens/cli/commands/BaseLensCommand.java > cdfdbe84c75640f438c887166ead8f64bbdda3f2 > lens-cli/src/main/java/org/apache/lens/cli/commands/LensCubeCommands.java > 9d835da2c30e096ca77558e075930f0d912e8c17 > lens-cli/src/test/java/org/apache/lens/cli/TestLensCubeCommands.java > 41f2c9336c3ec25ecfbe8ba289eab462d69844d5 > lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java > f22515508cd4ceaf7b67ba4cb40ce0df8bf2ac6a > lens-client/src/main/java/org/apache/lens/client/LensClient.java > 8f1b7e18c1cc1a44bfd3c34dc84d9aad86902c9f > lens-client/src/main/java/org/apache/lens/client/LensMetadataClient.java > 3462dc3d6af807ef248e9ee0fbdff27cd0137b2c > > Diff: https://reviews.apache.org/r/32871/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.150s] > [INFO] Lens .............................................. SUCCESS [2.268s] > [INFO] Lens API .......................................... SUCCESS [5.534s] > [INFO] Lens API for server and extensions ................ SUCCESS [6.637s] > [INFO] Lens Cube ......................................... SUCCESS [2:16.913s] > [INFO] Lens DB storage ................................... SUCCESS [10.299s] > [INFO] Lens Query Library ................................ SUCCESS [5.522s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:33.670s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [18.866s] > [INFO] Lens Server ....................................... SUCCESS [4:27.024s] > [INFO] Lens client ....................................... SUCCESS [20.632s] > [INFO] Lens CLI .......................................... SUCCESS [2:42.824s] > [INFO] Lens Examples ..................................... SUCCESS [0.848s] > [INFO] Lens Distribution ................................. SUCCESS [10.130s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:02.403s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.515s] > [INFO] Lens Regression ................................... SUCCESS [0.483s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 14:29.632s > [INFO] Finished at: Mon Apr 06 07:11:15 UTC 2015 > [INFO] Final Memory: 108M/944M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Rajat Khandelwal > >
