Merge lp:~mikemc/ubuntuone-control-panel/fix-1037904-run-tests into lp:ubuntuone-control-panel

Proposed by Mike McCracken
Status: Merged
Approved by: Mike McCracken
Approved revision: 345
Merged at revision: 345
Proposed branch: lp:~mikemc/ubuntuone-control-panel/fix-1037904-run-tests
Merge into: lp:ubuntuone-control-panel
Diff against target: 45 lines (+4/-8)
1 file modified
run-tests (+4/-8)
To merge this branch: bzr merge lp:~mikemc/ubuntuone-control-panel/fix-1037904-run-tests
Reviewer Review Type Date Requested Status
Manuel de la Peña (community) Approve
Roberto Alsina (community) Approve
dobey (community) Approve
Review via email: mp+120057@code.launchpad.net

Commit message

- Remove use of env vars for u1lint and u1trial from old buildout. (LP: #1037904)

Description of the change

- Remove use of env vars for u1lint and u1trial from old buildout. (LP: #1037904)

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :

The attempt to merge lp:~mikemc/ubuntuone-control-panel/fix-1037904-run-tests into lp:ubuntuone-control-panel failed. Below is the output from the failed tests.

*** Running DBus test suite ***

./run-tests: line 68: --reactor=glib: command not found

Revision history for this message
Manuel de la Peña (mandel) wrote :

Finally!

review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :

The attempt to merge lp:~mikemc/ubuntuone-control-panel/fix-1037904-run-tests into lp:ubuntuone-control-panel failed. Below is the output from the failed tests.

*** Running DBus test suite ***

./run-tests: line 68: --reactor=glib: command not found

345. By Mike McCracken

Fixed braino, missed an instance of

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'run-tests'
--- run-tests 2012-06-07 23:00:22 +0000
+++ run-tests 2012-08-20 18:51:09 +0000
@@ -32,7 +32,7 @@
32fi32fi
3333
34style_check() {34style_check() {
35 $u1lint --ignore ubuntuone/controlpanel/gui/qt/ui35 u1lint --ignore ubuntuone/controlpanel/gui/qt/ui
36 if [ -x `which pep8` ]; then36 if [ -x `which pep8` ]; then
37 pep8 --exclude '.svn,CVS,.bzr,.hg,.git,*_ui.py,*_rc.py' --repeat . bin/*37 pep8 --exclude '.svn,CVS,.bzr,.hg,.git,*_ui.py,*_rc.py' --repeat . bin/*
38 else38 else
@@ -52,8 +52,6 @@
52SYSNAME=`uname -s`52SYSNAME=`uname -s`
5353
54if [ "$SYSNAME" == "Darwin" ]; then54if [ "$SYSNAME" == "Darwin" ]; then
55 u1trial="python $u1trial"
56 u1lint="python $u1lint"
57 IGNORE_FILES="$WINDOWS_TESTS, $LINUX_TESTS"55 IGNORE_FILES="$WINDOWS_TESTS, $LINUX_TESTS"
58 IGNORE_DBUS="-p $DBUS_TESTS_PATH"56 IGNORE_DBUS="-p $DBUS_TESTS_PATH"
59 echo "*** Skipping DBus test suite ***"57 echo "*** Skipping DBus test suite ***"
@@ -64,21 +62,19 @@
64 if [ "x$XVFB_CMDLINE" == "x" ]; then62 if [ "x$XVFB_CMDLINE" == "x" ]; then
65 echo "WARNING: Could not find xvfb-run, prepare for visual spam."63 echo "WARNING: Could not find xvfb-run, prepare for visual spam."
66 fi64 fi
67 u1trial=u1trial
68 u1lint=u1lint
69 IGNORE_FILES="$WINDOWS_TESTS, $DARWIN_TESTS"65 IGNORE_FILES="$WINDOWS_TESTS, $DARWIN_TESTS"
70 IGNORE_DBUS=""66 IGNORE_DBUS=""
71 echo "*** Running DBus test suite ***"67 echo "*** Running DBus test suite ***"
72 $u1trial --reactor=glib "$DBUS_TESTS_PATH"68 u1trial --reactor=glib "$DBUS_TESTS_PATH"
73 REACTOR=gi69 REACTOR=gi
74fi70fi
7571
76echo "*** Running test suite for ""$MODULE"" ***"72echo "*** Running test suite for ""$MODULE"" ***"
77$u1trial --reactor=$REACTOR -p "$DBUS_TESTS_PATH, $QT_TESTS_PATH" -i "$IGNORE_FILES" "$MODULE"73u1trial --reactor=$REACTOR -p "$DBUS_TESTS_PATH, $QT_TESTS_PATH" -i "$IGNORE_FILES" "$MODULE"
7874
79echo "*** Running QT test suite for ""$MODULE"" ***"75echo "*** Running QT test suite for ""$MODULE"" ***"
80./setup.py build76./setup.py build
81$XVFB_CMDLINE $u1trial $IGNORE_DBUS -i "$IGNORE_FILES" --reactor=qt4 --gui "$MODULE"77$XVFB_CMDLINE u1trial $IGNORE_DBUS -i "$IGNORE_FILES" --reactor=qt4 --gui "$MODULE"
82rm -rf _trial_temp78rm -rf _trial_temp
83rm -rf build79rm -rf build
8480

Subscribers

People subscribed via source and target branches