test_id_list_default used even when running in parallel

Bug #1027042 reported by Jonathan Lange
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Testrepository
Fix Released
Critical
Robert Collins

Bug Description

Given this configuration::

  [DEFAULT]
  test_command=trial --reporter=subunit $LISTOPT $IDLIST
  test_id_list_default=twisted
  test_id_list_option=--list

`testr run --parallel` does::
  running=trial --reporter=subunit twisted twisted.trial

rather than listing it.

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

It also messes up --parallel. This is a defect that escaped into the wild.

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

Oh bah. So, extra parameters to run are just ignored, they aren't test ids. There is a separate bug about that I believe. I'm going to nab this bug for the parallel thing you found.

description: updated
summary: - test_id_list_default used even when id provided
+ test_id_list_default used even when running in parallel
Revision history for this message
Robert Collins (lifeless) wrote :

I've pushed a fix up. It wants a review of the commit, if you're interested. Its uhm shocking.

Changed in testrepository:
assignee: nobody → Robert Collins (lifeless)
milestone: none → next
Changed in testrepository:
status: Triaged → Fix Committed
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.

Other bug subscribers

Remote bug watches

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