Use constant BATTERY_ID instead of hardcoded number

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

tests/actor/test_battery_status.py

@@ -378,7 +378,7 @@ async def test_sync_blocking_feature(
 
             # message is not correct, component should not block.
             tracker._handle_status_set_power_result(
-                SetPowerResult(succeed={1}, failed={106})
+                SetPowerResult(succeed={1}, failed={BATTERY_ID})
             )
 
             assert tracker._get_new_status_if_changed() is None
@@ -392,15 +392,15 @@ async def test_sync_blocking_feature(
             for timeout in expected_blocking_timeout:
                 # message is not correct, component should not block.
                 tracker._handle_status_set_power_result(
-                    SetPowerResult(succeed={1}, failed={106})
+                    SetPowerResult(succeed={1}, failed={BATTERY_ID})
                 )
 
                 assert tracker._get_new_status_if_changed() is Status.UNCERTAIN
 
                 # Battery should be still blocked, nothing should happen
                 time.shift(timeout - 1)
                 tracker._handle_status_set_power_result(
-                    SetPowerResult(succeed={1}, failed={106})
+                    SetPowerResult(succeed={1}, failed={BATTERY_ID})
                 )
 
                 assert tracker._get_new_status_if_changed() is None
@@ -416,7 +416,7 @@ async def test_sync_blocking_feature(
 
             # should block for 30 sec
             tracker._handle_status_set_power_result(
-                SetPowerResult(succeed={1}, failed={106})
+                SetPowerResult(succeed={1}, failed={BATTERY_ID})
             )
 
             assert tracker._get_new_status_if_changed() is Status.UNCERTAIN
@@ -437,14 +437,14 @@ async def test_sync_blocking_feature(
 
             # should block for 30 sec
             tracker._handle_status_set_power_result(
-                SetPowerResult(succeed={1}, failed={106})
+                SetPowerResult(succeed={1}, failed={BATTERY_ID})
             )
             assert tracker._get_new_status_if_changed() is Status.UNCERTAIN
             time.shift(28)
 
             # If battery succeed, then it should unblock.
             tracker._handle_status_set_power_result(
-                SetPowerResult(succeed={106}, failed={206})
+                SetPowerResult(succeed={BATTERY_ID}, failed={206})
             )
             assert tracker._get_new_status_if_changed() is Status.WORKING
 
@@ -481,15 +481,15 @@ async def test_sync_blocking_interrupted_with_with_max_data(
             assert tracker._get_new_status_if_changed() is Status.WORKING
 
             tracker._handle_status_set_power_result(
-                SetPowerResult(succeed={1}, failed={106})
+                SetPowerResult(succeed={1}, failed={BATTERY_ID})
             )
             assert tracker._get_new_status_if_changed() is Status.UNCERTAIN
 
             expected_blocking_timeout = [1, 2, 4]
             for timeout in expected_blocking_timeout:
                 # message is not correct, component should not block.
                 tracker._handle_status_set_power_result(
-                    SetPowerResult(succeed={1}, failed={106})
+                    SetPowerResult(succeed={1}, failed={BATTERY_ID})
                 )
                 assert tracker._get_new_status_if_changed() is None
                 time.shift(timeout)
@@ -527,7 +527,7 @@ async def test_sync_blocking_interrupted_with_invalid_message(
         assert tracker._get_new_status_if_changed() is Status.WORKING
 
         tracker._handle_status_set_power_result(
-            SetPowerResult(succeed={1}, failed={106})
+            SetPowerResult(succeed={1}, failed={BATTERY_ID})
         )
         assert tracker._get_new_status_if_changed() is Status.UNCERTAIN
 
@@ -540,12 +540,12 @@ async def test_sync_blocking_interrupted_with_invalid_message(
         assert tracker._get_new_status_if_changed() is Status.NOT_WORKING
 
         tracker._handle_status_set_power_result(
-            SetPowerResult(succeed={1}, failed={106})
+            SetPowerResult(succeed={1}, failed={BATTERY_ID})
         )
         assert tracker._get_new_status_if_changed() is None
 
         tracker._handle_status_set_power_result(
-            SetPowerResult(succeed={106}, failed={})
+            SetPowerResult(succeed={BATTERY_ID}, failed={})
         )
         assert tracker._get_new_status_if_changed() is None