Code review comment for lp:~zyga/checkbox/fix-1306051

Revision history for this message
Daniel Manrique (roadmr) wrote :

+1, easy enough. No tests:

12:32 < zyga> roadmr: there are no tests as provider_manager has no unit tests and clearly
              has not enough intergration tests to make that change matter
12:32 < zyga> roadmr: I thought about adding more but I need to refactor some code first,
              to make creating testbeds like that easier

review: Approve

« Back to merge proposal