Ubuntu Software Center

Merge lp:~mvo/software-center/improve-dep8-feedback into lp:software-center

  1. improve-dep8-feedback
  2. Merge into trunk
Proposed by Michael Vogt
Status: Merged
Merged at revision: 3112
Proposed branch: lp:~mvo/software-center/improve-dep8-feedback
Merge into: lp:software-center
Diff against target: 27 lines (+10/-1)
2 files modified
debian/tests/run-tests (+8/-1)
run-tests.sh (+2/-0)
To merge this branch: bzr merge lp:~mvo/software-center/improve-dep8-feedback
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+120168@code.launchpad.net

Description of the change

Output failure logs for the dep8/ADT test runs on jenkins

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) wrote :

Cool!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/tests/run-tests'
2--- debian/tests/run-tests 2012-08-14 13:34:40 +0000
3+++ debian/tests/run-tests 2012-08-17 13:42:58 +0000
4@@ -1,2 +1,9 @@
5 #!/bin/sh
6-xvfb-run ./run-tests.sh
7+set -e
8+
9+if ! xvfb-run ./run-tests.sh; then
10+ # show all the failures for easier analyzing
11+ cat ./tests/output/*.FAIL
12+ exit 1
13+fi
14+
15
16=== modified file 'run-tests.sh'
17--- run-tests.sh 2012-06-27 20:00:07 +0000
18+++ run-tests.sh 2012-08-17 13:42:58 +0000
19@@ -76,6 +76,8 @@
20 if ! $PYTHON -v -c -b $TEST_PREFIX.$TEST_NAME > $OUTPUT/$TEST_NAME.out 2>&1; then
21 FAILED="$FAILED $TEST_NAME"
22 echo "[ FAIL ]"
23+ # add .FAIL symlink to make finding the broken ones trivial
24+ (cd $OUTPUT ; ln -s $TEST_NAME.out $TEST_NAME.out.FAIL)
25 else
26 echo "[ OK ]"
27 rm -f ${OUTPUT}/$file.out;

Subscribers

People subscribed via source and target branches