Reviewing the correct code now... I agree with John that a comment about the sleep should be added.
test_aimdns.py - typo at line 87, 164, 189 and 233 'ahve' -> 'have' - Given that this code is the same for all of the tests, why not put it in a method that they can all call? Otherwise, looking good. Thanks, Darren. On 26/08/2011 14:09, Matt Keenan wrote: > Hi, > > Can I get a review and blessing for fix for : > 7050547 ai_sd/aimdns unit tests failing on machine not configured as > AI server > http://monaco.sfbay.sun.com/detail.jsf?cr=7050547 > > Webrev: > https://cr.opensolaris.org/action/browse/caiman/mattman/7050547/ > > > Slim install unit tests tend to be run on a build machine which is not > configured as an install server, thus these tests fail. Nose provides a > SkipTest exception which when used will not report the tests themselves > as failed. > > For ai_sd if multicast DNS SMF is not available then the tests will fail. > > For aimdns if install/server SMF is not available 5 tests will fail. > > I also discovered that when a service is removed, it remains advertised > for a default time of 120 seconds, this results in time.sleep(120) calls > being required for tests to be successful. > > cheers > > Matt > _______________________________________________ > caiman-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
