Re: Review Request for PR 609

2015-07-22 Thread Rajani Karuturi
I ran them from intellij (right click on the test package, run tests) It also used to work from command line (mvn test from the ldap project) Its not working currently due to the commit ec32ea30f7b3e5351e661786955d9fa0929047bd which changed gmaven plugin version. I will revert that and update the

Re: Review Request for PR 609

2015-07-21 Thread Daan Hoogland
Lot of work Rajani, I don't understand the spec files in groovy, yet. How can I verify these are run? On Wed, Jul 22, 2015 at 6:39 AM, Rajani Karuturi wrote: > Can someone please review PR 609? Its an enhancement to LDAP import users > functionality > > https://github.com/apache/cloudstack/pull/6

Review Request for PR 609

2015-07-21 Thread Rajani Karuturi
Can someone please review PR 609? Its an enhancement to LDAP import users functionality https://github.com/apache/cloudstack/pull/609 Thanks, ~Rajani