Update redis cluster unit tests for assertRaisesRegexp

Bug #1491918 reported by Morgan Jones
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Low
sandy muñoz

Bug Description

The redis cluster unit tests in trove/tests/unittests/cluster/test_cluster_redis_controller.py should be updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated.

Changed in trove:
status: New → Triaged
tags: added: low-hanging-fruit
Changed in trove:
importance: Undecided → Low
milestone: none → ongoing
Changed in trove:
assignee: nobody → sandy muñoz (sandrely2611)
Revision history for this message
sandy muñoz (sandrely2611) wrote :

I'd appreciate more specific information about what is being asked in this report. What regexp should be asserted?

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

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

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

Reviewed: https://review.openstack.org/232126
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=2eb1902b14161d3957320c6da1064aaba5ef843a
Submitter: Jenkins
Branch: master

commit 2eb1902b14161d3957320c6da1064aaba5ef843a
Author: sandrely26 <email address hidden>
Date: Wed Oct 7 12:16:56 2015 -0500

    Fix redis cluster unit test for assertRaisesRegexp

    The redis cluster unit test was updated to use
    assertRaisesRegexp rather than assertRaises
    to ensure that the correct messages are being generated.

    Closes-Bug: #1491918
    Change-Id: I89f0da62df7933786424ce2e96815765a66625aa

Changed in trove:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/trove 5.0.0.0b1

This issue was fixed in the openstack/trove 5.0.0.0b1 development milestone.

Changed in trove:
status: Fix Committed → Fix Released
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.