[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: RFR: 8274082 : Wrong test case name specified in jtreg run tag for java/awt/print/PrinterJob/Swi
From: Alexandre Iline <shurailine () openjdk ! java ! net>
Date: 2022-03-29 21:33:48
Message-ID: cLsEoc2Gz-Aew_A0eYGOCA_RmjFnhOZWzobvwFRrHCw=.61391d08-a9cc-48d0-b187-a6f283ecd4ec () github ! com
[Download RAW message or body]
On Thu, 24 Mar 2022 20:24:35 GMT, lawrence.andrews <duke@openjdk.java.net> wrote:
> Following issues were fixed in this test
> 1) Fixed - Parser error due to yesno in @run main/manual=yesno
> 2) Fixed Wrong test name specified in @run jtreg
> @run main/manual=yesno PrintTextTest . It should be @run main/manual=yesno \
> SwingUIText 3) Instruction frame does not have 'Pass' or 'Fail' button to mark the \
> test as Pass or Fail after verifying Text on UI and print out. 4) Fixed - Test UI \
> ( test instruction frame, UI frame and Print Dialog) just gets disposed \
> automatically without interacting via running the test case with jtreg.
> @shurymury
test/jdk/java/awt/print/PrinterJob/SwingUIText.java line 96:
> 94: private static void createTestInstructionUI() {
> 95: testInstructionFrame = new JFrame("Test Instruction Frame");
> 96: final String INSTRUCTION = """
Would this make more sense to use the already existing manual test support from the \
A11Y tests?
test/jdk/java/awt/print/PrinterJob/SwingUIText.java line 101:
> 99: as seen on-screen, and that the spacing of the text is of,
> 100: reasonable even-ness. This latter part is very subjective and,
> 101: the comparison has to be with JDK1.5 GA, or JDK 1.6 GA.
Are test executors really to compare to JDK 1.5? Would it be possible to come up with \
a comprehensive criteria of some kind? Worst case, can some reference images be used \
for the test executor to compare the printout to?
-------------
PR: https://git.openjdk.java.net/jdk/pull/7946
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic