harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Zakharov" <alexei.zakha...@gmail.com>
Subject Re: svn commit: r472149 - /incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/PersistenceDelegateTest.java
Date Thu, 09 Nov 2006 10:13:40 GMT
Let me rephrase my question a little bit. What should we do in this
particular case when some test passes on J9 and HDK (DRLVM with
BUILD_CFG=release) and fails/crashes on DRLVM with BUILD_CFG=debug? I
have two examples of such tests already.

Thanks,

2006/11/9, Alexei Zakharov <alexei.zakharov@gmail.com>:
> Stepan, Tim,
>
> Well, I am not sure here. All tests passes both on J9 and DLRVM Relase
> build (HDK is a RELEASE build is far as I understand). But it crashes
> on DRLVM DEBUG - this makes our drlvm guys a bit nervous.  As far as I
> understand the reason is too complex to be fixed promptly. Probably
> the solution I have chosen is not the best one. So I am open for
> proposals here. Should we exclude the whole test? In my opinion DRLVM
> is not just "another_VM" - it is something we should care about. IMHO.
>
> Thanks,
>
> 2006/11/8, Tim Ellison <t.p.ellison@gmail.com>:
> > Stepan Mishura wrote:
> > > Hi Alexei,
> > >
> > > Sorry, I don't understand your logic. Is the test case valid? If there was
> > > another bug (for example: "[another_VM][unit] half of classlib beans tests
> > > crashes VM"), would you agree to comment out a half of beans tests?
> >
> > Thanks Stepan, I had also intended to point out this commit.  I agree
> > that it is masking a problem elsewhere which should be fixed, and I'd
> > also point out that commenting out test code is not the preferred way to
> > exclude a test since it will be very difficult to find the excluded test
> > and include it again.  We have been using the build scripts to exclude
> > test classes.



-- 
Alexei Zakharov,
Intel Enterprise Solutions Software Division

Mime
View raw message