Fix ServiceStateLogicTest

Previously `testManuallySettingIndicatorsOnApplicationsIsMoreComplicated` failed non-deterministically. Though this test name suggested the failure was because it wasn't complicated enough, the actual cause was mostly down to using `assertAttributeEqualsEventually` instead of `assertAttributeEquals` or a couple of wrong assertions `:-)`.

I added some `assertAttributeEqualsContinually` assertions for when we expect the state to remain the same after changing some other attribute or config. This is because the test failed non-deterministically (which could be made deterministic with some `sleep(1000)` added temporarily.

I renamed the test to `testManuallySettingIndicatorsOnApplications`. Longer term, it might well be worth splitting this big test into multiple small tests that each test a smaller number of cases.
Or `EntityTestUtils` I suppose. Perhaps we can enforce a convention that `Asserts` will have the base `assertSomethingHappensEventually(Entity entity, T thing)` sort of methods and `EntityTestUtils` has overloads that call the same-named method in `Asserts` but with _try_..._catch_ blocks that do tae `dumpInfo()` and rethrow, as here?