Fix warning messages due to nose test deprecation

Multiple warnings similar to the below are emitted during test execution:
```
tests/vnet_route_check_test.py::TestVnetRouteCheck::test_vnet_route_check
  /usr/lib/python3/dist-packages/_pytest/fixtures.py:901: PytestRemovedIn8Warning: Support for nose tests is deprecated and will be removed in a future release.
  tests/vnet_route_check_test.py::TestVnetRouteCheck::test_vnet_route_check is using nose-specific method: `setup(self)`
  To remove this warning, rename it to `setup_method(self)`
  See docs: https://docs.pytest.org/en/stable/deprecations.html#support-for-tests-written-for-nose
```

Rename `def setup(self)` to `def setup_method(self)` and `def teardown(self)` to
`def teardown_method(self)` as per documentation.

Signed-off-by: Brad House <bhouse@nexthop.ai>

Author: bhouse-nexthop

tests/clear_test.py

@@ -5,7 +5,7 @@
 from unittest.mock import patch, MagicMock
 
 class TestClear(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('clear.main.run_command')
@@ -115,12 +115,12 @@ def test_clear_switchcounters(self, run_command):
         assert result.exit_code == 0
         run_command.assert_called_with(['switchstat', '-c'])
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEAR DOWN')
 
 
 class TestClearQuaggav4(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('clear.main.run_command')
@@ -176,12 +176,12 @@ def test_clear_ipv4_quagga(self, run_command):
         assert result.exit_code == 0
         run_command.assert_called_with(['sudo', 'vtysh', '-c', "clear ip bgp 10.0.0.1 soft out"])
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEAR DOWN')
 
 
 class TestClearQuaggav6(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('clear.main.run_command')
@@ -237,12 +237,12 @@ def test_clear_ipv6_quagga(self, run_command):
         assert result.exit_code == 0
         run_command.assert_called_with(['sudo', 'vtysh', '-c', "clear ipv6 bgp 10.0.0.1 soft out"])
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEAR DOWN')
 
 
 class TestClearFrr(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('clear.main.run_command')
@@ -298,12 +298,12 @@ def test_clear_ipv6_frr(self, run_command):
         assert result.exit_code == 0
         run_command.assert_called_with(['sudo', 'vtysh', '-c', "clear bgp ipv6 10.0.0.1 soft out"])
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEAR DOWN')
 
 
 class TestClearFlowcnt(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('utilities_common.cli.run_command')
@@ -330,6 +330,6 @@ def test_flowcnt_route_route(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_with(['flow_counters_stat', '-c', '-t', 'route', '--prefix', '3.3.0.0/16', '--vrf', str('Vrf_1'), '-n', 'asic0'])
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEAR DOWN')
 

tests/config_test.py

@@ -402,7 +402,7 @@ def mock_run_command_side_effect_disabled_timer(*args, **kwargs):
 sonic_cfggen = load_module_from_source('sonic_cfggen', '/usr/local/bin/sonic-cfggen')
 
 class TestHelper(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('config.main.subprocess.Popen')
@@ -417,11 +417,11 @@ def test_get_device_type(self, mock_subprocess):
         mock_subprocess.assert_called_with(['/usr/local/bin/sonic-cfggen', '-m', '-v', 'DEVICE_METADATA.localhost.type'], text=True, stdout=-1)
         assert device_type == "Unknown"
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 class TestConfig(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('config.main.subprocess.check_call')
@@ -3463,7 +3463,7 @@ def teardown_class(cls):
 
 
 class TestConfigPfcwd(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3520,12 +3520,12 @@ def test_start_default(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_once_with(['pfcwd', 'start_default'], display_cmd=True)
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 
 class TestConfigAclUpdate(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3548,12 +3548,12 @@ def test_incremental(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_once_with(['acl-loader', 'update', 'incremental', file_name])
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 
 class TestConfigDropcounters(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3620,12 +3620,12 @@ def test_remove_reasons(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_once_with(['dropconfig', '-c', 'remove', '-n', str(counter_name), '-r', str(reasons)], display_cmd=True)
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 
 class TestConfigDropcountersMasic(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
         os.environ['UTILITIES_UNIT_TESTING'] = "1"
         os.environ["UTILITIES_UNIT_TESTING_TOPOLOGY"] = "multi_asic"
@@ -3746,7 +3746,7 @@ def teardown_class(cls):
         dbconnector.load_namespace_config()
 
 class TestConfigWatermarkTelemetry(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3759,12 +3759,12 @@ def test_interval(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_once_with(['watermarkcfg', '--config-interval', str(interval)])
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 
 class TestConfigZtp(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3794,7 +3794,7 @@ def test_enable(self, mock_run_command):
         assert result.exit_code == 0
         mock_run_command.assert_called_once_with(['ztp', 'enable'], display_cmd=True)
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 
@@ -3820,7 +3820,7 @@ def test_change_hostname(mock_run_command):
 
 
 class TestConfigInterface(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -3960,7 +3960,7 @@ def test_startup_shutdown_loopback(self):
         assert result.exit_code == 0
         assert db.cfgdb.get_table('LOOPBACK_INTERFACE')['Loopback0']['admin_status'] == 'up'
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
 
 

tests/disk_check_test.py

@@ -126,7 +126,7 @@ def swap_work(tc):
 
 
 class TestDiskCheck(object):
-    def setup(self):
+    def setup_method(self):
         pass
 
 

tests/dropconfig_test.py

@@ -11,7 +11,7 @@
 dropconfig = load_module_from_source('dropconfig', dropconfig_path)
 
 class TestDropConfig(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('builtins.print')
@@ -169,5 +169,5 @@ def test_invalid_dct_enable_monitor_error(self, mock_print):
         mock_print.assert_called_once_with(err_msg)
         assert e.value.code == 1
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEARDOWN')

tests/fast_reboot_filter_routes_test.py

@@ -5,7 +5,7 @@
 fast_reboot_filter_routes = importlib.import_module("scripts.fast-reboot-filter-routes")
 
 class TestFastRebootFilterRoutes(object):
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
 
     @patch('utilities_common.cli.run_command')
@@ -22,5 +22,5 @@ def test_get_connected_routes_command_failed(self, mock_run_command):
             fast_reboot_filter_routes.get_connected_routes()
         mock_run_command.assert_called_with(['sudo', 'vtysh', '-c', "show ip route connected json"], return_cmd=True)
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEAR DOWN")

tests/flock_test.py

@@ -28,7 +28,7 @@ def dummy_f2(bypass_lock=True):
 
 
 class TestFLock:
-    def setup(self):
+    def setup_method(self):
         print("SETUP")
         f0_exit.clear()
         f1_exit.clear()
@@ -180,7 +180,7 @@ def get_file_content(fd):
                 f2_exit.set()
                 thrd.join()
 
-    def teardown(self):
+    def teardown_method(self):
         print("TEARDOWN")
         f0_exit.clear()
         f1_exit.clear()

tests/fwutil_test.py

@@ -7,7 +7,7 @@
 import fwutil.lib as fwutil_lib
 
 class TestSquashFs(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('fwutil.lib.check_output_pipe')
@@ -75,12 +75,12 @@ def test_unmount_next_image_fs(self, mock_check_call, mock_exists, mock_rmdir):
             call(sqfs.fs_mountpoint)
         ]
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEARDOWN')
 
 
 class TestComponentUpdateProvider(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch("glob.glob", MagicMock(side_effect=[[], ['abc'], [], ['abc']]))
@@ -242,7 +242,7 @@ def test_regular_chassis_module_validation_error(self, mock_mkdir, mock_validate
             )
         assert "Module names mismatch" in str(excinfo.value)
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEARDOWN')
 
 

tests/route_check_test.py

@@ -216,7 +216,7 @@ def extract_namespace_from_args(args):
                 return args[i + 1]
         return DEFAULTNS
 
-    def setup(self):
+    def setup_method(self):
         pass
 
     def init(self):

tests/show_barefoot_test.py

@@ -7,7 +7,7 @@
 
 
 class TestShowBarefoot(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('subprocess.run')
@@ -48,6 +48,6 @@ def test_nondefault_profile(self, mock_run, mock_cmd):
         mock_open_file.assert_called_once_with('/usr/share/sonic/hwsku_dir/switch-tna-sai.conf')
         assert mock_cmd.call_args_list == expected_calls
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEARDOWN')
 

tests/show_mlnx_test.py

@@ -6,7 +6,7 @@
 
 
 class TestShowMlnx(object):
-    def setup(self):
+    def setup_method(self):
         print('SETUP')
 
     @patch('click.style')
@@ -113,6 +113,6 @@ def mock_return(*args, **kwargs):
         result = show.is_issu_status_enabled()
         assert result is status
 
-    def teardown(self):
+    def teardown_method(self):
         print('TEARDOWN')