Hi Roger,
On 2/26/2014 12:34 PM, roger riggs wrote:
The testlibrary for the jdk should be printing the values in the failed
assertions to make debugging easier and quicker.
The webrev adds the printing of the failed assertions and added methods
for formatting and unconditional fail methods.
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-testlibrary-asserts-8035889/
AssertsTest.java: line 28: @library doesn't look like it's needed.
There is no jdk/test/testlibrary directory and I think jdk.testlibrary.*
are found as relative to $test.src.
Otherwise, the change looks okay.
Now that jtreg supports TestNG and I wonder if this class should retire
some day (there are only about 10 existing tests using this class). Are
you writing new tests using this Asserts class?
Mandy
Bug:
8035889: jdk testlibrary - add printing of values of failed assertions
Thanks, Roger
[1] https://bugs.openjdk.java.net/browse/JDK-8035889