RFR: 8315990: Amend problemlisted tests to proper position [v3]
Alexey Ivanov
aivanov at openjdk.org
Tue Sep 12 10:33:41 UTC 2023
On Tue, 12 Sep 2023 03:32:04 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> test/jdk/ProblemList.txt line 471:
>>
>>> 469: java/awt/Mixing/AWT_Mixing/OpaqueOverlapping.java 8294264 windows-x64
>>> 470:
>>> 471: java/awt/dnd/MissingDragExitEventTest/MissingDragExitEventTest.java 8288839 windows-x64
>>
>> These tests are excluded on Windows, they don't seem to belong under the heading provided.
>>
>> Should they rather be placed closer to similar tests, in alphabetical order?
>
> There was a line gap which should signify these tests doesn't belong to that heading, but anyways I have placed those tests closer to similar tests..
Thanks.
I can't find any guidelines where newly added tests should be placed. Looking at the list, I have the impression that it was common to add tests to the same group, keeping the list sorted alphabetically.
However, as far as I can see, most new lines are added to an end of a section. This is easier, I guess…
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15655#discussion_r1322838901
More information about the client-libs-dev
mailing list