RFR: 8193154: methods in java.time's TCKZoneRules OpenJDK test miss @Test annotation (original) (raw)
Lance Andersen lance.andersen at oracle.com
Sun May 27 12:04:12 UTC 2018
- Previous message: RFR: 8193154: methods in java.time's TCKZoneRules OpenJDK test miss @Test annotation
- Next message: [11] RFR JDK-8203474: Update description of "Cyrillic Supplementary" block name in Character.UnicodeBlock class.
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Vivek,
I suspect the submitter of the issue, was not aware of the option of putting the annotation on the class. If you needed additional attributes for the annotation, then it makes sense to add them but given there are no attributes needed, the annotation on the class should be enough
Best Lance
On May 27, 2018, at 3:25 AM, Vivek Theeyarath <vivek.theeyarath at oracle.com> wrote:
Hi Lance, Thanks for the feedback. I did notice the @Test at the class level. The motivation behind the fix was a comment in the bug saying not having the annotation at the method level affects correct processing of TCKZoneRules and inclusion of test methods into master test list where this test has been ported to. So, I am assuming this would still help. But for that, I agree with you that this may not be needed. Regards Vivek From: Lance Andersen Sent: Saturday, May 26, 2018 6:06 PM To: Vivek Theeyarath <vivek.theeyarath at oracle.com <mailto:vivek.theeyarath at oracle.com>> Cc: core-libs-dev <core-libs-dev at openjdk.java.net <mailto:core-libs-dev at openjdk.java.net>> Subject: Re: RFR: 8193154: methods in java.time's TCKZoneRules OpenJDK test miss @Test annotation Hi Vivek, The original test has the @Test annotation on the class which indicates that all public methods are test methods: http://testng.org/doc/documentation-main.html#class-level <http://testng.org/doc/documentation-main.html#class-level> So adding @Test to each method is OK but not needed. HTH Best Lance
On May 26, 2018, at 3:51 AM, Vivek Theeyarath <vivek.theeyarath at oracle.com <mailto:vivek.theeyarath at oracle.com>> wrote: Hi All, Please review fix for https://bugs.openjdk.java.net/browse/JDK-8193154 <https://bugs.openjdk.java.net/browse/JDK-8193154> .
http://cr.openjdk.java.net/~vtheeyarath/8193154/webrev.00/ <http://cr.openjdk.java.net/~vtheeyarath/8193154/webrev.00/> The test ran fine with Mach5 http://java.se.oracle.com:10065/mdash/jobs/root-jdk-dev-20180526-0645-24095 <http://java.se.oracle.com:10065/mdash/jobs/root-jdk-dev-20180526-0645-24095> Regards Vivek <image001.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
- Previous message: RFR: 8193154: methods in java.time's TCKZoneRules OpenJDK test miss @Test annotation
- Next message: [11] RFR JDK-8203474: Update description of "Cyrillic Supplementary" block name in Character.UnicodeBlock class.
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]