Nitpick: as long as you're modifying this code, I much prefer the context manager form of the `assertRaises` statement:
```python
with self.assertRaises(ItemNotFoundError):
    modulestore().get_course(CourseLocator(course_id='GreekHero', branch='published'))
```
Much more readable, and you can execute multiple statements in the block.