[igt-dev] [PATCH i-g-t v2 1/2] xe_test_config.json: mark some fields are mandatory at subtest level

[Kamil Konieczny]

Hi Mauro,
On 2023-08-10 at 17:10:30 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> Some IGT Xe tests were added without proper documentation.
> Currently, igt_doc.py doesn't know what fields are mandatory or
> not, so it can't enforce it.
> 
> Add a field to mark those.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/xe/xe_test_config.json | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/tests/xe/xe_test_config.json b/tests/xe/xe_test_config.json
> index 08c39de74384..8fb65d02d840 100644
> --- a/tests/xe/xe_test_config.json
> +++ b/tests/xe/xe_test_config.json
> @@ -5,6 +5,7 @@
>      "fields": {
>          "Category": {
>              "_properties_": {
> +                "mandatory": true,
>                  "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
>              },
>              "Mega feature": {
> @@ -13,10 +14,12 @@
>                  },
>                  "Sub-category": {
>                      "_properties_": {
> +                        "mandatory": true,
>                          "description": "Contains the the technical feature/functionality"
>                      },
>                      "Functionality": {
>                          "_properties_": {
> +                            "mandatory": true,
>                              "description": "Groups page table tests on buckets containg more detailed functionality"
>                          },
>                          "Feature": {
> @@ -27,6 +30,7 @@
>                      },
>                      "Run type": {
>                          "_properties_": {
> +                            "mandatory": true,
>                              "description": "Defines what category of testlist it belongs",
>                              "order": [
>                                  "boot",
> @@ -75,6 +79,7 @@
>          },
>          "Description" : {
>              "_properties_": {
> +                "mandatory": true,
>                  "description": "Provides a description for the test/subtest."
>              }
>          }
> -- 
> 2.41.0
>