----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56013/#review163493 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/UpdateKerberosConfigsServerActionTest.java <https://reviews.apache.org/r/56013/#comment234964> Please make sure tests pass on `trunk`, too, not only on `branch-2.5`. ``` [INFO] --- maven-checkstyle-plugin:2.17:check (checkstyle) @ ambari-server --- [INFO] Starting audit... [ERROR] ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/UpdateKerberosConfigsServerActionTest.java:47: 'com.google.inject.AbstractModule' should be separated from previous imports. [ImportOrder] Audit done. ``` - Attila Doroszlai On Jan. 30, 2017, 11:30 a.m., Eugene Chekanskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56013/ > ----------------------------------------------------------- > > (Updated Jan. 30, 2017, 11:30 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Robert Levas. > > > Bugs: AMBARI-19741 > https://issues.apache.org/jira/browse/AMBARI-19741 > > > Repository: ambari > > > Description > ------- > > fixing filed tests > > > Diffs > ----- > > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/UpdateKerberosConfigsServerActionTest.java > e756491 > > Diff: https://reviews.apache.org/r/56013/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Eugene Chekanskiy > >
