<Sound Dev> RFR: 7188098 : TEST_BUG: closed/javax/sound/midi/Synthesizer/Receiver/bug6186488.java fails [v2]

[lawrence.andrews]

> 1) Testcase was failing with Parse Exception: Arguments to `manual' option not supported: yesno due to yesno
> 2) After removing =yesno once again test case was failing with following exception
> ----------System.err:(15/782)----------
> java.io.IOException: Illegal seek
> at java.io.FileInputStream.skip(Native Method)
> at java.io.BufferedInputStream.skip(BufferedInputStream.java:366)
> at bug6186488.main(bug6186488.java:36)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:474)
> at com.sun.javatest.regtest.MainWrapper$MainThread.run(MainWrapper.java:96)
> at java.lang.Thread.run(Thread.java:722)
> 
> 3) If test case is run independently user has to enter y or n to make the test case pass or fail that is when user heard the sound or no. The same action was able to perform via JTREG.  
> 4) Rewrote the testcase . Now user can listen to sound when he press "Start Test" and press "pass"  button if user hears the sound  and press "fail" button when user does not hear the sound. Test UI also shows a timer that get timeout if user does not perform any action.
> 5) Add try-with-resource to close the synth.
> 
> @shurymury

lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision:

  Added extra instruction for the user

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5338/files
  - new: https://git.openjdk.java.net/jdk/pull/5338/files/c64e9480..863171b0

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5338&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5338&range=00-01

  Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5338.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5338/head:pull/5338

PR: https://git.openjdk.java.net/jdk/pull/5338