Testrepository

testr run should support limiting ids

Bug #597060 reported by James Westby
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Testrepository
Fix Released
Wishlist
Robert Collins
Testrepository 0.0.8

Bug Description

testr run currently appends any of its arguments to the command with
the default $IDLIST etc. substituted. I expected it to substitute the arguments
as $IDLIST, to allow me to limit the tests that are run.

Thanks,

James

Tags: testr-run

Related branches

lp://qastaging/~james-w/testrepository/run-specify-ids
testrepository committers: Pending requested
Diff: 58 lines (+24/-1)
3 files modified
NEWS (+3/-0)
testrepository/commands/run.py (+6/-1)
testrepository/tests/commands/test_run.py (+15/-0)
Jonathan Lange (jml)
tags: added: testr-run
Revision history for this message
Jonathan Lange (jml) wrote :

Apparently related to the mysterious bug 531666.

Robert Collins (lifeless)
Changed in testrepository:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Robert Collins (lifeless) wrote :

So James' sketch is good. However I think its important to differentiate between 'here is something for the underlying app' and 'here is a test selector for testr'. Consider too the id vs regex-match UI aspect.

An interim step I'm adding is to support --load-list, which is in many ways superior to passing ids on the command line.

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

What do you think of this idea:
# pass to underlying commands
testr run -- foo bar
# testr interprets as regex matches
testr run foo bar

?

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

Further to that you'd have
testr run foo bar --load-list FILENAME

would apply foo and bar as regex matches further refining the tests listed by load-list ?

Robert Collins (lifeless)
Changed in testrepository:
status: Triaged → Fix Committed
assignee: nobody → Robert Collins (lifeless)
milestone: none → next
Robert Collins (lifeless)
Changed in testrepository:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.