RFR(M) : 8177507 : line number sensitive tests for jdi should be unified

[Igor Ignatyev]

looks like I have uploaded the webrev before I saved files... uploaded a new webrev w/ all mentioned changes: http://cr.openjdk.java.net/~iignatyev/8177507/webrev.02 <http://cr.openjdk.java.net/~iignatyev/8177507/webrev.02>

-- Igor

> On Apr 2, 2017, at 1:31 PM, David Holmes <david.holmes at oracle.com <mailto:david.holmes at oracle.com>> wrote:
> 
> Hi Igor,
> 
> On 3/04/2017 3:09 AM, Igor Ignatyev wrote:
>> Hi,
>> 
>> http://cr.openjdk.java.net/~iignatyev//8177507/webrev.01 <http://cr.openjdk.java.net/~iignatyev//8177507/webrev.01> is the next
>> iteration w/ copyright years fixed,
> 
> test/com/sun/jdi/BreakpointTest.java
> 
> also needs fixing - says 2015.
> 
>> LineNumberOnBraceTarg and
> 
> test/com/sun/jdi/LineNumberOnBraceTest.java - Ok.
> 
>> LambdaBreakpointTest made more unified w/ the rest.
> 
> I don't see any change here.
> 
> Thanks,
> David
> -----
> 
>> Thanks,
>> -- Igor
>> 
>>> On Mar 29, 2017, at 12:57 PM, serguei.spitsyn at oracle.com <mailto:serguei.spitsyn at oracle.com>
>>> <mailto:serguei.spitsyn at oracle.com <mailto:serguei.spitsyn at oracle.com>> wrote:
>>> 
>>> This one also does not look unified:
>>> http://cr.openjdk.java.net/~iignatyev/8177507/webrev.00/test/com/sun/jdi/LambdaBreakpointTest.java.udiff.html <http://cr.openjdk.java.net/~iignatyev/8177507/webrev.00/test/com/sun/jdi/LambdaBreakpointTest.java.udiff.html>
>> 
>>> On 3/29/17 11:08, Mikhailo Seledtsov wrote:
>>> 
>>> One style nit:
>>> 
>>> LineNumberOnBraceTarg:
>>>  All other Java tests use style of CAP_UNDERSCORE (e.g. STOP_LINE)
>>> for line number variables, but this test uses 'stopLine'.
>>> Consider changing it to STOP_LINE (and STOP_LINE_2) to be uniform.
>> 
>>> On Mar 28, 2017, at 6:37 PM, David Holmes <david.holmes at oracle.com <mailto:david.holmes at oracle.com>
>>> <mailto:david.holmes at oracle.com <mailto:david.holmes at oracle.com>>> wrote:
>>> 
>>> Two nits:
>>> - test/com/sun/jdi/FetchLocals.java
>>> - test/com/sun/jdi/LambdaBreakpointTest.java
>>> 
>>> Second copyright year should be 2017.
>> 
>>> On Mar 24, 2017, at 1:56 PM, Igor Ignatyev <igor.ignatyev at oracle.com <mailto:igor.ignatyev at oracle.com>
>>> <mailto:igor.ignatyev at oracle.com <mailto:igor.ignatyev at oracle.com>>> wrote:
>>> Hi all,
>>> 
>>> could you please review this fix for 8177507?
>>> 
>>> due to their nature, some of jdi tests are line number sensitive.
>>> unfortunately different tests indicate that differently, so it's quite
>>> easy to overlook that and incidentally break tests, for example by
>>> changing module dependency declaration or license modification. this
>>> fix unifies the way line number sensitivity is indicated and also
>>> improves readability/maintainability of some tests by using constant
>>> fields instead of magic numbers.
>>> 
>>> some of line number sensitive tests have been unexpectedly removed
>>> from execution because they had @test/nodynamiccopyright/ instead of
>>> @test tag. this changeset fixes and returns them to regular execution.
>>> 
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8177507/webrev.00 <http://cr.openjdk.java.net/~iignatyev/8177507/webrev.00>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8177507 <https://bugs.openjdk.java.net/browse/JDK-8177507>
>>> testing: test/com/sun/jdi

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20170403/f22d1489/attachment-0001.html>