Switchbot Cloud: Fix Roller Shade not work issue (home-assistant#152528)

Author: XiaoLing-git

homeassistant/components/switchbot_cloud/cover.py

@@ -109,15 +109,13 @@ class SwitchBotCloudCoverRollerShade(SwitchBotCloudCover):
 
     async def async_open_cover(self, **kwargs: Any) -> None:
         """Open the cover."""
-        await self.send_api_command(RollerShadeCommands.SET_POSITION, parameters=str(0))
+        await self.send_api_command(RollerShadeCommands.SET_POSITION, parameters=0)
         await asyncio.sleep(COVER_ENTITY_AFTER_COMMAND_REFRESH)
         await self.coordinator.async_request_refresh()
 
     async def async_close_cover(self, **kwargs: Any) -> None:
         """Close the cover."""
-        await self.send_api_command(
-            RollerShadeCommands.SET_POSITION, parameters=str(100)
-        )
+        await self.send_api_command(RollerShadeCommands.SET_POSITION, parameters=100)
         await asyncio.sleep(COVER_ENTITY_AFTER_COMMAND_REFRESH)
         await self.coordinator.async_request_refresh()
 
@@ -126,7 +124,7 @@ async def async_set_cover_position(self, **kwargs: Any) -> None:
         position: int | None = kwargs.get("position")
         if position is not None:
             await self.send_api_command(
-                RollerShadeCommands.SET_POSITION, parameters=str(100 - position)
+                RollerShadeCommands.SET_POSITION, parameters=(100 - position)
             )
             await asyncio.sleep(COVER_ENTITY_AFTER_COMMAND_REFRESH)
             await self.coordinator.async_request_refresh()

homeassistant/components/switchbot_cloud/entity.py

@@ -44,7 +44,7 @@ async def send_api_command(
         self,
         command: Commands,
         command_type: str = "command",
-        parameters: dict | str = "default",
+        parameters: dict | str | int = "default",
     ) -> None:
         """Send command to device."""
         await self._api.send_command(

tests/components/switchbot_cloud/test_cover.py

@@ -319,7 +319,7 @@ async def test_roller_shade_features(
             blocking=True,
         )
     mock_send_command.assert_called_once_with(
-        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", "0"
+        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", 0
     )
 
     await configure_integration(hass)
@@ -334,7 +334,7 @@ async def test_roller_shade_features(
             blocking=True,
         )
     mock_send_command.assert_called_once_with(
-        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", "100"
+        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", 100
     )
 
     await configure_integration(hass)
@@ -349,7 +349,7 @@ async def test_roller_shade_features(
             blocking=True,
         )
     mock_send_command.assert_called_once_with(
-        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", "50"
+        "cover-id-1", RollerShadeCommands.SET_POSITION, "command", 50
     )