Use "medium" instead of "med" for the medium fan mode in Coolmaster (home-assistant#157253)

Author: tan-lawrence

homeassistant/components/coolmaster/climate.py

@@ -8,6 +8,10 @@
 from pycoolmasternet_async import SWING_MODES
 
 from homeassistant.components.climate import (
+    FAN_AUTO,
+    FAN_HIGH,
+    FAN_LOW,
+    FAN_MEDIUM,
     ClimateEntity,
     ClimateEntityFeature,
     HVACMode,
@@ -31,7 +35,16 @@
 
 HA_STATE_TO_CM = {value: key for key, value in CM_TO_HA_STATE.items()}
 
-FAN_MODES = ["low", "med", "high", "auto"]
+CM_TO_HA_FAN = {
+    "low": FAN_LOW,
+    "med": FAN_MEDIUM,
+    "high": FAN_HIGH,
+    "auto": FAN_AUTO,
+}
+
+HA_FAN_TO_CM = {value: key for key, value in CM_TO_HA_FAN.items()}
+
+FAN_MODES = list(CM_TO_HA_FAN.values())
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -111,7 +124,7 @@ def hvac_mode(self):
     @property
     def fan_mode(self):
         """Return the fan setting."""
-        return self._unit.fan_speed
+        return CM_TO_HA_FAN[self._unit.fan_speed]
 
     @property
     def fan_modes(self):
@@ -138,7 +151,7 @@ async def async_set_temperature(self, **kwargs: Any) -> None:
     async def async_set_fan_mode(self, fan_mode: str) -> None:
         """Set new fan mode."""
         _LOGGER.debug("Setting fan mode of %s to %s", self.unique_id, fan_mode)
-        self._unit = await self._unit.set_fan_speed(fan_mode)
+        self._unit = await self._unit.set_fan_speed(HA_FAN_TO_CM[fan_mode])
         self.async_write_ha_state()
 
     async def async_set_swing_mode(self, swing_mode: str) -> None:

tests/components/coolmaster/test_climate.py

@@ -167,23 +167,28 @@ async def test_set_temperature(
     assert hass.states.get("climate.l1_100").attributes[ATTR_TEMPERATURE] == 30
 
 
+@pytest.mark.parametrize("target_fan_mode", FAN_MODES)
 async def test_set_fan_mode(
     hass: HomeAssistant,
     load_int: ConfigEntry,
+    target_fan_mode: str,
 ) -> None:
     """Test the Coolmaster climate set fan mode."""
     assert hass.states.get("climate.l1_100").attributes[ATTR_FAN_MODE] == FAN_LOW
+
     await hass.services.async_call(
         CLIMATE_DOMAIN,
         SERVICE_SET_FAN_MODE,
         {
             ATTR_ENTITY_ID: "climate.l1_100",
-            ATTR_FAN_MODE: FAN_HIGH,
+            ATTR_FAN_MODE: target_fan_mode,
         },
         blocking=True,
     )
     await hass.async_block_till_done()
-    assert hass.states.get("climate.l1_100").attributes[ATTR_FAN_MODE] == FAN_HIGH
+    assert (
+        hass.states.get("climate.l1_100").attributes[ATTR_FAN_MODE] == target_fan_mode
+    )
 
 
 async def test_set_swing_mode(