[PATCH i-g-t v1] tests/intel/i915_test_config: align JSON documentation with xe

[Kamil Konieczny]

Hi Mauro,
On 2024-02-19 at 08:08:30 +0100, Mauro Carvalho Chehab wrote:
> On Fri, 16 Feb 2024 16:52:07 +0100
> Kamil Konieczny <kamil.konieczny at linux.intel.com> wrote:
> 
> > From: Jari Tahvanainen <jari.tahvanainen at intel.com>
> > 
> > Restructure i915 JSON to align with Xe.
> > 
> > Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>
> > Cc: Mauro Carvalho Chehab <mchehab at kernel.org>
> > Signed-off-by: Jari Tahvanainen <jari.tahvanainen at intel.com>
> > Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> > ---
> >  tests/intel/i915_test_config.json | 45 ++++++++++++++++++++++---------
> >  1 file changed, 32 insertions(+), 13 deletions(-)
> > 
> > diff --git a/tests/intel/i915_test_config.json b/tests/intel/i915_test_config.json
> > index ad59677f8..b09f4244e 100644
> > --- a/tests/intel/i915_test_config.json
> > +++ b/tests/intel/i915_test_config.json
> > @@ -16,33 +16,52 @@
> >      "exclude_files": [ "kms_*", "gem_concurrent_all.c", "gem_concurrent_blit.c",
> >                         "xe_*" ],
> >      "fields": {
> > -	"Mega feature": {
> > +        "Category": {
> > +            "_properties_": {
> > +                "mandatory": true,
> > +                "description": "Contains the major group for the tested functionality, being hardware, software or firmware"
> > +            },
> > +            "Mega feature": {
> >                  "_properties_": {
> >                      "description": "Contains the mega feature for end to end use case, e. g. the 'sellable' feature."
> >                  },
> > -                "Category": {
> > -                "_properties_": {
> > -                    "description": "Contains the major group for the tested functionality"
> > -                },
> > -
> >                  "Sub-category": {
> >                      "_properties_": {
> > -                        "description": "Contains the minor group of the functionality"
> > +                        "mandatory": true,
> > +                        "description": "Contains the technical feature/functionality"
> >                      },
> >                      "Functionality": {
> >                          "_properties_": {
> > -                            "description": "Describes the functionality to be tested"
> > +                            "mandatory": true,
> > +                            "description": "Groups page table tests on buckets containg more detailed functionality"
> >                          },
> >                          "Feature": {
> >                              "_properties_": {
> >                                  "description": "Describes the lowest level feature bucket"
> > -                            },
> > -                            "Run type": {
> > -                                "_properties_": {
> > -                                    "description": "Defines what category of testlist it belongs"
> > -                                }
> >                              }
> >                          }
> > +                    },
> > +                    "Run type": {
> > +                        "_properties_": {
> > +                            "mandatory": true,
> > +                            "description": "Defines what category of testlist it belongs",
> > +                            "default-testlist": "FULL",
> > +                            "update-from-file": {
> > +                                "exclude-type": "regex-ignorecase",
> > +                                "append-value-if-not-excluded": "i915 FULL",
> > +                                "include": [
> > +                                    { "i915 BAT": "../intel-ci/fast-feedback.testlist" }
> > +                                ],
> > +                                "exclude": [
> > +                                    { "i915 BAT, 915 FULL": "../intel-ci/blacklist.txt" }
> 
> There is a typo at "915 FULL". It should be, instead: 
> 
> 	{ "i915 BAT, i915 FULL": "../intel-ci/blacklist.txt" }
> 
> or, even better:
> 
> 	{ "all": "../intel-ci/blacklist.txt" }

I will go with this,

Regards,
Kamil

> 
> With that:
> 
> Reviewed-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> 
> > +                                ]
> > +                            },
> > +                            "order": [
> > +                                "boot",
> > +                                "__all__",
> > +                                "kunit"
> > +                            ]
> > +                        }
> >                      }
> >                  }
> >              }