ironic.tests.base.TestCase uses database fixture while lots of tests don't need it

Bug #1376285 reported by Vladyslav Drok
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Vladyslav Drok

Bug Description

ironic.tests.base.TestCase uses database fixture while lots of tests don't need it. This fixture should be moved to a separate test case.

Vladyslav Drok (vdrok)
Changed in ironic:
assignee: nobody → Vladyslav Drok (vdrok)
Dmitry Tantsur (divius)
Changed in ironic:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/125361
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=d45eae7fe12bbdf3a47b151ad025d564fbdf2a3f
Submitter: Jenkins
Branch: master

commit d45eae7fe12bbdf3a47b151ad025d564fbdf2a3f
Author: Vladyslav Drok <email address hidden>
Date: Wed Oct 1 15:55:37 2014 +0300

    Move database fixture to a separate test case

    This change moves database fixture from ironic.tests.base.TestCase to
    ironic.tests.db.base.DbTestCase and adds dbapi field there. Also it
    adds ironic admin context to ironic.tests.base.TestCase, as context
    may be used without db fixture.

    Closes-bug: #1376285

    Change-Id: I008e44da315767afb6282000a01f0b39698f9768

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