test_id_list_default used even when running in parallel

Reported by Jonathan Lange on 2012-07-20
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Testrepository
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.

Related branches

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
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
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  Edit
Everyone can see this information.

Other bug subscribers