Merge lp:~pitti/autopilot-gtk/test-improvements into lp:autopilot-gtk

Proposed by Martin Pitt
Status: Merged
Approved by: Allan LeSage
Approved revision: 67
Merged at revision: 66
Proposed branch: lp:~pitti/autopilot-gtk/test-improvements
Merge into: lp:autopilot-gtk
Diff against target: 62 lines (+19/-0)
4 files modified
debian/control (+2/-0)
debian/rules (+2/-0)
debian/tests/autopilot (+4/-0)
debian/tests/control (+11/-0)
To merge this branch: bzr merge lp:~pitti/autopilot-gtk/test-improvements
Reviewer Review Type Date Requested Status
Allan LeSage (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+199510@code.launchpad.net

Commit message

* Respect DEB_BUILD_OPTIONS=nocheck.
* Add autopkgtest.
* Add missing at-spi2-core test dependency (caused tests to run 20 min instead of 40 s).

Description of the change

Fix DEB_BUILD_OPTIONS=nocheck and add autopkgtest.

The tests (both during package build and during autopkgtests) still take
awfully long (20 mins instead of 40 s) because of something that is missing in
those environments. I still need to track that down, but in the meantime these
tests work. I verified in sbuild (during package build, as they've run for
ages) and with run-adt-test.

To post a comment you must log in.
67. By Martin Pitt

Add at-spi2-core build/test dependency

The absence of this caused long D-BUS timeouts for starting applications, so
that our tests took 20 minutes instead of 40 seconds.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Allan LeSage (allanlesage) wrote :

Tested build under both check and nocheck conditions, tests are snappy (~40 sec.) as reported.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-11-06 11:11:53 +0000
3+++ debian/control 2013-12-18 17:48:01 +0000
4@@ -4,6 +4,7 @@
5 XSBC-Original-Maintainer: Allan LeSage <allan.lesage@canonical.com>
6 Build-Depends: autopilot-desktop (>= 1.4),
7 bamfdaemon,
8+ at-spi2-core,
9 cmake,
10 dbus-x11,
11 debhelper (>= 9),
12@@ -24,6 +25,7 @@
13 # and sync up the code again.
14 Vcs-Bzr: https://code.launchpad.net/~autopilot/autopilot-gtk/trunk
15 Vcs-Browser: https://bazaar.launchpad.net/~autopilot/autopilot-gtk/trunk/files
16+XS-Testsuite: autopkgtest
17
18 Package: libautopilot-gtk
19 Section: libs
20
21=== modified file 'debian/rules'
22--- debian/rules 2013-09-17 13:44:29 +0000
23+++ debian/rules 2013-12-18 17:48:01 +0000
24@@ -7,10 +7,12 @@
25 dh $@
26
27 override_dh_auto_test:
28+ifeq (, $(findstring nocheck, $(DEB_BUILD_OPTIONS)))
29 mkdir -p debian/tmp/home/run
30 env HOME=$(CURDIR)/debian/tmp/home XDG_RUNTIME_DIR=$(CURDIR)/debian/tmp/home/run \
31 xvfb-run -a -e $(CURDIR)/debian/tmp/X.log dh_auto_test || \
32 { echo "==== X.org log ===="; cat $(CURDIR)/debian/tmp/X.log; false; }
33+endif
34
35 override_dh_install:
36 dh_install --fail-missing
37
38=== added directory 'debian/tests'
39=== added file 'debian/tests/autopilot'
40--- debian/tests/autopilot 1970-01-01 00:00:00 +0000
41+++ debian/tests/autopilot 2013-12-18 17:48:01 +0000
42@@ -0,0 +1,4 @@
43+#!/bin/sh
44+set -eu
45+cd tests/autopilot
46+autopilot-sandbox-run -a -v tests
47
48=== added file 'debian/tests/control'
49--- debian/tests/control 1970-01-01 00:00:00 +0000
50+++ debian/tests/control 2013-12-18 17:48:01 +0000
51@@ -0,0 +1,11 @@
52+Tests: autopilot
53+Depends: @,
54+ autopilot-desktop (>= 1.4),
55+ bamfdaemon,
56+ at-spi2-core,
57+ xvfb,
58+ dbus-x11,
59+ gnome-system-log,
60+ gsettings-desktop-schemas,
61+ python-gi,
62+ gir1.2-gtk-3.0

Subscribers

People subscribed via source and target branches

to all changes: