Use context manager for assertRaises

[pferate]

In the unit tests, update the assertRaises calls to use the context manager.

self.assertRaises(SomeException, do_something, arg1)

change to:

with self.assertRaises(SomeException):
    do_something(arg1)

[pferate]

According to the Python docs, this was added in 2.7.

There are some tests that are already using this syntax, so I'm curious to see what's going on there that doesn't have them fail in py26. I will look into this later.

If this is incompatible with 2.6, then this may need to wait until 2.6 support is dropped in September.

[theacodes]

unittest2 probably fixes compatibility.

[pferate]

Ahhh, that looks like it. Thanks. It will probably be next week before I
could work on it.

On Thu, Jun 30, 2016, 2:46 PM Jon Wayne Parrott notifications@github.com
wrote:

unittest2 probably fixes compatibility.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#536 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAckK8ff5ZNux-4N2vNdlKRBHGv4lPROks5qRDfCgaJpZM4JCm4O
.