Code review comment for ~bettyl/plainbox-provider-certification-client:add-auto-test-plan

Revision history for this message
Jonathan Cave (jocave) wrote :

Thanks having the links in the MR helps with auditing.

I think it would appropriate to be more descriptive in _name & _description - for example if these where to shown in the test plan browser of checkbox just "Generic Auto" really tells you very little about what the test plan is. The description could indicate where it is likely to be used something like "the default set of automated tests for PC OEM and Sanity testing"..

Could you come up with something?

« Back to merge proposal