<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 27, 2025 at 11:37 AM Dean Marx <<a href="mailto:dmarx@iol.unh.edu" target="_blank">dmarx@iol.unh.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Fix a bug in the port stats check test suite that was causing<br>
the DTS doc generation to fail.<br>
<br>
Fixes: 8f21210b1d50 ("dts: add port stats check test suite")<br>
<br>
Signed-off-by: Dean Marx <<a href="mailto:dmarx@iol.unh.edu" target="_blank">dmarx@iol.unh.edu</a>><br>
---<br>
 dts/tests/TestSuite_port_stats_checks.py | 13 +++++++++----<br>
 1 file changed, 9 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/dts/tests/TestSuite_port_stats_checks.py b/dts/tests/TestSuite_port_stats_checks.py<br>
index 2a3fb06946..491c2263b6 100644<br>
--- a/dts/tests/TestSuite_port_stats_checks.py<br>
+++ b/dts/tests/TestSuite_port_stats_checks.py<br>
@@ -51,10 +51,15 @@ class TestPortStatsChecks(TestSuite):<br>
<br>
     #: Length of the packet being sent including the IP and frame headers.<br>
     total_packet_len: ClassVar[int] = 100<br>
-    #: Packet to send during testing.<br>
-    send_pkt: ClassVar[Packet] = (<br>
-        Ether() / IP() / Raw(b"X" * (total_packet_len - ip_header_len - ether_header_len))<br>
-    )<br>
+<br>
+    @property<br>
+    def send_pkt(self) -> Packet:<br>
+        """Packet to send during testing."""<br>
+        return (<br>
+            Ether()<br>
+            / IP()<br>
+            / Raw(b"X" * (self.total_packet_len - self.ip_header_len - self.ether_header_len))<br>
+        )<br></blockquote><div><br></div><div>Looks good. In my opinion this should be a standalone patch, since it's unrelated to patch 1 and 2, but it's not functionally relevant since there is no such thing as a series in git, so you don't need to resubmit. Keep in mind for future series though that they should be logically grouped.</div><div><br></div><div>Reviewed-by: Patrick Robb <<a href="mailto:probb@iol.unh.edu">probb@iol.unh.edu</a>> </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
     def extract_noise_information(<br>
         self, verbose_out: list[TestPmdVerbosePacket]<br>
-- <br>
2.49.0<br>
<br>
</blockquote></div></div>
</div>