Pin tornado version to <5

[timcnicholls]

This PR pins the requirement for tornado to <5, to avoid issues with installation and hanging test runs. Also minor modifications to some tests to complete coverage of recent access logging changes.

[ajgdls]

Tests completed. Version of tornado installed was less than 5.

[timcnicholls]

@ajgdls Thanks for testing. Related Q: excalibur-detector/control has this repo tag 0.2 during setup script so fails its own tests due to the tornado issue above. Possible fixes are for it to re-tag (.e.g 0.4.0 - but I know you recently tagged 0.3.0), decouple excalibur-detector/control from a specific release so it checks out head, or have a 'soft' tag that we move as necessary. Thoughts?

[ajgdls]

Hi @timcnicholls I don't mind if you think we should make new releases. The modules have all now been released for DLS and so there is no issue with making new commits/releases. Another release of excalibur-detector will be made at DLS soon enough I'm sure.
We could make it 0.3.1 if we think it is minor?

[timcnicholls]

@ajgdls OK thanks, I'll merge the PR and tag 0.3.1.