Merge lp:~pitti/friends/fix-autopkgtest into lp:friends

Proposed by Martin Pitt
Status: Merged
Approved by: Ken VanDine
Approved revision: 164
Merged at revision: 162
Proposed branch: lp:~pitti/friends/fix-autopkgtest
Merge into: lp:friends
Diff against target: 35 lines (+13/-2)
3 files modified
debian/changelog (+10/-0)
debian/tests/control (+1/-1)
debian/tests/unit (+2/-1)
To merge this branch: bzr merge lp:~pitti/friends/fix-autopkgtest
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ken VanDine Approve
Review via email: mp+153749@code.launchpad.net

Commit message

* debian/tests/unit: Redirect stderr to stdout, as unittest discover writes
  to stderr by default. Succeeding autopkgtests must not write anything
  there.
* Run autopkgtests under xvfb, otherwise D-BUS autospawn fails.
* Add dbus-x11 test dependency, for D-BUS autospawning.

Description of the change

friends' autopgkgtests are failing for multiple reasons:
http://10.98.0.1:8080/view/Raring/view/AutoPkgTest/job/raring-adt-friends/

This branch fixes them all, so that this works now:

  run-adt-test -sS lp:~pitti/friends/fix-autopkgtest friends

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:164
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~pitti/friends/fix-autopkgtest/+merge/153749/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/friends-ci/7/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/friends-raring-amd64-ci/7/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/friends-ci/7/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks great, thanks!

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-03-11 10:43:36 +0000
3+++ debian/changelog 2013-03-18 10:55:26 +0000
4@@ -1,3 +1,13 @@
5+friends (0.1.2daily13.03.11.1-0ubuntu2) UNRELEASED; urgency=low
6+
7+ * debian/tests/unit: Redirect stderr to stdout, as unittest discover writes
8+ to stderr by default. Succeeding autopkgtests must not write anything
9+ there.
10+ * Run autopkgtests under xvfb, otherwise D-BUS autospawn fails.
11+ * Add dbus-x11 test dependency, for D-BUS autospawning.
12+
13+ -- Martin Pitt <martin.pitt@ubuntu.com> Mon, 18 Mar 2013 11:41:27 +0100
14+
15 friends (0.1.2daily13.03.11.1-0ubuntu1) raring; urgency=low
16
17 [ Robert Bruce Park ]
18
19=== modified file 'debian/tests/control'
20--- debian/tests/control 2012-10-22 15:31:06 +0000
21+++ debian/tests/control 2013-03-18 10:55:26 +0000
22@@ -1,3 +1,3 @@
23 Tests: unit
24-Depends: @, python3, python3-distutils-extra, python3-pkg-resources, python3-setuptools, python3-gobject, python3-gi, python3-dbus, python3-mock, python3-oauthlib, gir1.2-dee-1.0, gir1.2-soup-2.4, gir1.2-glib-2.0, gir1.2-gdkpixbuf-2.0, gir1.2-signon-1.0, gir1.2-networkmanager-1.0, libaccounts-glib-dev, libsignon-glib1, shared-mime-info
25+Depends: @, python3, python3-distutils-extra, python3-pkg-resources, python3-setuptools, python3-gobject, python3-gi, python3-dbus, python3-mock, python3-oauthlib, gir1.2-dee-1.0, gir1.2-soup-2.4, gir1.2-glib-2.0, gir1.2-gdkpixbuf-2.0, gir1.2-signon-1.0, gir1.2-networkmanager-1.0, libaccounts-glib-dev, libsignon-glib1, shared-mime-info, xvfb, dbus-x11
26
27
28=== modified file 'debian/tests/unit'
29--- debian/tests/unit 2012-10-22 13:21:40 +0000
30+++ debian/tests/unit 2013-03-18 10:55:26 +0000
31@@ -1,2 +1,3 @@
32 #! /bin/sh
33-make check
34+# succeeding test must not write anything to stderr, as per DEP-8
35+xvfb-run make check 2>&1

Subscribers

People subscribed via source and target branches

to all changes: