Regarding ReferenceableTest, this can stay in JMS tests for now, since I believe this is actually an old test, not a new one that Diego added.
Yes.. there was only one new test on ReferenceableTest, added to test the JCA Adaptor as referenceable, which is now commented out.
All the others were already there before.
If the commented out test code is no longer needed, it should be removed.
I commented out because the build was broken. I didn't know if the test was valid or not.
If it's not valid as we don't require Referencable on the RA adapter, we can just remove it then.
Ah... duh... Sorry Tim...
I read it wrong berfore.. I read it again now.. I thought you were blaming me for having it commented out...
I'm paranoid.. it's monday.. so.. ignore me :-)
BTW: I have removed the test.
We can recover it if we find a good reason later