Comment 7 for bug 1557584

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/296081
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=726943f9f83c75557d6e35fc6c6b2b4f24a4c75c
Submitter: Jenkins
Branch: master

commit 726943f9f83c75557d6e35fc6c6b2b4f24a4c75c
Author: Diana Clarke <email address hidden>
Date: Tue Mar 22 17:42:19 2016 -0400

    Clean up the TestGlanceClientWrapper retry tests

    I promised to follow up the fix for Bug #1557584 with a patch to clean
    up all the TestGlanceClientWrapper retry tests rather than just the one
    I added. This patch breaks the retry tests out into a dedicated test
    class (TestGlanceClientWrapperRetries), makes sure spec is declared for
    all MagicMocks, adds a couple custom asserts and helper methods, and
    otherwise makes the group of tests more consistent.

    Change-Id: I8533f5a99d6fd7a232cd65dae7aae895a7fd00aa
    Related-Bug: #1557584