* [PATCH] doc: elaborate on per test suite configuration dts docs
@ 2025-05-16 20:26 Nicholas Pratte
2025-06-05 18:54 ` Dean Marx
2025-06-16 16:15 ` Luca Vizzarro
0 siblings, 2 replies; 3+ messages in thread
From: Nicholas Pratte @ 2025-05-16 20:26 UTC (permalink / raw)
To: luca.vizzarro, thomas, probb, dmarx, paul.szczepanek,
Honnappa.Nagarahalli, thomas.wilks, yoan.picchi, stephen
Cc: dev, Nicholas Pratte
Introduce some additional documentation explaing the per test suite
configuration component of DTS as the current dts documentation does
not provide a quick access reference guide or explanation.
Signed-off-by: Nicholas Pratte <npratte@iol.unh.edu>
---
doc/guides/tools/dts.rst | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/doc/guides/tools/dts.rst b/doc/guides/tools/dts.rst
index fcc6d22036..951b5462e2 100644
--- a/doc/guides/tools/dts.rst
+++ b/doc/guides/tools/dts.rst
@@ -405,6 +405,13 @@ There are four types of methods that comprise a test suite:
If there's any functionality or logic missing from the framework,
it should be implemented so that the test suites can use one of these two ways.
+ Test suites may also be configured individually using a provided configuration file,
+ provided at the command line, for custom configuration. The configuration file is a simple
+ mapping of test suite names to their corresponding configurations.
+
+ Any test suite can be designed to require custom configuration attributes or optional ones.
+ Any optional attributes should supply a default value for the test suite to use.
+
#. **Test case verification**
Test case verification should be done with the ``verify`` method, which records the result.
@@ -507,3 +514,18 @@ And they both have two network ports which are physically connected to each othe
.. literalinclude:: ../../../dts/nodes.example.yaml
:language: yaml
:start-at: # Define
+
+Additionally, an example configuration file is provided to demonstrate custom test suite configuration:
+
+.. note::
+ You do not neeed to supply configurations for all test suites, and not all test suites will support
+ or need additional configuration.
+
+.. _tests_config_example:
+
+``dts/tests_config.example.yaml``
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+.. literalinclude:: ../../../dts/tests_config.example.yaml
+ :language: yaml
+ :start-at: # Define
\ No newline at end of file
--
2.47.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] doc: elaborate on per test suite configuration dts docs
2025-05-16 20:26 [PATCH] doc: elaborate on per test suite configuration dts docs Nicholas Pratte
@ 2025-06-05 18:54 ` Dean Marx
2025-06-16 16:15 ` Luca Vizzarro
1 sibling, 0 replies; 3+ messages in thread
From: Dean Marx @ 2025-06-05 18:54 UTC (permalink / raw)
To: dev
Cc: luca.vizzarro, thomas, probb, paul.szczepanek,
Honnappa.Nagarahalli, thomas.wilks, yoan.picchi, stephen
On Fri, May 16, 2025 at 4:26 PM Nicholas Pratte <npratte@iol.unh.edu> wrote:
>
> Introduce some additional documentation explaing the per test suite
> configuration component of DTS as the current dts documentation does
> not provide a quick access reference guide or explanation.
>
> Signed-off-by: Nicholas Pratte <npratte@iol.unh.edu>
explaining is misspelled in the commit message
<snip>
> + Test suites may also be configured individually using a provided configuration file,
> + provided at the command line, for custom configuration. The configuration file is a simple
> + mapping of test suite names to their corresponding configurations.
This part is a little odd, the words configured/configuration and
provided are used a lot. Maybe something like this would be better:
"Test suites may also be configured individually using a file provided
at the command line. This file is a simple mapping of test suite names
to their corresponding configurations."
Reviewed-by: Dean Marx <dmarx@iol.unh.edu>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] doc: elaborate on per test suite configuration dts docs
2025-05-16 20:26 [PATCH] doc: elaborate on per test suite configuration dts docs Nicholas Pratte
2025-06-05 18:54 ` Dean Marx
@ 2025-06-16 16:15 ` Luca Vizzarro
1 sibling, 0 replies; 3+ messages in thread
From: Luca Vizzarro @ 2025-06-16 16:15 UTC (permalink / raw)
To: Nicholas Pratte, thomas, probb, dmarx, paul.szczepanek,
Honnappa.Nagarahalli, thomas.wilks, yoan.picchi, stephen
Cc: dev
Agree with Dean's changes. Also the start marker is broken, but can
easily be added.
Reviewed-by: Luca Vizzarro <luca.vizzarro@arm.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-06-16 16:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-05-16 20:26 [PATCH] doc: elaborate on per test suite configuration dts docs Nicholas Pratte
2025-06-05 18:54 ` Dean Marx
2025-06-16 16:15 ` Luca Vizzarro
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).