Scott Wilson

Yup, agreed. Sorry, I somehow had the notion that TestDocument was a more extensive class as added in the other PR.

@brianlheim just to confirm: #4582 was merged to develop, so no need to merge 3.10 to develop before rebasing this?

Yes, sorry it's likely trivial, but I've not had time.

> No problem, just checking what's the status currently. No worries. My work pattern with SC these days is periods of availability for semi-intensive work, followed by being swamped by...

Hi @joshpar Yes ideally. If it’s fine without the test can be merged. Otherwise I might be able to add it tomorrow.

Okay this has now been rebased on latest develop (resolved a conflict) and added @jamshark70's test method. All seems to run fine. I believe this still shows as needing a...

Anything I need to do to move this along?

I'm at bit of a loss with these travis failures. Can someone advise?

> you need to format your code properly Sorry @brianlheim! I tried to use the instructions for fixing PRs, but it ends up marking a ton of stuff modified and...

This is ready for review now, btw