> On Лис. 10, 2015, 1:19 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/python/ambari_server/serverConfiguration.py, line 983 > > <https://reviews.apache.org/r/40134/diff/1/?file=1121705#file1121705line983> > > > > Should we add an unit test for future regressions?
We don't have UT for serverConfiguration.py at all. And i don't think this is the critical place which can be corrupted. This code changes very rarely. But if you want i can add test for this method. - Vitalyi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40134/#review105850 ----------------------------------------------------------- On Лис. 10, 2015, 1:04 після полудня, Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40134/ > ----------------------------------------------------------- > > (Updated Лис. 10, 2015, 1:04 після полудня) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Dmytro > Sen. > > > Bugs: AMBARI-13809 > https://issues.apache.org/jira/browse/AMBARI-13809 > > > Repository: ambari > > > Description > ------- > > PROBLEM: When going through 'ambari-server setup' and trying to change Oracle > JDK version, option for JDK 1.8 does not exist > > > Diffs > ----- > > ambari-server/src/main/python/ambari_server/serverConfiguration.py 95d59d8 > > Diff: https://reviews.apache.org/r/40134/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >
