> 1) Removed =yesno that was causing the test to fail with following exception > test result: Error. Parse Exception: Arguments to `manual' option not > supported: yesno > After removing =yesno, test was just passing without user interaction so > fixed the following > > Add the following so that user can user interact with the test > a) Added pass/fail button and instruction for the user to know what he/she is > going to test and what is expected. > b) Added Dialog with textarea to enter the reason for the testcase failure. > This will help the user to understanding the reason why the test was failed > while analyzing the results. > > @shurymury
lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: Removed RenderingHints.VALUE_TEXT_ANTIALIAS_GASP and fixed the review comments ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7275/files - new: https://git.openjdk.java.net/jdk/pull/7275/files/5b667fa6..f42940cd Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7275&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7275&range=02-03 Stats: 84 lines in 1 file changed: 25 ins; 36 del; 23 mod Patch: https://git.openjdk.java.net/jdk/pull/7275.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7275/head:pull/7275 PR: https://git.openjdk.java.net/jdk/pull/7275