jmx-dev JDK-7120365 DiffHBTest.java fails due to ConcurrentModificationException
Alan Bateman
Alan.Bateman at oracle.com
Thu Dec 27 23:39:21 PST 2012
On 28/12/2012 03:22, Stuart Marks wrote:
> :
>
> Alan, you've fixed a bunch of tests that were missing @run tags at
> least twice in the past [1], [2]. I observe that this recent changeset
> [3] removed an @run tag that was necessary to run the test. It's not
> quite the same pathology, but it demonstrates that it's pretty easy to
> get the jtreg tags wrong.
>
> Sorry Shanliang, I'm using your changeset as a springboard for this
> discussion of jtreg tag style. I really do recommend that each test
> include an explicit @run tag, since the tags are so easy to get wrong.
That's fine, my main observation (after fixing several of these) is that
the mistake happen once it becomes necessary to use @build or @compile.
It is really easy to not observe that the test isn't actually executed
if there isn't an @run. Anyway, I think this is something for a regular
checking script to check for, it just needs to have a list of the
compile-only tests so that it doesn't flag as missing a tag.
-Alan
More information about the serviceability-dev
mailing list