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
test/jdk/javax/sound/midi/Synthesizer/Receiver/bug6186488.java line 104:
> 102: private static void createAndShowTestDialog() {
> 103: String testInstruction = "This test verify that software Java
> Syntesizer processed non-ShortMessage-derived messages. \n" +
> 104: "Click on 'Start Test' button. If you listen a sound
> then test pass else test fail.";
Probably it will be useful to ask the user to "Make sure that the speakers are
connected and the volume is up."? Some other tests use this suggestion.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5338
