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

[Sergey Bylokhov]

On Wed, 1 Sep 2021 17:34:14 GMT, lawrence.andrews <github.com+87324768+lawrence-andrew at 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