sascha/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-11-01 07:31:51 +00:00
Code Issues Releases Wiki Activity

Merge branch 'initial_option_template_select' into integration

Browse Source
This commit is contained in:
J. Nick Koston
2025-10-25 10:31:37 -07:00
parent 3d020d5c6f 3a49103584
commit bff02daa6c
3 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
esphome/components/template/select/__init__.py
View File

@@ -75,6 +75,9 @@ async def to_code(config):
else:
cg.add(var.set_optimistic(config[CONF_OPTIMISTIC]))
initial_option_index = config[CONF_OPTIONS].index(config[CONF_INITIAL_OPTION])
# Only set if non-zero to avoid bloating setup() function
# (initial_option_index_ is zero-initialized in the header)
if initial_option_index != 0:
cg.add(var.set_initial_option_index(initial_option_index))
if CONF_RESTORE_VALUE in config:

4
esphome/components/template/select/template_select.cpp
View File

@@ -60,13 +60,11 @@ void TemplateSelect::dump_config() {
LOG_UPDATE_INTERVAL(this);
if (this->f_.has_value())
return;
auto initial_option = this->at(this->initial_option_index_);
ESP_LOGCONFIG(TAG,
" Optimistic: %s\n"
" Initial Option: %s\n"
" Restore Value: %s",
YESNO(this->optimistic_),
initial_option.has_value() ? initial_option.value().c_str() : LOG_STR_LITERAL("unknown"),
YESNO(this->optimistic_), this->at(this->initial_option_index_).value().c_str(),
YESNO(this->restore_value_));
}

2
tests/integration/test_host_mode_empty_string_options.py
View File

@@ -129,7 +129,7 @@ async def test_host_mode_empty_string_options(
initial_state = states[initial_option_test.key]
assert initial_state.state == "Third", (
f"Expected initial state 'Third' but got '{initial_state.state}' - "
f"initial_option_index optimization may not be working correctly"
f"initial_option not correctly applied"
)
# The main test is that we got here without protobuf errors