[igt-dev] [PATCH v2 i-g-t 2/5] tests/kms_hdmi_inject: Provide igt_descriptions

[Ser, Simon]

On Tue, 2019-06-18 at 13:36 +0300, Arkadiusz Hiler wrote:
> This test is quite simple which makes it perfect for the first real
> world example of igt_describe usage.
> 
> v2: keep the original comment (Simon)
> 
> Cc: Simon Ser <simon.ser at intel.com>
> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Works for me!

Reviewed-by: Simon Ser <simon.ser at intel.com>

> ---
>  tests/kms_hdmi_inject.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_hdmi_inject.c b/tests/kms_hdmi_inject.c
> index 8c0d1333..c518c5d8 100644
> --- a/tests/kms_hdmi_inject.c
> +++ b/tests/kms_hdmi_inject.c
> @@ -33,7 +33,10 @@
>  #define HDISPLAY_4K	3840
>  #define VDISPLAY_4K	2160
>  
> -IGT_TEST_DESCRIPTION("Tests 4K and audio HDMI injection.");
> +IGT_TEST_DESCRIPTION("Test that in-kernel EDID parsing is producing "
> +		     "expected results by forcing a disconnected HDMI "
> +		     "connector with a known EDID and checking that the "
> +		     "metadata exposed to user space matches.");
>  
>  /**
>   * This collection of tests performs EDID and status injection tests. Injection
> @@ -210,9 +213,13 @@ igt_main
>  		igt_require(connector);
>  	}
>  
> +	igt_describe("Make sure that 4K modes exposed by DRM match the "
> +                     "forced EDID and modesetting using it succeed.");
>  	igt_subtest("inject-4k")
>  		hdmi_inject_4k(drm_fd, connector);
>  
> +	igt_describe("Make sure that audio information exposed by ALSA "
> +		     "match the forced EDID.");
>  	igt_subtest("inject-audio")
>  		hdmi_inject_audio(drm_fd, connector);
>