jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chris Custine <notificati...@github.com>
Subject Re: [jclouds-labs-google] JCLOUDS-643: Fix Google and OAuth tests (#37)
Date Fri, 01 Aug 2014 00:17:34 GMT
> @@ -33,6 +33,7 @@
>  import com.google.common.collect.ImmutableSet;
>  import com.google.common.collect.Lists;
>  
> +@Test(groups = "live", testName = "FirewallApiLiveTest")

This is another thing I learned about testng recently.  The default testName is "", and testng
will actually group together anything without testName in the annotation as all being part
of the same test when it does its parallelization and ordering for dependencies.  So what
happens is that test methods from different test classes are interleaved in a single thread.
 In the case of these tests, it meant that there were too many networks created and the interleaving
bottlenecked the last test method which deleted the network, so it maxed the 5 network quota
on gce.

That was a mouthful, but I will explain more of this when I submit my other testng cleanup
PR :-)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/37/files#r15677803
Mime
View raw message