<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">That aside, Phil, Sergey, I would really like your opinion on test selection. Did I run the right tests to verify the fix?<div class=""><br class=""></div><div class="">Shura<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Nov 7, 2018, at 3:05 PM, Alexandre (Shura) Iline <<a href="mailto:alexandre.iline@oracle.com" class="">alexandre.iline@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html; charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Phil, Igor,<div class=""><br class=""></div><div class="">A quick check shows that indeed most of cases where a “lib” library used by relative path (which may or may not be the top level library) are in the client tests.</div><div class=""><br class=""></div><div class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo;" class=""><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">$ egrep -r '@library.* (../)+lib' --include "*.java" . | cut -d ':' -f 1 | cut -d / -f 2-3 | uniq -c</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     59 java/awt</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">      1 tools/jimage</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     15 tools/jlink</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     43 com/sun</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     39 javax/swing</span></div><div class=""><span style="font-variant-ligatures: no-common-ligatures" class=""><br class=""></span></div></div><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo;" class=""><span style="font-variant-ligatures: no-common-ligatures" class=""><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo;" class=""><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">$ egrep -r '@library.* (../)+lib' --include "*.java" com/sun | cut -d ':' -f 1 | cut -d / -f 2-3 | uniq -c</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     43 sun/jndi</span></div></div></div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><br class=""></div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class="">Further, almost all of such cases, in client tests, is included into my last fix:</div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><br class=""></div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo;" class=""><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">$ egrep -r '@library.* (../)+lib/client' --include "*.java" . | cut -d ':' -f 1 | cut -d / -f 2-3 | uniq -c</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     56 java/awt</span></div><div style="margin: 0px; font-stretch: normal; line-height: normal;" class=""><span style="font-variant-ligatures: no-common-ligatures" class="">     39 javax/swing</span></div></div><div class=""><br class=""></div></div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class="">The is to say, I will update the webrev with using absolute path to lib/client.</div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><br class=""></div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class="">Shura</div><div style="font-family: "Helvetica Neue"; font-size: 12px;" class=""><br class=""></div></span></div><div class=""><br class=""><blockquote type="cite" class=""><div class="">On Nov 6, 2018, at 5:30 PM, Philip Race <<a href="mailto:philip.race@oracle.com" class="">philip.race@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type" class="">
  
  <div bgcolor="#FFFFFF" text="#000000" class="">
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1" class="">
    I don't understand why the relative path is used  in some two cases
    and<br class="">
    an absolute path in others. OK you didn't change that but does it
    make<br class="">
    sense to standardise on what I hope is the absolute path which is
    easier<br class="">
    to get right and will withstand the tests being moved ? <br class="">
    Can we do the same for "helpers" ?<br class="">
    <br class="">
    -phil.<br class="">
    <br class="">
test/jdk/java/awt/Modal/ModalExclusionTests/ApplicationExcludeDialogFileTest.java<br class="">
    <br class="">
    @@ -29,11 +29,11 @@<br class="">
      * @bug 8047179 8044429<br class="">
      * @summary Check whether a FileDialog blocks an application
    modality excluded Dialog<br class="">
      *          (it shouldn't). Checks also whether setting a parent
    dialog to be<br class="">
      *          modality excluded excludes its children from being
    blocked too.<br class="">
      *<br class="">
    - * @library ../helpers /lib/testlibrary/<br class="">
    + * @library ../helpers /lib/client/<br class="">
      * @library /test/lib<br class="">
      * @build ExtendedRobot<br class="">
      * @build Flag<br class="">
      * @build TestDialog<br class="">
      * @build TestWindow<br class="">
    <br class="">
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1" class="">
test/jdk/java/awt/Modal/ModalDialogOrderingTest/ModalDialogOrderingTest.java<br class="">
    <br class="">
    @@ -33,11 +33,11 @@<br class="">
      * @test<br class="">
      * @key headful<br class="">
      * @bug 8008728<br class="">
      * @summary [macosx] Swing. JDialog. Modal dialog goes to
    background<br class="">
      * @author Alexandr Scherbatiy<br class="">
    - * @library ../../../../lib/testlibrary<br class="">
    + * @library ../../../../lib/client<br class="">
      * @build ExtendedRobot<br class="">
      * @run main ModalDialogOrderingTest<br class="">
      */<br class="">
     public class ModalDialogOrderingTest {<br class="">
     <br class="">
    <br class="">
    -phil.<br class="">
    <br class="">
    On 11/6/18, 5:21 PM, Alexandre (Shura) Iline wrote:
    <blockquote cite="mid:78863B1F-9B4C-4507-BBFC-680965D62984@oracle.com" type="cite" class="">
      <pre wrap="" class="">Ping.

</pre>
      <blockquote type="cite" class="">
        <pre wrap="" class="">On Oct 29, 2018, at 3:02 PM, Alexandre (Shura) Iline <a class="moz-txt-link-rfc2396E" href="mailto:alexandre.iline@oracle.com"><alexandre.iline@oracle.com></a> wrote:

Hi.

Please take a look on this fix for JDK-8186549.

For testing I have selected tests and groups of tests which use the ExtendedRobot class. The tests show the same pass rate. There are still few tests which use lib/testlibrary, they are not affected by this fix.

Task: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8186549">https://bugs.openjdk.java.net/browse/JDK-8186549</a>
Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~shurailine/8186549/webrev.00">http://cr.openjdk.java.net/~shurailine/8186549/webrev.00</a>
Tests used for testing the fix: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~shurailine/8186549/extended_robot_tests">http://cr.openjdk.java.net/~shurailine/8186549/extended_robot_tests</a>

Shura

</pre>
      </blockquote>
      <pre wrap="" class=""></pre>
    </blockquote>
  </div>

</div></blockquote></div><br class=""></div></div></div></blockquote></div><br class=""></div></body></html>