Changing `expect(x.indexOf(y)).toBeGraterThan(-1)` to be `expect(x).toContain(y)`. Some usages of the former slipped into our tests along the way and have since been propagating.
Fixed DownloadPanelItemTemplate test as it was incorrect (it did not test what the author had intended).
Merge pull request #1286 from aodn/tidying_jasmine_tests

Tidying jasmine tests