Summary

Work around assertRaisesMessage not working on the latest Python 2.7.x.

Review Request #7466 — Created June 28, 2015 and submitted July 1, 2015, 4:51 p.m.

Information

Owner

chipx86

Repository

Djblets

Branch

release-0.8.x

Bugs

Depends On

Commit

a81c288...

Reviewers

Groups

djblets

People

Description

There's a bug with Django's assertRaisesMessage and Python 2.7.9/2.7.10,
which causes all usage as a context manager to fail. Python has reverted
the change for 2.7.11, but Django has no plans to work around that.

The workaround is easy, though. We just don't want to pass a default
value for the callable down to assertRaisesRegex. This change provides
our own implementation that fixes this issue.

Testing Done

All formerly-failing unit tests now pass.

Issues

Description	From	Last Updated
undefined name 'six'	reviewbot	June 30, 2015, 2:56 p.m.

Tool: PEP8 Style Checker
Processed Files:
    djblets/testing/testcases.py



Tool: Pyflakes
Processed Files:
    djblets/testing/testcases.py

djblets/testing/testcases.py (Diff revision 1)
The issue has been resolved. Show all issues
```
 undefined name 'six'
```

Change Summary:

Missed the six import when doing git add -p.

Commit:

-	70f016dfeefc146df7549f90a18b2ddea26ce00a
+	a81c2886bf4272ed3bfdf2845bafa7e2b98e973f

Diff:

Revision 2 (+17)

Show changes

djblets/testing/testcases.py

Tool: Pyflakes
Processed Files:
    djblets/testing/testcases.py



Tool: PEP8 Style Checker
Processed Files:
    djblets/testing/testcases.py

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-0.8.x (236c77c)