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

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
=== modified file 'debian/tests/run-tests'
--- debian/tests/run-tests 2012-08-14 13:34:40 +0000
+++ debian/tests/run-tests 2012-08-17 13:42:58 +0000
@@ -1,2 +1,9 @@
1#!/bin/sh1#!/bin/sh
2xvfb-run ./run-tests.sh2set -e
3
4if ! xvfb-run ./run-tests.sh; then
5 # show all the failures for easier analyzing
6 cat ./tests/output/*.FAIL
7 exit 1
8fi
9
310
=== modified file 'run-tests.sh'
--- run-tests.sh 2012-06-27 20:00:07 +0000
+++ run-tests.sh 2012-08-17 13:42:58 +0000
@@ -76,6 +76,8 @@
76 if ! $PYTHON -v -c -b $TEST_PREFIX.$TEST_NAME > $OUTPUT/$TEST_NAME.out 2>&1; then76 if ! $PYTHON -v -c -b $TEST_PREFIX.$TEST_NAME > $OUTPUT/$TEST_NAME.out 2>&1; then
77 FAILED="$FAILED $TEST_NAME"77 FAILED="$FAILED $TEST_NAME"
78 echo "[ FAIL ]"78 echo "[ FAIL ]"
79 # add .FAIL symlink to make finding the broken ones trivial
80 (cd $OUTPUT ; ln -s $TEST_NAME.out $TEST_NAME.out.FAIL)
79 else81 else
80 echo "[ OK ]"82 echo "[ OK ]"
81 rm -f ${OUTPUT}/$file.out;83 rm -f ${OUTPUT}/$file.out;

Subscribers

People subscribed via source and target branches