I'm not a huge fan of this structure. The syntax is such that it's difficult to see what is being called, and I'm not bothered about the exact contents of the error message being tested. I would prefer:
```python
with self.assertRaises(ValueError):
    func_that_should_raise()
```