Replace try/except/else with self.assertraises for expected exception tests.

Registered by suresh Govinda Rao

Recently community reached a general consensus on not to use the try/except/else format for testing expected exceptions, instead to use self.assertRaises & addCleanUp for finally part. This proposal is to modify or update the tests as per the above requirements.

Blueprint information

Status:
Complete
Approver:
Jay Pipes
Priority:
Medium
Drafter:
None
Direction:
Approved
Assignee:
Donald Ngo
Definition:
Approved
Series goal:
None
Implementation:
Implemented
Milestone target:
None
Started by
David Kranz
Completed by
David Kranz

Related branches

Sprints

Whiteboard

List of tests required this fix listing here:

https://etherpad.openstack.org/update-expected-exception-tests

Gerrit topic: https://review.openstack.org/#q,topic:bp/update-expected-exception-tests,n,z

Addressed by: https://review.openstack.org/24044
    Replace try/except/else with self.assertRaises.

Addressed by: https://review.openstack.org/24140
    Replace try/except/else with self.assertRaises.

Addressed by: https://review.openstack.org/24141
    Replace try/except/else with self.assertRaises.

Addressed by: https://review.openstack.org/24281
    Replace try/except/else with self.assertRaises.

Addressed by: https://review.openstack.org/24403
    Convert try/expect/else as per new Tempest style.

Addressed by: https://review.openstack.org/24503
    Convert try/expect/else as per new Tempest style.

Addressed by: https://review.openstack.org/25669
    Replace try/except/else with self.assertRaises

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.