Age | Commit message (Collapse) | Author | Files | Lines |
|
Conflicts:
xts5/Makefile.am
Moved the DISPLAY check to the top-level Makefile.am where check-local
is now defined.
|
|
Fixes error message
"_xgeEventToWire: Unknown extension -1, this should never happen."
on Xlib12 XSendEvent 21.
This requires expanding the global event_type array to LASTEvent - 2 instead
of the current hardcoded 32. Since the first two opcodes are used for
replies and errors, two minus the last event opcode is sufficient.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
All tests require DISPLAY to be set and it is rather annoying when a test
runs four hours just to have all the results complain about a missing
DISPLAY environment variable.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
NS_button contains the static button mappings (for whatever reason). Don't
try to access it nbuttons() time because that will return the number of
buttons configured on the device (usually 12). Instead, use the size of the
array we're traversing through.
Aside from running OOB on an array being considered impolite, it also gives
us BadValue when trying to XTestFakeButtonEvent a Button 0, which the test
suite doesn't like.
Fixes UNRESOLVED on test case 10/39.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Optimistic, I know. Life's too short to be pessimistic.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Since most values are optional, set a couple of reasonable defaults, thus
making the need for TET_CONFIG obsolete.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
After parsing the config file, run again through the environment and
check if any of the variables (notably DISPLAY, added here) was set
there. If so, overwrite the config value.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
pt sets up some of the environment vital to the execution of the test.
Add a -gdb flag that simply starts the program through gdb with that
enviroment. The user just has to hit "run" in gdb to debug.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
This patch adds two changes:
Instead of defaulting to a ./Test binary (these aren't built anymore),
take the non-parameter argument as program name. If multiple are
specified, the last one will be picked - don't do that.
export TET_RESFILE as <programname>.log by default unless it's already
set in the environment.
Example use case now:
$> pt XGrabButton
will generate XGrabButton.log
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
The numbers were swapped in the failed/total tests message.
Not sure this is actually useful now that the report is generated, so
it might get removed entirely.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
This partially reverts commit 830f253234c94636c197617466c4b2910a39e574.
We want to keep the part where each individual test prints its status to
stdout, but the part where the test case prints the composite status and
the code to support it is unnecessary.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
This reverts commit 9ae6abd455e9b52bbdf88bb5a03874dc1ad7e26a.
The rationale for this commit was that automake would call the scenario
with a "./" prefix. Since we're no longer using the automake test
framework, we can drop this hack.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Although it would be nice to assume that xts is ready for a milestone
1.0 release, the reality is that it's definitely still rough and will
need a couple prereleases to settle down.
Apologies to anyone who tried to package this already.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
This being the X Test Suite, and not XTest the extension, it is more
appropriate to have the package name be "xts". That's what it's called
internally, anyway.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Commit a2826324 only actually changed the Xproto tests to use the xts
shared libraries. This does it for the rest of the tests. On my x86_64
linux box compiling at the default -g -O2, the installed size has gone
from 237MB to 72MB. Yay.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Making libapi shared saves quite a bit of space since it's linked into
every test program. However, parts of the API are intended to be linked
statically into the program, so libapi and libtcm have been split into
static and shared (_s) portions.
In order to satisy undefined extern variables with shared libraries,
tet_pname and tet_thistest have been added to tcc and exported.
Hopefully, more code can be moved into the shared _s portion with just
small stubs linked statically into the tests.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Now that the regular test cycle generates both the journal and the
summary report, there's really no need for the logfile hack.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Having all the commands and environment variable overrides for in-tree
testing in the Makefile was nice, but a wrapper script means you can do
in-tree testing with or without the "make check" machinery.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
warnings.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
There are still two left:
XDrawRectangle/XDrawRectangle.c:4211: warning: unused variable ‘jssav’
XDrawRectangles/XDrawRectangles.c:4292: warning: unused variable ‘jssav’
These are a little harder to fix because the declaration is generated but jssav
is only used in some of the tests.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
'Artistic' license.
From the license:
3. You may otherwise modify your copy of this Package in any way,
provided that you **insert a prominent notice in each changed file
stating how and when you changed that file**, and provided that you do
at least ONE of the following:
[...]
(emphasis mine)
Most, but not all, of the files in src/tet3 do not have the X11 license at the
top and so are covered by this clause.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Add a TET_NORETURN attribute and tag various fatal functions with it. That
fixes a few "int x; switch(...) { case C: x = blah; break; default: fatal() }"
cases that result in uninitialized variable warnings.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Example:
maperr.c: In function ‘tet_maperrno’:
/home/aaron/src/x/xtest/src/tet3/dtet2lib/maperr.c|61| warning: suggest explicit braces to avoid ambiguous ‘else’
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
These global arrays are used by the error-reporting macros to define the source
file. I guess it's because using __FILE__ everywhere generated tons of copies
of the file path in .data on ancient compilers. Nowadays, they end up with only
one copy in .rodata. To facilitate that, make the arguments to the error
reporting functions const. Propagate constness to eliminate additional
warnings.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
For example:
addarg.c:18: error: ‘sccsid’ defined but not used
These strings are seriously outdated, unused, and cause warnings.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
tprogs lists the test programs without $(EXEEXT), so substituting
the $(EXEEXT) with .c to get BUILD_SOURCES fails to work correctly
when $(EXEEXT) isn't empty
Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
|
|
Allows moving the location of the results directory without having to
recreate the dated subdirectory. For example, "xts-run -o ~/xts-results"
will give you results in ~/xts-results/$date and subsequent runs will
not conflict.
|
|
Collect all the results under one directory instead of cluttering the
current directory. Also, add the seconds to the date in the directory
name to reduce the chance of conflict.
|
|
When using an older automake, the AM_SILENT_RULES macro is not used,
but the custom silent rule for mc uses the variable AM_DEFAULT_VERBOSITY.
Make sure it's defined even without AM_SILENT_RULES.
Tested-by: Pat Kane <pekane52@gmail.com>
|
|
Give users a quick summary of the test results with the xts-report
"no details" mode. This will appear next to the journal as summary.
|
|
Use the package version for reporting the version of xts-run and
xts-report.
|
|
|
|
I still can't find the issue, but AM_V_mc_ was causing make to stop on a
recursive variable. Now it's basically AM_V_GEN with s/GEN/MC/.
|
|
|
|
xts-report makes a lot more sense than vswrpt since everything else is
called xts and you probably couldn't even figure out what vswrpt is.
A few fixes along the way:
- use DEFAULT_TET_ROOT when $TET_ROOT is not set and use it throughout
the code
- fix the argv[0] scanner to also accept xts-report
- comment out a random form feed (wtf?) and a useless error message
- install the docs so xts-report can strip the release data
|
|
Not used yet, but will be much better than constant getenv("TET_ROOT").
|
|
Hopefully by the year 2100 XTS will not be necessary...
|
|
|
|
This will be renamed to xts-report, but it has some clever argv[0]
matching that prevents it for now.
Also fixes a broken comment from an RCS ID removal.
|
|
|
|
|
|
|
|
Linking the xts libraries statically into each test program results in a
massive size for the collection of programs. The libraries have been
changed to private libtool modules that default to shared, resulting in
a much smaller installed size.
One consequence of linking to shared objects is that the linker errors
on undefined extern variables. The obsolete and unused startup routines
in linkstart.c had one of these, so they have been removed from the
build.
Next is to create the tet library shared.
|
|
We want to use the tcc built with xts, so default to setting the full
path instead of hoping to find it in $PATH.
|
|
More tests can be run when using XTEST.
|
|
The XP check for protocol tests in configure was grabbing the same
packages as the general XTS check. Consolidate them.
|