> 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

Reply via email to