Fix checkers failures

Author: benle7

tests/show_platform_test.py

@@ -225,11 +225,10 @@ def test_bmc_summary_regular_output(self):
         mock_bmc.get_eeprom.return_value = self.TEST_BMC_EEPROM_INFO
         mock_bmc.get_version.return_value = self.TEST_BMC_VERSION
 
-        sys.modules['sonic_platform'].platform.Platform.return_value = mock_platform
-
-        result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['summary'], [])
-        assert result.exit_code == 0, result.output
-        assert result.output == textwrap.dedent(expected_output)
+        with mock.patch('sonic_platform.platform.Platform', return_value=mock_platform):
+            result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['summary'], [])
+            assert result.exit_code == 0, result.output
+            assert result.output == textwrap.dedent(expected_output)
 
     def test_bmc_summary_json_output(self):
         """Test 'show platform bmc summary' with JSON output"""
@@ -251,12 +250,11 @@ def test_bmc_summary_json_output(self):
         mock_bmc.get_eeprom.return_value = self.TEST_BMC_EEPROM_INFO
         mock_bmc.get_version.return_value = self.TEST_BMC_VERSION
 
-        sys.modules['sonic_platform'].platform.Platform.return_value = mock_platform
-
-        result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['summary'], ['--json'])
-        assert result.exit_code == 0, result.output
-        output_json = json.loads(result.output)
-        assert output_json == expected_json
+        with mock.patch('sonic_platform.platform.Platform', return_value=mock_platform):
+            result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['summary'], ['--json'])
+            assert result.exit_code == 0, result.output
+            output_json = json.loads(result.output)
+            assert output_json == expected_json
 
     def test_bmc_eeprom_regular_output(self):
         """Test 'show platform bmc eeprom' with regular output"""
@@ -282,11 +280,10 @@ def test_bmc_eeprom_regular_output(self):
         mock_chassis.get_bmc.return_value = mock_bmc
         mock_bmc.get_eeprom.return_value = self.TEST_BMC_EEPROM_INFO
 
-        sys.modules['sonic_platform'].platform.Platform.return_value = mock_platform
-
-        result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['eeprom'], [])
-        assert result.exit_code == 0, result.output
-        assert result.output == textwrap.dedent(expected_output)
+        with mock.patch('sonic_platform.platform.Platform', return_value=mock_platform):
+            result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['eeprom'], [])
+            assert result.exit_code == 0, result.output
+            assert result.output == textwrap.dedent(expected_output)
 
     def test_bmc_eeprom_json_output(self):
         """Test 'show platform bmc eeprom' with JSON output"""
@@ -306,9 +303,8 @@ def test_bmc_eeprom_json_output(self):
         mock_chassis.get_bmc.return_value = mock_bmc
         mock_bmc.get_eeprom.return_value = self.TEST_BMC_EEPROM_INFO
 
-        sys.modules['sonic_platform'].platform.Platform.return_value = mock_platform
-
-        result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['eeprom'], ['--json'])
-        assert result.exit_code == 0, result.output
-        output_json = json.loads(result.output)
-        assert output_json == expected_json
+        with mock.patch('sonic_platform.platform.Platform', return_value=mock_platform):
+            result = CliRunner().invoke(show.cli.commands['platform'].commands['bmc'].commands['eeprom'], ['--json'])
+            assert result.exit_code == 0, result.output
+            output_json = json.loads(result.output)
+            assert output_json == expected_json