----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29130/#review65295 -----------------------------------------------------------
ambari-admin/src/main/resources/ui/admin-web/app/views/stackVersions/create.html <https://reviews.apache.org/r/29130/#comment108375> Can we include unit tests for this validation rule? - Yusaku Sako On Dec. 17, 2014, 12:56 a.m., Richard Zang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29130/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2014, 12:56 a.m.) > > > Review request for Ambari, Andrii Tkach and Yusaku Sako. > > > Bugs: AMBARI-8744 > https://issues.apache.org/jira/browse/AMBARI-8744 > > > Repository: ambari > > > Description > ------- > > 1. Only show stack version having upgrade_packs. > 2. Right align stack version pull down. > 3. Add repo version client side validation. > 4. Add client-side validation to make sure at least one OS is selected. > 5. Fix unit test. > > > Diffs > ----- > > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/stackVersions/StackVersionsCreateCtrl.js > e6cc743 > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/stackVersions/StackVersionsListCtrl.js > 034537f > ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/Stack.js > aa574c8 > ambari-admin/src/main/resources/ui/admin-web/app/styles/main.css 8136088 > > ambari-admin/src/main/resources/ui/admin-web/app/views/stackVersions/create.html > 20fe4c4 > > ambari-admin/src/main/resources/ui/admin-web/test/unit/controllers/stackVersions/StackversionsListCtrl_test.js > PRE-CREATION > > ambari-admin/src/main/resources/ui/admin-web/test/unit/controllers/stackVersions/StackversoinsListCtrl_test.js > 8865180 > > Diff: https://reviews.apache.org/r/29130/diff/ > > > Testing > ------- > > Manually tested in live cluster. Unit test fixed. > > > Thanks, > > Richard Zang > >
