db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mike Matrigali <mikem_...@sbcglobal.net>
Subject Re: [jira] Commented: (DERBY-999) test SURTest.junit fails with ibm142
Date Thu, 02 Mar 2006 21:11:27 GMT
Yes I think it is fine to add a test which you know is going to
cause an error, if you don't add it to the nightly suite.  As you
say it is the way to naturally share the test with others.

I didn't realize this one was one particular version of jdk, I was
assuming that it was a generic 1.4.2 issue as I knew there were issues
with jdbc4.0 vs. 1.4.2.

Daniel John Debrunner wrote:
> Mike Matrigali wrote:
>>New tests that break regression suite should be backed out of suite. 
> This case is interesting because the test does "pass" on Sun's jdk
> 1.4.2, that I think is reasonable for test that truly passes to be an
> addition to the derbyall regression suite. [ Assuming of course the test
> is not intentionally written to only run on a single jvm. ]
> Now if a valid test fails on IBM's 1.4.2, that's a problem for whoever
> has the itch for derby to run on that vm, same as if the test failed on
> Jrockit.
> But in this case, the test is actually failing, but running clean since
> the failures are in the master file. Since the failures are not expected
> in the long run, I assume not a lot of work has gone into making the
> failure output consistent across jvms. I'm not sure what should be done
> about this. Seems to be wait for the patch that implements the
> functionality or remove it from the suites.
> I know that it wastes my time, having this consistent failure in my
> runs, also conditions me to ignore that test in the future. If I'm
> breaking that in some way, and don't realize it's meant to run clean,
> then I'm going to ignore it's error and checkin a bad fix.
> I just added a test that I knew showed an error
> (AggregateClassLoading.java) but didn't add it to the suites until I had
> the fix, seemed more natural to me. Makes the test available for others
> to use, doesn't break anyone.
> Dan.

View raw message