watcher

Add tests to Actions

Bug #1523513 reported by Jean-Emile DARTOIS
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
watcher
Fix Released
Medium
Vincent Françoise
watcher mitaka-3 "m3"

Bug Description

The applier has some Actions
 such as Migrate, ChangePowerstate , ... (watcher/applier/actions/...) and there is no unit tests.

See original description
Jean-Emile DARTOIS (jed56)
tags: added: low-hanginf
tags: added: low-hanging-fruit
removed: low-hanginf
Steve Wilkerson (srwilker)
Changed in watcher:
assignee: nobody → Steve Wilkerson (sw5822)
Steve Wilkerson (srwilker)
Changed in watcher:
status: New → In Progress
Antoine Cabot (antoinecabot)
Changed in watcher:
importance: Undecided → Medium
milestone: none → mitaka-2
Steve Wilkerson (srwilker)
Changed in watcher:
assignee: Steve Wilkerson (sw5822) → nobody
Jean-Emile DARTOIS (jed56)
Changed in watcher:
assignee: nobody → Jean-Emile DARTOIS (jed56)
Jean-Emile DARTOIS (jed56)
summary: - Add tests to primitive
+ Add tests to Actions
description: updated
Jean-Emile DARTOIS (jed56)
Changed in watcher:
milestone: mitaka-2 → mitaka-3
Jean-Emile DARTOIS (jed56)
Changed in watcher:
assignee: Jean-Emile DARTOIS (jed56) → nobody
Vincent Françoise (vincent-francoise)
Changed in watcher:
assignee: nobody → Vincent Françoise (vincent-francoise)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to watcher (master)

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

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

Reviewed: https://review.openstack.org/278380
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=55aeb783e3bfbd024cbd1ffa1b4c6e098a6ee933
Submitter: Jenkins
Branch: master

commit 55aeb783e3bfbd024cbd1ffa1b4c6e098a6ee933
Author: Vincent Françoise <email address hidden>
Date: Wed Feb 10 14:59:14 2016 +0100

    Added unit tests on actions

    As we had a low test coverage on actions, I added some more tests
    with this patchset. This actually revealed a small bug (typo) in
    "change_nova_service_state" which has been fixed in here.

    Note that Tempest test also cover these action via the
    basic_consolidation strategy.

    Change-Id: I2d7116a6fdefee82ca254512a9cf50fc61e3c80e
    Closes-Bug: #1523513

Changed in watcher:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.