unit test search unnecessary directories for test cases

Bug #1368509 reported by Can Zhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Can Zhang

Bug Description

Currently in `.testr.conf`, the test_command is written as `${PYTHON:-python} -m subunit.run discover ceilometer -t . $LISTOPT $IDOPTION`, which actually searches `ceilometer/ceilometer/*`. But as all unit tests lie in `ceilometer/ceilometer/tests`, the test_command should use `ceilometer/tests` instead of `ceilometer`.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/120969

Changed in ceilometer:
assignee: nobody → Can Zhang (acme-ican)
status: New → In Progress
gordon chung (chungg)
Changed in ceilometer:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/120969
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=dd772619e8722612a68f0b27787e6cc662e7654a
Submitter: Jenkins
Branch: master

commit dd772619e8722612a68f0b27787e6cc662e7654a
Author: Can ZHANG <email address hidden>
Date: Fri Sep 12 12:01:11 2014 +0800

    Eliminate unnecessary search for test cases

    As all unit tests lie in `ceilometer/ceilometer/tests`,
    the test_command should use `ceilometer/tests` instead of `ceilometer`.

    Change-Id: Idb1dade44a06c71fb5e64ca875b71633def110a5
    Closes-Bug: 1368509

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → kilo-1
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: kilo-1 → 2015.1.0
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.