Remove doctest from testing

[SamuelBrand1]

We have moved towards only checking that the documented code blocks run without failure rather than returning a specific value/string.

The documentation build action already covers this testing purpose, and the correctness of the modelling is covered elsewhere (e.g. simulation data checking reasonable parameters are inferred).

I think we can remove the doctest step in the testing (along with the Documenter dep).