I'm all in favour of adding more tests to ci_hsc, but we should be aware that it's not currently deployed as part of CI and likely to be extensively rewritten before it gets that far. It's a handy tool for us to use locally, but I don't think it's a substitute for other testing.
On reflection, I think even a noddy test (does that make sense in American? "Simplistic", I guess) which literally checks that the aperture corrections are enabled in the task configs would be worthwhile. That would, for example, have answered my second point above (if the unit tests were asserting that it should be enabled for ForcedPhotCcd, it would unambiguously be intentional), and it would cover us against trivial mistakes like
That said, I think you can go ahead and merge, whether or not you decide to add such a test.