fix(bme280): Address Copilot review comments on PR 314.

Author: nedseb

lib/bme280/README.md

@@ -13,7 +13,7 @@ The BME280 is a high-precision environmental sensor suitable for applications su
 
 ---
 
-# Features
+## Features
 
 * I2C communication
 * device identification
@@ -31,7 +31,7 @@ The BME280 is a high-precision environmental sensor suitable for applications su
 
 ---
 
-# Sensor Overview
+## Sensor Overview
 
 | Feature                | Value                   |
 | ---------------------- | ----------------------- |
@@ -46,7 +46,7 @@ The BME280 is a high-precision environmental sensor suitable for applications su
 
 ---
 
-# I2C Address
+## I2C Address
 
 The sensor can use two I2C addresses depending on the **SDO pin**:
 
@@ -59,7 +59,7 @@ The default address used by the driver is **0x76**.
 
 ---
 
-# Basic Usage
+## Basic Usage
 
 ```python
 from machine import I2C
@@ -71,7 +71,7 @@ i2c = I2C(1)
 sensor = BME280(i2c)
 
 while True:
-    temperature, pressure, humidity = sensor.read()
+    temperature, pressure, humidity = sensor.read_one_shot()
 
     print("T:", temperature, "C")
     print("P:", pressure, "hPa")
@@ -83,7 +83,7 @@ while True:
 
 ---
 
-# API Reference
+## API Reference
 
 ## Initialization
 
@@ -267,7 +267,7 @@ Performs a soft reset, re-reads calibration data, and re-applies default configu
 
 ---
 
-# Examples
+## Examples
 
 | Example              | Description                                        |
 | -------------------- | -------------------------------------------------- |
@@ -276,6 +276,6 @@ Performs a soft reset, re-reads calibration data, and re-applies default configu
 
 ---
 
-# References
+## References
 
 * [BME280 Datasheet](https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bme280-ds002.pdf)

lib/bme280/examples/basic_reader.py

@@ -13,7 +13,7 @@
 print("Device ID:", hex(sensor.device_id()))
 
 for _ in range(10):
-    temperature, pressure, humidity = sensor.read()
+    temperature, pressure, humidity = sensor.read_one_shot()
 
     print(
         "T: {:.1f} C  P: {:.1f} hPa  H: {:.1f} %RH".format(

tests/scenarios/bme280.yaml

@@ -420,50 +420,54 @@ tests:
   # ----- Hardware tests -----
 
   - name: "Temperature in plausible range"
-    action: call
-    method: temperature
-    expect_range: [10.0, 45.0]
+    action: script
+    script: |
+      t, _, _ = dev.read_one_shot()
+      result = 10.0 <= t <= 45.0
+    expect_true: true
     mode: [hardware]
 
   - name: "Pressure in plausible range"
-    action: call
-    method: pressure_hpa
-    expect_range: [900.0, 1100.0]
+    action: script
+    script: |
+      _, p, _ = dev.read_one_shot()
+      result = 900.0 <= p <= 1100.0
+    expect_true: true
     mode: [hardware]
 
   - name: "Humidity in plausible range"
-    action: call
-    method: humidity
-    expect_range: [5.0, 95.0]
+    action: script
+    script: |
+      _, _, h = dev.read_one_shot()
+      result = 5.0 <= h <= 95.0
+    expect_true: true
     mode: [hardware]
 
-  - name: "read() returns three plausible values"
+  - name: "read_one_shot returns three plausible values"
     action: script
     script: |
-      t, p, h = dev.read()
+      t, p, h = dev.read_one_shot()
       result = 10.0 <= t <= 45.0 and 900.0 <= p <= 1100.0 and 5.0 <= h <= 95.0
     expect_true: true
     mode: [hardware]
 
-  - name: "read_one_shot returns three plausible values"
+  - name: "Continuous mode produces plausible values"
     action: script
     script: |
-      t, p, h = dev.read_one_shot()
+      from time import sleep_ms
+      dev.set_continuous()
+      sleep_ms(100)
+      t, p, h = dev.read()
+      dev.power_off()
       result = 10.0 <= t <= 45.0 and 900.0 <= p <= 1100.0 and 5.0 <= h <= 95.0
     expect_true: true
     mode: [hardware]
 
   - name: "Readings feel correct"
     action: manual
     display:
-      - method: temperature
-        label: "Temperature"
-        unit: "C"
-      - method: pressure_hpa
-        label: "Pressure"
-        unit: "hPa"
-      - method: humidity
-        label: "Humidity"
-        unit: "%RH"
+      - method: read_one_shot
+        label: "T, P, H"
     prompt: "Do the values look reasonable?"
+    expect_true: true
     mode: [hardware]