Cinder

Unit tests belong in cinder/tests/unit/*

Bug #1462106 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
John Griffith
Cinder 7.0.0 "liberty"

Bug Description

Change: f09c4b1aa0d76d5063d63b12a597da7fead0b93e was approved and merged but the test is in the root test directory instead of the tests/unit directory where it belongs.

Review: https://review.openstack.org/#/c/177665/

John Griffith (john-griffith)
Changed in cinder:
assignee: nobody → John Griffith (john-griffith)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: John Griffith (john-griffith) → Walt Boring (walter-boring)
status: New → In Progress
assignee: Walt Boring (walter-boring) → John Griffith (john-griffith)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Walter A. Boring IV (hemna) (<email address hidden>) on branch: master
Review: https://review.openstack.org/188601
Reason: jdg put in a patch to fix this as well.

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

Reviewed: https://review.openstack.org/188603
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=cc00b33d19bc637d4ad1261a7232da72f84466e9
Submitter: Jenkins
Branch: master

commit cc00b33d19bc637d4ad1261a7232da72f84466e9
Author: John Griffith <email address hidden>
Date: Thu Jun 4 15:39:13 2015 -0600

    Move Dothill tests out of root test directory

    The Dothill code merged but unfortunately had it's
    unit tests in the base tests directory.

    This patch just moves it to tests/unit/volume/drivers

    Change-Id: I010ed2f784384e272599ec3139957989f8838e6f
    Closes-Bug: #1462106

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
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.