[igt-dev] [PATCH i-g-t] tests/kms_lease: Add subtest description

[Petri Latvala]

On Fri, May 07, 2021 at 06:20:39PM +0530, Arundhati Hagaragi wrote:
> Added description for subtest
> 
> Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi at intel.com>
> ---
>  tests/kms_lease.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/kms_lease.c b/tests/kms_lease.c
> index 6397ef8f..957af1e0 100644
> --- a/tests/kms_lease.c
> +++ b/tests/kms_lease.c
> @@ -1307,6 +1307,7 @@ igt_main
>  
>  	for (f = funcs; f->name; f++) {
>  
> +		igt_describe("Tests the valid crtc/connector combinations for given function");


That doesn't help at all. You need to document what each of those
functions do, separately.

Add a description to each of the elements in the funcs array so you can do

for (f = funcs; f->name; f++) {
  igt_describe("%s", f->desc);
  igt_subtest_f("%s", f->name) {
     run_test(&data, f->func);
  }
}


-- 
Petri Latvala