RFR: 8318573: The nsk.share.jpda.SocketConnection should fail if socket was closed. [v2]

[Leonid Mesnik]

> There are several failure with reason
> "Cannot invoke "String.equals(Object)" because "<local1>" is null"
> which really caused by reading the command from debugee via socket.
> Like:
> 
>         String command = pipe.readln();
> 
>         if (!command.equals(AbstractDebuggeeTest.COMMAND_READY)) {
>             setSuccess(false);
>             log.complain("TEST BUG: unknown debuggee's command: " + command);
> 
>             return false;
>         }
> 
> The command is null when socket fails with EOFException which usually means that debugee has been crashed/exited or killed by timeout handler.
> 
> However, it is not clear from the error log. So
> https://bugs.openjdk.org/browse/JDK-8310940 has different problems with the same NPE symptom.
> 
> The fix update error handling to improve logging.
> 
> So output loos like:
> 
> _open_test_hotspot_jtreg_vmTestbase_nsk_jdi_VirtualMachine_dispose_dispose002_TestDescription_java/tmp' \
>         -Dtest.compiler.opts= \
>         -Dtest.java.opts= \
>         -Dtest.jdk=/home/lmesnik/ws/jdk-vmTestbase/build/linux-x64/images/jdk \
>         -Dcompile.jdk=/home/lmesnik/ws/jdk-vmTestbase/build/linux-x64/images/jdk \
>         -Dtest.timeout.factor=4.0 \
>         -Dtest.nativepath=/home/lmesnik/ws/jdk-vmTestbase/build/linux-x64/images/test/hotspot/jtreg/native \
>         -Dtest.root=/home/lmesnik/ws/jdk-vmTestbase/open/test/hotspot/jtreg \
>         -Dtest.name=vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002/TestDescription.java \
>         -Dtest.file=/home/lmesnik/ws/jdk-vmTestbase/open/test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002/TestDescription.java \
>         -Dtest.src=/home/lmesnik/ws/jdk-vmTestbase/open/test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002 \
>         -Dtest.src.path=/home/lmesnik/ws/jdk-vmTestbase/open/test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002:/home/lmesnik/ws/jdk-vmTestbase/open/test/hotspot/jtreg/vmTestbase:/home/lmesnik/ws/jdk-vmTestbase/open/test/lib \
>         -Dtest.classes=/home/lmesnik/ws/jdk-vmTestbase/build/linux-x64/test-support/jtreg_open_test_hotspot_jtreg_vmTestbase_nsk_jdi_VirtualMachine_dispose_dispose002_TestDescription_java/classes/0/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002/TestDescription.d \
>         -Dtest.class.path=/home/lmesnik/ws/jdk-vmTestbase/build/linux-x64/test-support/jtreg_open_test_hotspot_jtreg_vmTestbase_nsk_jdi_VirtualMachine_dispose_dispose002_TestDescription_java/classes/0/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002/Te...

Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:

  Update test/hotspot/jtreg/vmTestbase/nsk/share/jpda/SocketConnection.java
  
  Co-authored-by: Chris Plummer <chris.plummer at oracle.com>

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/16280/files
  - new: https://git.openjdk.org/jdk/pull/16280/files/d305b0ea..48a7cf9c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16280&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16280&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/16280.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16280/head:pull/16280

PR: https://git.openjdk.org/jdk/pull/16280