Add test cases

tests/test_rfc2737.py

@@ -32,13 +32,13 @@ def test_PhysicalTableMIBUpdater_exception_in_reinit_data_wont_block_reinit_iter
               mock.patch('sonic_ax_impl.mibs.ietf.rfc2737.ThermalCacheUpdater.reinit_data') as mocked_thermal_reinit_data,
               mock.patch('sonic_ax_impl.mibs.cancel_redis_pubsub') as mocked_cancel_redis_pubsub):
             updater.reinit_data()
-        assert str(excinfo.value) == "mocked error"
-        mocked_xcvr_reinit_data.assert_called()
-        mocked_psu_reinit_data.assert_called()
-        mocked_fan_drawer_reinit_data.assert_called()
-        mocked_fan_cache_reinit_data.assert_called()
-        mocked_thermal_reinit_data.assert_called()
-        mocked_cancel_redis_pubsub.assert_called()
+            assert str(excinfo.value) == "mocked error"
+            mocked_xcvr_reinit_data.assert_called()
+            mocked_psu_reinit_data.assert_called()
+            mocked_fan_drawer_reinit_data.assert_called()
+            mocked_fan_cache_reinit_data.assert_called()
+            mocked_thermal_reinit_data.assert_called()
+            mocked_cancel_redis_pubsub.assert_called()
 
     # Given: 5 physical updaters are register into reinit of PhysicalTableMIBUpdater
     # When: The last updater(ThermalCacheUpdater) raises exception in the reinit
@@ -67,7 +67,7 @@ def test_PhysicalTableMIBUpdater_exception_in_reinit_data_wont_block_reinit_iter
     # When: The first updater(XcvrCacheUpdater) raises Runtime exception in the reinit
     # Then: The remaining updaters should execute reinit without any affection, and the un-subscription should be called
     @mock.patch('sonic_ax_impl.mibs.ietf.rfc2737.ThermalCacheUpdater.reinit_data', side_effect=RuntimeError('mocked runtime error'))
-    def test_PhysicalTableMIBUpdater_runtime_exc_in_reinit_data_wont_block_reinit_iteration_first(self, mock_xcvr_reinit_data):
+    def test_PhysicalTableMIBUpdater_runtime_exc_in_reinit_data_wont_block_reinit_iteration_first(self, mock_thermal_reinit_data):
         updater = PhysicalTableMIBUpdater()
 
         with (pytest.raises(RuntimeError) as excinfo,
@@ -87,7 +87,7 @@ def test_PhysicalTableMIBUpdater_runtime_exc_in_reinit_data_wont_block_reinit_it
             mocked_fan_cache_reinit_data.assert_called()
             mocked_thermal_reinit_data.assert_called()
             mocked_cancel_redis_pubsub.assert_called()
-        assert str(excinfo.value) == "mocked runtime error"
+            assert str(excinfo.value) == "mocked runtime error"
 
     # Given: 5 physical updaters are register into reinit of PhysicalTableMIBUpdater
     # When: The last updater(XcvrCacheUpdater) raises Runtime exception in the reinit
@@ -113,7 +113,7 @@ def test_PhysicalTableMIBUpdater_runtime_exc_in_reinit_data_wont_block_reinit_it
             mocked_fan_cache_reinit_data.assert_called()
             mocked_thermal_reinit_data.assert_called()
             mocked_cancel_redis_pubsub.assert_called()
-        assert str(excinfo.value) == "mocked runtime error"
+            assert str(excinfo.value) == "mocked runtime error"
 
     # Given: 5 physical updaters are register into reinit of PhysicalTableMIBUpdater
     # When: The first(XcvrCacheUpdater) and last updater(ThermalCacheUpdater)
@@ -141,4 +141,4 @@ def test_PhysicalTableMIBUpdater_multi_exception(self, mock_xcvr_reinit_data, mo
             mocked_fan_cache_reinit_data.assert_called()
             mocked_thermal_reinit_data.assert_called()
             mocked_cancel_redis_pubsub.assert_called()
-        assert str(excinfo.value) == "mocked runtime error"
+            assert str(excinfo.value) == "mocked runtime error"