RFR(XXS): 8141146: Add Error hierarchy to test library

[Christian Tornqvist]

Hi Kirill,

>I think it indicates that we need a separate error for test bug/environment issue. It will allow us to identify test/env issues easily and assign them directly to SQE avoiding spending any dev time.

I'm still very sceptic how you'd reliably separate the issues into test/environment/product issues and what "SQE avoiding spending any dev time" means?

Again, I'd love to see some examples of why this would be a worthwhile addition. 

Thanks,
Christian

-----Original Message-----
From: Kirill Zhaldybin [mailto:kirill.zhaldybin at oracle.com] 
Sent: Monday, November 2, 2015 1:57 PM
To: Christian Tornqvist <christian.tornqvist at oracle.com>; 'hotspot-dev' <hotspot-dev at openjdk.java.net>
Subject: Re: RFR(XXS): 8141146: Add Error hierarchy to test library

Hi Christian,

We already have more than 2k statements like "throw new testbug" in hotspot/test.
I think it indicates that we need a separate error for test bug/environment issue. It will allow us to identify test/env issues easily and assign them directly to SQE avoiding spending any dev time.

I created a sub-task on new Asserts framework here -
https://bugs.openjdk.java.net/browse/JDK-8141204
We are really appreciate your comments.

Thank you.

Regards, Kirill


On 02.11.2015 21:20, Christian Tornqvist wrote:
> Hi Kirill,
>
> I'm still doubting how useful this would be, can you give some examples of where these different exceptions would be used and what the benefit would be? How would this be used with the current Assert class?
>
> Thanks,
> Christian
>
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On 
> Behalf Of Kirill Zhaldybin
> Sent: Monday, November 2, 2015 12:56 PM
> To: hotspot-dev <hotspot-dev at openjdk.java.net>
> Subject: RFR(XXS): 8141146: Add Error hierarchy to test library
>
> Hi,
>
> Could somebody please review this change?
> It adds EnvironmentIssue and TestBug classes for environment issue and test bug correspondingly to the test library.
> It is the first part of JDK-8065651: Enhance Asserts to throw different exceptions for different types of assertions.
>
> CR: https://bugs.openjdk.java.net/browse/JDK-8141146
> WebRev: 
> http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8141146/webrev.00/
>
> Thank you.
>
> Regards, Kirill
>