----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30661/#review72341 -----------------------------------------------------------
test/integration/component/test_ldap.py <https://reviews.apache.org/r/30661/#comment118431> It is better to skip the test when there are no required fields. test/integration/component/test_ldap.py <https://reviews.apache.org/r/30661/#comment118432> remove this if it is no more required - SrikanteswaraRao Talluri On Feb. 5, 2015, 11:10 a.m., suresh sadhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30661/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2015, 11:10 a.m.) > > > Review request for cloudstack, sanjeev n and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-8218 > https://issues.apache.org/jira/browse/CLOUDSTACK-8218 > > > Repository: cloudstack-git > > > Description > ------- > > added missing scenrio and additional checks for better debugging > > > Diffs > ----- > > test/integration/component/test_ldap.py 3464022 > > Diff: https://reviews.apache.org/r/30661/diff/ > > > Testing > ------- > > Yes > > > Thanks, > > suresh sadhu > >
