From: "Zbigniew Kempczyński" <zbigniew.kempczynski@intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [igt-dev] [PATCH i-g-t 13/18] xe/xe_test_config.json: cleanup the field struct
Date: Fri, 10 Mar 2023 09:23:21 +0100 [thread overview]
Message-ID: <20230310082326.12088-14-zbigniew.kempczynski@intel.com> (raw)
In-Reply-To: <20230310082326.12088-1-zbigniew.kempczynski@intel.com>
From: Mauro Carvalho Chehab <mchehab@kernel.org>
Originally, the JSON file was conceived to also validate fields
and only allow certain values when others are present.
It turns that it doesn't make much sense to do such validations,
as it makes harder to change the hierarchy. So, remove the
parts of the fields struct that doesn't contain "is_field".
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Acked-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/xe/xe_test_config.json | 80 ++++--------------------------------
1 file changed, 9 insertions(+), 71 deletions(-)
diff --git a/tests/xe/xe_test_config.json b/tests/xe/xe_test_config.json
index 05ba71c6b8..6c474d6053 100644
--- a/tests/xe/xe_test_config.json
+++ b/tests/xe/xe_test_config.json
@@ -4,128 +4,66 @@
"fields": {
"Category": {
"_properties_": {
- "is_field": true,
"description": "Contains the major group for the tested functionality"
},
- "Hardware": {
+ "Sub-category": {
"_properties_": {
- "description": "Harware-supported build blocks"
+ "description": "Contains the minor group of the functionality"
},
- "Sub-category": {
+ "Functionality": {
"_properties_": {
- "is_field": true,
- "description": "Contains the minor group of the functionality"
- },
- "Page table": {
- "Functionality": {
- "_properties_": {
- "is_field": true,
- "description": "Groups page table tests per functionality"
- }
- }
- },
- "Unified Shared Memory building block": {
- "Functionality": {
- "_properties_": {
- "is_field": true,
- "description": "Groups page table tests per functionality"
- }
- }
- },
- "Compression": {
- "Functionality": {
- "_properties_": {
- "is_field": true
- }
- }
+ "description": "Groups page table tests per functionality"
}
- }
- },
- "Software building block": {
- "_properties_": {
- "description": "Software-based building blocks"
},
- "Sub-category": {
- "_properties_": {
- "is_field": true,
- "description": "Contains the minor group of the functionality"
- }
- }
- },
- "Software feature": {
- "Sub-category": {
+ "Run type": {
"_properties_": {
- "is_field": true,
- "description": "Contains the minor group of the functionality"
+ "description": "Defines what category of testlist it belongs"
}
}
},
- "End to end use case": {
- "Sub-category": {
- "_properties_": {
- "is_field": true,
- "description": "Contains the minor group of the functionality"
- }
- },
- "Mega feature": {
- "_properties_": {
- "is_field": true,
- "description": "Contains the mega feature for E2E use case"
- }
+ "Mega feature": {
+ "_properties_": {
+ "description": "Contains the mega feature for E2E use case"
}
}
},
"Test category": {
"_properties_": {
- "is_field": true,
"description": "Defines the test category. Usually used at subtest level."
}
},
"Test requirement": {
"_properties_": {
- "is_field": true,
"description": "Defines Kernel parameters required for the test to run"
}
},
- "Run type": {
- "_properties_": {
- "is_field": true,
- "description": "Defines the test primary usage. Usually used at subtest level."
- }
- },
"Issue": {
"_properties_": {
- "is_field": true,
"description": "If the test is used to solve an issue, point to the URL containing the issue."
}
},
"GPU excluded platform": {
"_properties_": {
- "is_field": true,
"description": "Provides a list of GPUs not capable of running the subtest (or the test as a hole)."
}
},
"GPU requirement": {
"_properties_": {
- "is_field": true,
"description": "Describes any GPU-specific requrirement, like requiring multi-tiles."
}
},
"Depends on" : {
"_properties_": {
- "is_field": true,
"description": "List other subtests that are required to not be skipped before calling this one."
}
},
"TODO": {
"_properties_": {
- "is_field": true,
"description": "Point to known missing features at the test or subtest."
}
},
"Description" : {
"_properties_": {
- "is_field": true,
"description": "Provides a description for the test/subtest."
}
}
--
2.34.1
next prev parent reply other threads:[~2023-03-10 8:24 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-10 8:23 [igt-dev] [PATCH i-g-t 00/18] Add igt_doc.py Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 01/18] scripts/igt_doc.py: beautify its code Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 02/18] scripts/igt_doc.py: add JSON file output Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 03/18] scripts/igt_doc.py: dynamically create fields array from a JSON file Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 04/18] scripts/igt_doc.py: add support to specify numeric values Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 05/18] scripts/igt_doc.py: improve --show-subtests logic Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 06/18] scripts/igt_doc.py: add error handler for subprocess Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 07/18] scripts/igt_doc.py: improve multi-line logic Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 08/18] scripts/igt_doc.py: don't use ":=" operator Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 09/18] scripts/igt_doc.py: make it compatible with Python 3.6 Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 10/18] scripts/igt_doc.py: sets the minimal version to run the script Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 11/18] scripts/igt_doc.py: use a different logic to get IGT prefix Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 12/18] testplan/meson.build: add targets to build Xe test documentation Zbigniew Kempczyński
2023-03-10 8:23 ` Zbigniew Kempczyński [this message]
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 14/18] xe/xe_huc_copy: add GPU dependency to its documentation Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 15/18] xe/xe_*: add TEST/SUBTEST documentation Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 16/18] xe-fast-feedback.testlist: update debugfs tests Zbigniew Kempczyński
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 17/18] xe_mmap: skip VRAM tests if no VRAM is found Zbigniew Kempczyński
2023-03-10 10:52 ` Das, Nirmoy
2023-03-13 6:48 ` Zbigniew Kempczyński
2023-03-14 12:26 ` Das, Nirmoy
2023-03-10 8:23 ` [igt-dev] [PATCH i-g-t 18/18] meson: replace "igt@xe/" by "igt@" Zbigniew Kempczyński
2023-03-10 10:23 ` [igt-dev] ✓ Fi.CI.BAT: success for Add igt_doc.py Patchwork
2023-03-13 1:18 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230310082326.12088-14-zbigniew.kempczynski@intel.com \
--to=zbigniew.kempczynski@intel.com \
--cc=igt-dev@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox