I feel like these tests would be less racy if we could pass achan time.Time between the agent being tested and the unit test itself.

I think the entire metadata test could use a re-think, we already know when the agent posts updates due to injected function so we could utilize that knowledge. AFAICT there’s no race in the service banner test though and the intent in this PR was to remove the conditional test code to avoid such use in the future.