From 4995a58f998d3bdd055e49e919945bc0aa939be4 Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Mon, 20 Oct 2025 18:52:10 +0200 Subject: T7948: always call setUp() and tearDown() base class methods While working on task T7664 (FRR 10.4 upgrade), I identified the need for additional validation and safeguards around the FRR management daemon. The most appropriate place for this logic is in the setUp() and tearDown() methods of the smoketest base class, VyOSUnitTestSHIM. However, during implementation, it became apparent that test cases do not consistently invoke the base class's setup and teardown methods. This inconsistency complicates the process of capturing the FRR mgmtd PID at the start of a test and verifying that it remains unchanged by the end - a key step in detecting crashes or unexpected terminations (e.g., SIGSEGV) of the FRR management daemon during tests. --- smoketest/scripts/cli/test_interfaces_input.py | 2 ++ 1 file changed, 2 insertions(+) (limited to 'smoketest/scripts/cli/test_interfaces_input.py') diff --git a/smoketest/scripts/cli/test_interfaces_input.py b/smoketest/scripts/cli/test_interfaces_input.py index 737138b0a..fc1a3bdb8 100755 --- a/smoketest/scripts/cli/test_interfaces_input.py +++ b/smoketest/scripts/cli/test_interfaces_input.py @@ -33,6 +33,8 @@ class InputInterfaceTest(VyOSUnitTestSHIM.TestCase): def tearDown(self): self.cli_delete(base_path) self.cli_commit() + # always forward to base class + super().tearDown() def test_01_description(self): # Check if PPPoE dialer can be configured and runs -- cgit v1.2.3