Launchpad itself

Merge lp:~jelmer/launchpad/testr into lp:launchpad

  1. testr
  2. Merge into devel
Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~jelmer/launchpad/testr
Merge into: lp:launchpad
Diff against target: 7 lines (+3/-0)
1 file modified
.testr.conf (+3/-0)
To merge this branch: bzr merge lp:~jelmer/launchpad/testr
Reviewer Review Type Date Requested Status
Robert Collins (community) Approve
Review via email: mp+20511@code.launchpad.net

Commit message

Add a configuration fiile for easier use with testrepository.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

As requested, a testr configuration file for Launchpad :-)

Revision history for this message
Robert Collins (lifeless) wrote :

On Wed, 2010-03-03 at 02:51 +0000, Jelmer Vernooij wrote:

> === added file '.testr.conf'
> --- .testr.conf 1970-01-01 00:00:00 +0000
> +++ .testr.conf 2010-03-03 02:51:19 +0000
> @@ -0,0 +1,3 @@
> +[DEFAULT]
> +test_command=./bin/test --subunit $IDLIST
> +test_id_list_default=

test_id_list_default defaults to '' anyway.

You may be better off with
test_command=./bin/testt --subunit $IDOPTION
test_id_option=--load-list $IDFILE

- I think bin/test has a load-list option now (Jml and I put one
together a while back). This will be more precise at loading things.

-Rob

Revision history for this message
Robert Collins (lifeless) wrote :

Looks fine to me now, especially as you say you've tested it ;)

Revision history for this message
Robert Collins (lifeless) wrote :

big green button.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file '.testr.conf'
2--- .testr.conf 1970-01-01 00:00:00 +0000
3+++ .testr.conf 2010-03-04 02:04:27 +0000
4@@ -0,0 +1,3 @@
5+[DEFAULT]
6+test_command=./bin/test --subunit $IDOPTION
7+test_id_option=--load-list $IDFILE