Add tests to Actions

Bug #1523513 reported by Jean-Emile DARTOIS on 2015-12-07
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
watcher
Medium
Vincent Françoise

Bug Description

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

tags: added: low-hanginf
tags: added: low-hanging-fruit
removed: low-hanginf
Changed in watcher:
assignee: nobody → Steve Wilkerson (sw5822)
Changed in watcher:
status: New → In Progress
Changed in watcher:
importance: Undecided → Medium
milestone: none → mitaka-2
Changed in watcher:
assignee: Steve Wilkerson (sw5822) → nobody
Changed in watcher:
assignee: nobody → Jean-Emile DARTOIS (jed56)
summary: - Add tests to primitive
+ Add tests to Actions
description: updated
Changed in watcher:
milestone: mitaka-2 → mitaka-3
Changed in watcher:
assignee: Jean-Emile DARTOIS (jed56) → nobody
Changed in watcher:
assignee: nobody → Vincent Françoise (vincent-francoise)

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  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers