Get healthyness passed through and add it to tests
Pass environment and application into initialization
Remove the parameters to the method which are now passed into the constructor
Remove the application_model class, folding it into the live model resolver.

The application model class was anemic already (it only held a list, and the only users of it
just grabbed the list straight out) + the live model resolver also held this list (and was
responsible for constructing it.

Ergo it seemed sane to just fold everything into the live model creator, as now the live
model creator can be refactored to update (rather than replace) the models, which gives
us an easy path to having more knowledge about what we did before (and why it didn't work).

This now of course means that a whole chunk of the naming for things is insane, however
I'll be planning to clean this up later, once I see what shape things take when I've
moved more stuff around
Stop passing no longer needed things to constructor
Pass additional parts into live model creator
Move all the resolution into the application model also.

I'm less sure that this is the right place for this functionality to be folded into, so I'm going to delay
cleaning up / removing Orc::Engine (which could theoretcially be done now) until I've actually
got some of the functionality I'm going for working.

We do now have all the code needed to implement the memory of previous states (and ergo remember when
we entered a state) and new wait for healthy action all in the same place, ergo I should be able
to now work on and hopefully complete what I want to do inside the live model resolver, before
then finishing cleaning everything up..
Remove the client/statuses class.

This was again just an array holder. In this case, it had a pile more methods on it,
which would have made you think the class was used, however we again construct it just to
call .instances on it, and that's the only operation we ever do - ergo the class existing
at all is kinda pointless, so I've removed it
Remove some confusion as we now have 1 class, not two...
Ensure execute always returns true
Hold if an action failed in the action itself
Remove the second instance variable which isn't needed