----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52911/#review152870 -----------------------------------------------------------
pom.xml (line 273) <https://reviews.apache.org/r/52911/#comment222040> Had to change this because of Hamcrest sharelib/oozie/pom.xml (line 66) <https://reviews.apache.org/r/52911/#comment222041> mockito-all contains hamcrest as well, so using core only to avoid class loading problems - Peter Bacsko On okt. 17, 2016, 1:49 du, Peter Bacsko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52911/ > ----------------------------------------------------------- > > (Updated okt. 17, 2016, 1:49 du) > > > Review request for oozie, András Piros, Attila Sasvari, Peter Cseh, and > Robert Kanter. > > > Repository: oozie-git > > > Description > ------- > > New unit tests for LauncherAM class. > > These tests already provide good coverage of the class (around ~80%). Some > code paths are missing but the most important scenarios are covered. > > > Diffs > ----- > > pom.xml ef22b39 > sharelib/oozie/pom.xml 3ea10a5 > > sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/LauncherAMTestMainClass.java > PRE-CREATION > > sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java > PRE-CREATION > > sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestLauncherAM.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/52911/diff/ > > > Testing > ------- > > > Thanks, > > Peter Bacsko > >
