Low test coverage in CollectQueueingHistoryService
The Problem
I was reviewing a (mostly) frontend MR !131797 (merged) that technically had an undercoverage violation. The MR itself only added a simple feature flag check, and it turns out the method hasn't really had test coverage to begin with.
We went ahead with the original MR to keep the scope down, but we should really have test coverage for this.
A Proposal
Add specs!
Edited by drew stachon