<div dir="ltr"><div dir="ltr">On Wed, Aug 21, 2024 at 10:53 AM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">A test case or suite may require certain capabilities to be present in<br>
the tested environment. Add the basic infrastructure for checking the<br>
support status of capabilities:<br>
* The Capability ABC defining the common capability API<br>
* Extension of the TestProtocol with required capabilities (each test<br>
  suite or case stores the capabilities it requires)<br>
* Integration with the runner which calls the new APIs to get which<br>
  capabilities are supported.<br>
<br>
Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech><br></blockquote><div><br></div><div>Looks all good to me, it was interesting to see how you've used abstract methods in the Capability class. The only thing I noticed was it seems like you wrote "dst" instead of "dts" in the commit message, otherwise:</div><div><br></div><div>Reviewed-by: Dean Marx <<a href="mailto:dmarx@iol.unh.edu">dmarx@iol.unh.edu</a>></div></div></div>