Not sure .We'll have to wait for Rajesh to answer this . Perhaps he has them ready but probably hasn't sent them for review.
-----Original Message----- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Tuesday, March 19, 2013 12:08 AM To: cloudstack-dev@incubator.apache.org Cc: Rajesh Battala; Brian Federle; Animesh Chaturvedi; Haroon Abdelrahman Subject: Re: [Merged] - CloudStack -664:AWS Health Check feature into Master On Mon, Mar 18, 2013 at 06:35:33PM +0000, Pranav Saxena wrote: > Hi Chip , > > I guess there was a discussion about the reason behind not having unit tests > for this feature now - > (http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.mbox/%3c97f4356aea71904482cd192135c038f9011cc0d20...@banpmailbox01.citrite.net%3E > ) . Not sure , if they hold good now as well otherwise Rajesh would need to > add them . > > Thanks, > Pranav Indeed we did discuss this. It had been over a month and a half, so I was assuming that the comment about Rajesh working on some tests would have meant that they were being worked on... Any reason to not at least have marvin tests?