whitebox tests should be converted to nova unit tests

Bug #1222852 reported by Sean Dague
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Adalberto Medeiros
tempest
Fix Released
High
Matthew Treinish

Bug Description

The whitebox tests in tempest are actually basically state change tests for nova, which could be better done as nova unit tests. This is especially true because changing the database in tempest is somewhat a verboten thing. It would also solve the nightly fails on tempest-all by removing them.

Tags: testing
Sean Dague (sdague)
Changed in tempest:
status: New → Confirmed
milestone: none → havana-rc
importance: Undecided → High
Revision history for this message
Matthew Treinish (treinish) wrote :

I added nova to the bug so we can track adding the new unit tests that mirror the whitebox coverage separately from pulling whitebox out of tempest.

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

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

Changed in tempest:
assignee: nobody → Matthew Treinish (treinish)
status: Confirmed → In Progress
Matt Riedemann (mriedem)
tags: added: testing
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/46116
Committed: http://github.com/openstack/tempest/commit/de8378b65be5e4a8d7de71b769ea4aba71a9aee7
Submitter: Jenkins
Branch: master

commit de8378b65be5e4a8d7de71b769ea4aba71a9aee7
Author: Matthew Treinish <email address hidden>
Date: Wed Sep 11 14:19:46 2013 -0400

    Remove Whitebox tests

    This commit removes the whitebox tests from tempest. The proper place
    for this type of testing is in nova unit tests not in tempest.

    Fixes bug 1222852

    Change-Id: I0fac7b030c51985f9d6d93129bf9bab75c18cd11

Changed in tempest:
status: In Progress → Fix Committed
Sean Dague (sdague)
Changed in nova:
importance: Undecided → High
milestone: none → havana-rc1
importance: High → Medium
assignee: nobody → Adalberto Medeiros (adalbas)
Changed in nova:
milestone: havana-rc1 → ongoing
Changed in nova:
status: New → Confirmed
Sean Dague (sdague)
Changed in tempest:
status: Fix Committed → Fix Released
Revision history for this message
Joe Gordon (jogo) wrote :

 sdague> mark it as fix released or invalid

Changed in nova:
status: Confirmed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: ongoing → none
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.