Comment 8 for bug 2013084

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Hi Michal, thanks for the patch.

While the patch LGTM and the fix seems quite straightforward, the [Test Plan] seems to be missing a more substantial description of the affected and expected behaviors. I also understand testing this issue may not be a trivial task and I see the bug reporter agreed to test the potential fix.

Keha, would you be willing to come up with such a test plan for SRU documentation purposes?

Otherwise, I am leaving the test plan discussions/assessment for the SRU team and am proceeding with the fix upload here.