Add a test-suite for in-server unit-testing.
This patch adds a test/ directory that contains the setup for a unit-testing suite designed for in-server unit-testing. All functions available to the X server are available to the test binaries through static linking. This test suite uses the glib testing framework. Do not use glib calls outside of the test/ directory. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
parent
987579c930
commit
4124c465a8
12
configure.ac
12
configure.ac
|
@ -439,6 +439,9 @@ AC_ARG_ENABLE(werror, AS_HELP_STRING([--enable-werror],
|
|||
AC_ARG_ENABLE(debug, AS_HELP_STRING([--enable-debug],
|
||||
[Enable debugging (default: disabled)]),
|
||||
[DEBUGGING=$enableval], [DEBUGGING=no])
|
||||
AC_ARG_ENABLE(unit-tests, AS_HELP_STRING([--enable-unit-tests],
|
||||
[Enable unit-tests (default: enabled)]),
|
||||
[UNITTESTS=$enableval], [UNITTESTS=yes])
|
||||
AC_ARG_WITH(int10, AS_HELP_STRING([--with-int10=BACKEND], [int10 backend: vm86, x86emu or stub]),
|
||||
[INT10="$withval"],
|
||||
[INT10="$DEFAULT_INT10"])
|
||||
|
@ -1141,6 +1144,14 @@ if test "x$DEBUGGING" = xyes; then
|
|||
fi
|
||||
AM_CONDITIONAL(DEBUG, [test "x$DEBUGGING" = xyes])
|
||||
|
||||
if test "x$UNITTESTS" = xyes; then
|
||||
AC_DEFINE(UNITTESTS, 1, [Enable unit tests])
|
||||
PKG_CHECK_MODULES([GLIB], [glib-2.0 >= 2.16])
|
||||
AC_SUBST([GLIB_LIBS])
|
||||
AC_SUBST([GLIB_CFLAGS])
|
||||
fi
|
||||
AM_CONDITIONAL(UNITTESTS, [test "x$UNITTESTS" = xyes])
|
||||
|
||||
AC_DEFINE(XTEST, 1, [Support XTest extension])
|
||||
AC_DEFINE(XSYNC, 1, [Support XSync extension])
|
||||
AC_DEFINE(XCMISC, 1, [Support XCMisc extension])
|
||||
|
@ -1980,5 +1991,6 @@ hw/kdrive/fbdev/Makefile
|
|||
hw/kdrive/linux/Makefile
|
||||
hw/kdrive/sdl/Makefile
|
||||
hw/kdrive/src/Makefile
|
||||
test/Makefile
|
||||
xorg-server.pc
|
||||
])
|
||||
|
|
39
test/Makefile.am
Normal file
39
test/Makefile.am
Normal file
|
@ -0,0 +1,39 @@
|
|||
if UNITTESTS
|
||||
check_PROGRAMS = xkb
|
||||
check_LTLIBRARIES = libxservertest.la
|
||||
|
||||
TESTS=$(check_PROGRAMS)
|
||||
|
||||
AM_CFLAGS = $(DIX_CFLAGS) $(GLIB_CFLAGS) @XORG_CFLAGS@
|
||||
INCLUDES = @XORG_INCS@
|
||||
TEST_LDADD=libxservertest.la $(XORG_SYS_LIBS) $(XSERVER_SYS_LIBS) $(GLIB_LIBS)
|
||||
|
||||
xkb_LDADD=$(TEST_LDADD)
|
||||
|
||||
|
||||
libxservertest_la_LIBADD = \
|
||||
$(XSERVER_LIBS) \
|
||||
$(top_builddir)/hw/xfree86/loader/libloader.la \
|
||||
$(top_builddir)/hw/xfree86/os-support/libxorgos.la \
|
||||
$(top_builddir)/hw/xfree86/common/libcommon.la \
|
||||
$(top_builddir)/hw/xfree86/parser/libxf86config.la \
|
||||
$(top_builddir)/hw/xfree86/dixmods/libdixmods.la \
|
||||
$(top_builddir)/hw/xfree86/modes/libxf86modes.la \
|
||||
$(top_builddir)/hw/xfree86/ramdac/libramdac.la \
|
||||
$(top_builddir)/hw/xfree86/ddc/libddc.la \
|
||||
$(top_builddir)/hw/xfree86/i2c/libi2c.la \
|
||||
$(top_builddir)/hw/xfree86/dixmods/libxorgxkb.la \
|
||||
$(top_builddir)/hw/xfree86/libxorg.la \
|
||||
$(top_builddir)/mi/libmi.la \
|
||||
$(top_builddir)/os/libos.la \
|
||||
@XORG_LIBS@
|
||||
|
||||
CLEANFILES=libxservertest.c
|
||||
|
||||
libxservertest.c:
|
||||
touch $@
|
||||
|
||||
all:
|
||||
@echo "Run 'make check' to run the test suite"
|
||||
|
||||
endif
|
36
test/README
Normal file
36
test/README
Normal file
|
@ -0,0 +1,36 @@
|
|||
X server test suite
|
||||
|
||||
This suite contains a set of tests to verify the behaviour of functions used
|
||||
internally to the server. This test suite is based on glib's testing
|
||||
framework [1].
|
||||
|
||||
= How it works =
|
||||
Through some automake abuse, we link the test programs with the same static
|
||||
libraries as the Xorg binary. The test suites can then call various functions
|
||||
and verify their behaviour - without the need to start the server or connect
|
||||
clients.
|
||||
|
||||
This testing only works for functions that do not rely on a particular state
|
||||
of the X server. Unless the test suite replicates the expected state, which
|
||||
may be difficult.
|
||||
|
||||
= How to run the tests =
|
||||
Run "make check" the test directory. This will compile the tests and execute
|
||||
them in the order specified in the TESTS variable in test/Makefile.am.
|
||||
|
||||
Each set of tests related to a subsystem are available as a binary that can be
|
||||
executed directly. For example, run "xkb" to perform some xkb-related tests.
|
||||
|
||||
== Adding a new test ==
|
||||
When adding a new test, ensure that you add a short description of what the
|
||||
test does and what the expected outcome is. If the test reproduces a
|
||||
particular bug, using g_test_bug().
|
||||
|
||||
== Misc ==
|
||||
|
||||
The programs "gtester" and "gtester-report" may be used to generate XML/HTML
|
||||
log files of tests succeeded and failed.
|
||||
|
||||
---------
|
||||
|
||||
[1] http://library.gnome.org/devel/glib/stable/glib-Testing.html
|
173
test/xkb.c
Normal file
173
test/xkb.c
Normal file
|
@ -0,0 +1,173 @@
|
|||
/**
|
||||
* Copyright © 2009 Red Hat, Inc.
|
||||
*
|
||||
* Permission is hereby granted, free of charge, to any person obtaining a
|
||||
* copy of this software and associated documentation files (the "Software"),
|
||||
* to deal in the Software without restriction, including without limitation
|
||||
* the rights to use, copy, modify, merge, publish, distribute, sublicense,
|
||||
* and/or sell copies of the Software, and to permit persons to whom the
|
||||
* Software is furnished to do so, subject to the following conditions:
|
||||
*
|
||||
* The above copyright notice and this permission notice (including the next
|
||||
* paragraph) shall be included in all copies or substantial portions of the
|
||||
* Software.
|
||||
*
|
||||
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
||||
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
||||
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
||||
* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
||||
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
|
||||
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
|
||||
* DEALINGS IN THE SOFTWARE.
|
||||
*/
|
||||
|
||||
#ifdef HAVE_DIX_CONFIG_H
|
||||
#include <dix-config.h>
|
||||
#endif
|
||||
|
||||
#include <xkb-config.h>
|
||||
|
||||
#include <stdio.h>
|
||||
#include <stdlib.h>
|
||||
#include <ctype.h>
|
||||
#include <unistd.h>
|
||||
#include <math.h>
|
||||
#include <X11/X.h>
|
||||
#include <X11/Xproto.h>
|
||||
#include <X11/keysym.h>
|
||||
#include <X11/Xatom.h>
|
||||
#include "misc.h"
|
||||
#include "inputstr.h"
|
||||
#include "opaque.h"
|
||||
#include "property.h"
|
||||
#define XKBSRV_NEED_FILE_FUNCS
|
||||
#include <xkbsrv.h>
|
||||
#include "../xkb/xkbgeom.h"
|
||||
#include <X11/extensions/XKMformat.h>
|
||||
#include "xkbfile.h"
|
||||
#include "../xkb/xkb.h"
|
||||
|
||||
#include <glib.h>
|
||||
|
||||
/**
|
||||
* Initialize an empty XkbRMLVOSet.
|
||||
* Call XkbGetRulesDflts to obtain the default ruleset.
|
||||
* Compare obtained ruleset with the built-in defaults.
|
||||
*
|
||||
* Result: RMLVO defaults are the same as obtained.
|
||||
*/
|
||||
static void xkb_get_rules_test(void)
|
||||
{
|
||||
XkbRMLVOSet rmlvo = { NULL};
|
||||
XkbGetRulesDflts(&rmlvo);
|
||||
|
||||
|
||||
g_assert(rmlvo.rules);
|
||||
g_assert(rmlvo.model);
|
||||
g_assert(rmlvo.layout);
|
||||
g_assert(rmlvo.variant);
|
||||
g_assert(rmlvo.options);
|
||||
g_assert(strcmp(rmlvo.rules, XKB_DFLT_RULES) == 0);
|
||||
g_assert(strcmp(rmlvo.model, XKB_DFLT_MODEL) == 0);
|
||||
g_assert(strcmp(rmlvo.layout, XKB_DFLT_LAYOUT) == 0);
|
||||
g_assert(strcmp(rmlvo.variant, XKB_DFLT_VARIANT) == 0);
|
||||
g_assert(strcmp(rmlvo.options, XKB_DFLT_OPTIONS) == 0);
|
||||
}
|
||||
|
||||
/**
|
||||
* Initialize an random XkbRMLVOSet.
|
||||
* Call XkbGetRulesDflts to obtain the default ruleset.
|
||||
* Compare obtained ruleset with the built-in defaults.
|
||||
* Result: RMLVO defaults are the same as obtained.
|
||||
*/
|
||||
static void xkb_set_rules_test(void)
|
||||
{
|
||||
XkbRMLVOSet rmlvo = {
|
||||
.rules = "test-rules",
|
||||
.model = "test-model",
|
||||
.layout = "test-layout",
|
||||
.variant = "test-variant",
|
||||
.options = "test-options"
|
||||
};
|
||||
XkbRMLVOSet rmlvo_new = { NULL };
|
||||
|
||||
XkbSetRulesDflts(&rmlvo);
|
||||
XkbGetRulesDflts(&rmlvo_new);
|
||||
|
||||
/* XkbGetRulesDflts strdups the values */
|
||||
g_assert(rmlvo.rules != rmlvo_new.rules);
|
||||
g_assert(rmlvo.model != rmlvo_new.model);
|
||||
g_assert(rmlvo.layout != rmlvo_new.layout);
|
||||
g_assert(rmlvo.variant != rmlvo_new.variant);
|
||||
g_assert(rmlvo.options != rmlvo_new.options);
|
||||
|
||||
g_assert(strcmp(rmlvo.rules, rmlvo_new.rules) == 0);
|
||||
g_assert(strcmp(rmlvo.model, rmlvo_new.model) == 0);
|
||||
g_assert(strcmp(rmlvo.layout, rmlvo_new.layout) == 0);
|
||||
g_assert(strcmp(rmlvo.variant, rmlvo_new.variant) == 0);
|
||||
g_assert(strcmp(rmlvo.options, rmlvo_new.options) == 0);
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Get the default RMLVO set.
|
||||
* Set the default RMLVO set.
|
||||
* Get the default RMLVO set.
|
||||
* Repeat the last two steps.
|
||||
*
|
||||
* Result: RMLVO set obtained is the same as previously set.
|
||||
*/
|
||||
static void xkb_set_get_rules_test(void)
|
||||
{
|
||||
/* This test failed before XkbGetRulesDftlts changed to strdup.
|
||||
We test this twice because the first time using XkbGetRulesDflts we obtain
|
||||
the built-in defaults. The unexpected free isn't triggered until the second
|
||||
XkbSetRulesDefaults.
|
||||
*/
|
||||
XkbRMLVOSet rmlvo = { NULL };
|
||||
XkbRMLVOSet rmlvo_backup;
|
||||
|
||||
XkbGetRulesDflts(&rmlvo);
|
||||
|
||||
/* pass 1 */
|
||||
XkbSetRulesDflts(&rmlvo);
|
||||
XkbGetRulesDflts(&rmlvo);
|
||||
|
||||
/* Make a backup copy */
|
||||
rmlvo_backup.rules = strdup(rmlvo.rules);
|
||||
rmlvo_backup.layout = strdup(rmlvo.layout);
|
||||
rmlvo_backup.model = strdup(rmlvo.model);
|
||||
rmlvo_backup.variant = strdup(rmlvo.variant);
|
||||
rmlvo_backup.options = strdup(rmlvo.options);
|
||||
|
||||
/* pass 2 */
|
||||
XkbSetRulesDflts(&rmlvo);
|
||||
|
||||
/* This test is iffy, because strictly we may be comparing against already
|
||||
* freed memory */
|
||||
g_assert(strcmp(rmlvo.rules, rmlvo_backup.rules) == 0);
|
||||
g_assert(strcmp(rmlvo.model, rmlvo_backup.model) == 0);
|
||||
g_assert(strcmp(rmlvo.layout, rmlvo_backup.layout) == 0);
|
||||
g_assert(strcmp(rmlvo.variant, rmlvo_backup.variant) == 0);
|
||||
g_assert(strcmp(rmlvo.options, rmlvo_backup.options) == 0);
|
||||
|
||||
XkbGetRulesDflts(&rmlvo);
|
||||
g_assert(strcmp(rmlvo.rules, rmlvo_backup.rules) == 0);
|
||||
g_assert(strcmp(rmlvo.model, rmlvo_backup.model) == 0);
|
||||
g_assert(strcmp(rmlvo.layout, rmlvo_backup.layout) == 0);
|
||||
g_assert(strcmp(rmlvo.variant, rmlvo_backup.variant) == 0);
|
||||
g_assert(strcmp(rmlvo.options, rmlvo_backup.options) == 0);
|
||||
}
|
||||
|
||||
|
||||
int main(int argc, char** argv)
|
||||
{
|
||||
g_test_init(&argc, &argv,NULL);
|
||||
g_test_bug_base("https://bugzilla.freedesktop.org/show_bug.cgi?id=");
|
||||
|
||||
g_test_add_func("/xkb/set-get-rules", xkb_set_get_rules_test);
|
||||
g_test_add_func("/xkb/get-rules", xkb_get_rules_test);
|
||||
g_test_add_func("/xkb/set-rules", xkb_set_rules_test);
|
||||
|
||||
return g_test_run();
|
||||
}
|
Loading…
Reference in New Issue
Block a user