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.
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.