From cd76747b259d4a5164f90f161770c8a2f0277db2 Mon Sep 17 00:00:00 2001
From: "J. Nick Koston" <nick@home-assistant.io>
Date: Thu, 8 Jan 2026 22:19:52 -1000
Subject: [PATCH] tests

---
 tests/components/fan/common.yaml | 38 ++++++++++++++++++++++++++------
 1 file changed, 31 insertions(+), 7 deletions(-)

diff --git a/tests/components/fan/common.yaml b/tests/components/fan/common.yaml
index bbd5be327a..ccc822be5a 100644
--- a/tests/components/fan/common.yaml
+++ b/tests/components/fan/common.yaml
@@ -11,23 +11,47 @@ fan:
     speed_count: 3
 
 # Test lambdas using get_preset_mode() which returns std::string_view
+# These examples match the migration guide in the PR description
 binary_sensor:
   - platform: template
     id: fan_has_preset
     name: "Fan Has Preset"
     lambda: |-
-      // Test has_preset_mode() method
-      if (!id(test_fan).has_preset_mode()) {
-        return false;
+      // Migration guide: Checking if preset mode is set
+      // Use empty() or has_preset_mode()
+      if (!id(test_fan).get_preset_mode().empty()) {
+        // preset is set
       }
-      // Test .empty() on string_view
-      if (id(test_fan).get_preset_mode().empty()) {
-        return false;
+      if (id(test_fan).has_preset_mode()) {
+        // preset is set
       }
-      // Test == comparison with string literal
+
+      // Migration guide: Comparing preset mode
+      // Use == operator directly (safe, works even when empty)
       if (id(test_fan).get_preset_mode() == "Eco") {
         return true;
       }
+
+      // Migration guide: Checking for no preset
+      if (id(test_fan).get_preset_mode().empty()) {
+        // no preset
+      }
+      if (!id(test_fan).has_preset_mode()) {
+        // no preset
+      }
+
+      // Migration guide: Getting as std::string
+      std::string preset = std::string(id(test_fan).get_preset_mode());
+
+      // Migration guide: Logging option 1
+      // Use .data() - works because string_view points to null-terminated string in traits
+      ESP_LOGD("test", "Preset: %s", id(test_fan).get_preset_mode().data());
+
+      // Migration guide: Logging option 2
+      // Use %.*s format (safer, no null-termination assumption)
+      auto preset_view = id(test_fan).get_preset_mode();
+      ESP_LOGD("test", "Preset: %.*s", (int)preset_view.size(), preset_view.data());
+
       // Test != comparison
       if (id(test_fan).get_preset_mode() != "Sleep") {
         return true;