The test is functionally good enough to be merged now once the parser bug is fixed. But personally I still find the `setUp` method messy and would split up into more methods, or at least move more of the stuff that's related to the presentations to the `test_get_presentations` method.

IMO, clean code would be something like

```python
def test_get_presentations(self):
    feed_url = 'http://fosslc.org/drupal/presentations_rss/summercamp2010'
    mock_request(feed_url, 'presentation_feed.rss')
    actual_presentations = FeedParser().get_presentations(feed_url)
    expected_presentations = load_json_presentations('presentation_feed.json')
    self.assertEqual(actual_presentations, expected_presentations)
```

I'm sure it could even be a lot nicer than that. But like I said, the code is mergeable as is, but it could be nicer.