- Issue created by @Wim Leers
- Status changed to Needs review
10 months ago 3:17pm 23 November 2023 - π§πͺBelgium Wim Leers Ghent π§πͺπͺπΊ
I think we should add explicit test coverage for all methods on
TypedConfigManager
, because it's unclear which code paths work correctly and which ones fail in the way described in the issue summary and demonstrated in the basic test coverage already in the MR. - Status changed to Needs work
10 months ago 7:53pm 24 November 2023 - πΊπΈUnited States smustgrave
The test added seemed to work. Not sure if this issue is meant to include all test coverage or if that should be a follow up. Could that be noted in IS?
- π§πͺBelgium borisson_ Mechelen, π§πͺ
I think the plan is to include all test coverage in this issue, I think that would make the most sense?
- π§πͺBelgium Wim Leers Ghent π§πͺπͺπΊ
Bumping to given the data corruption/pollution nature of the problem.