feat(bme280): Implement _ensure_data() auto-trigger pattern.

Author: nedseb

lib/bme280/bme280/device.py

@@ -239,6 +239,21 @@ def humidity_ready(self):
     # Forced measurement trigger
     # --------------------------------------------------
 
+    def _is_sleep_mode(self):
+        """Return True if the sensor is in sleep mode."""
+        return (self._read_reg(REG_CTRL_MEAS) & MODE_MASK) == MODE_SLEEP
+
+    def _ensure_data(self):
+        """Trigger a forced measurement if the sensor is in sleep mode.
+
+        In normal mode this is a no-op. In sleep mode it triggers a
+        single conversion and waits for completion so that subsequent
+        register reads return fresh data.
+        """
+        if self._is_sleep_mode():
+            self.trigger_one_shot()
+            self._wait_measurement()
+
     def trigger_one_shot(self):
         """Trigger a single forced measurement. Poll data_ready() for completion."""
         ctrl = self._read_reg(REG_CTRL_MEAS)
@@ -323,24 +338,44 @@ def _compensate_humidity(self, raw_hum):
     # --------------------------------------------------
 
     def temperature(self):
-        """Return compensated temperature in °C (float)."""
+        """Return compensated temperature in °C (float).
+
+        If the sensor is in sleep mode, a forced measurement is triggered
+        automatically before reading.
+        """
+        self._ensure_data()
         raw_temp, _, _ = self._read_raw()
         return self._compensate_temperature(raw_temp) / 100.0
 
     def pressure_hpa(self):
-        """Return compensated pressure in hPa (float)."""
+        """Return compensated pressure in hPa (float).
+
+        If the sensor is in sleep mode, a forced measurement is triggered
+        automatically before reading.
+        """
+        self._ensure_data()
         raw_temp, raw_press, _ = self._read_raw()
         self._compensate_temperature(raw_temp)
         return self._compensate_pressure(raw_press) / 25600.0
 
     def humidity(self):
-        """Return compensated relative humidity in %RH (float)."""
+        """Return compensated relative humidity in %RH (float).
+
+        If the sensor is in sleep mode, a forced measurement is triggered
+        automatically before reading.
+        """
+        self._ensure_data()
         raw_temp, _, raw_hum = self._read_raw()
         self._compensate_temperature(raw_temp)
         return self._compensate_humidity(raw_hum) / 1024.0
 
     def read(self):
-        """Return (temperature_c, pressure_hpa, humidity_rh) tuple."""
+        """Return (temperature_c, pressure_hpa, humidity_rh) tuple.
+
+        If the sensor is in sleep mode, a forced measurement is triggered
+        automatically before reading.
+        """
+        self._ensure_data()
         raw_temp, raw_press, raw_hum = self._read_raw()
         temp_c = self._compensate_temperature(raw_temp) / 100.0
         press_hpa = self._compensate_pressure(raw_press) / 25600.0

tests/scenarios/bme280.yaml

@@ -471,3 +471,69 @@ tests:
     prompt: "Do the values look reasonable?"
     expect_true: true
     mode: [hardware]
+
+  # ----- Auto-trigger (_ensure_data) -----
+
+  - name: "_is_sleep_mode returns True after init"
+    action: script
+    script: |
+      result = dev._is_sleep_mode()
+    expect_true: true
+    mode: [mock]
+
+  - name: "_is_sleep_mode returns False in normal mode"
+    action: script
+    script: |
+      dev.power_on()
+      result = not dev._is_sleep_mode()
+    expect_true: true
+    mode: [mock]
+
+  - name: "_ensure_data triggers forced mode in sleep"
+    action: script
+    script: |
+      dev.power_off()
+      i2c.clear_write_log()
+      dev._ensure_data()
+      log = i2c.get_write_log()
+      triggered = any(reg == 0xF4 and (data[0] & 0x03) == 0x01 for reg, data in log)
+      result = triggered
+    expect_true: true
+    mode: [mock]
+
+  - name: "_ensure_data is no-op in normal mode"
+    action: script
+    script: |
+      dev.power_on()
+      i2c.clear_write_log()
+      dev._ensure_data()
+      log = i2c.get_write_log()
+      # In normal mode, only a read of ctrl_meas (no write to 0xF4)
+      wrote_forced = any(reg == 0xF4 and (data[0] & 0x03) == 0x01 for reg, data in log)
+      result = not wrote_forced
+    expect_true: true
+    mode: [mock]
+
+  - name: "temperature() auto-triggers in sleep mode"
+    action: script
+    script: |
+      dev.power_off()
+      i2c.clear_write_log()
+      t = dev.temperature()
+      log = i2c.get_write_log()
+      triggered = any(reg == 0xF4 and (data[0] & 0x03) == 0x01 for reg, data in log)
+      result = triggered and abs(t - 25.08) < 0.1
+    expect_true: true
+    mode: [mock]
+
+  - name: "read() auto-triggers in sleep mode"
+    action: script
+    script: |
+      dev.power_off()
+      i2c.clear_write_log()
+      t, p, h = dev.read()
+      log = i2c.get_write_log()
+      triggered = any(reg == 0xF4 and (data[0] & 0x03) == 0x01 for reg, data in log)
+      result = triggered and abs(t - 25.08) < 0.1
+    expect_true: true
+    mode: [mock]