End to end test coverage of Terms Controller

See #650, #581
@danielbachhuber if this is more of an end-to-end test then I think it should include the authentication. On the other had, if you are just testing the internals of `get_items` it should probably use as little superfluous stuff as possible.

I'm not a big fan of large tests for that reason - it gets very brittle and is also pretty slow to run, so we should keep these to a minimum. It can be temping to make your test run through a huge amount of code with a kind of "the more I cover the higher the chance of catching an error" thinking.