On Wed, 1 Sep 2021 17:34:14 GMT, lawrence.andrews <github.com+87324768+lawrence-and...@openjdk.org> wrote:
> 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 This pull request has now been integrated. Changeset: c640fe42 Author: lawrence.andrews <lawrence.andr...@oracle.com> Committer: Sergey Bylokhov <s...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/c640fe42c2b5e6668a2a875678be44443942c868 Stats: 182 lines in 1 file changed: 153 ins; 22 del; 7 mod 7188098: TEST_BUG: closed/javax/sound/midi/Synthesizer/Receiver/bug6186488.java fails Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/5338
