On 6/17/2016 4:17 PM, Lindenmaier, Goetz wrote:
Hi Alexandr,
yes, you are right, the test you mention is missing in this change.
There are others, too, and we still have lots of failures for other
reasons.
We are currently working on getting all the tests green in
our test environment where we test linuxppc64, linuxppc64le
and aixppc64 (and, for reference, the Oracle platforms).
So I will address all the remaining issues at some point.
If you basically agree on this change, I would appreciate if we
could push this one and I make a follow up change. Handling
changes with this many files is a pain point. But I can also
extend this change so that we get all of them at once.
As I see there are areas like jdk_beans or jdk_imageio which usually
does not require headful environment and jdk_awt or jdk_swing which
usually requires it. It seems that ordinary AWT/Swing tests require the
"headful" keyword.
May be it is more appropriate to have "headful" keyword for areas
like jdk_beans and "headless" keyword for areas like jdk_awt and
jdk_swing? This will allow to mark only small part of tests with
necessary keyword for each area.
In other way almost all AWT/Swing tests should be marked by "headful"
keyword.
Thanks,
Alexandr.
Best regards,
Goetz.
-----Original Message-----
From: Alexandr Scherbatiy [mailto:alexandr.scherba...@oracle.com]
Sent: Freitag, 17. Juni 2016 14:53
To: Volker Simonis <volker.simo...@gmail.com>
Cc: Lindenmaier, Goetz <goetz.lindenma...@sap.com>; swing-
d...@openjdk.java.net; awt-...@openjdk.java.net
Subject: Re: <Swing Dev> <AWT Dev> RFR(L): 8159690: [TESTBUG] Mark
headful tests with @key headful.
On 6/17/2016 11:36 AM, Volker Simonis wrote:
On Thu, Jun 16, 2016 at 5:55 PM, Alexandr Scherbatiy
<alexandr.scherba...@oracle.com> wrote:
On 6/16/2016 5:40 PM, Lindenmaier, Goetz wrote:
Hi,
we have test machines without X server. On these many of the jdk
jtreg tests fail with a headless exception.
We grepped for this exception in the test output and identified
about 450 tests.
In these tests, we added with another script "@key headful".
What is a number of tests which passe in headless mode?
I would expect that an ordinary client test which use Frame and fails in
headless mode does not require a special key by default.
Hi Alexandr,
I don't quite understand your concerns, but the purpose of this change
is to make it possible to simply exclude all tests which require a
"headful" environment from a jtreg run.
There are AWT/Swing tests which can be run even without X server. For
example:
java/awt/image/DrawImage/DrawImageCoordsTest.java
Others, like for example:
java/awt/image/DrawImage/EABlitTest.jtr
will throw a Headless exception and fail:
java.awt.HeadlessException:
No X11 DISPLAY variable was set, but this program performed an
operation which requires it.
With Goetz's change we can simply exclude all the test which require a
headful environment by specifying "-keywords:\!headful" to jtreg.
After all, I think that's the purpose why the "headful" keyword has
been introduced.
If there's any other simple way of excluding all tests which require a
headful environment, please let us now.
For example, the test
jdk/test/javax/swing/AbstractButton/6711682/bug6711682.java when it is
run with -Djava.awt.headless=true option fails with exception
"java.awt.AWTException: headless environment".
The test is not listed in the proposed patch. Is it correct that this
test requires the "headful" environment and should be marked with the
"headful" keyword?
Thanks,
Alexandr.
Regards,
Volker
Thanks,
Alexandr.
So that the script generates better output, I adapted the
formatting of some of the test descriptions.
see also the text in the webrev, where I posted some incremental diffs
of the changes I more or less edited by hand. I hope this eases
reviewing :)
Last, I updated the Copyrights with the script by Coleen.
Please review this change.
http://cr.openjdk.java.net/~goetz/wr16/8159690-headful/webrev.01/
Best regards,
Goetz.
