> So worth adding a check in the test that ensures both apps are in the
> main stage
I could, but that'd be a bit redundant with this check:
compare(app1.requestedState, ApplicationInfoInterface.RequestedSuspended);
Which is what we're really interested in. The test was poor before, because the checked requested state was the same as the running state, and so it would need to look for secondary characteristics to confirm the expected state but didn't.
But now that we compare for the suspended state, I don't think it's necessary to add further checks. But they'd be harmless to add if you like.
> So worth adding a check in the test that ensures both apps are in the
> main stage
I could, but that'd be a bit redundant with this check: app1.requestedS tate, ApplicationInfo Interface. RequestedSuspen ded);
compare(
Which is what we're really interested in. The test was poor before, because the checked requested state was the same as the running state, and so it would need to look for secondary characteristics to confirm the expected state but didn't.
But now that we compare for the suspended state, I don't think it's necessary to add further checks. But they'd be harmless to add if you like.